[cvs] bogofilter/src bogoconfig.c,1.136,1.137 configfile.c,1.33,1.34 configfile.h,1.10,1.11

Matthias Andree matthias.andree at gmx.de
Mon Jan 19 04:39:14 CET 2004


On Sun, 18 Jan 2004, Stefan Bellon wrote:

> Matthias Andree wrote:
> > On Sat, 17 Jan 2004, relson at users.sourceforge.net wrote:
> 
> [snip]
> 
> > > Add '--option=value' to permit config file options on the command
> > > line.
> > > 
> > > +	case '-':
> > > +	    process_config_option(optarg, true, PR_COMMAND);
> > > +	    break;
> > > +
> 
> > Is this the right thing to do? I doubt it. I'd think we'd use
> > gnugetopt to parse "--option"s and use a dedicated option (yes, yet
> > another) to expose configuration file options.
> 
> So, why not use '-' as the dedicated option that tells that now a
> configuration file option follows? The first '-' is the command line
> switch and the second '-' is your dedicated option. What's wrong with
> that?

We'll tie command line options to configuration file options. I wonder
if that's desirable. Some options make sense on the command line only
(such as the configuration file name). We're exposing all with that
single-option approach.

-- 
Matthias Andree

Encrypt your mail: my GnuPG key ID is 0x052E7D95




More information about the bogofilter-dev mailing list