I'll try to keep to the case....
On Wed, 26 Aug 2009, Brian Cameron wrote:
>
> Randy:
>
> > On Wed, 26 Aug 2009, Brian Cameron wrote:
> >
> > >
> > > Jedy/Sherry:
> > >
> > > > Sherry gave us the whole story. The logic you mentioned will be
> > > > implemented in ConsoleKit which will be integrated after b128. So
> > > > currently, we have 2 options. The first one is to wait ConsoleKit's
> > > > integration. The second one is to integrate the current fast reboot
> > > > support of GNOME restart dialog at b124 and then update it after
> > > > ConsoleKit's integration after b128.
> > >
> > > Note that ConsoleKit currently does not support the ability to respond
> > > to different types of reboot options. So, I assume part of the plan to
> > > use ConsoleKit in the future is to enhance it so it can support doing
> > > this.
> >
> > IMHO, it is sufficient for the restart dialog to only have
> > 'Restart", as other choices are effectively TMI. The majority of
> > users will gladly take the default policy as to what restart() means
> > (fast, through BIOS, whatever). Users that have a clue as to the
> > difference between 'reboot -f' and 'reboot -p' will be happy to do
> > that on a CLI. And as long as there is a decent API to select what
> > the default reboot properties should be, I don't really see that the
> > added complexity is worth it.
>
> Yes, perhaps ConsoleKit could be modified to check an SMF property to
> determine what the default boot preference is, and call the right
> command, whether it is "reboot -f" or "reboot -p" instead of calling
> init. This would eliminate the need for the ConsoleKit D-Bus interfaces
> needing to be expanded so that reboot options can be passed in.
>
Sorry if it wasn't clear, but as I see it, there is no need to check
SMF and do something based on the result. Just reboot (or 'init 6'),
and the right thing will occur. I don't see that an extra button to
decide if "fast" or "BIOS/Prom" is necessary, as the vast majority of
users either won't care or won't understand the difference (and the
latter will only add to their confusion).
For folks who understand the behavior, they know how to change the
default, or make the correct call in other ways.
And while the project team is at it, they should consider putting a
placeholder in for "Suspend", as I see that as a legitimate extension.
Cheers!
---- Randy
