Bug#507868: please handle VERBOSE via log_daemon_msg/log_end_msg

2008-12-05 Thread Guido Günther
Package: initscripts
Version: 2.86.ds1-61
Severity: wishlist

Hi,
the current /etc/init.d/skeleton prefixes every call to the lsb log_*
functions with:
 
 [ $VERBOSE != no ] 

This looks like overkill and just reduces the readability. Wouldn't it
be better if the the lsb log_* functions would look at an env var
(LSB_VERBOSE or similar)?
Cheers,
 -- Guido



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#507868: [Pkg-sysvinit-devel] Bug#507868: please handle VERBOSE via log_daemon_msg/log_end_msg

2008-12-05 Thread Petter Reinholdtsen
[Guido Günther]
 This looks like overkill and just reduces the readability. Wouldn't
 it be better if the the lsb log_* functions would look at an env var
 (LSB_VERBOSE or similar)?

I believe each script writer need to decide on a per message basis if
the message should be visible when VERBOSE=no (or kernel option quiet)
is used, and thus believe it is the wrong approach to move the test to
the log_* functions.  The skeleton example just happen to believe all
its messages should be hidden in quiet mode.

Happy hacking,
-- 
Petter Reinholdtsen



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#507868: [Pkg-sysvinit-devel] Bug#507868: please handle VERBOSE via log_daemon_msg/log_end_msg

2008-12-05 Thread Guido Günther
On Fri, Dec 05, 2008 at 09:59:56AM +0100, Petter Reinholdtsen wrote:
 [Guido Günther]
  This looks like overkill and just reduces the readability. Wouldn't
  it be better if the the lsb log_* functions would look at an env var
  (LSB_VERBOSE or similar)?
 
 I believe each script writer need to decide on a per message basis if
 the message should be visible when VERBOSE=no (or kernel option quiet)
 is used, and thus believe it is the wrong approach to move the test to
 the log_* functions.  The skeleton example just happen to believe all
 its messages should be hidden in quiet mode.
But wouldn't it be better to make all the log_* functions look at
LSB_VERBOSE by default and have a special set of functions that ignores
$LSB_VERBOSE?
I agree that it should be left to the script maintainer to decide what
is important and what isn't but the way we do it looks a bit like
overkill.
 -- Guido



--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]