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

[Michael Stone]

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.

Reactions?

Michael