Padraig: > See CR 6734778. When gnome-volume-manager is run from the command line with > > /usr/lib/gnome-volume-manager --sm-disable --no-daemon > > it outputs > > manager.c/3846: ConsoleKit.GetSeats request failed to reply > manager.c/3846: ConsoleKit.GetSeats request failed to reply > > Do we intend to deliver ConsoleKit as part of vermillion?
Looking at the code, it seems that it is using ConsoleKit to find out if the user is local or not. This seems to only affect whether the restart_style is GNOME_RESTART_ANYWAY (if local) or GNOME_RESTART_NEVER (if not local). It also looks like it has some fallabck code in gvm_user_is_local that gets called if ConsoleKit isn't available. This fallback code tries to figure this out based on the Xserver being used. I am not sure how robust this code is. I think the only problem would be that we would set restart to never when we should set it to anyway. So if it crashes I guess it won't start back up. Perhaps not a huge problem. I would recommend testing the code to make sure the fallback logic works and that the restart value is getting set properly. ConsoleKit only knows about what seats are running if GDM informs it. So, if we integrate it, this would make the warning go away, but ConsoleKit wouldn't have the right data to pass back, and wouldn't work right. Therefore, we should wait to integrate ConsoleKit with the new rewritten GDM, which uses ConsoleKit. Brian
