[Sugar-devel] [PATCH] use ConsoleKit instead of HAL for
michael at laptop.org
Wed Apr 21 00:34:42 EDT 2010
> james wrote:
> > On Tue, Apr 20, 2010 at 07:55:45PM -0400, Chris Ball wrote:
> > > Hi James/Sascha,
> > > > Reviewed-by: James Cameron <quozl at laptop.org>
> > > Did you test on XO-1 or XO-1.5? I'm curious how much of a backwards-
> > > compatibility break this is.
> > No, I only did a code review and cross-check against ConsoleKit API
> > documentation.
> > However, I've just applied the patch on os119 on XO-1.5, restarted Sugar
> > and tested Restart and Shutdown options, and they function correctly.
> > Restart causes a UL screen and reboot. Shutdown causes a UL screen and
> > power off.
> so, can someone tell me (gently) why either of these techniques
> is better than simply invoking /bin/reboot or /bin/shutdown?
> (other than the fact that those will work even if hal isn't
For what it's worth, I would prefer to see a patch which simply called "sudo
/sbin/shutdown -h now" and "sudo /sbin/shutdown -r now". I don't see the value
that D-Bus and ConsoleKit are providing here.
More information about the Sugar-devel