Paul wrote: > james wrote: > > On Tue, Apr 20, 2010 at 07:55:45PM -0400, Chris Ball wrote: > > > Hi James/Sascha, > > > > Reviewed-by: James Cameron <[email protected]> > > > 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. Reactions? Michael _______________________________________________ Sugar-devel mailing list [email protected] http://lists.sugarlabs.org/listinfo/sugar-devel
