Gentoo Archives: gentoo-commits

From: Jay Faulkner <jay@×××.cc>
To: gentoo-commits@l.g.o
Subject: [gentoo-commits] repo/proj/guru:dev commit in: sys-power/tlp/
Date: Fri, 02 Sep 2022 02:11:25
Message-Id: 1662084564.4c77f8f888abd2553fc03a6c54d6c0a2a0424b69.jayf@gentoo
1 commit: 4c77f8f888abd2553fc03a6c54d6c0a2a0424b69
2 Author: Jay Faulkner <jay <AT> jvf <DOT> cc>
3 AuthorDate: Fri Sep 2 01:47:15 2022 +0000
4 Commit: Jay Faulkner <jay <AT> jvf <DOT> cc>
5 CommitDate: Fri Sep 2 02:09:24 2022 +0000
6 URL: https://gitweb.gentoo.org/repo/proj/guru.git/commit/?id=4c77f8f8
7
8 sys-power/tlp: Remove unneeded use flags
9
10 The systemd and elogind toggles in the tlp build only toggle
11 installation of unit file and sleep triggers. There's no need to
12 conditionally install those per PG#0301.
13
14 Signed-Off-By: Jay Faulkner <jay <AT> jvf.cc>
15
16 sys-power/tlp/tlp-1.5.0.ebuild | 15 ++++++---------
17 1 file changed, 6 insertions(+), 9 deletions(-)
18
19 diff --git a/sys-power/tlp/tlp-1.5.0.ebuild b/sys-power/tlp/tlp-1.5.0.ebuild
20 index 0fc6f0d01..bf18bbbb7 100644
21 --- a/sys-power/tlp/tlp-1.5.0.ebuild
22 +++ b/sys-power/tlp/tlp-1.5.0.ebuild
23 @@ -12,25 +12,22 @@ S="${WORKDIR}/TLP-${PV}"
24 LICENSE="GPL-2"
25 SLOT="0"
26 KEYWORDS="~amd64"
27 -IUSE="bash-completion elogind systemd"
28 RESTRICT="mirror"
29 +# It's uncertain if elogind/systemd is actually required, however, without the sleep
30 +# hooks working, which require one of them, it doesn't seem like this app is very useful.
31 RDEPEND="virtual/udev
32 - bash-completion? ( app-shells/bash app-shells/bash-completion )
33 - elogind? ( sys-auth/elogind )
34 - systemd? ( sys-apps/systemd )"
35 + || ( sys-auth/elogind sys-apps/systemd )"
36 DEPEND="${RDEPEND}"
37 -REQUIRED_USE="?? ( elogind systemd )"
38
39 src_install() {
40 emake \
41 DESTDIR="${D}" \
42 TLP_NO_INIT=1 \
43 - TLP_NO_BASHCOMP=$(usex bash-completion 0 1) \
44 - TLP_WITH_ELOGIND=$(usex elogind 1 0) \
45 - TLP_WITH_SYSTEMD=$(usex systemd 1 0) \
46 + TLP_WITH_ELOGIND=1 \
47 + TLP_WITH_SYSTEMD=1 \
48 install install-man
49
50 - chmod 444 "${D}/usr/share/tlp/defaults.conf" # manpage says this file should not be edited
51 + fperms 444 "/usr/share/tlp/defaults.conf" # manpage says this file should not be edited
52 newinitd "${FILESDIR}/tlp.init" tlp
53 keepdir "/var/lib/tlp" # created by Makefile, probably important
54 }