public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
* [gentoo-dev] enew{user,group}: killing off [extra] argument
@ 2011-11-02 22:13 Mike Frysinger
  2011-11-03 15:30 ` Mike Frysinger
  0 siblings, 1 reply; 5+ messages in thread
From: Mike Frysinger @ 2011-11-02 22:13 UTC (permalink / raw
  To: gentoo-dev

[-- Attachment #1: Type: text/plain, Size: 1289 bytes --]

when i first wrote enew{user,group} oh-so-long-ago, the reason for the [extra] 
arguments was the assumption that i am short sighted.  i figured someone would 
come up with some creative need for passing additional flags that i couldn't 
possibly think of.

however, in the ~9 years since, all i have to report is disappointment in the 
collective lack of creativity when it comes to abusing open ended interfaces.  
two flags have surfaced in the main tree with enewuser, and only in a handful 
of ebuilds:
	-c <comment>
	-G <groups>
enewgroup hasn't had any extra args !

when it comes to -c, the custom comment in the ebuilds isn't all that better 
than the default one that enewuser already adds, so i've punted them.

when it comes to -G, it's a matter of the existing argument not being 
correctly utilized.  enewuser already takes care of splitting the <groups> 
argument into -g/-G.  so i've converted the packages using this.

back to the point of this e-mail ... i plan on dropping the [extra] part of 
enew{user,group} and changing this to a `die`.  this simplifies the interface, 
and makes it more portable -- the darwin port has already been calling `die` 
when people pass in extra args.

now is the time to whine if you want to keep this !
-mike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2011-11-09 21:25 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-11-02 22:13 [gentoo-dev] enew{user,group}: killing off [extra] argument Mike Frysinger
2011-11-03 15:30 ` Mike Frysinger
2011-11-06 18:33   ` Petteri Räty
2011-11-06 22:54     ` Mike Frysinger
2011-11-09 21:24       ` Doug Goldstein

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox