[Buildroot] [PATCH 1/1] package/libmodsecurity: fix build with libmaxminddb
Yann E. MORIN
yann.morin.1998 at free.fr
Sun Jul 18 09:25:17 UTC 2021
Fabrice, All,
On 2021-07-18 10:33 +0200, Fabrice Fontaine spake thusly:
> Le dim. 18 juil. 2021 à 10:08, Yann E. MORIN <yann.morin.1998 at free.fr> a écrit :
[--SNIP--]
> > > > diff --git a/package/libmodsecurity/libmodsecurity.mk b/package/libmodsecurity/libmodsecurity.mk
> > > > index b2637257b6..10338dad7c 100644
> > > > --- a/package/libmodsecurity/libmodsecurity.mk
> > > > +++ b/package/libmodsecurity/libmodsecurity.mk
> > > > @@ -14,6 +14,24 @@ LIBMODSECURITY_CPE_ID_VENDOR = trustwave
> > > > LIBMODSECURITY_CPE_ID_PRODUCT = modsecurity
> > > >
> > > > LIBMODSECURITY_DEPENDENCIES = pcre
> > > > +
> > > > +# configure.ac will call git to get various version strings, from
> > > > +# libmodsecurity itself, but also from bundled dependencies, just
> > > > +# for the sake of siplaying them at the end of configure.
> > > > +# Fake a git that returns the version for libmodescurity, to quiesce
> > > > +# warnings...
> > > > +define LIBMODSECURITY_FAKE_GIT
> > > > + $(Q)mkdir -p $(@D)/.buildroot
> > > > + $(Q)printf '#!/usr/bin/env sh\necho "$(LIBMODSECURITY_VERSION)"\n' \
> > > > + >$(@D)/.buildroot/git
> > > > + @(Q)chmod 755 $(@D)/.buildroot/git
> > > > +endef
> > > > +LIBMODSECURITY_POST_EXTRACT_HOOKS += LIBMODSECURITY_FAKE_GIT
> > > > +
> > > > +# 0001-Revert-Fix-maxminddb-link-on-FreeBSD.patch
> > > > +LIBMODSECURITY_AUTORECONF = YES
> > > > +LIBMODSECURITY_AUTORECONF_ENV = PATH="$(@D)/.buildroot:$(BR_PATH)"
Becarefull here: BR_PATH is already quoted:
Makefile:478:BR_PATH = "$(HOST_DIR)/bin:$(HOST_DIR)/sbin:$(PATH)"
How do other packages do? Hmm... It sems they usually do not quote
anything, e.g.:
package/nodejs/nodejs.mk:91: PATH=$(@D)/bin:$(BR_PATH)
package/qt5/qt5webkit/qt5webkit.mk:33:QT5WEBKIT_CONF_ENV = PATH=$(@D)/host-bin:$(BR_PATH)
So, no quoting...
However, wireshark got it wrong:
package/wireshark/wireshark.mk:24: PATH="$(@D)/bin:$(BR_PATH)"
Regards,
Yann E. MORIN.
> > > > +
> > > > LIBMODSECURITY_CONF_OPTS = \
> > > > --with-pcre="$(STAGING_DIR)/usr/bin/pcre-config" \
> > > > --disable-examples \
> >
> > Regards,
> > Yann E. MORIN.
> >
> > --
> > .-----------------.--------------------.------------------.--------------------.
> > | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
> > | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
> > | +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
> > | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
> > '------------------------------^-------^------------------^--------------------'
> Best Regards,
>
> Fabrice
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list