Skip Menu |

This queue is for tickets about the Setup-Unix-User CPAN distribution.

Report information
The Basics
Id: 70202
Status: resolved
Priority: 0/
Queue: Setup-Unix-User

People
Owner: Nobody in particular
Requestors: AGORMAN [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 0.06
Fixed in: (no value)



Thank you very much for writing this module and taking the time to read this bug report. I'm on irc.perl.org if you would like to discuss. The documentation for member_of doesn't match the implementation. The documentation reads as follows: --------------------------------------------------------- member_of => array List of Unix group names that the user must be member of. The first element will be used as the primary group. If a group doesn't exist, it will be ignored. If not specified, the default is one group having the same name as the user. The group will be created if not already exists. The code on line 156 always adds the username as a group regardless of what is specified in the member_of ArrayRef. --------------------------------------------------------- push @$member_of, $name unless $name ~~ @$member_of; It's desirable for me to be able to add a user to an existing group and not have that a group be created with the same name as that user.
On Wed Aug 10 17:11:06 2011, AGORMAN wrote: Show quoted text
> Thank you very much for writing this module and taking the time to
read Show quoted text
> this bug report. I'm on irc.perl.org if you would like to discuss. > > > > The documentation for member_of doesn't match the implementation. > > The documentation reads as follows: > --------------------------------------------------------- > member_of => array > > List of Unix group names that the user must be member of. > > The first element will be used as the primary group. If a group
doesn't Show quoted text
> exist, it will be ignored. > > If not specified, the default is one group having the same name as the > user. The group will be created if not already exists. > > > > The code on line 156 always adds the username as a group regardless of > what is specified in the member_of ArrayRef. > --------------------------------------------------------- > > push @$member_of, $name unless $name ~~ @$member_of; > > > It's desirable for me to be able to add a user to an existing group
and Show quoted text
> not have that a group be created with the same name as that user.
You're welcome. Implemented in 0.07 via a new option 'primary_group'. User needs to be a member of at least one group, but now you can set it to another group other than the one with the same name as the user. -- sh
On Wed Aug 10 22:18:46 2011, SHARYANTO wrote: Show quoted text
> On Wed Aug 10 17:11:06 2011, AGORMAN wrote:
> > Thank you very much for writing this module and taking the time to
> read
> > this bug report. I'm on irc.perl.org if you would like to discuss. > > > > > > > > The documentation for member_of doesn't match the implementation. > > > > The documentation reads as follows: > > --------------------------------------------------------- > > member_of => array > > > > List of Unix group names that the user must be member of. > > > > The first element will be used as the primary group. If a group
> doesn't
> > exist, it will be ignored. > > > > If not specified, the default is one group having the same name as the > > user. The group will be created if not already exists. > > > > > > > > The code on line 156 always adds the username as a group regardless of > > what is specified in the member_of ArrayRef. > > --------------------------------------------------------- > > > > push @$member_of, $name unless $name ~~ @$member_of; > > > > > > It's desirable for me to be able to add a user to an existing group
> and
> > not have that a group be created with the same name as that user.
> > You're welcome. > > Implemented in 0.07 via a new option 'primary_group'. User needs to be > a member of at least one group, but now you can set it to another group > other than the one with the same name as the user. > > -- > sh
Nailed it. Thank you!