Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 236905

Summary: dev-util/catalyst-2.0.6: argument "stage4/users" not recognized
Product: Gentoo Hosted Projects Reporter: Peter Abrahamsen <rainhead>
Component: CatalystAssignee: Gentoo Catalyst Developers <catalyst>
Status: RESOLVED FIXED    
Severity: minor CC: bkohler, majoron, proneet11
Priority: High    
Version: unspecified   
Hardware: AMD64   
OS: Linux   
URL: https://github.com/gentoo/catalyst/pull/8
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: enable 'stage4/users' feature
enable 'stage4/users' feature
bug fix

Description Peter Abrahamsen 2008-09-06 23:18:43 UTC
Hi,

I read about an spec file option "stage4/users" in a spec from kyron's SoC SSI project, but when I try to use it, I get:

!!! catalyst: Argument "stage4/users" not recognized.

My apologies if this isn't supposed to be a supported option, but if it's simple to make it work again, that'd be awesome.

Reproducible: Always
Comment 1 Andrew Gaffney (RETIRED) gentoo-dev 2008-09-15 03:10:07 UTC
Well, this option is in the example stage4 spec, and there's no reason it shouldn't be supported. However, the option isn't in the list of valid options in stage4, and the code to handle it is in livecdfs-udpate.sh.
Comment 2 Proneet Verma 2014-03-27 20:12:37 UTC
Created attachment 373660 [details, diff]
enable 'stage4/users' feature

This patch would enable 'stage4/users' feature while building stage4 archive.
Comment 3 Proneet Verma 2014-03-28 10:51:54 UTC
FYI: The patch I have attached before would work only with [3.0] branch. Not supported with version <3.0
Comment 4 Proneet Verma 2014-04-03 22:19:17 UTC
Created attachment 374208 [details, diff]
enable 'stage4/users' feature
Comment 5 David Palao 2019-10-28 10:32:37 UTC
Hello,
In the (stable) version on my system (dev-util/catalyst-3.0.5) the "stage4/users" is still not recognized:

ERROR   : CatalystError: Argument "stage4/users" not recognized.

Is there an important reason why the patch is not yet, after 5 years, in catalyst?

Best
Comment 6 Ben Kohler gentoo-dev 2019-10-28 14:08:13 UTC
I tried this patch (after a bit of work to port it to newest codebase) and it didn't work as-is.  So that's why I personally haven't done anything with it.

Can you test & confirm that it works for you?
Comment 7 David Palao 2019-10-29 11:48:27 UTC
Hi,
After looking at the code, I don't see why this patch should work in the current codebase...

I'll try to implement it.

What testing strategy is followed in the development of Catalyst? I don't find much info...
Comment 8 David Palao 2019-11-29 13:46:57 UTC
Created attachment 597844 [details, diff]
bug fix

At least it works for me. I think there must be 100 things to improve though.
Comment 9 David Palao 2019-12-02 07:52:46 UTC
There is at least one issue: the docs (man catalyst-spec, section miscellaneous) claims that <target>/users creates users belonging "to the following groups: users, wheel, audio, games, cdrom, and usb"

This patch does not do that because the group "games" does not exist.

Should I create the group "games" too, to comply with the docs, or should the man page be modified?
Comment 10 Larry the Git Cow gentoo-dev 2022-04-02 23:50:45 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/proj/catalyst.git/commit/?id=a6522008e915abbbf7cddc057c9f7951e7d667a6

commit a6522008e915abbbf7cddc057c9f7951e7d667a6
Author:     Patrice Clement <monsieurp@gentoo.org>
AuthorDate: 2022-03-08 21:11:13 +0000
Commit:     Matt Turner <mattst88@gentoo.org>
CommitDate: 2022-04-02 23:50:27 +0000

    example: document new options
    
    Closes: https://bugs.gentoo.org/236905
    Signed-off-by: Patrice Clement <monsieurp@gentoo.org>
    Signed-off-by: Matt Turner <mattst88@gentoo.org>

 examples/stage4_template.spec | 34 ++++++++++++++++++++++++++++------
 1 file changed, 28 insertions(+), 6 deletions(-)

Additionally, it has been referenced in the following commit(s):

https://gitweb.gentoo.org/proj/catalyst.git/commit/?id=5be6069bcbd5a7fa3f114f28366597bc5ddbb891

commit 5be6069bcbd5a7fa3f114f28366597bc5ddbb891
Author:     Patrice Clement <monsieurp@gentoo.org>
AuthorDate: 2022-03-08 21:02:55 +0000
Commit:     Matt Turner <mattst88@gentoo.org>
CommitDate: 2022-04-02 23:50:16 +0000

    catalyst: support 3 new options
    
    * stage4/groups: create a a list of groups.
    * stage4/users: create a list of users. users can also be added to
      groups using the "foo.bar=wheel,audio,baz" format.
    * stage4/ssh_public_keys: copy an SSH public key into the stage4 user's home
      (.ssh/authorized_keys) and set the file permission to 0644.
    
    Bug: https://bugs.gentoo.org/236905
    Signed-off-by: Patrice Clement <monsieurp@gentoo.org>
    Signed-off-by: Matt Turner <mattst88@gentoo.org>

 catalyst/base/stagebase.py | 70 ++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 70 insertions(+)