Re: [bug-inetutils] [PATCH 2/7] argp'fied argument parsing & tiny fixes

[Alfred M. Szmidt]

   +static struct argp_option argp_options[] = {
   +#define GRP 0
   +  {"debug", 'D', "LEVEL", OPTION_ARG_OPTIONAL, "Set debugging level", 
GRP+1},
   +  {"authmode", 'a', "AUTHMODE", 0, "Specify what mode to use for "
   +    "authentication", GRP+1},
   +  {"exec-login", 'E', "STRING", 0, "Set program to be executed instead "
   +    "of /bin/login", GRP+1},
   +  {"no-hostinfo", 'h', NULL, 0, "Do not print host information before login 
"
   +    "has been completed", GRP+1},
   +  {"linemode", 'l', "MODE", OPTION_ARG_OPTIONAL, "Set line mode", GRP+1},
   +  {"no-keepalive", 'n', NULL, 0, "Disable TCP keep-alives", GRP+1},
   +  {"reverse-lookup", 'U', NULL, 0, "Refuse  connections from addresses that 
"
   +    "cannot be mapped back into a symbolic name", GRP+1},
   +#ifdef AUTHENTICATION
   +  {"disable-auth-type", 'X', "AUTHTYPE", 0, "Disable the use of given "
   +    "authentication option", GRP+1},
   +#endif

This is a bad idea, the option should exist even if maybe it is not
supported.  If it is not supported, then we should get a error message
saying so.