[Pm-utils] Release?

[Michael Biebl]

2008/3/16, Victor Lowther <victor.lowther at gmail.com>:
> On Sat, 2008-03-15 at 18:16 -0500, Victor Lowther wrote:
>  > On Sat, 2008-03-15 at 23:43 +0100, Michael Biebl wrote:
>  > > 2008/3/15, Victor Lowther <victor.lowther at gmail.com>:
>  > > More bugs:
>  > > Running pm-is-supported as not root:
>  > >
>  > > # pm-is-supported --suspend && echo yes
>  > > mkdir: cannot create directory `/var/run/pm-utils/storage': Permission denied
>  > > yes
>  > > # pm-is-supported --help
>  > > mkdir: cannot create directory `/var/run/pm-utils/storage': Permission denied
>  > > pm-is-supported [--suspend | --hibernate | --suspend-hybrid ]
>  >
>  > Fixing that the right way will be slightly tricker.  I will postpone
>  > looking at this until tomorrow --  I have a party to get ready for.
>
>
> Fix comitted and pushed upstream.  Now we only try to parse blacklist
>  entries and load default parameters after we have taken the suspend
>  lock.

Hm, seems I missed to review your vlowther-dynamic-hook-disable changes.
Honestly, I'm a bit sceptical about adding this new /etc/pm/parameters
and /etc/pm/blacklists interface.

We already have a (documented) mechanism for users to disable a hook
(which is, to create a non-executable file in /etc/pm/sleep.d/. Why do
we need another mechanism?

Second, do we really need to pass parameters to hooks? I haven't
needed it so far. Imho we should only add functionality which is
actually used.

Besides, those two new interfaces are completely undocumented.

Cheers,
Michael


-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?