[Buildroot] [PATCH for-next 2/3] package/swig: enable PCRE regex matching in host-swig

[Peter Korsgaard]

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni at bootlin.com> writes:

 > Hello,
 > +Arnout, Yann, Peter in Cc.

 > On Wed, 29 May 2019 17:16:54 +0200
 > Romain Naour <romain.naour at gmail.com> wrote:

 >> The upcoming ogre package needs a host swig binary with PCRE
 >> regex support.
 >> 
 >> Signed-off-by: Romain Naour <romain.naour at gmail.com>
 >> ---
 >> package/swig/swig.mk | 4 ++--
 >> 1 file changed, 2 insertions(+), 2 deletions(-)

 > I never really like the addition of mandatory host package dependencies
 > just for the sake of one use case. I was considering adding a hidden
 > BR2_PACKAGE_HOST_PCRE option for this, but since pcre takes only 17
 > seconds to build, maybe it was not worth the effort. So I've applied to
 > next as-is, but I'd be interested to know what Arnout/Peter/Yann thing
 > about adding a hidden BR2_PACKAGE_HOST_PCRE hidden option for that kind
 > of situation. We had decided to not systematically add such options in
 > all host packages, but that we could add some in a case-by-case basis.
 > This case-by-case basis however will look "weird": ogre would have
 > host-swig in its <pkg>_DEPENDENCIES, but in Config.in would select
 > BR2_PACKAGE_HOST_PCRE and not BR2_PACKAGE_HOST_SWIG. This could start
 > getting a bit clunky.

I think just having it unconditionally enabled is OK for now.

-- 
Bye, Peter Korsgaard