Hi Christoph,

Thanks for the feedback, but are you really suggested to replace
an informative message (which points to what exactly needs to be changed)
with a generic 'enabled/disabled' action (whenever there is no ENABLED
variable at all) suggestion?

On Thu, 27 May 2010, Christoph Anton Mitterer wrote:
> Hi.

> This is really a very minor wishlisht bug only.

> I'm currently looking through some of Debians init-scripts, as
> some of them are really too chatty and nearly output documentation.

> For /etc/init.d/edac one could think whether this also applies
> for the following:
> log_daemon_msg "Not enabling ${DESC} since EDAC_DRIVER is not set"

> It might be enough to simply say something like
> "Not enabling EDAC as it has been disabled."
> Ok that text is not so good, but what I mean is, the EDAC_DRIVER
> thingy might be too descriptive.

> But as said,.. this is rather perfectionism than really a problem.

> Cheers,
> Chris.
-- 
                                  .-.
=------------------------------   /v\  ----------------------------=
Keep in touch                    // \\     (yoh@|www.)onerussian.com
Yaroslav Halchenko              /(   )\               ICQ#: 60653192
                   Linux User    ^^-^^    [175555]





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

Reply via email to