Gentoo Archives: gentoo-dev

From: David Seifert <soap@g.o>
To: gentoo-dev@l.g.o
Cc: base-system@g.o
Subject: Re: [gentoo-dev] [PATCH] fixheadtails.eclass: move sed to BDEPEND for EAPI 7
Date: Fri, 20 Mar 2020 20:35:21
Message-Id: 6a2b03bf6f8eb7ea06606b555331adfcfba57dfc.camel@gentoo.org
In Reply to: [gentoo-dev] [PATCH] fixheadtails.eclass: move sed to BDEPEND for EAPI 7 by David Michael
1 On Fri, 2020-03-20 at 14:19 -0400, David Michael wrote:
2 > It executes sed at build time, so it should be installed in /.
3 >
4 > Signed-off-by: David Michael <fedora.dm0@×××××.com>
5 > ---
6 >
7 > Hi,
8 >
9 > Here is another simple dependency move to put a required program in the
10 > correct ROOT so it can be executed during the build. It's basically the
11 > same as 814ab1294edf3565fc02fe63d15d6fa7ca886429.
12 >
13 > Thanks.
14 >
15 > David
16 >
17 > eclass/fixheadtails.eclass | 7 +++++--
18 > 1 file changed, 5 insertions(+), 2 deletions(-)
19 >
20 > diff --git a/eclass/fixheadtails.eclass b/eclass/fixheadtails.eclass
21 > index c19d33924aa..5d221ca678c 100644
22 > --- a/eclass/fixheadtails.eclass
23 > +++ b/eclass/fixheadtails.eclass
24 > @@ -1,4 +1,4 @@
25 > -# Copyright 1999-2014 Gentoo Foundation
26 > +# Copyright 1999-2020 Gentoo Authors
27 > # Distributed under the terms of the GNU General Public License v2
28 >
29 > # @ECLASS: fixheadtails.eclass
30 > @@ -8,7 +8,10 @@
31 > # Original author John Mylchreest <johnm@g.o>
32 > # @BLURB: functions to replace obsolete head/tail with POSIX compliant ones
33 >
34 > -DEPEND=">=sys-apps/sed-4"
35 > +case "${EAPI:-0}" in
36 > + [0-6]) DEPEND=">=sys-apps/sed-4" ;;
37 > + *) BDEPEND=">=sys-apps/sed-4" ;;
38 > +esac
39 >
40 > _do_sed_fix() {
41 > einfo " - fixed $1"
42
43 This is an ancient dependency, just kill it altogether.