org.freedesktop.PowerManagement

[Holger Macht]

On Fri 30. Mar - 17:41:00, David Zeuthen wrote:
> On Fri, 2007-03-30 at 23:19 +0200, Lubos Lunak wrote:
> > On pá 30. března 2007, David Zeuthen wrote:
> > > On Fri, 2007-03-30 at 17:47 +0200, Lubos Lunak wrote:
> > > > > Well, you could contribute to the discussion, or bash GNOME
> > > > > applications. g-p-m uses a lot of libraries, and loads only the
> > > > > gobjects for the hardware you have on your system. Try compiling
> > > > > without DPMS and see the memory reduction.
> > > >
> > > >  That's not bashing, that's reality. I'm not going to recompile anything,
> > > > because first of all that's a strange way of quitting something I don't
> > > > need and second it's not going to help anyway (unless your DPMS
> > > > implementation needs 1MB heap). KPowersave is not different, that's part
> > > > of today's reality as well. There are good reasons not to run either of
> > > > them.
> > >
> > > This is all very interesting but perhaps it would be a bit more useful
> > > if you stayed on topic and gave feedback to the org.fd.PM interface.
> > 
> >  I thought supporting an opinion did not require one to repeat it, 
> 
> I didn't ask you to repeat your opinion; I just thought it was off-topic
> to discuss the implementation of g-p-m. Sorry if I didn't make that
> clear.
> 
> > but if you 
> > want: Shutdown/Reboot are basic functionality but power management features 
> > are an add-on that not everybody may be interested in having (see above for 
> > my feedback on that part). Since a DBUS interface cannot be split that makes 
> > the whole org.fd.PM interface optional and therefore Shutdown/Reboot should 
> > not be part of it.
> 
> The problem here is that Inhibit() on org.fd.PM affects the
> implementation on of Shutdown() and Reboot() - e.g. if I want to be able
> to make sure that the user gets this dialog
> 
>  Some app $APP is preventing shut down because: $REASON.
> 
>                                [Cancel] [Shut down anyway]
> 
> where $APP may be "k3b" and reason may be "burning DVD". Don't you want
> this kind of interaction?
> 
> And if you care about memory consumption no-one is forcing you to run
> g-p-m or kpowersave; just run other stuff instead and make it return
> NotSupported for anything anything but Shutdown() and Reboot(). That's
> not really too much to ask I think. Thoughts?

So beside anything else, _only_ Shutdown() and Reboot() should be
compulsory.

Having the other methods return NotSupported is actually the same for them
as being optional. So please make that clear in the spec.

Regards,
	Holger