Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support for 'm' in sysusers file? #2816

Closed
mlschroe opened this issue Dec 14, 2023 · 3 comments
Closed

Support for 'm' in sysusers file? #2816

mlschroe opened this issue Dec 14, 2023 · 3 comments
Assignees
Labels
Milestone

Comments

@mlschroe
Copy link
Collaborator

Does it make sense to add support for 'm' lines in sysusers config files? This would mean the following changes:

  • add a provides for every m line, e.g. "group-member(groupname) = base64_line"
  • feed all such provides to systemd_sysusers tool (after all the user/group lines)

The sysusers.sh scripts already has support for 'm' lines.

@mlschroe
Copy link
Collaborator Author

Should the m lines generate user()/group() requires?

@pmatilai
Copy link
Member

Supporting 'm' would be nice of course. It was left out of the initial implementation due to lack of time/energy to think about how to properly implement it, rather than a "never" decision.

The implicit user/group creation is indeed something that would cause problems when an explicit entry exists, so I think adding hard requires for both the user and the group is needed. Alternatively we'd need some special logic for dependency resolution of these items, where we'd favor a user/group with an EVR over one without it.

@pmatilai pmatilai added the RFE label Jan 17, 2024
@pmatilai pmatilai added this to RPM Jan 17, 2024
@pmatilai pmatilai moved this from Backlog to Todo in RPM Jan 17, 2024
@github-project-automation github-project-automation bot moved this to Backlog in RPM Jan 17, 2024
@pmatilai pmatilai removed their assignment Jan 22, 2024
@pmatilai pmatilai added this to the 4.20.0 milestone Jan 22, 2024
@pmatilai pmatilai moved this from Todo to Priority in RPM Feb 21, 2024
@ffesti ffesti self-assigned this Feb 21, 2024
@ffesti ffesti moved this from Priority to In Progress in RPM Feb 21, 2024
@pmatilai
Copy link
Member

On a related note, we should probably also generate a group require on a case like this:
u cyrus 76:mail "Cyrus IMAP Server" /var/lib/imap /sbin/nologin

ffesti added a commit to ffesti/rpm that referenced this issue Mar 25, 2024
m user group

Create Requires/Recommends for both the user and the group.

Resolves: rpm-software-management#2816
@ffesti ffesti moved this from In Progress to In Review in RPM Mar 25, 2024
ffesti added a commit to ffesti/rpm that referenced this issue Mar 26, 2024
m user group

Create Requires/Recommends for both the user and the group.

Resolves: rpm-software-management#2816
ffesti added a commit to ffesti/rpm that referenced this issue Apr 3, 2024
m user group

Add Provides with groupmember(user/group) and Requires/Recommends for both the
user and the group.

Extend RPMTAG_SYSUSERS and recognize the groupmember provides during
installation.

Resolves: rpm-software-management#2816
ffesti added a commit to ffesti/rpm that referenced this issue Apr 3, 2024
m user group

Add Provides with groupmember(user/group) and Requires/Recommends for both the
user and the group.

Extend RPMTAG_SYSUSERS and recognize the groupmember provides during
installation.

Resolves: rpm-software-management#2816
ffesti added a commit to ffesti/rpm that referenced this issue Apr 4, 2024
m user group

Add Provides with groupmember(user/group) and Requires/Recommends for both the
user and the group.

Extend RPMTAG_SYSUSERS and recognize the groupmember provides during
installation.

Resolves: rpm-software-management#2816
@github-project-automation github-project-automation bot moved this from In Review to Done in RPM Apr 4, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
Status: Done
Development

No branches or pull requests

3 participants