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

[Peter Robinson]

On Wed, Apr 21, 2010 at 1:32 AM, James Cameron <quozl at laptop.org> 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.
>
> Tested-by: James Cameron <quozl at laptop.org>
>
> Now, if one also stops HAL, the shutdown or restart options don't work,
> and shell.log says:

I thought the point of these patches was to move from HAL to
ConsoleKit so surely it should work whether HAL is there or not
especially with everyone moving away from HAL to
ConsoleKit/upower/udisk/udev etc/

Peter