Message ID | 1439732249-14832-1-git-send-email-yann.morin.1998@free.fr |
---|---|
State | Changes Requested |
Headers | show |
Hi Yann, On Sun, Aug 16, 2015 at 03:37:29PM +0200, Yann E. MORIN wrote: > +# popt links with -lintl and specifies it in its popt.pc, and has done > +# so for the past 6+ years, but oprofile does not use pkconfig to find > +# popt, so we need to tell it to link with -lintl for a static build. > +ifeq ($(BR2_PACKAGE_GETTEXT)$(BR2_STATIC_LIBS),yy) > +OPROFILE_CONF_OPTS += LIBS=-lintl > +endif Why not run $(PKG_CONFIG_HOST_BINARY) directly like we do in package/logrotate/logrotate.mk for popt? baruch
Baruch, All, On 2015-08-16 16:43 +0300, Baruch Siach spake thusly: > On Sun, Aug 16, 2015 at 03:37:29PM +0200, Yann E. MORIN wrote: > > +# popt links with -lintl and specifies it in its popt.pc, and has done > > +# so for the past 6+ years, but oprofile does not use pkconfig to find > > +# popt, so we need to tell it to link with -lintl for a static build. > > +ifeq ($(BR2_PACKAGE_GETTEXT)$(BR2_STATIC_LIBS),yy) > > +OPROFILE_CONF_OPTS += LIBS=-lintl > > +endif > > Why not run $(PKG_CONFIG_HOST_BINARY) directly like we do in > package/logrotate/logrotate.mk for popt? Good idea, I'll do. Thanks! :-) Regards, Yann E. MORIN.
diff --git a/package/oprofile/oprofile.mk b/package/oprofile/oprofile.mk index 5876307..3a5f807 100644 --- a/package/oprofile/oprofile.mk +++ b/package/oprofile/oprofile.mk @@ -41,6 +41,13 @@ ifeq ($(BR2_PACKAGE_LIBPFM4),y) OPROFILE_DEPENDENCIES += libpfm4 endif +# popt links with -lintl and specifies it in its popt.pc, and has done +# so for the past 6+ years, but oprofile does not use pkconfig to find +# popt, so we need to tell it to link with -lintl for a static build. +ifeq ($(BR2_PACKAGE_GETTEXT)$(BR2_STATIC_LIBS),yy) +OPROFILE_CONF_OPTS += LIBS=-lintl +endif + define OPROFILE_INSTALL_TARGET_CMDS $(INSTALL) -d -m 755 $(TARGET_DIR)/usr/bin $(INSTALL) -d -m 755 $(TARGET_DIR)/usr/share/oprofile
oprofile links with popt, which links with -lintl and specifies it in its popt.pc, and has done so for the past 6+ years, but oprofile does not use pkconfig to find popt, so we need to tell it to link with -lintl for a static build. Fixes: http://autobuild.buildroot.org/results/a48/a485f2171a3fcbfa2e0e5aec8e653cf88512615d/ http://autobuild.buildroot.org/results/b37/b374874f1915ab1d7aa6262604ccc8db102bb0b0/ http://autobuild.buildroot.org/results/261/2616a63081ed8c55c7ca1a6879b1a6a14e19f918/ http://autobuild.buildroot.org/results/73f/73f34ab59d54ded68ff60608a0212a6e5a5190e5/ ... Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr> --- package/oprofile/oprofile.mk | 7 +++++++ 1 file changed, 7 insertions(+)