[PATCH - RESEND] addgroup code cleanup, size reduction and feature enhancement

Tito farmatito at tiscali.it
Thu Apr 5 04:44:58 PDT 2007


On Thursday 05 April 2007 10:43:10 you wrote:
> On Thu, 2007-04-05 at 08:33 +0200, Tito wrote:
> 
> > 2) crippled function removal:
> > the current addgroup applet when called with 2 arguments
> > permits only to add an existing user to an still to be created group.
> > This seems to me a crippled behavior as it could be used just one time
> > at group creation time: 
> > 
> > root at localhost:~/Desktop/busybox.new# ./busybox_old addgroup prova tito
> > root at localhost:~/Desktop/busybox.new# ./busybox_old addgroup prova dina
> > addgroup: prova: group already in use
> > 
> > 3) the above mentioned feature was re-added as optional
> >     function enabled by ENABLE_FEATURE_ADDUSER_TO_GROUP.
> >     Now    if  called  with two non-option arguments, addgroup will add an existing
> >     user to an existing group like the full blown addgroup does (at least on my system).
> 
> What system do you use? None of mine systems (Gentoo, Ubuntu, Debian,
> FreeBSD, OpenBSD) have an addgroup (or groupadd) that allows me to add a
> user to an existing group.

(k)Ubuntu 6.10

ls -la /usr/sbin/addgroup lrwxrwxrwx 1 root root 7 2007-03-06 15:13 /usr/sbin/addgroup -> adduser

Ciao,
Tito
> Anyway... the patch is better than what we have today so it would be
> nice to see it committed.
> 
> Natanael Copa
> 
> 




More information about the busybox mailing list