[Lcdproc] Re: Bug#397072: lcdproc: LCDd fails to start due to incorrect argument syntax in init.d script.

[Peter Marschall]

Hi Joris,

On Sunday, 5. November 2006 10:33, Joris Robijn wrote:
> On 4 Nov 2006 at 18:06, Lennart Sorensen wrote:
> > The init.d script for LCDd contains the options -s -f -c /etc/LCDd.conf,
> > while in the past I have also seen -s 1 -f 1 -c /etc/LCDd.conf.  Both are
> > wrong since -s requires an argument and -f doesn't take one.  The options
> > -s 1 -f -c /etc/LCDd.conf work and allow LCDd to actually start from the
> > init.d script.
>
> The reason why the values after the options were introduced was because
> you can otherwise not override such an option from the command line once
> it is in the config file.

I am perfectly aware of that.
And if you look at the current option syntax for LCDd in CVS, an argument
is still required for -s (If it is missing in the official Debian package,
it is a bug introduced there).

For -f, I decided to change the behaviour a bit: If the config file states
Foreground=false, you can override it with the command line option -f
to cause LCDd to not daemonize.
But I disabled the other way 'round for a few reasons:
- with all clients in the package having a config file, there is no 
  need to explicitely tell them "Do deamonize" on the command line.
  The default behavour can be specified using the config file.
- with config files present and the ability to use alternative config 
  files as well, I cannot imagine a single use case where you have
  "Foreground=no" in your config file, but want to daemonize using
  command line options.
- most other daemons I know have a similar option for debugging purposes
  that tells the deamon to stay in the foreground.
  For "compatibility" with other deamons I prefer the non-argument option.
- it saves me typing 2 keys when debugging ;-)

Regards
Peter

-- 
Peter Marschall
peter@adpm.de