Opened 7 years ago

Closed 6 years ago

#7266 closed defect (fixed)

can't restart/shutdown system from sugar with olpc3

Reported by: dsd Owned by: jg
Priority: normal Milestone: 8.2.0 (was Update.2)
Component: distro Version: olpc-3
Keywords: rel-8.2.0:? olpc3-17:- olpc3-19:- olpc3-20:- joyride-2072:+ Cc: dgilmore
Blocked By: Blocking:
Deployments affected: Action Needed:
Verified: no

Description

Running olpc3-17. Hovering mouse over XO icon and selecting the "Restart" option doesn't do anything at all (I'm returned to a fully functional sugar interface).

Attachments (1)

shell.log (1.8 KB) - added by dsd 7 years ago.
sugar log showing exception

Download all attachments as: .zip

Change History (10)

comment:1 Changed 7 years ago by dsd

  • Keywords rel-8.2.0:? olpc3-17:- added

comment:2 Changed 7 years ago by dsd

  • Keywords olpc3-19:- added
  • Summary changed from can't restart system from sugar with olpc3 to can't restart/shutdown system from sugar with olpc3

shutdown is broken too

Changed 7 years ago by dsd

sugar log showing exception

comment:3 Changed 7 years ago by dsd

  • Cc dgilmore added

This is fixed in more recent sugar versions: http://dev.laptop.org/git?p=sugar;a=commitdiff;h=508eb82f64eb1b8445779391c5959083a15085db

Dennis, any chance we could have a newer sugar tagged in the olpc3 stream?

comment:4 Changed 7 years ago by dsd

  • Keywords olpc3-20:- added

we have a new sugar but things are still broken, different error, will work on this...

comment:5 Changed 7 years ago by dsd

This is broken because HAL asks PolicyKit if sugar is allowed to mount removable drives, but PolicyKit says no. PK asks ConsopleKit if sugar is part of the active session, CK says no, and the default PK rules specify that only active users can mount stuff. We either need to modify the PolicyKit rules or properly integrate olpc-dm with ConsoleKit (requires some rearchitecting - Erik is looking into this).

comment:6 Changed 6 years ago by marco

comment:7 Changed 6 years ago by marco

I sent a patch here:

http://lists.laptop.org/pipermail/code-review/2008-June/000022.html

To test it, make sure that you don't have any other user logged in on the console. The current policy doesn't allow to shutdown in that case. It seems like we should tweak that policy for the XO since it's single user... but that's a separate bug.

comment:8 Changed 6 years ago by marco

Tomeu tested that it works in joyride.

comment:9 Changed 6 years ago by marco

  • Keywords joyride-2072:+ added
  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.