[PATCH] users: new applet

Denys Vlasenko vda.linux at googlemail.com
Tue Aug 9 02:38:24 UTC 2011


On Wednesday 27 July 2011 17:12, Tito wrote:
> On Wednesday 27 July 2011 16:43:34 Tito wrote:
> > On Wednesday 27 July 2011 13:10:49 gotrunks at gmail.com wrote:
> > > On Wed, Jul 27, 2011 at 10:29 AM, Tito <farmatito at tiscali.it> wrote:
> > > > On Wednesday 27 July 2011 00:33:37 gotrunks at gmail.com wrote:
> > > >> Hi,
> > > >>
> > > >> patch attached.
> > > >>
> > > >> Regards,
> > > >>
> > > >
> > > > Hi,
> > > > printf(ut->ut_user);  maybe adding a format string here would be a good thing.
> > > 
> > > Ok.
> > > 
> > > > BTW.: this code looks very similar to what the who applet does, couldn't
> > > > they be merged?
> > > >
> > > 
> > > That was the first thing I tried, but I don't know how to do it
> > > without unsightly impact the who applet.
> > > 
> > > > Ciao,
> > > > Tito
> > > > _______________________________________________
> > > > busybox mailing list
> > > > busybox at busybox.net
> > > > http://lists.busybox.net/mailman/listinfo/busybox
> > > >
> > > 
> > > Regards,
> > > 
> > 
> > Hi,
> > attached an example patch on how who and users could be merged.
> > Denys will love my obfuscated code ;-)
> > 
> > Ciao,
> > Tito
> > 
> > --- coreutils/who.c.orig	2011-06-22 14:29:58.000000000 +0200
> > +++ coreutils/who.c	2011-07-27 16:32:09.000000000 +0200
> > @@ -18,6 +18,31 @@
> >   */
> >  /* BB_AUDIT SUSv3 _NOT_ compliant -- missing options -b, -d, -l, -m, -p, -q, -r, -s, -t, -T, -u; Missing argument 'file'.  */
> >  
> > +//config:config WHO
> > +//config:      bool "who"
> > +//config:      default y
> > +//config:      depends on FEATURE_UTMP
> > +//config:      help
> > +//config:        who is used to show who is logged on.
> > +
> > +//config:config USERS
> > +//config:      bool "users"
> > +//config:      default y
> > +//config:      depends on FEATURE_UTMP
> > +//config:      help
> > +//config:        Print users currently logged on.
> > +
> > +//applet:IF_USERS(APPLET_ODDNAME(users, who, BB_DIR_USR_BIN, BB_SUID_DROP, users))
> > +//applet:IF_WHO(  APPLET(  who, BB_DIR_USR_BIN, BB_SUID_DROP))
> > +
> > +//kbuild:lib-$(CONFIG_USERS) += who.o
> > +//kbuild:lib-$(CONFIG_WHO) += who.o
> > +
> > +//usage:#define users_trivial_usage
> > +//usage:       ""
> > +//usage:#define users_full_usage "\n\n"
> > +//usage:       "Print the users currently logged on"
> > +
> >  //usage:#define who_trivial_usage
> >  //usage:       "[-a]"
> >  //usage:#define who_full_usage "\n\n"
> > @@ -49,9 +74,10 @@
> >  {
> >  	struct utmp *ut;
> >  	unsigned opt;
> > +	int do_users = (ENABLE_USERS && (!ENABLE_WHO || applet_name[0] == 'u'));
> >  
> >  	opt_complementary = "=0";
> > -	opt = getopt32(argv, "aH");
> > +	opt = getopt32(argv, do_users ? "" : "aH");
> >  	if (opt & 2) // -H
> >  		printf("USER\t\tTTY\t\tIDLE\tTIME\t\t HOST\n");
> >  
> > @@ -60,6 +86,7 @@
> >  		if (ut->ut_user[0]
> >  		 && ((opt & 1) || ut->ut_type == USER_PROCESS)
> >  		) {
> > +		if (!do_users) {
> >  			char str6[6];
> >  			char name[sizeof("/dev/") + sizeof(ut->ut_line) + 1];
> >  			struct stat st;
> > @@ -88,8 +115,15 @@
> >  					ctime(&seconds) + 4,
> >  					(int)sizeof(ut->ut_host), ut->ut_host
> >  			);
> > +			} else {
> > +				/* use opt as flag for first time */
> > +				printf(" %s" + !opt, ut->ut_user);
> > +				opt = 1;
> 				/* we don't want to trigger (opt & 1) */
> 				opt = 2;
> > +			}
> >  		}
> >  	}
> > +	if (do_users)
> > +		bb_putchar('\n');
> >  	if (ENABLE_FEATURE_CLEAN_UP)
> >  		endutent();
> >  	return EXIT_SUCCESS;
> > 
> 
> Hi,
> minor fix on previous patch. We probably don't want set opt = 1, so let us use 2 instead.

Applied, thanks!
-- 
vda


More information about the busybox mailing list