[Sugar-devel] [PATCH] use ConsoleKit instead of HAL for

[Paul Fox]

michael wrote:
 > Paul wrote:
 > > 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
 > > running?)
 > 
 > Sascha, Paul,
 > 
 > 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.

that would certainly solve OLPC's problem, but i suspect sugar
wants something more portable than that in the codebase.

paul
=---------------------
 paul fox, pgf at laptop.org