yara: update to 1.7

[a.schuster@…]

Updated YARA to version 1.7. Added variants to build either with re2 or pcre support.

[cooljeanius (Eric Gallager)]

Diff between this and the current Portfile?

[a.schuster@…]

Replying to egall@…:

Diff between this and the current Portfile?

.. is now attached to the ticket.

[ryandesign (Ryan Carsten Schmidt)]

Thanks. Some observations:

• The $Id$ line should not be removed.
• When updating to a new version, the revision line should be removed or set to 0.
• "universal_variant yes" is the default and does not need to be mentioned explicitly.
• The pcre variant just adds a dependency; that's usually not sufficient. The port needs to ensure that the pcre library will not be used if the pcre variant is not selected.

[a.schuster@…]

Replying to ryandesign@…:

Thanks. Some observations:

Thanks for your helpful comments! I've attached a revised Portfile.

• The $Id$ line should not be removed.
• When updating to a new version, the revision line should be removed or set to 0.
• "universal_variant yes" is the default and does not need to be mentioned explicitly.

done.

• The pcre variant just adds a dependency; that's usually not sufficient. The port needs to ensure that the pcre library will not be used if the pcre variant is not selected.

Now there's only one variant that enables re2, and at the same time disables pcre.

[ryandesign (Ryan Carsten Schmidt)]

Replying to a.schuster@…:

Now there's only one variant that enables re2, and at the same time disables pcre.

Ok, that works.

I didn't mind the two conflicting variants, in fact I usually prefer that when there is a choice of two different libraries for a feature (vs. when it's just a choice of whether to enable or disable a feature, in which case a single variant should be used). What I minded was that you left open the possibility that the user could deselect both of the variants, which would then still use pcre if it was installed but not declare a dependency on it.

[ryandesign (Ryan Carsten Schmidt)]

Replying to a.schuster@…:

Thanks for your helpful comments! I've attached a revised Portfile.

r105138