[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Bug#203588: acpid: Shell script has nothing to do in /etc



On Sat, 2003-08-02 at 03:38, Pierre THIERRY wrote:

> >> Tags: patch
> > You forgot to attach it :-)
> 
> Shit. And the BTS doesn't seem to have noticed the patch tag...
> 
You meant to put "tags 203588 patch", "thanks" and Bcc
control@bugs.debian.org, didn't you? :-)

> > Event-handling from cardmgr, hotplug, usbmgr, acpid, apmd etc. are
> > really useful to be able to be customised by power users.
> 
> I think I'm something like a power user, and I hate having to read and
> understand a script (being shell, perl or anything else) to customize a
> package to my needs.
> 
Then you're not what I'd consider a UNIX power user :-)

> And I love a well-documented configuration file, where I just have to
> change some paramters, without having to understand everything behing
> it. The power user might want to focus on its work, not on the
> custimozation of every signle package he installs...
> 
Then they probably don't care what happens when the power button is
pushed!

> > You've assumed they want the power button to *be* a power button, it's
> > entirely likely that they might want it to (for example) switch the
> > user into single user mode instead.
> 
> I didn't assume anything, and my version of the script just need th
> change ACTION=halt to ACTION=single to achieve this. And if the script
> is rewritten or modified to be just better, an apt-get upgrade won't
> erase all the customizations made by the sysadmin, because it is in a
> configuration file that have little reasons to change...
> 
How do I configure your script to restart apache when the power button
is pushed?

> > Shell scripts run by event daemons are the power-user's configuration
> > files. Leave them be.
> 
> They are a very bad manner to provide configuration files to the
> power-user, IMHO... And I still think this bug is an RC one.
> 
It's not an RC bug.  If shell scripts weren't allowed in /etc -- init.d
would be a bit of a problem :-)

Scott

Attachment: signature.asc
Description: This is a digitally signed message part


Reply to: