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

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

 



On Thu, Mar 31, 2011 at 16:30, Martin Pitt <martin.pitt@xxxxxxxxxx> wrote:
> 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.

There is no real need to have the long and short ones in sync.

Most of the short options are only used on the commandline during
development, and none of them are documented, or there are any real
guarantees about them to stay as they are.

Only the documented long options should be used for anything that is
not a human on the commandline.

Kay
--
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