svn commit: samba r19510 - in branches/SAMBA_4_0/source/torture: . basic nbench raw

tridge at samba.org tridge at samba.org
Mon Oct 30 00:35:21 GMT 2006


Jelmer,

 > Make smbtorture's `nprocs' parameter a parametric option.

I'd like this option to remain as a normal command line option.

parametric options don't show up in --help, so options that users of a
utility will commonly want should not be parametric options. 

I don't mind parametric options being used for things that users
rarely want to change, or for enabling unusual debugging paths, but
they should not be used for things that users commonly will want to
set. The --num-progs option is probably the most commonly used option
with some parts of smbtorture (for example the NBENCH test), so it
should not be hidden from users.

It's ok to make it parametric internally if you like, if the aim is to
make it not a global variable, but please leave it in the popt table,
and have the popt code set the parametric option using
lp_set_cmdline().

Cheers, Tridge


More information about the samba-cvs mailing list