[Top][All Lists]

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

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

From: Alfred M. Szmidt
Subject: Re: [bug-inetutils] [PATCH 2/7] argp'fied argument parsing & tiny fixes
Date: Sat, 23 Aug 2008 05:03:42 -0400

   +static struct argp_option argp_options[] = {
   +#define GRP 0
   +  {"debug", 'D', "LEVEL", OPTION_ARG_OPTIONAL, "Set debugging level", 
   +  {"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},
   +  {"disable-auth-type", 'X', "AUTHTYPE", 0, "Disable the use of given "
   +    "authentication option", GRP+1},

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.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]