Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 713680 - sys-apps/sed-4.8: use BDEPEND with EAPI 7
Summary: sys-apps/sed-4.8: use BDEPEND with EAPI 7
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-03-20 19:45 UTC by David Michael
Modified: 2020-03-21 20:16 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Michael 2020-03-20 19:45:32 UTC
The ebuild for 4.8 moved to EAPI 7 but doesn't use BDEPEND.  It looks like the host gettext is needed for NLS, so it can be taken out of DEPEND to reduce needless dependencies in a sysroot build.

Reproducible: Always




I tested this change:

--- sys-apps/sed/sed-4.8.ebuild
+++ sys-apps/sed/sed-4.8.ebuild
@@ -22,13 +22,13 @@
 	)
 "
 DEPEND="${RDEPEND}
-	nls? ( sys-devel/gettext )
 	static? (
 		acl? ( virtual/acl[static-libs(+)] )
 		nls? ( virtual/libintl[static-libs(+)] )
 		selinux? ( sys-libs/libselinux[static-libs(+)] )
 	)
 "
+BDEPEND="nls? ( sys-devel/gettext )"
 
 src_configure() {
 	local myconf=()
Comment 1 Larry the Git Cow gentoo-dev 2020-03-21 20:16:28 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=f70a46c3ede2106696249295b2f9463199abf9c3

commit f70a46c3ede2106696249295b2f9463199abf9c3
Author:     David Michael <fedora.dm0@gmail.com>
AuthorDate: 2020-03-21 20:14:18 +0000
Commit:     Lars Wendler <polynomial-c@gentoo.org>
CommitDate: 2020-03-21 20:16:22 +0000

    sys-apps/sed: Move sys-devel/gettext to BDEPEND
    
    Closes: https://bugs.gentoo.org/713680
    Package-Manager: Portage-2.3.94, Repoman-2.3.21
    Signed-off-by: Lars Wendler <polynomial-c@gentoo.org>

 sys-apps/sed/sed-4.8.ebuild | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)