Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/2] package/powertop: picutils is optional, not mandatory
@ 2023-09-29 16:12 Fabrice Fontaine
  2023-09-29 16:12 ` [Buildroot] [PATCH 2/2] package/powertop: bump to version 2.15 Fabrice Fontaine
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Fabrice Fontaine @ 2023-09-29 16:12 UTC (permalink / raw)
  To: buildroot; +Cc: Steven Noonan, Fabrice Fontaine

pciutils is optional, not mandatory since version 2.1 and the addition
of the package in commit 0adc0e24eed72efca0097c7567319f07b8af6d94 and
https://github.com/fenrus75/powertop/commit/98ca605e7cf856795facc4a3a4f9c5b9baf1cab1

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 package/powertop/Config.in   | 1 -
 package/powertop/powertop.mk | 3 ++-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/powertop/Config.in b/package/powertop/Config.in
index 0f323a7bfb..fd88a3d0eb 100644
--- a/package/powertop/Config.in
+++ b/package/powertop/Config.in
@@ -4,7 +4,6 @@ config BR2_PACKAGE_POWERTOP
 	depends on BR2_TOOLCHAIN_HAS_THREADS # libnl
 	depends on BR2_USE_WCHAR
 	select BR2_PACKAGE_NCURSES
-	select BR2_PACKAGE_PCIUTILS
 	select BR2_PACKAGE_LIBNL
 	help
 	  A tool to diagnose issues with power consumption and power
diff --git a/package/powertop/powertop.mk b/package/powertop/powertop.mk
index c3e8d2549c..681f5fda46 100644
--- a/package/powertop/powertop.mk
+++ b/package/powertop/powertop.mk
@@ -6,7 +6,8 @@
 
 POWERTOP_VERSION = 2.13
 POWERTOP_SITE = https://01.org/sites/default/files/downloads
-POWERTOP_DEPENDENCIES = pciutils ncurses libnl host-pkgconf \
+POWERTOP_DEPENDENCIES = ncurses libnl host-pkgconf \
+	$(if $(BR2_PACKAGE_PCIUTILS),pciutils) \
 	$(TARGET_NLS_DEPENDENCIES)
 POWERTOP_LICENSE = GPL-2.0
 POWERTOP_LICENSE_FILES = COPYING
-- 
2.40.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [Buildroot] [PATCH 2/2] package/powertop: bump to version 2.15
  2023-09-29 16:12 [Buildroot] [PATCH 1/2] package/powertop: picutils is optional, not mandatory Fabrice Fontaine
@ 2023-09-29 16:12 ` Fabrice Fontaine
  2023-09-30 17:19   ` Yann E. MORIN
  2023-09-30 17:19 ` [Buildroot] [PATCH 1/2] package/powertop: picutils is optional, not mandatory Yann E. MORIN
  2023-10-13  6:21 ` Peter Korsgaard
  2 siblings, 1 reply; 5+ messages in thread
From: Fabrice Fontaine @ 2023-09-29 16:12 UTC (permalink / raw)
  To: buildroot; +Cc: Steven Noonan, Fabrice Fontaine

- Switch site to get latest version
- Replace patch by an upstreamable one

https://github.com/fenrus75/powertop/compare/v2.13...v2.15

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 .checkpackageignore                           |  1 -
 ...1-add-disable-stack-protector-option.patch | 55 +++++++++++++++++++
 ...dont-force-stack-smashing-protection.patch | 18 ------
 package/powertop/Config.in                    |  2 +-
 package/powertop/powertop.hash                |  6 +-
 package/powertop/powertop.mk                  | 16 ++++--
 6 files changed, 70 insertions(+), 28 deletions(-)
 create mode 100644 package/powertop/0001-add-disable-stack-protector-option.patch
 delete mode 100644 package/powertop/0001-dont-force-stack-smashing-protection.patch

diff --git a/.checkpackageignore b/.checkpackageignore
index 0c7fae9409..d6873fb5a3 100644
--- a/.checkpackageignore
+++ b/.checkpackageignore
@@ -1110,7 +1110,6 @@ package/poke/0002-lib-getrandom.c-fix-build-with-uclibc-1.0.35.patch Upstream
 package/policycoreutils/0001-Add-DESTDIR-to-all-paths-that-use-an-absolute-path.patch Upstream
 package/policycoreutils/0002-Add-PREFIX-to-host-paths.patch Upstream
 package/postgresql/S50postgresql Variables
-package/powertop/0001-dont-force-stack-smashing-protection.patch Upstream
 package/pppd/0001-pppd-Fix-compilation-with-older-glibc-or-kernel-headers.patch Upstream
 package/pppd/0002-pppd-eap-tls.c-fix-build-with-libressl.patch Upstream
 package/pptp-linux/0001-susv3-legacy.patch Upstream
diff --git a/package/powertop/0001-add-disable-stack-protector-option.patch b/package/powertop/0001-add-disable-stack-protector-option.patch
new file mode 100644
index 0000000000..01a030522d
--- /dev/null
+++ b/package/powertop/0001-add-disable-stack-protector-option.patch
@@ -0,0 +1,55 @@
+From 95382246ddd889839633aa0da800a03936b93986 Mon Sep 17 00:00:00 2001
+From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+Date: Sun, 31 Oct 2021 18:26:01 +0100
+Subject: [PATCH] add --disable-stack-protector option
+
+Allow the user to disable stack-protector as not all toolchains support
+this feature
+
+Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+Upstream: https://github.com/fenrus75/powertop/pull/138
+---
+ configure.ac    | 4 ++++
+ src/Makefile.am | 5 ++++-
+ 2 files changed, 8 insertions(+), 1 deletion(-)
+
+diff --git a/configure.ac b/configure.ac
+index 37c1304..69160d5 100644
+--- a/configure.ac
++++ b/configure.ac
+@@ -43,6 +43,10 @@ AX_ADD_FORTIFY_SOURCE
+ AX_CXX_COMPILE_STDCXX([11], [noext], [mandatory])
+ PKG_PROG_PKG_CONFIG
+ 
++AC_ARG_ENABLE([stack-protector],
++	AS_HELP_STRING([--disable-stack-protector], [Disable stack-protector]))
++AM_CONDITIONAL([ENABLE_STACK_PROTECTOR], [test x$enable_stack_protector != xno])
++
+ # Checks for libraries.
+ AX_PTHREAD([
+     LIBS="$PTHREAD_LIBS $LIBS"
+diff --git a/src/Makefile.am b/src/Makefile.am
+index 6b523f6..ca30d20 100644
+--- a/src/Makefile.am
++++ b/src/Makefile.am
+@@ -138,13 +138,16 @@ powertop_CXXFLAGS = \
+ 	-Wformat \
+ 	-Wshadow \
+ 	-fno-omit-frame-pointer \
+-	-fstack-protector \
+ 	$(GLIB2_CFLAGS) \
+ 	$(LIBNL_CFLAGS) \
+ 	$(NCURSES_CFLAGS) \
+ 	$(PCIUTILS_CFLAGS) \
+ 	$(PTHREAD_CFLAGS)
+ 
++if ENABLE_STACK_PROTECTOR
++powertop_CXXFLAGS += \
++	-fstack-protector
++endif
+ 
+ powertop_CPPFLAGS = \
+ 	-DLOCALEDIR=\"$(localedir)\" \
+-- 
+2.33.0
+
diff --git a/package/powertop/0001-dont-force-stack-smashing-protection.patch b/package/powertop/0001-dont-force-stack-smashing-protection.patch
deleted file mode 100644
index 75ed2d1c24..0000000000
--- a/package/powertop/0001-dont-force-stack-smashing-protection.patch
+++ /dev/null
@@ -1,18 +0,0 @@
-Do not pass -fstack-protector unconditionally
-
-Using -fstack-protector only works when the toolchain has SSP support.
-
-Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
-Index: b/src/Makefile.am
-===================================================================
---- a/src/Makefile.am
-+++ b/src/Makefile.am
-@@ -127,7 +127,6 @@
- 	-Wformat \
- 	-Wshadow \
- 	-fno-omit-frame-pointer \
--	-fstack-protector \
- 	$(GLIB2_CFLAGS) \
- 	$(LIBNL_CFLAGS) \
- 	$(NCURSES_CFLAGS) \
diff --git a/package/powertop/Config.in b/package/powertop/Config.in
index fd88a3d0eb..2f7ba2bb9d 100644
--- a/package/powertop/Config.in
+++ b/package/powertop/Config.in
@@ -9,7 +9,7 @@ config BR2_PACKAGE_POWERTOP
 	  A tool to diagnose issues with power consumption and power
 	  management
 
-	  https://01.org/powertop/
+	  https://github.com/fenrus75/powertop/
 
 comment "powertop needs a toolchain w/ C++, threads, wchar"
 	depends on !BR2_INSTALL_LIBSTDCPP || !BR2_TOOLCHAIN_HAS_THREADS || \
diff --git a/package/powertop/powertop.hash b/package/powertop/powertop.hash
index fc8f3cc52f..810b80a514 100644
--- a/package/powertop/powertop.hash
+++ b/package/powertop/powertop.hash
@@ -1,5 +1,3 @@
-# Locally calculated after checking pgp signature
-# https://01.org/sites/default/files/downloads/powertop-2.13.tar.gz.asc.txt
-# using key 22E8F306C8FA4BAA2A5F36F3A0303B060918941C
-sha256  a65f992ca4a419bc73b623651060eb9fc00c5a86fa03556358cd9db011ef3178  powertop-2.13.tar.gz
+# Locally calculated
+sha256  e58ab3fd7b8ff5f4dd0d17f11848817e7d83c0a6918145ac81de03b5dccf8f49  powertop-2.15.tar.gz
 sha256  b499eddebda05a8859e32b820a64577d91f1de2b52efa2a1575a2cb4000bc259  COPYING
diff --git a/package/powertop/powertop.mk b/package/powertop/powertop.mk
index 681f5fda46..41403b0918 100644
--- a/package/powertop/powertop.mk
+++ b/package/powertop/powertop.mk
@@ -4,16 +4,24 @@
 #
 ################################################################################
 
-POWERTOP_VERSION = 2.13
-POWERTOP_SITE = https://01.org/sites/default/files/downloads
-POWERTOP_DEPENDENCIES = ncurses libnl host-pkgconf \
+POWERTOP_VERSION = 2.15
+POWERTOP_SITE = $(call github,fenrus75,powertop,v$(POWERTOP_VERSION))
+POWERTOP_DEPENDENCIES = host-autoconf-archive ncurses libnl host-pkgconf \
 	$(if $(BR2_PACKAGE_PCIUTILS),pciutils) \
 	$(TARGET_NLS_DEPENDENCIES)
 POWERTOP_LICENSE = GPL-2.0
 POWERTOP_LICENSE_FILES = COPYING
 POWERTOP_CONF_ENV = LIBS=$(TARGET_NLS_LIBS)
-# 0001-dont-force-stack-smashing-protection.patch
+POWERTOP_CONF_OPTS = --disable-stack-protector
+# 0001-add-disable-stack-protector-option.patch
 POWERTOP_AUTORECONF = YES
+POWERTOP_AUTORECONF_OPTS = --include=$(HOST_DIR)/share/autoconf-archive
+
+# fix missing config.rpath (needed for autoreconf) in the codebase
+define POWERTOP_TOUCH_CONFIG_RPATH
+	touch $(@D)/config.rpath
+endef
+POWERTOP_PRE_CONFIGURE_HOOKS += POWERTOP_TOUCH_CONFIG_RPATH
 
 # Help powertop at finding the right ncurses library depending on
 # which one is available.
-- 
2.40.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [Buildroot] [PATCH 1/2] package/powertop: picutils is optional, not mandatory
  2023-09-29 16:12 [Buildroot] [PATCH 1/2] package/powertop: picutils is optional, not mandatory Fabrice Fontaine
  2023-09-29 16:12 ` [Buildroot] [PATCH 2/2] package/powertop: bump to version 2.15 Fabrice Fontaine
@ 2023-09-30 17:19 ` Yann E. MORIN
  2023-10-13  6:21 ` Peter Korsgaard
  2 siblings, 0 replies; 5+ messages in thread
From: Yann E. MORIN @ 2023-09-30 17:19 UTC (permalink / raw)
  To: Fabrice Fontaine; +Cc: Steven Noonan, buildroot

Fabrice, All,

On 2023-09-29 18:12 +0200, Fabrice Fontaine spake thusly:
> pciutils is optional, not mandatory since version 2.1 and the addition
> of the package in commit 0adc0e24eed72efca0097c7567319f07b8af6d94 and
> https://github.com/fenrus75/powertop/commit/98ca605e7cf856795facc4a3a4f9c5b9baf1cab1
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Applied to master, after slightly reorganinsing the _DEPENDENCIES,
thanks.

Regards,
Yann E. MORIN.

> ---
>  package/powertop/Config.in   | 1 -
>  package/powertop/powertop.mk | 3 ++-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/package/powertop/Config.in b/package/powertop/Config.in
> index 0f323a7bfb..fd88a3d0eb 100644
> --- a/package/powertop/Config.in
> +++ b/package/powertop/Config.in
> @@ -4,7 +4,6 @@ config BR2_PACKAGE_POWERTOP
>  	depends on BR2_TOOLCHAIN_HAS_THREADS # libnl
>  	depends on BR2_USE_WCHAR
>  	select BR2_PACKAGE_NCURSES
> -	select BR2_PACKAGE_PCIUTILS
>  	select BR2_PACKAGE_LIBNL
>  	help
>  	  A tool to diagnose issues with power consumption and power
> diff --git a/package/powertop/powertop.mk b/package/powertop/powertop.mk
> index c3e8d2549c..681f5fda46 100644
> --- a/package/powertop/powertop.mk
> +++ b/package/powertop/powertop.mk
> @@ -6,7 +6,8 @@
>  
>  POWERTOP_VERSION = 2.13
>  POWERTOP_SITE = https://01.org/sites/default/files/downloads
> -POWERTOP_DEPENDENCIES = pciutils ncurses libnl host-pkgconf \
> +POWERTOP_DEPENDENCIES = ncurses libnl host-pkgconf \
> +	$(if $(BR2_PACKAGE_PCIUTILS),pciutils) \
>  	$(TARGET_NLS_DEPENDENCIES)
>  POWERTOP_LICENSE = GPL-2.0
>  POWERTOP_LICENSE_FILES = COPYING
> -- 
> 2.40.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Buildroot] [PATCH 2/2] package/powertop: bump to version 2.15
  2023-09-29 16:12 ` [Buildroot] [PATCH 2/2] package/powertop: bump to version 2.15 Fabrice Fontaine
@ 2023-09-30 17:19   ` Yann E. MORIN
  0 siblings, 0 replies; 5+ messages in thread
From: Yann E. MORIN @ 2023-09-30 17:19 UTC (permalink / raw)
  To: Fabrice Fontaine; +Cc: Steven Noonan, buildroot

Fabrice, All,

On 2023-09-29 18:12 +0200, Fabrice Fontaine spake thusly:
> - Switch site to get latest version
> - Replace patch by an upstreamable one
> 
> https://github.com/fenrus75/powertop/compare/v2.13...v2.15
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Applied to master, thanks.

Regards,
Yann E. MORIN.

> ---
>  .checkpackageignore                           |  1 -
>  ...1-add-disable-stack-protector-option.patch | 55 +++++++++++++++++++
>  ...dont-force-stack-smashing-protection.patch | 18 ------
>  package/powertop/Config.in                    |  2 +-
>  package/powertop/powertop.hash                |  6 +-
>  package/powertop/powertop.mk                  | 16 ++++--
>  6 files changed, 70 insertions(+), 28 deletions(-)
>  create mode 100644 package/powertop/0001-add-disable-stack-protector-option.patch
>  delete mode 100644 package/powertop/0001-dont-force-stack-smashing-protection.patch
> 
> diff --git a/.checkpackageignore b/.checkpackageignore
> index 0c7fae9409..d6873fb5a3 100644
> --- a/.checkpackageignore
> +++ b/.checkpackageignore
> @@ -1110,7 +1110,6 @@ package/poke/0002-lib-getrandom.c-fix-build-with-uclibc-1.0.35.patch Upstream
>  package/policycoreutils/0001-Add-DESTDIR-to-all-paths-that-use-an-absolute-path.patch Upstream
>  package/policycoreutils/0002-Add-PREFIX-to-host-paths.patch Upstream
>  package/postgresql/S50postgresql Variables
> -package/powertop/0001-dont-force-stack-smashing-protection.patch Upstream
>  package/pppd/0001-pppd-Fix-compilation-with-older-glibc-or-kernel-headers.patch Upstream
>  package/pppd/0002-pppd-eap-tls.c-fix-build-with-libressl.patch Upstream
>  package/pptp-linux/0001-susv3-legacy.patch Upstream
> diff --git a/package/powertop/0001-add-disable-stack-protector-option.patch b/package/powertop/0001-add-disable-stack-protector-option.patch
> new file mode 100644
> index 0000000000..01a030522d
> --- /dev/null
> +++ b/package/powertop/0001-add-disable-stack-protector-option.patch
> @@ -0,0 +1,55 @@
> +From 95382246ddd889839633aa0da800a03936b93986 Mon Sep 17 00:00:00 2001
> +From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> +Date: Sun, 31 Oct 2021 18:26:01 +0100
> +Subject: [PATCH] add --disable-stack-protector option
> +
> +Allow the user to disable stack-protector as not all toolchains support
> +this feature
> +
> +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> +Upstream: https://github.com/fenrus75/powertop/pull/138
> +---
> + configure.ac    | 4 ++++
> + src/Makefile.am | 5 ++++-
> + 2 files changed, 8 insertions(+), 1 deletion(-)
> +
> +diff --git a/configure.ac b/configure.ac
> +index 37c1304..69160d5 100644
> +--- a/configure.ac
> ++++ b/configure.ac
> +@@ -43,6 +43,10 @@ AX_ADD_FORTIFY_SOURCE
> + AX_CXX_COMPILE_STDCXX([11], [noext], [mandatory])
> + PKG_PROG_PKG_CONFIG
> + 
> ++AC_ARG_ENABLE([stack-protector],
> ++	AS_HELP_STRING([--disable-stack-protector], [Disable stack-protector]))
> ++AM_CONDITIONAL([ENABLE_STACK_PROTECTOR], [test x$enable_stack_protector != xno])
> ++
> + # Checks for libraries.
> + AX_PTHREAD([
> +     LIBS="$PTHREAD_LIBS $LIBS"
> +diff --git a/src/Makefile.am b/src/Makefile.am
> +index 6b523f6..ca30d20 100644
> +--- a/src/Makefile.am
> ++++ b/src/Makefile.am
> +@@ -138,13 +138,16 @@ powertop_CXXFLAGS = \
> + 	-Wformat \
> + 	-Wshadow \
> + 	-fno-omit-frame-pointer \
> +-	-fstack-protector \
> + 	$(GLIB2_CFLAGS) \
> + 	$(LIBNL_CFLAGS) \
> + 	$(NCURSES_CFLAGS) \
> + 	$(PCIUTILS_CFLAGS) \
> + 	$(PTHREAD_CFLAGS)
> + 
> ++if ENABLE_STACK_PROTECTOR
> ++powertop_CXXFLAGS += \
> ++	-fstack-protector
> ++endif
> + 
> + powertop_CPPFLAGS = \
> + 	-DLOCALEDIR=\"$(localedir)\" \
> +-- 
> +2.33.0
> +
> diff --git a/package/powertop/0001-dont-force-stack-smashing-protection.patch b/package/powertop/0001-dont-force-stack-smashing-protection.patch
> deleted file mode 100644
> index 75ed2d1c24..0000000000
> --- a/package/powertop/0001-dont-force-stack-smashing-protection.patch
> +++ /dev/null
> @@ -1,18 +0,0 @@
> -Do not pass -fstack-protector unconditionally
> -
> -Using -fstack-protector only works when the toolchain has SSP support.
> -
> -Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> -
> -Index: b/src/Makefile.am
> -===================================================================
> ---- a/src/Makefile.am
> -+++ b/src/Makefile.am
> -@@ -127,7 +127,6 @@
> - 	-Wformat \
> - 	-Wshadow \
> - 	-fno-omit-frame-pointer \
> --	-fstack-protector \
> - 	$(GLIB2_CFLAGS) \
> - 	$(LIBNL_CFLAGS) \
> - 	$(NCURSES_CFLAGS) \
> diff --git a/package/powertop/Config.in b/package/powertop/Config.in
> index fd88a3d0eb..2f7ba2bb9d 100644
> --- a/package/powertop/Config.in
> +++ b/package/powertop/Config.in
> @@ -9,7 +9,7 @@ config BR2_PACKAGE_POWERTOP
>  	  A tool to diagnose issues with power consumption and power
>  	  management
>  
> -	  https://01.org/powertop/
> +	  https://github.com/fenrus75/powertop/
>  
>  comment "powertop needs a toolchain w/ C++, threads, wchar"
>  	depends on !BR2_INSTALL_LIBSTDCPP || !BR2_TOOLCHAIN_HAS_THREADS || \
> diff --git a/package/powertop/powertop.hash b/package/powertop/powertop.hash
> index fc8f3cc52f..810b80a514 100644
> --- a/package/powertop/powertop.hash
> +++ b/package/powertop/powertop.hash
> @@ -1,5 +1,3 @@
> -# Locally calculated after checking pgp signature
> -# https://01.org/sites/default/files/downloads/powertop-2.13.tar.gz.asc.txt
> -# using key 22E8F306C8FA4BAA2A5F36F3A0303B060918941C
> -sha256  a65f992ca4a419bc73b623651060eb9fc00c5a86fa03556358cd9db011ef3178  powertop-2.13.tar.gz
> +# Locally calculated
> +sha256  e58ab3fd7b8ff5f4dd0d17f11848817e7d83c0a6918145ac81de03b5dccf8f49  powertop-2.15.tar.gz
>  sha256  b499eddebda05a8859e32b820a64577d91f1de2b52efa2a1575a2cb4000bc259  COPYING
> diff --git a/package/powertop/powertop.mk b/package/powertop/powertop.mk
> index 681f5fda46..41403b0918 100644
> --- a/package/powertop/powertop.mk
> +++ b/package/powertop/powertop.mk
> @@ -4,16 +4,24 @@
>  #
>  ################################################################################
>  
> -POWERTOP_VERSION = 2.13
> -POWERTOP_SITE = https://01.org/sites/default/files/downloads
> -POWERTOP_DEPENDENCIES = ncurses libnl host-pkgconf \
> +POWERTOP_VERSION = 2.15
> +POWERTOP_SITE = $(call github,fenrus75,powertop,v$(POWERTOP_VERSION))
> +POWERTOP_DEPENDENCIES = host-autoconf-archive ncurses libnl host-pkgconf \
>  	$(if $(BR2_PACKAGE_PCIUTILS),pciutils) \
>  	$(TARGET_NLS_DEPENDENCIES)
>  POWERTOP_LICENSE = GPL-2.0
>  POWERTOP_LICENSE_FILES = COPYING
>  POWERTOP_CONF_ENV = LIBS=$(TARGET_NLS_LIBS)
> -# 0001-dont-force-stack-smashing-protection.patch
> +POWERTOP_CONF_OPTS = --disable-stack-protector
> +# 0001-add-disable-stack-protector-option.patch
>  POWERTOP_AUTORECONF = YES
> +POWERTOP_AUTORECONF_OPTS = --include=$(HOST_DIR)/share/autoconf-archive
> +
> +# fix missing config.rpath (needed for autoreconf) in the codebase
> +define POWERTOP_TOUCH_CONFIG_RPATH
> +	touch $(@D)/config.rpath
> +endef
> +POWERTOP_PRE_CONFIGURE_HOOKS += POWERTOP_TOUCH_CONFIG_RPATH
>  
>  # Help powertop at finding the right ncurses library depending on
>  # which one is available.
> -- 
> 2.40.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Buildroot] [PATCH 1/2] package/powertop: picutils is optional, not mandatory
  2023-09-29 16:12 [Buildroot] [PATCH 1/2] package/powertop: picutils is optional, not mandatory Fabrice Fontaine
  2023-09-29 16:12 ` [Buildroot] [PATCH 2/2] package/powertop: bump to version 2.15 Fabrice Fontaine
  2023-09-30 17:19 ` [Buildroot] [PATCH 1/2] package/powertop: picutils is optional, not mandatory Yann E. MORIN
@ 2023-10-13  6:21 ` Peter Korsgaard
  2 siblings, 0 replies; 5+ messages in thread
From: Peter Korsgaard @ 2023-10-13  6:21 UTC (permalink / raw)
  To: Fabrice Fontaine; +Cc: Steven Noonan, buildroot

>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

 > pciutils is optional, not mandatory since version 2.1 and the addition
 > of the package in commit 0adc0e24eed72efca0097c7567319f07b8af6d94 and
 > https://github.com/fenrus75/powertop/commit/98ca605e7cf856795facc4a3a4f9c5b9baf1cab1

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Committed to 2023.02.x and 2023.08.x, thanks.

-- 
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2023-10-13  6:22 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-09-29 16:12 [Buildroot] [PATCH 1/2] package/powertop: picutils is optional, not mandatory Fabrice Fontaine
2023-09-29 16:12 ` [Buildroot] [PATCH 2/2] package/powertop: bump to version 2.15 Fabrice Fontaine
2023-09-30 17:19   ` Yann E. MORIN
2023-09-30 17:19 ` [Buildroot] [PATCH 1/2] package/powertop: picutils is optional, not mandatory Yann E. MORIN
2023-10-13  6:21 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox