[gentoo-commits] repo/gentoo:master commit in: sys-apps/acl/

["David Seifert" <soap@gentoo.org>]

commit:     ed777fea32b3a9b73daa34055ea86a4a4b3ad72b
Author:     Sam James <sam <AT> gentoo <DOT> org>
AuthorDate: Thu May 13 13:54:52 2021 +0000
Commit:     David Seifert <soap <AT> gentoo <DOT> org>
CommitDate: Thu May 13 13:54:52 2021 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=ed777fea

sys-apps/acl: drop unused toolchain-funcs, move filter-flags

* Drop unused toolchain-funcs inherit

* Move filter-flags call from pkg_setup to src_configure (multilib)
  which is more conventional given it only takes effect during
  source builds (not binpkgs).

* Minor style changes to aid readability.

Signed-off-by: Sam James <sam <AT> gentoo.org>
Signed-off-by: David Seifert <soap <AT> gentoo.org>

 sys-apps/acl/acl-2.2.53-r1.ebuild | 23 ++++++++++++-----------
 sys-apps/acl/acl-2.3.0.ebuild     | 23 ++++++++++++-----------
 sys-apps/acl/acl-2.3.1.ebuild     | 23 ++++++++++++-----------
 3 files changed, 36 insertions(+), 33 deletions(-)

diff --git a/sys-apps/acl/acl-2.2.53-r1.ebuild b/sys-apps/acl/acl-2.2.53-r1.ebuild
index 1c5fd400559..5ab955ac836 100644
--- a/sys-apps/acl/acl-2.2.53-r1.ebuild
+++ b/sys-apps/acl/acl-2.2.53-r1.ebuild
@@ -3,9 +3,9 @@
 
 EAPI=7
 
-inherit flag-o-matic libtool toolchain-funcs multilib-minimal usr-ldscript
+inherit flag-o-matic libtool multilib-minimal usr-ldscript
 
-DESCRIPTION="access control list utilities, libraries and headers"
+DESCRIPTION="Access control list utilities, libraries, and headers"
 HOMEPAGE="https://savannah.nongnu.org/projects/acl"
 SRC_URI="mirror://nongnu/${PN}/${P}.tar.gz"
 
@@ -20,37 +20,38 @@ RDEPEND="
 DEPEND="${RDEPEND}"
 BDEPEND="nls? ( sys-devel/gettext )"
 
-pkg_setup() {
-	# filter out -flto flags as they break getfacl/setfacl binaries
-	# (bug #667372)
-	filter-flags -flto*
-}
-
 src_prepare() {
 	default
-	elibtoolize #580792
+
+	# bug #580792
+	elibtoolize
 }
 
 multilib_src_configure() {
+	# Filter out -flto flags as they break getfacl/setfacl binaries
+	# bug #667372
+	filter-flags -flto*
+
 	local myeconfargs=(
 		--bindir="${EPREFIX}"/bin
 		$(use_enable static-libs static)
 		--libexecdir="${EPREFIX}"/usr/$(get_libdir)
 		$(use_enable nls)
 	)
+
 	ECONF_SOURCE="${S}" econf "${myeconfargs[@]}"
 }
 
 multilib_src_test() {
 	# Tests call native binaries with an LD_PRELOAD wrapper
-	# https://bugs.gentoo.org/772356
+	# bug #772356
 	multilib_is_native_abi && default
 }
 
 multilib_src_install() {
 	default
 
-	# move shared libs to /
+	# Move shared libs to /
 	gen_usr_ldscript -a acl
 }
 

diff --git a/sys-apps/acl/acl-2.3.0.ebuild b/sys-apps/acl/acl-2.3.0.ebuild
index b802802db8a..7ffef6cfc23 100644
--- a/sys-apps/acl/acl-2.3.0.ebuild
+++ b/sys-apps/acl/acl-2.3.0.ebuild
@@ -3,9 +3,9 @@
 
 EAPI=7
 
-inherit flag-o-matic libtool toolchain-funcs multilib-minimal usr-ldscript
+inherit flag-o-matic libtool multilib-minimal usr-ldscript
 
-DESCRIPTION="access control list utilities, libraries and headers"
+DESCRIPTION="Access control list utilities, libraries, and headers"
 HOMEPAGE="https://savannah.nongnu.org/projects/acl"
 SRC_URI="mirror://nongnu/${PN}/${P}.tar.gz"
 
@@ -20,37 +20,38 @@ RDEPEND="
 DEPEND="${RDEPEND}"
 BDEPEND="nls? ( sys-devel/gettext )"
 
-pkg_setup() {
-	# filter out -flto flags as they break getfacl/setfacl binaries
-	# (bug #667372)
-	filter-flags -flto*
-}
-
 src_prepare() {
 	default
-	elibtoolize #580792
+
+	# bug #580792
+	elibtoolize
 }
 
 multilib_src_configure() {
+	# Filter out -flto flags as they break getfacl/setfacl binaries
+	# bug #667372
+	filter-flags -flto*
+
 	local myeconfargs=(
 		--bindir="${EPREFIX}"/bin
 		$(use_enable static-libs static)
 		--libexecdir="${EPREFIX}"/usr/$(get_libdir)
 		$(use_enable nls)
 	)
+
 	ECONF_SOURCE="${S}" econf "${myeconfargs[@]}"
 }
 
 multilib_src_test() {
 	# Tests call native binaries with an LD_PRELOAD wrapper
-	# https://bugs.gentoo.org/772356
+	# bug #772356
 	multilib_is_native_abi && default
 }
 
 multilib_src_install() {
 	default
 
-	# move shared libs to /
+	# Move shared libs to /
 	gen_usr_ldscript -a acl
 }
 

diff --git a/sys-apps/acl/acl-2.3.1.ebuild b/sys-apps/acl/acl-2.3.1.ebuild
index b802802db8a..7ffef6cfc23 100644
--- a/sys-apps/acl/acl-2.3.1.ebuild
+++ b/sys-apps/acl/acl-2.3.1.ebuild
@@ -3,9 +3,9 @@
 
 EAPI=7
 
-inherit flag-o-matic libtool toolchain-funcs multilib-minimal usr-ldscript
+inherit flag-o-matic libtool multilib-minimal usr-ldscript
 
-DESCRIPTION="access control list utilities, libraries and headers"
+DESCRIPTION="Access control list utilities, libraries, and headers"
 HOMEPAGE="https://savannah.nongnu.org/projects/acl"
 SRC_URI="mirror://nongnu/${PN}/${P}.tar.gz"
 
@@ -20,37 +20,38 @@ RDEPEND="
 DEPEND="${RDEPEND}"
 BDEPEND="nls? ( sys-devel/gettext )"
 
-pkg_setup() {
-	# filter out -flto flags as they break getfacl/setfacl binaries
-	# (bug #667372)
-	filter-flags -flto*
-}
-
 src_prepare() {
 	default
-	elibtoolize #580792
+
+	# bug #580792
+	elibtoolize
 }
 
 multilib_src_configure() {
+	# Filter out -flto flags as they break getfacl/setfacl binaries
+	# bug #667372
+	filter-flags -flto*
+
 	local myeconfargs=(
 		--bindir="${EPREFIX}"/bin
 		$(use_enable static-libs static)
 		--libexecdir="${EPREFIX}"/usr/$(get_libdir)
 		$(use_enable nls)
 	)
+
 	ECONF_SOURCE="${S}" econf "${myeconfargs[@]}"
 }
 
 multilib_src_test() {
 	# Tests call native binaries with an LD_PRELOAD wrapper
-	# https://bugs.gentoo.org/772356
+	# bug #772356
 	multilib_is_native_abi && default
 }
 
 multilib_src_install() {
 	default
 
-	# move shared libs to /
+	# Move shared libs to /
 	gen_usr_ldscript -a acl
 }