Skip to content

Conversation

@ashgit23
Copy link
Contributor

This PR updates the help text for the -U option in groupadd to improve clarity and reduce ambiguity.

Old text:
-U, --users USERS list of user members of this group
New text:
-U, --users USERS list of users to assign to the group

The previous phrasing could be misinterpreted as querying existing group members. The updated version makes it clear that this option is used to assign users when creating a new group.

This change affects only the English help output in src/groupadd.c. No functional behavior is modified

@alejandro-colomar
Copy link
Collaborator

Thanks! Can you please squash the commits?

Reviewed-by: Alejandro Colomar <alx@kernel.org>

Align wording with groupmod to reduce ambiguity in -U option description.
@ashgit23 ashgit23 force-pushed the clarify-groupadd-U-help branch from 7d22424 to 6e60f0e Compare November 18, 2025 12:14
@ashgit23
Copy link
Contributor Author

Thank you for the feedback ! Commits squashed as requested. PR is now a single commit with updated message.

Copy link
Collaborator

@alejandro-colomar alejandro-colomar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

@alejandro-colomar alejandro-colomar merged commit e7ccd3d into shadow-maint:master Nov 18, 2025
9 of 10 checks passed
@ashgit23 ashgit23 deleted the clarify-groupadd-U-help branch December 28, 2025 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants