Gentoo Archives: gentoo-dev

From: Alec Warner <antarus@g.o>
To: Gentoo Dev <gentoo-dev@l.g.o>
Subject: Re: [gentoo-dev] [PATCH] acct-user.eclass: Support var overrides for user properties
Date: Mon, 04 Jan 2021 17:36:05
Message-Id: CAAr7Pr8NQCvLve8q41LMVrVv6knfGCeERhRjwVvjcQ59w+cc0w@mail.gmail.com
In Reply to: Re: [gentoo-dev] [PATCH] acct-user.eclass: Support var overrides for user properties by Thomas Deutschmann
1 On Mon, Jan 4, 2021 at 9:15 AM Thomas Deutschmann <whissi@g.o> wrote:
2 >
3 > On 2021-01-04 18:08, Michał Górny wrote:
4 > > Introduce a few variables to allow easy overrides of common user account
5 > > proprerties, that is:
6 > >
7 > > - ACCT_USER_<username>_SHELL
8 > > - ACCT_USER_<username>_HOME
9 > > - ACCT_USER_<username>_HOME_OWNER
10 > > - ACCT_USER_<username>_HOME_PERMS
11 > > - ACCT_USER_<username>_GROUPS
12 > > - ACCT_USER_<username>_GROUPS_ADD
13 > >
14 > > The first five variables override the respective ACCT_USER_* variables,
15 > > with ACCT_USER_*_GROUPS being a space-separated list. The *_GROUPS_ADD
16 > > variable appends to groups present in the ebuild, as this seems a common
17 > > necessity.
18 > >
19 > > We do realize that the original requirement of overriding ebuilds
20 > > in a local repository was inconvenient. This new logic should permit
21 > > easy updates via make.conf. Additionally, it has the advantage
22 > > of clearly reporting the changes made in the build logs.
23 > >
24 > > This does not preclude other solutions to the problem. However, this
25 > > is probably the best one and it should become the current
26 > > recommendation.
27 >
28 > This will improve the overlay situation and can be seen as overall
29 > improvement but it doesn't address any shared concerns nor is it a
30 > replacement for the proposed 'acct-user.eclass: don't modify existing
31 > user by default' patch.
32 >
33
34 Same response from me, merge it but please also merge the other patch.
35
36 -A
37
38 >
39 > --
40 > Regards,
41 > Thomas Deutschmann / Gentoo Linux Developer
42 > fpr: C4DD 695F A713 8F24 2AA1 5638 5849 7EE5 1D5D 74A5
43 >