/srv/irclogs.ubuntu.com/2009/12/02/#ayatana.txt

=== MacSlow is now known as MacSlow|lunch
=== MacSlow|lunch is now known as MacSlow
=== beuno is now known as beuno-lunch
alkisgHi, in previous Ubuntu versions, there was a patch for fusa that was hiding the reboot and shutdown menu items in LTSP clients. I think that patch was dropped in Karmic, along with the switch to indicator-applet.16:21
alkisgWould it be possible for me to suggest a smaller patch for indicator-applet, i.e. to set a xatom with the action that the user selected (reboot, shutdown etc) so that the LTSP display manager would read it and act appropriately?16:21
mac_v_tedg: ^ ;)16:26
tedgalkisg: No, but for a good reason :)16:27
tedgalkisg: We're now checking with the ConsoleKit settings to see if the user can do that.16:27
tedgalkisg: So you should be patching/fixing consolekit so that it happens all over the desktop.16:27
tedgalkisg: Including the session menu.16:28
alkisgtedg: the user dbus session is on the server, so that won't work for the ltsp client16:28
alkisgAh, consolekit, not gnome-session... hmm...16:28
tedgalkisg: ConsoleKit is on the system dbus.16:28
tedgalkisg: And if it's on teh server, it should definitely not be giving users permission to shut it down! :)16:28
tedgHiding the menu item is one thing, but there shouldn't be a way for them to be able to craft a dbus command to do it either.16:29
alkisgLTSP currently doesn't install policykit1 on the chroot... so if we installed it, and called the appropriate method, would that work?16:29
alkisg(I tried to call the CK method for restart, but it complained about PK)16:29
tedgalkisg: I'm not sure about PK here.  We're just calling the CK method "CanRestart" and "CanStop".16:30
tedgalkisg: I'm not sure how you configure/modify those in ConsoleKit.16:30
alkisgtedg: Anyway I got it, thanks a lot, I'll try to see how that would work with LTSP16:31
tedgnp16:31
tedgmac_v_: Thanks for the ping!16:31
alkisg(i.e. I'll try to somehow make CK.restart work... thanks again)16:31
mac_v_hehe , actually i wanted to ping you for something else and caught the chat ;)16:32
mac_v_tedg: any news about inkscape ppa ?16:32
tedgmac_v_: I haven't had time to work on it :(16:33
mac_v_hmm , :( ... 16:33
tedgmac_v_: I need to migrate the branches from SVN to BZR16:33
tedgmac_v_: Well, they were already in BZR, but now that we've moved the version numbers don't line up.16:33
tedgmac_v_: I'm planning to work on it tonight, if it goes smoothly it'll be done -- but this stuff rarely goes smoothly :)16:34
mac_v_tedg: the "save" is killing me... thats the only problem ... np... take you time :)16:34
mac_v_yay16:34
=== mac_v_ is now known as mac_v
=== MacSlow is now known as MacSlow|break
=== beuno-lunch is now known as beuno
alkisgtedg: could you please direct me to the package/source file that calls CK for canrestart/restart etc?17:10
* tedg looks17:11
tedgalkisg: Hmm, I can't find it... I may have lied. I swear I wrote that code...17:13
alkisgtedg: heh, np, I'll look myself, thanks. Is that indicator-session?17:13
tedgalkisg: Yes.  bzr branch lp:indicator-session17:14
alkisgThanks a lot! :)17:14
alkisgsrc/gtk-dialog/logout-dialog.c17:16
alkisgtedg: if I understand the sources correctly, ck-pk-helper.c asks CK/PK if the user can shutdown the system, but the actual action is in gtk-logout-helper.c, e.g. res = dbus_g_proxy_call_with_timeout (sm_proxy, "RequestShutdown", INT_MAX, &error,...17:52
alkisgThe action happens on the session dbus, which is on the server.17:52
alkisgI think that LTSP needs some help on the action part, i.e. if LTSP_CLIENT is set in the environment, set an xprop for LDM to know if it's supposed to reboot/shutdown etc. Would you consider accepting such a (small) patch?17:52
alkisgAs it turns out, there's already code in LTSP that reads a file named /tmp/ldm-logout-action, and if it contains "shutdown" or "reboot", it acts appropriately. So a small patch in indicator-session that would write those values to that file if the LTSP_CLIENT env var is present, would be most welcomed by LTSP users (instead of the xprop I mentioned before).19:06
alkisg(and btw, we teachers also miss the real user name instead of the login name :))19:10
beunoalkisg, I think that letting people configure their display name would be a good thing to do19:14
beunoalkisg, would you like to bring that up on the ayatana mailing list?19:15
alkisgbeuno: sure, thanks.19:15
alkisgI've filed a launchpad bug for this already19:15
alkisg(some days ago)19:15
beunoalkisg, great, will help us track it. I think the mailing list is a better forum for discussions, and decisions  :)19:16
alkisgSure, will do ;)19:16
beunoand we still have time to change it in Lucid!19:16
alkisgLucid (and all LTS releases) are very important for schools, as they don't usually upgrade every 6 months... :)19:17
beunoyeah, there is a lot riding on Ubuntu this cycle19:22
=== MacSlow|break is now known as MacSlow
mac_vbeuno: is it really essential to show the user name in a single user setup ? :(  20:02
beunomac_v, I don't think it's the name that is essential, but the actions under it20:02
beunowhat else would you display?20:02
mac_vthough it makes sense in alkisg's scenario  , its kinda wasting space on the panel ... i already know my name;)20:03
alkisgmac_v: an option for it would suffice, it doesn't need to be default... but even on my standalone PC I don't like seeing "alkisg", I prefer seeing my Greek name.20:03
mac_vbeuno: IMO , just the icon would suffice , not sure why the name is needed20:03
beunomac_v, so, you now have something to reply on the list  ;)20:04
beunoI don't think I feel strongly about it, but it seems to feel right20:04
beunoand I know there are some plans for the future in that area, that integrate/group more features under there20:04
alkisgAlso, is the top panel space important? It's usually empty :)20:05
mac_valkisg: i use only 1 panel :)20:06
mac_valkisg: if you ask the dekstop team , they are planning on removing the bottom panel and a lot of widgets to reduce the boot time20:07
mac_vdesktop*20:07
beunoI think 1 panel is the future20:07
alkisgmac_v: well, then you are not using the default setup anyway....  Ah. Ouch. :)20:07
beunoI'd have to check of it's Lucid future, or Lucid+120:07
mac_valkisg: they are going to make it the default or atleast try to convince UX , , since their alloted time is 4secs ;)20:07
beunobut it's near future20:07
beunodjsiegel1, do you know?20:07
tgpraveen1mac_v: 1 panel would be a pretty radical change.i doubt UX will approve.20:08
mac_vsure they wont ;p  , but lot of the applets are gonna be removed 20:09
djsiegel1I do not think we are changing anything about the panels20:10
mac_vkenvandine knows more whats going/planned to be removed ;)20:11
kenvandinebeuno, it isn't set in stone or anything, but i think the goal was to try it 20:14
kenvandineand see how it impacted startup time and see what UX thought20:15
kenvandinebeuno, you should really talk to rickspencer320:15
beunothanks20:15
alkisgtedg: adding those 4 lines to gtk-logout-helper.c make LTSP work fine with reboot/shutdown: http://paste.ubuntu.com/333393/21:17
alkisgUnfortunately, LTSP won't have client<=>server dbus integration for some time to come, so I think that that hack is the best that LTSP users can have right now...21:17
alkisgTested, works fine, an additional check "if LTSP_CLIENT env var is defined" would be better though.21:17
tedgalkisg: Where is the gnome-session running?21:18
alkisgOn the server21:19
tedgalkisg: Where is indicator-session-service running?21:19
alkisgOn the server21:19
tedgalkisg: Then why is client <=> server dbus integration an issue?21:19
alkisgBecause we want shutdown to shutdown the client, not the server21:20
tedgalkisg: So it seems to me the session manager should be doing that, eh?21:20
tedgI'm not trying to blow you off here, it just seems like changing the menu items is just dealing with the surface of the problem.21:21
alkisgtedg: The LTSP devs are planning to see if they can make the session dbus talk with the system dbus *on the client* instead of the one on the server21:21
alkisgBut that won't happen for a long time (if it's ever going to happen)21:21
alkisgtedg: so, ltsp users now have reboot/shutdown menu items that just logoff21:22
tedgYes.  But right now we have  "indicator-session-service -> session-manager -> console-kit" it seems like it should go to "indicator-session-service -> session-manager -> ltsp-tool"21:22
alkisgWith those 4 lines you can make them actually reboot/shutdown...21:22
alkisgtedg: I understand that it's a hack. But LTSP can't do better right now.21:23
tedg? Why can't gnome-session be fixed?21:23
alkisggnome-session runs on the server, it has no control over the client21:24
tedgSo does indicator-session-service, right?21:25
alkisgRight. So it would just send a "signal" to the ltsp display manager, so that when the session is over, LDM shuts off the PC21:26
tedgSo it seems like gnome-session could send that same signal to LDM.21:26
alkisgI don't think patching gnome would be any easier...21:26
tedgYes, but when other people use that gnome-session shutdown/restart interface it would work.21:27
tedgLike when you press the soft power button on the outside of the case.21:27
alkisgYou're right on that. But I don't think I'll ever be able to get such a patch accepted.21:28
alkisgSo Lucid, like Karmic, will go out with LTSP users unable to reboot/shutdown... :(21:28
tedgalkisg: Well, you only have to get the patch into the package.21:29
alkisgWell, I'll try. Thanks a lot for your time, and your advice! :)21:29
tedgalkisg: Yeah, okay, I checked and that whole interface is a distro patch anyway.21:30
tedgalkisg: So you just need to edit 95_dbus_request_shutdown.patch21:30
tedgalkisg: I believe that the guy who can help you the most there is chrisccoulson21:30
alkisgtedg: nice, you gave me hope!!! I'll try that tomorrow, too late here to go on :)21:30
tedgalkisg: He wrote that patch.21:31
tedgalkisg: Good night!21:31
alkisgGood night, and thanks again :)21:31
chrisccoulsonhello :)21:31
alkisgAh, hi chrisccoulson!21:31
alkisgDo I have any chances? :)21:31
chrisccoulsonhi alkisg21:31
chrisccoulsoni haven't read the scrollback yet - it might be quicker to ask your question again ;)21:32
alkisg(a small patch for LTSP users to be able to reboot/shutdown the clients)21:32
alkisgg_spawn_command_line_async("ltsp-localapps \"/bin/sh -c 'echo shutdown > /tmp/ldm-logout-action'\"", &error);21:32
tedgInstead of calling ConsoleKit in gnome-session.21:32
alkisgThat command should be executed if LTSP_CLIENT is defined in the local environment, so that the LTSP display manager would then actually poweroff the pc21:32
tedg(CK would shutdown the server instead of the client)21:32
chrisccoulsonthis is a patch to gnome-session right?21:33
alkisgRight21:33
tedgIt's a patch to your patch :)21:33
alkisgI haven't looked at the sources yet, I was trying to put it to indicator-session, but tedg corrected me by telling it's best to go to gnome-session...21:34
chrisccoulsonoh, ok :)21:34
alkisg(and it indeed is)21:34
chrisccoulsonyes, it's probably best somewhere like that, but i don't really know enough about how LTSP works to have much of an opinion21:34
alkisgchrisccoulson: I think I could have stgraber and ogra to verify that it works21:35
alkisg(ubuntu devs)21:35
chrisccoulsonyeah, no problem:)21:35
alkisgThanks! I'll try it tomorrow, and come back with a full "case" :)21:36
alkisgThanks a lot to both of you, good night.21:37
=== kwwii_ is now known as kwwii

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!