[Buildroot] [PATCH v2] dropwatch: fix avr32 build failure

Peter Korsgaard jacmet at uclibc.org
Tue Nov 5 13:25:39 UTC 2013


>>>>> "spdawson" == spdawson  <spdawson at gmail.com> writes:

 > From: Simon Dawson <spdawson at gmail.com>
 > On avr32, dropwatch needs libiberty to avoid build failures such as

 >   http://autobuild.buildroot.net/results/cd5/cd5e22fb5c9b0fc5d396bc85a5e253a1a65054da/

 > Also test built on ARM.

 > Signed-off-by: Simon Dawson <spdawson at gmail.com>
 > ---
 >  v2: Handle using LDFLAGS in package makefile, rather than by patching. This
 >      was suggested by Peter Korsgaard.

 >  package/dropwatch/dropwatch.mk | 6 +++++-
 >  1 file changed, 5 insertions(+), 1 deletion(-)

 > diff --git a/package/dropwatch/dropwatch.mk b/package/dropwatch/dropwatch.mk
 > index 5d65d86..9d49d1e 100644
 > --- a/package/dropwatch/dropwatch.mk
 > +++ b/package/dropwatch/dropwatch.mk
 > @@ -13,8 +13,12 @@ DROPWATCH_LICENSE_FILES = COPYING
 
 >  # libbfd may be linked to libintl
 >  # Ugly... but LDFLAGS are hardcoded anyway
 > +#
 > +# Also: always need to add -liberty to hardcoded LDFLAGS for avr32
 >  ifeq ($(BR2_NEEDS_GETTEXT_IF_LOCALE),y)
 > -DROPWATCH_LDFLAGS = LDFLAGS="-lintl -lbfd -lreadline -lnl-3 -lnl-genl-3"
 > +DROPWATCH_LDFLAGS = LDFLAGS="-lintl -lbfd -liberty -lreadline -lnl-3 -lnl-genl-3"
 > +else
 > +DROPWATCH_LDFLAGS = LDFLAGS="-lbfd -liberty -lreadline -lnl-3 -lnl-genl-3"
 >  endif

I've changed this so the GETTEXT_IF_LOCALE case appends to the LDFLAGS
instead of having this twice.

Committed with that fixed, thanks.

-- 
Bye, Peter Korsgaard


More information about the buildroot mailing list