* [Buildroot] [PATCH] dropwatch: fix static link
@ 2017-06-30 4:48 Baruch Siach
2017-07-01 10:44 ` Thomas Petazzoni
0 siblings, 1 reply; 2+ messages in thread
From: Baruch Siach @ 2017-06-30 4:48 UTC (permalink / raw)
To: buildroot
Use LDLIBS for the list of libraries, since LDFLAGS appear before the source
files in the link command, and the upstream LDLIBS lacks -lncurses which is a
dependency of readline. We already pass target LDFLAGS in
TARGET_CONFIGURE_OPTS, so drop that part.
Fixes:
http://autobuild.buildroot.net/results/ce4/ce436bdd7e7950c801edac01f311b89110d87d62/
http://autobuild.buildroot.net/results/691/691d61d13849150c3616b1c74c964f50118ff1d9/
http://autobuild.buildroot.net/results/291/2918c75e473c5a2434e72f2e409becc5b4d07908/
Cc: Tzu-Jung Lee <roylee17@gmail.com>
Signed-off-by: Baruch Siach <baruch@tkos.co.il>
---
package/dropwatch/dropwatch.mk | 11 +++++------
1 file changed, 5 insertions(+), 6 deletions(-)
diff --git a/package/dropwatch/dropwatch.mk b/package/dropwatch/dropwatch.mk
index bb91fd784737..e08a0c1357bf 100644
--- a/package/dropwatch/dropwatch.mk
+++ b/package/dropwatch/dropwatch.mk
@@ -11,18 +11,17 @@ DROPWATCH_LICENSE = GPL-2.0
DROPWATCH_LICENSE_FILES = COPYING
# libbfd may be linked to libintl
-# Ugly... but LDFLAGS are hardcoded anyway
-DROPWATCH_LDFLAGS = \
- $(TARGET_LDFLAGS) -lbfd -lreadline -lnl-3 -lnl-genl-3 \
- -lpthread -lncurses -lm
+# Ugly... but LDLIBS are hardcoded anyway
+DROPWATCH_LDLIBS = \
+ -lbfd -lreadline -lnl-3 -lnl-genl-3 -lpthread -lncurses -lm
ifeq ($(BR2_NEEDS_GETTEXT_IF_LOCALE),y)
-DROPWATCH_LDFLAGS += -lintl
+DROPWATCH_LDLIBS += -lintl
endif
define DROPWATCH_BUILD_CMDS
$(TARGET_CONFIGURE_OPTS) $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) \
- LDFLAGS="$(DROPWATCH_LDFLAGS)" build
+ LDLIBS="$(DROPWATCH_LDLIBS)" build
endef
define DROPWATCH_INSTALL_TARGET_CMDS
--
2.11.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [Buildroot] [PATCH] dropwatch: fix static link
2017-06-30 4:48 [Buildroot] [PATCH] dropwatch: fix static link Baruch Siach
@ 2017-07-01 10:44 ` Thomas Petazzoni
0 siblings, 0 replies; 2+ messages in thread
From: Thomas Petazzoni @ 2017-07-01 10:44 UTC (permalink / raw)
To: buildroot
Hello,
On Fri, 30 Jun 2017 07:48:34 +0300, Baruch Siach wrote:
> Use LDLIBS for the list of libraries, since LDFLAGS appear before the source
> files in the link command, and the upstream LDLIBS lacks -lncurses which is a
> dependency of readline. We already pass target LDFLAGS in
> TARGET_CONFIGURE_OPTS, so drop that part.
>
> Fixes:
> http://autobuild.buildroot.net/results/ce4/ce436bdd7e7950c801edac01f311b89110d87d62/
> http://autobuild.buildroot.net/results/691/691d61d13849150c3616b1c74c964f50118ff1d9/
> http://autobuild.buildroot.net/results/291/2918c75e473c5a2434e72f2e409becc5b4d07908/
>
> Cc: Tzu-Jung Lee <roylee17@gmail.com>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> ---
> package/dropwatch/dropwatch.mk | 11 +++++------
> 1 file changed, 5 insertions(+), 6 deletions(-)
Applied to master, thanks.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-07-01 10:44 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-06-30 4:48 [Buildroot] [PATCH] dropwatch: fix static link Baruch Siach
2017-07-01 10:44 ` Thomas Petazzoni
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.