* [RFC PATCH] Drop powertop
@ 2022-11-07 12:24 Peter Müller
2022-11-07 16:44 ` Michael Tremer
0 siblings, 1 reply; 2+ messages in thread
From: Peter Müller @ 2022-11-07 12:24 UTC (permalink / raw)
To: development
[-- Attachment #1: Type: text/plain, Size: 6025 bytes --]
This has been discussed briefly in the telephone conference of
September: powertop is considered to be unnecessary, as IPFire is
optimized for performance, thus interfering with possible power
consumption reducing switches. Also, the need for powertop has been
diminished, given that x86 platforms are highly likely not to run on
batteries, and we are phasing out 32-bit ARM, where this could have been
the case.
Therefore, this patch proposes to drop powertop.
Signed-off-by: Peter Müller <peter.mueller(a)ipfire.org>
---
config/rootfiles/packages/powertop | 23 --------
lfs/powertop | 90 ------------------------------
make.sh | 1 -
3 files changed, 114 deletions(-)
delete mode 100644 config/rootfiles/packages/powertop
delete mode 100644 lfs/powertop
diff --git a/config/rootfiles/packages/powertop b/config/rootfiles/packages/powertop
deleted file mode 100644
index 3ec61dd8e..000000000
--- a/config/rootfiles/packages/powertop
+++ /dev/null
@@ -1,23 +0,0 @@
-usr/sbin/powertop
-#usr/share/bash-completion/completions/powertop
-#usr/share/locale/ca/LC_MESSAGES/powertop.mo
-#usr/share/locale/cs_CZ/LC_MESSAGES/powertop.mo
-#usr/share/locale/de_DE
-#usr/share/locale/de_DE/LC_MESSAGES
-#usr/share/locale/de_DE/LC_MESSAGES/powertop.mo
-#usr/share/locale/en_GB/LC_MESSAGES/powertop.mo
-#usr/share/locale/en_US/LC_MESSAGES/powertop.mo
-#usr/share/locale/es_ES
-#usr/share/locale/es_ES/LC_MESSAGES
-#usr/share/locale/es_ES/LC_MESSAGES/powertop.mo
-#usr/share/locale/hu_HU
-#usr/share/locale/hu_HU/LC_MESSAGES
-#usr/share/locale/hu_HU/LC_MESSAGES/powertop.mo
-#usr/share/locale/id_ID
-#usr/share/locale/id_ID/LC_MESSAGES
-#usr/share/locale/id_ID/LC_MESSAGES/powertop.mo
-#usr/share/locale/nl_NL
-#usr/share/locale/nl_NL/LC_MESSAGES
-#usr/share/locale/nl_NL/LC_MESSAGES/powertop.mo
-#usr/share/locale/zh_TW/LC_MESSAGES/powertop.mo
-#usr/share/man/man8/powertop.8
diff --git a/lfs/powertop b/lfs/powertop
deleted file mode 100644
index c909a73bf..000000000
--- a/lfs/powertop
+++ /dev/null
@@ -1,90 +0,0 @@
-###############################################################################
-# #
-# IPFire.org - A linux based firewall #
-# Copyright (C) 2007-2018 IPFire Team <info(a)ipfire.org> #
-# #
-# This program is free software: you can redistribute it and/or modify #
-# it under the terms of the GNU General Public License as published by #
-# the Free Software Foundation, either version 3 of the License, or #
-# (at your option) any later version. #
-# #
-# This program is distributed in the hope that it will be useful, #
-# but WITHOUT ANY WARRANTY; without even the implied warranty of #
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the #
-# GNU General Public License for more details. #
-# #
-# You should have received a copy of the GNU General Public License #
-# along with this program. If not, see <http://www.gnu.org/licenses/>. #
-# #
-###############################################################################
-
-###############################################################################
-# Definitions
-###############################################################################
-
-include Config
-
-SUMMARY = A Linux Tool to Find out What is Using Power on a Laptop
-
-VER = 2.14
-
-THISAPP = powertop-$(VER)
-DL_FILE = $(THISAPP).tar.gz
-DL_FROM = $(URL_IPFIRE)
-DIR_APP = $(DIR_SRC)/$(THISAPP)
-TARGET = $(DIR_INFO)/$(THISAPP)
-PROG = powertop
-PAK_VER = 4
-
-DEPS =
-
-SERVICES =
-
-###############################################################################
-# Top-level Rules
-###############################################################################
-
-objects = $(DL_FILE)
-
-$(DL_FILE) = $(DL_FROM)/$(DL_FILE)
-
-$(DL_FILE)_BLAKE2 = 5f53f81b01e83004f6436f497818d2a552b5e7ffda5007749aad415773adeab2a256aeffbacbf9e5fe803ff343a0c1954b5264aebdffa841bb70406b8b9611bc
-
-install : $(TARGET)
-
-check : $(patsubst %,$(DIR_CHK)/%,$(objects))
-
-download :$(patsubst %,$(DIR_DL)/%,$(objects))
-
-b2 : $(subst %,%_BLAKE2,$(objects))
-
-dist:
- $(PAK)
-
-###############################################################################
-# Downloading, checking, b2sum
-###############################################################################
-
-$(patsubst %,$(DIR_CHK)/%,$(objects)) :
- @$(CHECK)
-
-$(patsubst %,$(DIR_DL)/%,$(objects)) :
- @$(LOAD)
-
-$(subst %,%_BLAKE2,$(objects)) :
- @$(B2SUM)
-
-###############################################################################
-# Installation Details
-###############################################################################
-
-$(TARGET) : $(patsubst %,$(DIR_DL)/%,$(objects))
- @$(PREBUILD)
- @rm -rf $(DIR_APP) && cd $(DIR_SRC) && tar zxf $(DIR_DL)/$(DL_FILE)
- $(UPDATE_AUTOMAKE)
- cd $(DIR_APP) && ./autogen.sh
- cd $(DIR_APP) && ./configure --prefix=/usr
- cd $(DIR_APP) && make $(MAKETUNING)
- cd $(DIR_APP) && make install
- @rm -rf $(DIR_APP)
- @$(POSTBUILD)
diff --git a/make.sh b/make.sh
index 58220d046..ce88d9c19 100755
--- a/make.sh
+++ b/make.sh
@@ -1592,7 +1592,6 @@ buildipfire() {
lfsmake2 zerofree
lfsmake2 minicom
lfsmake2 ddrescue
- lfsmake2 powertop
lfsmake2 parted
lfsmake2 swig
lfsmake2 dtc
--
2.35.3
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [RFC PATCH] Drop powertop
2022-11-07 12:24 [RFC PATCH] Drop powertop Peter Müller
@ 2022-11-07 16:44 ` Michael Tremer
0 siblings, 0 replies; 2+ messages in thread
From: Michael Tremer @ 2022-11-07 16:44 UTC (permalink / raw)
To: development
[-- Attachment #1: Type: text/plain, Size: 6876 bytes --]
Hello Peter,
I am not quite sure what to respond to this patch.
I don’t have any reasons to keep powertop for myself. I didn’t use it in many years.
However, it might be helpful to debug some hardware issues. But do people do that these days?
In order to keep the distribution small and so on, let’s drop it. If someone wants it back, I wouldn’t mind having it back.
-Michael
Reviewed-by: Michael Tremer <michael.tremer(a)ipfire.org>
> On 7 Nov 2022, at 12:24, Peter Müller <peter.mueller(a)ipfire.org> wrote:
>
> This has been discussed briefly in the telephone conference of
> September: powertop is considered to be unnecessary, as IPFire is
> optimized for performance, thus interfering with possible power
> consumption reducing switches. Also, the need for powertop has been
> diminished, given that x86 platforms are highly likely not to run on
> batteries, and we are phasing out 32-bit ARM, where this could have been
> the case.
>
> Therefore, this patch proposes to drop powertop.
>
> Signed-off-by: Peter Müller <peter.mueller(a)ipfire.org>
> ---
> config/rootfiles/packages/powertop | 23 --------
> lfs/powertop | 90 ------------------------------
> make.sh | 1 -
> 3 files changed, 114 deletions(-)
> delete mode 100644 config/rootfiles/packages/powertop
> delete mode 100644 lfs/powertop
>
> diff --git a/config/rootfiles/packages/powertop b/config/rootfiles/packages/powertop
> deleted file mode 100644
> index 3ec61dd8e..000000000
> --- a/config/rootfiles/packages/powertop
> +++ /dev/null
> @@ -1,23 +0,0 @@
> -usr/sbin/powertop
> -#usr/share/bash-completion/completions/powertop
> -#usr/share/locale/ca/LC_MESSAGES/powertop.mo
> -#usr/share/locale/cs_CZ/LC_MESSAGES/powertop.mo
> -#usr/share/locale/de_DE
> -#usr/share/locale/de_DE/LC_MESSAGES
> -#usr/share/locale/de_DE/LC_MESSAGES/powertop.mo
> -#usr/share/locale/en_GB/LC_MESSAGES/powertop.mo
> -#usr/share/locale/en_US/LC_MESSAGES/powertop.mo
> -#usr/share/locale/es_ES
> -#usr/share/locale/es_ES/LC_MESSAGES
> -#usr/share/locale/es_ES/LC_MESSAGES/powertop.mo
> -#usr/share/locale/hu_HU
> -#usr/share/locale/hu_HU/LC_MESSAGES
> -#usr/share/locale/hu_HU/LC_MESSAGES/powertop.mo
> -#usr/share/locale/id_ID
> -#usr/share/locale/id_ID/LC_MESSAGES
> -#usr/share/locale/id_ID/LC_MESSAGES/powertop.mo
> -#usr/share/locale/nl_NL
> -#usr/share/locale/nl_NL/LC_MESSAGES
> -#usr/share/locale/nl_NL/LC_MESSAGES/powertop.mo
> -#usr/share/locale/zh_TW/LC_MESSAGES/powertop.mo
> -#usr/share/man/man8/powertop.8
> diff --git a/lfs/powertop b/lfs/powertop
> deleted file mode 100644
> index c909a73bf..000000000
> --- a/lfs/powertop
> +++ /dev/null
> @@ -1,90 +0,0 @@
> -###############################################################################
> -# #
> -# IPFire.org - A linux based firewall #
> -# Copyright (C) 2007-2018 IPFire Team <info(a)ipfire.org> #
> -# #
> -# This program is free software: you can redistribute it and/or modify #
> -# it under the terms of the GNU General Public License as published by #
> -# the Free Software Foundation, either version 3 of the License, or #
> -# (at your option) any later version. #
> -# #
> -# This program is distributed in the hope that it will be useful, #
> -# but WITHOUT ANY WARRANTY; without even the implied warranty of #
> -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the #
> -# GNU General Public License for more details. #
> -# #
> -# You should have received a copy of the GNU General Public License #
> -# along with this program. If not, see <http://www.gnu.org/licenses/>. #
> -# #
> -###############################################################################
> -
> -###############################################################################
> -# Definitions
> -###############################################################################
> -
> -include Config
> -
> -SUMMARY = A Linux Tool to Find out What is Using Power on a Laptop
> -
> -VER = 2.14
> -
> -THISAPP = powertop-$(VER)
> -DL_FILE = $(THISAPP).tar.gz
> -DL_FROM = $(URL_IPFIRE)
> -DIR_APP = $(DIR_SRC)/$(THISAPP)
> -TARGET = $(DIR_INFO)/$(THISAPP)
> -PROG = powertop
> -PAK_VER = 4
> -
> -DEPS =
> -
> -SERVICES =
> -
> -###############################################################################
> -# Top-level Rules
> -###############################################################################
> -
> -objects = $(DL_FILE)
> -
> -$(DL_FILE) = $(DL_FROM)/$(DL_FILE)
> -
> -$(DL_FILE)_BLAKE2 = 5f53f81b01e83004f6436f497818d2a552b5e7ffda5007749aad415773adeab2a256aeffbacbf9e5fe803ff343a0c1954b5264aebdffa841bb70406b8b9611bc
> -
> -install : $(TARGET)
> -
> -check : $(patsubst %,$(DIR_CHK)/%,$(objects))
> -
> -download :$(patsubst %,$(DIR_DL)/%,$(objects))
> -
> -b2 : $(subst %,%_BLAKE2,$(objects))
> -
> -dist:
> - $(PAK)
> -
> -###############################################################################
> -# Downloading, checking, b2sum
> -###############################################################################
> -
> -$(patsubst %,$(DIR_CHK)/%,$(objects)) :
> - @$(CHECK)
> -
> -$(patsubst %,$(DIR_DL)/%,$(objects)) :
> - @$(LOAD)
> -
> -$(subst %,%_BLAKE2,$(objects)) :
> - @$(B2SUM)
> -
> -###############################################################################
> -# Installation Details
> -###############################################################################
> -
> -$(TARGET) : $(patsubst %,$(DIR_DL)/%,$(objects))
> - @$(PREBUILD)
> - @rm -rf $(DIR_APP) && cd $(DIR_SRC) && tar zxf $(DIR_DL)/$(DL_FILE)
> - $(UPDATE_AUTOMAKE)
> - cd $(DIR_APP) && ./autogen.sh
> - cd $(DIR_APP) && ./configure --prefix=/usr
> - cd $(DIR_APP) && make $(MAKETUNING)
> - cd $(DIR_APP) && make install
> - @rm -rf $(DIR_APP)
> - @$(POSTBUILD)
> diff --git a/make.sh b/make.sh
> index 58220d046..ce88d9c19 100755
> --- a/make.sh
> +++ b/make.sh
> @@ -1592,7 +1592,6 @@ buildipfire() {
> lfsmake2 zerofree
> lfsmake2 minicom
> lfsmake2 ddrescue
> - lfsmake2 powertop
> lfsmake2 parted
> lfsmake2 swig
> lfsmake2 dtc
> --
> 2.35.3
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-11-07 16:44 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-07 12:24 [RFC PATCH] Drop powertop Peter Müller
2022-11-07 16:44 ` Michael Tremer
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox