Skip to contentSkip to conversation

package/powertop: needs gettextize, 2nd try

PATCH New Bernd Kuhls 2 participants 2 messages Mar 27, 2015

lore
Share & tools
lore

Bernd Kuhls

author
#0 lore raw

PATCH 1/1 package/powertop: needs gettextize, 2nd try

If host-gettext was compiled before powertop it is picked up by autoreconf which leads to a broken compile. To fix this unconditionally include host-gettext as dependency and move

POWERTOP_GETTEXTIZE = YES

outside the if-clause for BR2_NEEDS_GETTEXT

Fixes
http://autobuild.buildroot.net/results/24e/24e312271cf07a8dd966b43f153a06ddc858a786/
http://autobuild.buildroot.net/results/511/511a1c76bfd0ebf637e5abcd6fecc4d6a683ed3e/
http://autobuild.buildroot.net/results/652/65246f6f71e67c2637efbe8bb6e501750629d5fb/

and many others

Signed-off-by Bernd Kuhls <[email protected]>
---
 package/powertop/powertop.mk |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
package/powertop/powertop.mk +2 -2
index 4446bc9..331edc9 100644
@@ -6,14 +6,14 @@
 
 POWERTOP_VERSION = 2.7
 POWERTOP_SITE = https://01.org/sites/default/files/downloads/powertop/
-POWERTOP_DEPENDENCIES = pciutils ncurses libnl host-pkgconf
+POWERTOP_DEPENDENCIES = pciutils ncurses libnl host-gettext host-pkgconf
 POWERTOP_LICENSE = GPLv2
 POWERTOP_LICENSE_FILES = COPYING
 # We're patching Makefile.am
 POWERTOP_AUTORECONF = YES
+POWERTOP_GETTEXTIZE = YES
 
 ifeq ($(BR2_NEEDS_GETTEXT),y)
-POWERTOP_GETTEXTIZE = YES
 POWERTOP_DEPENDENCIES += gettext
 POWERTOP_CONF_ENV += LIBS='-lintl'
 endif
Signature · 1 line
1.7.10.4
Applied
#1 lore raw

PATCH 1/1 package/powertop: needs gettextize, 2nd try

Dear Bernd Kuhls,

Quoted 8 lines · Bernd Kuhls

If host-gettext was compiled before powertop it is picked up by autoreconf which leads to a broken compile. To fix this unconditionally include host-gettext as dependency and move

POWERTOP_GETTEXTIZE = YES

outside the if-clause for BR2_NEEDS_GETTEXT

Quoted 8 lines

Fixes
http://autobuild.buildroot.net/results/24e/24e312271cf07a8dd966b43f153a06ddc858a786/
http://autobuild.buildroot.net/results/511/511a1c76bfd0ebf637e5abcd6fecc4d6a683ed3e/
http://autobuild.buildroot.net/results/652/65246f6f71e67c2637efbe8bb6e501750629d5fb/

and many others

Signed-off-by Bernd Kuhls <[email protected]>

Applied, thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com