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
 }
 

Reply via email to