[Buildroot] [PATCH] libxmlrpc: fix static build
Baruch Siach
baruch at tkos.co.il
Fri Jun 19 04:34:21 UTC 2015
Hi Thomas,
On Sun, Jun 14, 2015 at 11:00:49PM +0200, Thomas Petazzoni wrote:
> On Sun, 14 Jun 2015 22:07:38 +0300, Baruch Siach wrote:
> > Unfortunately, these make variables are set explicitly in common.mk. So
> > passing them in make environment has no effect.
>
> Argh, ok.
>
> > Now that I think about it, why don't we have a similar thing for MAKE_OPTS,
> > say, $(PKG)_MAKE_VARS. Something like (untested, host packages missing):
> >
> > diff --git a/package/pkg-autotools.mk b/package/pkg-autotools.mk
> > index 1d694f0618fc..27ff16235f27 100644
> > --- a/package/pkg-autotools.mk
> > +++ b/package/pkg-autotools.mk
> > @@ -154,10 +154,10 @@ endif
> > $(2)_CONF_ENV ?=
> > $(2)_CONF_OPTS ?=
> > $(2)_MAKE_ENV ?=
> > -$(2)_MAKE_OPTS ?=
> > -$(2)_INSTALL_OPTS ?= install
> > -$(2)_INSTALL_STAGING_OPTS ?= DESTDIR=$$(STAGING_DIR) install
> > -$(2)_INSTALL_TARGET_OPTS ?= DESTDIR=$$(TARGET_DIR) install
> > +$(2)_MAKE_OPTS ?= $$($(2)_MAKE_VARS)
> > +$(2)_INSTALL_OPTS ?= $$($(2)_MAKE_VARS) install
> > +$(2)_INSTALL_STAGING_OPTS ?= $$($(2)_MAKE_VARS) DESTDIR=$$(STAGING_DIR) install
> > +$(2)_INSTALL_TARGET_OPTS ?= $$($(2)_MAKE_VARS) DESTDIR=$$(TARGET_DIR) install
>
> Why not. Though I'd be more convinced if there is some evidence that it
> would also be useful for other packages.
I looked around and I found a few more packages that might benefit from
_MAKE_VARS. But I thing the gain is not enough to worth the trouble. Adding
$(2)_MAKE_VARS might also lead to confusion when a user sets both _MAKE_VARS
and one or more other _OPTS, thus silently overriding _MAKE_VARS in a way that
is not apparent to the reader.
So, I suggest to just apply the original patch.
baruch
--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
More information about the buildroot
mailing list