Gentoo Archives: gentoo-dev

From: Mike Gilbert <floppym@g.o>
To: gentoo-dev@l.g.o
Cc: James Le Cuirot <chewi@g.o>
Subject: Re: [gentoo-dev] [PATCH 1/4] acct-group.eclass: Fix for when building in a rooted prefix (EROOT)
Date: Tue, 06 Dec 2022 23:54:55
Message-Id: CAJ0EP42CR_O8OK--PtpRjef+aO9ck6RWm5_R4Ab=dAm=pcoJfw@mail.gmail.com
In Reply to: [gentoo-dev] [PATCH 1/4] acct-group.eclass: Fix for when building in a rooted prefix (EROOT) by James Le Cuirot
1 On Tue, Dec 6, 2022 at 5:24 PM James Le Cuirot <chewi@g.o> wrote:
2 >
3 > Groups are largely irrelevant for prefix, but we still don't want the
4 > build to break.
5 >
6 > Signed-off-by: James Le Cuirot <chewi@g.o>
7 > ---
8 > eclass/acct-group.eclass | 2 +-
9 > 1 file changed, 1 insertion(+), 1 deletion(-)
10 >
11 > diff --git a/eclass/acct-group.eclass b/eclass/acct-group.eclass
12 > index 590a2f20ed8e..ada5fe386693 100644
13 > --- a/eclass/acct-group.eclass
14 > +++ b/eclass/acct-group.eclass
15 > @@ -176,7 +176,7 @@ acct-group_pkg_preinst() {
16 > fi
17 >
18 > if [[ -n ${ROOT} ]]; then
19
20 You should probably change this to [[ -n ${EROOT} ]]. Same goes for
21 acct-user.eclass.
22
23 Also see bug 779181; I'm not sure updating ${EROOT}/etc/group and
24 ${EROOT}/etc/passwd makes any sense at all.

Replies