2008/3/16, Victor Lowther <[EMAIL PROTECTED]>: > 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 <[EMAIL PROTECTED]>: > > > 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? _______________________________________________ Pm-utils mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/pm-utils
