NetCube-Systems-Austria/kumquat-buildroot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
f497900bfd
kumquat-buildroot/package/powertop/powertop.mk

30 lines
867 B
Makefile
Raw Normal View History

powertop: new package [Thomas: - fix commit title - powertop wants libintl unconditionally, so make sure BR2_PACKAGE_GETTEXT is selected when BR2_NEEDS_GETTEXT is set, and add gettext to the dependencies. - add missing comment about thread dependency. - add missing dependency on host-pkgconf, without which powertop cannot find libnl. - patch src/Makefile.am to not pass -fstack-protector, which fails to build if the toolchain does not have SSP support. - rename patch powertop-autotune.patch to confirm to the patch naming convention.] Signed-off-by: Steven Noonan <steven@uplinklabs.net> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
2015-03-19 06:40:21 +01:00
################################################################################
#
# powertop
#
################################################################################
POWERTOP_VERSION = 2.7
package: Remove trailing slash from all package site URLs The recommended form is without the trailing slash, and will become mandatory in a coming commit. This avoids the need for the $$($(2)_SITE:/=) magic in package/pkg-generic.mk to avoid double slashes in download URLs, like "https://mosh.mit.edu//mosh-1.2.5.tar.gz". ^^ Note: this work has already been done in b0b9606530dfc6de4030 a few months ago and earlier in c7f4b964718bc5a3329b and 4a9eb20de817fa64, but no check has been added at that time to avoid new slashes to slip in, and so they did. This time a patch will follow immediately to prevent future mistakes from being unnoticed. Mass-replaced with the following command: git grep -l '_SITE.*/$' | xargs sed -i '/_SITE.*=/s|/$||' Signed-off-by: Luca Ceresoli <luca@lucaceresoli.net> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
2015-09-28 18:10:31 +02:00
POWERTOP_SITE = https://01.org/sites/default/files/downloads/powertop
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 <bernd.kuhls@t-online.de> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
2015-03-27 15:20:30 +01:00
POWERTOP_DEPENDENCIES = pciutils ncurses libnl host-gettext host-pkgconf
powertop: new package [Thomas: - fix commit title - powertop wants libintl unconditionally, so make sure BR2_PACKAGE_GETTEXT is selected when BR2_NEEDS_GETTEXT is set, and add gettext to the dependencies. - add missing comment about thread dependency. - add missing dependency on host-pkgconf, without which powertop cannot find libnl. - patch src/Makefile.am to not pass -fstack-protector, which fails to build if the toolchain does not have SSP support. - rename patch powertop-autotune.patch to confirm to the patch naming convention.] Signed-off-by: Steven Noonan <steven@uplinklabs.net> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
2015-03-19 06:40:21 +01:00
POWERTOP_LICENSE = GPLv2
POWERTOP_LICENSE_FILES = COPYING
# We're patching Makefile.am
POWERTOP_AUTORECONF = YES
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 <bernd.kuhls@t-online.de> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
2015-03-27 15:20:30 +01:00
POWERTOP_GETTEXTIZE = YES
powertop: new package [Thomas: - fix commit title - powertop wants libintl unconditionally, so make sure BR2_PACKAGE_GETTEXT is selected when BR2_NEEDS_GETTEXT is set, and add gettext to the dependencies. - add missing comment about thread dependency. - add missing dependency on host-pkgconf, without which powertop cannot find libnl. - patch src/Makefile.am to not pass -fstack-protector, which fails to build if the toolchain does not have SSP support. - rename patch powertop-autotune.patch to confirm to the patch naming convention.] Signed-off-by: Steven Noonan <steven@uplinklabs.net> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
2015-03-19 06:40:21 +01:00
powertop: only link against -lintl when needed BR2_PACKAGE_GETTEXT=y might be enabled even if BR2_NEEDS_GETTEXT is not set, and in this case, we should not link against libintl, because it may not exist, for example with glibc toolchains. Fixes: http://autobuild.buildroot.net/results/270/270018e2c9e3845a4015faa155325eea2cabe3d9/ Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
2015-03-21 09:39:22 +01:00
ifeq ($(BR2_NEEDS_GETTEXT),y)
powertop: new package [Thomas: - fix commit title - powertop wants libintl unconditionally, so make sure BR2_PACKAGE_GETTEXT is selected when BR2_NEEDS_GETTEXT is set, and add gettext to the dependencies. - add missing comment about thread dependency. - add missing dependency on host-pkgconf, without which powertop cannot find libnl. - patch src/Makefile.am to not pass -fstack-protector, which fails to build if the toolchain does not have SSP support. - rename patch powertop-autotune.patch to confirm to the patch naming convention.] Signed-off-by: Steven Noonan <steven@uplinklabs.net> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
2015-03-19 06:40:21 +01:00
POWERTOP_DEPENDENCIES += gettext
POWERTOP_CONF_ENV += LIBS='-lintl'
endif
package/powertop: handle ncursesw properly and depend on wchar The powertop code uses some wide-char functions, so the package should depend on wide-char support. Also, when wide-char support is available, the wide-char variant of ncurses may be built, but powertop detection logic isn't quite up to the game, so we help him a little bit. Thanks to Gustavo Zacarias for doing some additional investigation and proposing the solution implemented in this patch. Fixes http://autobuild.buildroot.net/results/913/913cea22f8a8f5902d8da5f64c3fce056d66790f/ http://autobuild.buildroot.net/results/6e1/6e11fa2a7405a69c59ced046b92ff08660c4aab7/ http://autobuild.buildroot.net/results/1d3/1d3323b2afaefa7989854dbccf92015731199e66/ http://autobuild.buildroot.net/results/1e3/1e31d412d8b3a38a375ad0be8f696bee993ec297/ http://autobuild.buildroot.net/results/2ff/2ff511eb8d00b94aca68427446e2d0f6e4317a5a/ and maybe others [Thomas: - do not force select NCURSES_WCHAR, instead improve the detection logic to properly use it when available. Solution provided by Gustavo. - improve the commit log.] Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
2015-03-27 17:01:36 +01:00
# Help powertop at finding the right ncurses library depending on
# which one is available.
ifeq ($(BR2_PACKAGE_NCURSES_WCHAR),y)
POWERTOP_CONF_ENV += ac_cv_search_delwin="-lncursesw"
else
POWERTOP_CONF_ENV += ac_cv_search_delwin="-lncurses"
endif
powertop: new package [Thomas: - fix commit title - powertop wants libintl unconditionally, so make sure BR2_PACKAGE_GETTEXT is selected when BR2_NEEDS_GETTEXT is set, and add gettext to the dependencies. - add missing comment about thread dependency. - add missing dependency on host-pkgconf, without which powertop cannot find libnl. - patch src/Makefile.am to not pass -fstack-protector, which fails to build if the toolchain does not have SSP support. - rename patch powertop-autotune.patch to confirm to the patch naming convention.] Signed-off-by: Steven Noonan <steven@uplinklabs.net> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
2015-03-19 06:40:21 +01:00
$(eval $(autotools-package))
Reference in New Issue Copy Permalink