Re: [PATCH] udevd.c: Add 'N:' to optstring in getopt_long

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Thomas,

Thomas Egerer [2011-03-31 15:06 +0200]:
> -		option = getopt_long(argc, argv, "c:deDthV", options, NULL);
> +		option = getopt_long(argc, argv, "c:deDthN:V", options, NULL);

This is certainly an inconsistency, but I wonder if we shouldn't
instead just remove the other short options from the optstring?
Neither --help nor the manpage document the short versions, so they
could be considered a kind of implementation detail?

I. e. we should perhaps only support the documented long options.

Martin

-- 
Martin Pitt                        | http://www.piware.de
Ubuntu Developer (www.ubuntu.com)  | Debian Developer  (www.debian.org)
--
To unsubscribe from this list: send the line "unsubscribe linux-hotplug" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel]     [Linux DVB]     [Asterisk Internet PBX]     [DCCP]     [Netdev]     [X.org]     [Util Linux NG]     [Fedora Women]     [ALSA Devel]     [Linux USB]

  Powered by Linux