[00:00] <jbicha> robert_ancell: hey, did seb128 ping you about renaming gnome-calculator.desktop back to gcalctool.desktop?
[00:00] <robert_ancell> jbicha, not recently
[00:03] <jbicha> robert_ancell: http://irclogs.ubuntu.com/2013/02/28/%23ubuntu-desktop.html#t17:24
[00:04] <jbicha> desrt: gconf oddities? gnome-themes-standard should be fixed if that's what you meant
[00:05] <robert_ancell> jbicha, yeah, so either distro-patch it or convince the maintainer (I've passed it on)
[00:06] <jbicha> robert_ancell: but I thought you were the maintainer... same problem for the games though
[00:07] <robert_ancell> jbicha, I was but no longer. I also don't have a good answer - as an upstream you want the names to change but I see the migration issue
[00:10] <desrt> jbicha: and gnome-shell?
[00:12] <desrt> jbicha: it declared a dependency only (as far as i can tell) because it installed a migration script
[00:35] <jbicha> desrt: ok I dropped the gconf dependency for gnome-shell & pushed to raring & the GNOME3 PPA, I guess you don't need to use tweak tool?
[00:39] <desrt> jbicha: ya... tweak tool is kinda rough
[00:39] <desrt> not sure what we can do about that
[00:44] <desrt> jbicha: actually, it looks like nothing in here is even using GConf anymore...
[00:44] <desrt> there is an abstraction for GConf, but nothing currently using it...
[00:44] <jbicha> desrt: I think it just uses gconf for backwards compatibility, I believe the maintainer uses a stable Ubuntu release instead of trying to run the alpha stuff
[00:44] <desrt> jbicha: i can't find any setting that this thing tweaks that's stored in gconf
[00:45] <desrt> at least in git master version of g-t-t
[00:48] <jbicha> yeah, you should open a bug and ask him to kill the gconf support
[01:03] <RAOF> Huh. What's caused https://launchpad.net/ubuntu/+source/libgusb to be in the ‘publication pending’ state for 24h?
[01:06] <jbicha> RAOF: https://launchpad.net/ubuntu/raring/+queue?queue_state=0
[01:16] <RAOF> jbicha: Ah, of course! Binary NEW. Silly me.
[01:17] <RAOF> The reason I uploaded that was to add the new binary package!
[04:20] <jbicha> RAOF: you got a bug and a merge proposal :) bug 1143448
[04:21] <ubot2`> Launchpad bug 1143448 in colord (Ubuntu) "colord postinst fails with "27: [: which: unexpected operator"" [Undecided,New] https://launchpad.net/bugs/1143448
[04:22]  * RAOF is curious as to how the package installed fine locally
[04:31] <RAOF> jbicha: Thanks.
[05:59] <pitti> Good morning
[06:53] <didrocks> good morning
[09:07] <Laney> hey!
[09:08] <czajkowski> ello
[09:08] <czajkowski> does anyone else find chroium a bit over sensative for scrolling atm? every time I attempt to scroll I end up on a different application
[09:10] <lifeless> czajkowski: is your mouse over the tab list ?
[09:10] <lifeless> czajkowski: or perhaps ctrl key down ?
[09:10] <czajkowski> nope
[09:10] <czajkowski> it's only happening on chromium as well which makes it a little bit more annoying
[09:10] <czajkowski> lifeless: morning btw
[09:11] <lifeless> czajkowski: morning :)
[09:12] <chrisccoulson> hi desktop team!
[09:13] <seb128> hey desktopers
[09:13] <seb128> Laney, chrisccoulson: hey, had a good w.e?
[09:13] <chrisccoulson> seb128, yeah, not too bad thanks. and you?
[09:13] <seb128> chrisccoulson, mine was quite ok, thanks ;-)
[09:14] <Laney> yeah, pretty quiet
[09:14] <chrisccoulson> ah, crap, chromium failed to link on i386 because the linker hits the virtual address space limit :/
[09:14] <chrisccoulson> so we can have chromium as long as we don't want any debug symbols and crash reports
[09:15] <seb128> chrisccoulson, still having fun with chromium I see
[09:15] <mlankhorst> :>
[09:15] <chrisccoulson> seb128, yeah, it's becoming a pain
[09:15] <mlankhorst> chrisccoulson: can't you do a frankenbuild?
[09:15] <chrisccoulson> seb128, it's too big ;)
[09:15] <mlankhorst> install 64-bits kernel and gcc with multiarch
[09:15] <mlankhorst> erm linker, probably don't need gcc of amd64
[09:15] <chrisccoulson> mlankhorst, on the archive builders?
[09:15] <mlankhorst> :P
[09:16] <mlankhorst> yeah probably a bad idea
[09:16] <chrisccoulson> that's the problem. we can't build it in the archive ;)
[09:18] <mlankhorst> does building all chromium things shared instead of linking into a single blob help?
[09:18] <mlankhorst> oh right probably not, it's just too big
[09:19] <chrisccoulson> we may have to start splitting things out of the main binary, but i suspect that's not going to be an insignificant amount of work
[09:19] <chrisccoulson> and fairly unsupportable in the long term too
[09:19] <mlankhorst> yeah I just thought you could build some part of it shared
[09:19] <mlankhorst> but even then it's probably too big
[09:19] <mlankhorst> too many c++ debug symbols
[09:19] <seb128> chrisccoulson, just build without debug symbols on i386...
[09:19] <chrisccoulson> seb128, that kind of sucks though :(
[09:19] <seb128> it does
[09:20] <seb128> but only so much we can with the resources we have
[09:20] <mlankhorst> chrisccoulson: http://code.google.com/p/chromium/wiki/LinuxFasterBuilds
[09:21] <mlankhorst> you could try the shared thing, see if it works or not
[09:21] <mlankhorst> or remove just the webkit debug symbols
[09:22] <chrisccoulson> mlankhorst, oh, i didn't know about the "component-shared_library" option. it looks like that might do what i want (still using bundled libraries but splitting them out)
[09:22] <chrisccoulson> i'll give it a try
[10:20] <Laney> xnox!
[10:20] <Laney> !xonx
[10:20] <ubot2`> Factoid 'xonx' not found
[10:21] <xnox> Laney: hola! =)
[10:21] <Laney> hello!
[10:21] <Laney> so, you know the latest upstart package from stgraber? how it moves its Xsession.d file to 99upstart?
[10:22] <xnox> Laney: yeah....
[10:22] <Laney> it overwrites all this lovely stuff (except dbus obviously): http://paste.ubuntu.com/5584714/
[10:22] <Laney> how do we go about getting it back
[10:30] <xnox> Laney: right, so even before we were chewing some of these things. let me check what it is currently doing after the move.
[10:31] <Laney> chewing?
[10:31] <Laney> those things can generally work by setting some environment variables to be available in the session
[10:32] <Laney> but I don't think making gnome-session be start on started somehugelistofthings works
[10:39] <Laney> UDS session? ^o)
[10:42] <ogra_> to produce a buglist ?
[10:49] <xnox> Laney: the point is that all of those snippets should be handled as upstart jobs and/or be sensitive to UPSTART environment variable: aka there is UPSTART environment variable and I do ship an upstart job, so let's not polute the STARTUP var, vs I have no idea about upstart and will polute STARTUP var.
[10:52] <xnox> Laney: we have gnome-session & dbus, the bits that are missing are the ssh & gpg agents.
[10:52] <Laney> and im-config
[10:52] <xnox> yes.
[10:54] <Laney> how do you express that in upstart-fu then? so that e.g. $GPG_AGENT_INFO is available to the gnome-session
[10:54] <xnox> Laney: looking at other STARTUP scripts we are also potentially missing consolekit session?! although we will be planning to move to logind at the vUDS.
[10:55] <xnox> Laney: so in user-session upstart mode we support inheriting environment and setting/modifying global environment variables for _all_ jobs from any job.
[10:55] <xnox> specifically for these type of weird jobs ;-)
[10:55] <Laney> but you have to make gnome-session wait on those jobs then don't you?
[10:56] <ogra_> you have to make X wait for them
[10:57] <ogra_> (thats why it is callled Xsession :P )
[10:57] <Laney> hmm?
[10:57] <Laney> that's done implicitly by the STARTUP mechanism
[10:57] <xnox> Laney: gnome-session already waits on dbus.
[10:57] <Laney> yes
[10:57] <Laney> but not e.g. gpg-agent
[10:57] <ogra_> Xsession is executed on startx as well as on DM startup
[10:57] <xnox> and dbus is on-startup.
[10:57] <ogra_> not on login
[10:58] <Laney> ogra_: the new user jobs stuff shortcuts out the Xsession files
[10:58] <xnox> so your gpg-agent job should be "on starting dbus" and ideally a task, thus blocking dbus job, without explicetely modifying the dbus job.
[10:58] <ogra_> right
[10:59] <Laney> oh, on starting blocks?
[11:00] <Laney> start on starting foo blocks foo until i've started?
[11:00] <Laney> and thus if I initctl set-env --global some stuff foo will see it?
[11:01] <ogra_> it blocks until the depending process emitted the "starting" event
[11:03] <Laney> ah, 'task' was a hint
[11:05] <xnox> Laney: yeah, so in the task job one would want to export all the vars. The question is whether it's fast enough. E.g. I'm not sure if dbus job will notice the new environment vars, but gnome-session will.
[11:06] <Laney> I was thinking they'd be start on starting gnome-session, not dbus
[11:06] <Laney> that would be different to the original order though
[11:06] <Laney> hmm
[11:07] <Laney> maybe not then
[11:07] <xnox> Laney: maybe. Reading cookbook it seems like task is not necessory and normal jobs can block as well.
[11:07] <xnox> http://upstart.ubuntu.com/cookbook/#block-another-job-until-yours-has-started
[11:07] <xnox> Laney: as long as nothing tries to access gpg/ssh-agents over dbus and expect dbus to know about them.....
[11:08] <xnox> which would be weird. And I don't expect order to mater. But hey it's all sequential at the moment.
[11:11] <Laney> let me try it
[11:11]  * ogra_ still wonders how we plan to do such big transitions in a rolling release
[11:12] <ogra_> stuff will break and will stay broken over a certain amount of time until we have shaken out everything
[11:13] <Laney> it's a bit tough considering that we made having users run proposed be an antigoal
[11:13] <Laney> otherwise maybe we could do such things there with migration blocks
[11:13] <Laney> maybe we just have to use PPAs more/better
[11:17] <ogra_> well, if your transition touched half of the distro PPAs will get hairy
[11:17] <ogra_> C++ ABI bump in libc anyone ?
[11:34] <Laney> so glad I've never been around for one of those
[11:37] <seb128> Laney, we can ask Ryan to break the glib api if you want some fun :p
[11:37] <Laney> ah you know I love a good transition anyway
[11:37]  * Laney snuggles ghc
[11:38] <mlankhorst> Laney: one x1.14  coming up? :D
[11:53] <Laney> xnox: seems 'task' is required
[11:53] <Laney> 'sleep 5' doesn't block startup without it
[11:53] <Laney> good news is the env variable is set in my session
[11:55] <xnox> Laney: hm. I'd think that pre-start start portion will block, the exec/script will not.
[11:55] <xnox> Laney: as the "starting" event is emitted after pre-start.
[11:56] <Laney> yeah
[11:56] <Laney> I suppose it won't work for daemons
[11:56] <Laney> as task means that it has to transition back to stopped before unblocking
[12:52] <ogra_> qengho, wow, looks like you were really close with your last chromium build
[13:01] <Sweetshark> seb128: ping?
[13:02] <xclaesse> how can I disable ALT to open the search bar into app menu ?
[13:02] <Sweetshark> seb128: any opinion on making libreoffice-core C/P/R libreoffice-filter-binfilter ?
[13:02] <xclaesse> just can't get used to it, it always open when I don't want it :(
[13:02] <seb128> Sweetshark, does anything has a versioned depends on libreoffice-filter-binfilter?
[13:03] <seb128> xclaesse, system settings -> keyboard -> keybindings -> launcher -> the bottom line
[13:04] <xclaesse> seb128, perfect, thx :)
[13:04] <seb128> xclaesse, yw
[13:04] <xclaesse> btw, when opening settings focus should go to search bar ;-)
[13:04] <Sweetshark> seb128: the 'libreoffice' metapackage used to have a dep on it. Apart from that: no.
[13:05] <seb128> Sweetshark, I would just use C,R,P, we can decide to replace it by a dummy transitional package if update-manager gets confused and that proves needed
[13:05] <seb128> but let's start with the easy solution
[13:06] <Sweetshark> seb128: k. currently its just an unversioned breaks. I guess rene want people to explicitly remove binfilter ...
[13:07] <seb128> well, C,R,P will lead to it being removed as well
[13:08] <Sweetshark> seb128: sure, but the R is a lie and -core doesnt do what binfilter did ;)
[13:10] <seb128> oh, just use the versionned breaks then...
[13:10] <ogra_> just make sure you have proted it to surfaceflinger before we switch the desktop :P
[13:10] <ogra_> https://soltesza.wordpress.com/2013/03/03/ubuntu-may-switch-to-android-technologies-to-keep-the-linux-desktop-competitive/
[14:14] <desrt> i guess we may have to teach localed about how to install packages on ubuntu as well...
[14:14] <desrt> if we want to go that route
[14:15] <seb128> desrt, rodrigo has a branch in git by then for that iirc
[14:15] <desrt> oh nice
[14:15] <seb128> well, maybe it was not localed
[14:15] <desrt> is that new work, or from before?
[14:15] <seb128> but the region capplet
[14:15] <desrt> ah
[14:16] <desrt> lennart said that he'd be willing to accept patches upstream that made my life easier
[14:16] <seb128> well, we have been looking at replacing language-selector by the region capplet for a while
[14:16] <seb128> nice
[14:16] <desrt> so if we need a package install hook or something, probably we can get it
[14:17] <seb128> excellent
[14:18] <desrt> so we have a long-term plan to move ubuntu to the upstream one?
[14:18] <desrt> fwiw, the ibus stuff is starting to get sane by 3.8
[14:18] <desrt> they realise "oh ya... maybe we actually need per-window keyboard layout after all"
[14:18] <seb128> they started to add lot of options back, that's good
[14:18] <seb128> like the "never blank the screen" is back as well
[14:18] <stgraber> Laney, xnox: right, my plan was to update the Xsession script for ssh-agent, gpg-agent, ... to be no-ops if $UPSTART is set and then have those be upstart jobs that start before gnome-session (currently "start on starting gnome-session" is good but we may have to change that if we ever want to support something other than gnome-session in there)
[14:18] <desrt> and some of the options are annoyingly changed
[14:19] <seb128> which we had as a distro patch since GNOME3
[14:19] <desrt> but in a way, better
[14:19] <desrt> like, that xkb options window is completely gone
[14:19] <seb128> desrt, long term we do plan to move to the upstream region panel yes
[14:19] <desrt> but a lot of the more common choices are moved to a better place
[14:19] <seb128> we just need to sort out the ibus story, which includes having an indicator written on our side
[14:19] <seb128> and we need to plug the langpacks somewhere in there
[14:20] <desrt> i was about to volunteer until you added the part about 'having an indicator written' :)
[14:20] <seb128> ;-)
[14:20] <desrt> anyway
[14:20] <seb128> you can try to push larsu to volunteer for that one :p
[14:20] <desrt> today i'll sort out hostnamed
[14:20] <desrt> then i'll start looking into locale stuff
[14:20] <desrt> attente: maybe you're interested as well?
[14:20] <seb128> great
[14:21] <desrt> seems like we unify a little bit more with upstream this cycle...
[14:21] <desrt> are we still planning for u-c-c?
[14:21] <seb128> desrt, btw, how do you handle the fact that ubuntu-system-services conflicts with systemd-services atm?
[14:22] <desrt> i don't know
[14:22] <seb128> k
[14:22] <desrt> i guess i uninstalled ubuntu-system-services :)
[14:22] <desrt> did we have a hostname-changer in there before?
[14:22] <seb128> I've a deb of u-s-s without the systemd stuff dropped
[14:22] <seb128> no we didn't
[14:22] <desrt> so that was another g-s-d mechanism?
[14:22] <seb128> we didn't even have the prettyname stuff
[14:22] <seb128> no
[14:22] <seb128> we just didn't have an interface to change hostnames
[14:22] <seb128> missing feature
[14:23] <desrt> ahh
[14:23] <seb128> it's not like it was the most important feature ever ;-)
[14:23] <thumper> hi seb128, desrt
[14:23] <desrt> i bet there's just a patch in there
[14:23] <desrt> that needs to be dropped
[14:23] <desrt> thumper: hey!
[14:23] <seb128> thumper, hey
[14:23] <seb128> thumper, what are you doing up at this time?
[14:23] <desrt> nice to see you around these parts :)
[14:23] <thumper> seb128: I'm in atlanta
[14:23] <seb128> desrt, u-c-c ... yes, planned, after we land GNOME 3.8
[14:23] <desrt> huh
[14:23] <thumper> juju sprint
[14:23] <seb128> thumper, I see ;-)
[14:24] <desrt> my dream come true and suddenly i am unsatisfied :)
[14:24] <seb128> lol
[14:24] <desrt> seb128: i have systemd-services and ubuntu-system-service both installed
[14:25] <desrt> no conflict here, apparently?
[14:25] <seb128> desrt, k, so there is a chance that org.freedesktop.hostname1 is claimed by u-s-s
[14:25] <seb128> that's the case for me
[14:25] <seb128> according to d-feet
[14:25] <desrt> neat
[14:26] <desrt> so we implement the interface
[14:26] <desrt> but the UI doesn't work
[14:26] <seb128> well, it works, it gets the hostname
[14:26] <seb128> we don't implement the edit part
[14:26] <desrt> ah
[14:26] <desrt> so we need also a 5th upload then
[14:26] <desrt> stop having u-s-s do this
[14:27] <desrt> otherwise it's gonna muck up hostnamed
[14:28] <seb128> desrt, yeah, let me get that upload, I did hack on that during the London sprint week
[14:28] <desrt> desktop team ppa gets an early warm-up to the week :)
[14:28] <seb128> ;-)
[14:28] <seb128> desrt, btw I might wait for gtk 3.8 to upload the new serie to raring
[14:28] <desrt> i wonder if there is conflicting policy installed by those two services
[14:28] <seb128> debian guys pointed that DND is broken with the current version
[14:28] <desrt> seb128: i had a feeling this might happen
[14:28] <desrt> that's fine
[14:29] <seb128> desrt, @conflicting policy, systemd-service Replaces u-s-s
[14:29] <seb128> desrt, so it overrides its dbus files
[14:29] <desrt> if it replaces, why do i still have both installed?
[14:29] <seb128> Replaces means "you can overwrite files"
[14:29] <desrt> huh
[14:29] <desrt> til
[14:30] <seb128> desrt, u-s-s is still needed for a few things
[14:30] <seb128> like it has a "set system proxy" interface
[14:31] <desrt> seems weird to be living in a state where one package is replacing files of another, though
[14:31] <seb128> desrt, right, that's buggy and transitional
[14:31] <seb128> desrt, https://launchpad.net/~ubuntu-desktop/+archive/ppa/+sourcepub/3009976/+listing-archive-extra
[14:32] <desrt> nice
[14:32] <seb128> desrt, with that upload they stop conflicting
[14:32] <desrt> hostnamed is weird
[14:32] <desrt> or rather, the panel is weird
[14:35] <xnox> stgraber: Laney: well there is lxde-session starter thing... maybe they all should have a common even they start on and a dummy job that emits "about-to-start-session-starting-thing".....
[14:35] <xnox> sounds ugly.
[14:36] <desrt> so i guess we somehow have a different polkit setup than fedora does
[14:36] <desrt> because on fedora the hostname is changeable as soon as the dialog opens
[14:36] <desrt> and for ubuntu it's not
[14:37] <desrt> and it seems that nobody thought to add an unlock button to that dialog incase different people have different policy
[14:37] <seb128> desrt, could be that we need to update /var/lib/polkit-1/localauthority/10-vendor.d/com.ubuntu.desktop.pkla
[14:38] <desrt> erm
[14:38] <desrt> this is somehow totally different in 3.8
[14:38] <seb128> oh right
[14:38] <seb128> the new js based polkit
[14:38] <desrt> https://bugzilla.gnome.org/show_bug.cgi?id=687772
[14:38] <ubot2`> Gnome bug 687772 in Sharing "Implement the Sharing panel designs" [Normal,Resolved: fixed]
[14:38] <seb128> the security team is making nightmares over it
[14:39] <desrt> specifically http://bugzilla-attachments.gnome.org/attachment.cgi?id=233431
[14:39] <seb128> let's make polkit depends on a js interpreter...
[14:39] <stgraber> xnox: yeah, I started to think about it a bit. Basically have an xsession-init.conf job that's "start on startup", then have this one emit "xsession SESSION=$NAME"
[14:39] <stgraber> xnox: as a result, gnome-session would then be changed to be "start on xsession SESSION=gnome-session and started dbus"
[14:40] <seb128> desrt, http://lists.freedesktop.org/archives/polkit-devel/2012-June/000368.html
[14:40] <desrt> seb128: i know about this
[14:40] <seb128> k
[14:40] <seb128> desrt, well, we didn't do that update yet either
[14:40] <desrt> we discussed it quite a lot at FOSDEM
[14:40] <desrt> the RHEL security guys flipped their lid about this as well
[14:40] <seb128> what was the outcome?
[14:40] <desrt> seems that it's a change that nobody likes
[14:40] <desrt> i don't know
[14:41] <seb128> k
[14:41] <desrt> pitti knows more about it
[14:41] <desrt> someone proposed that we may end up getting the old format back alongside the new format, somehow?
[14:41] <seb128> that's an update I'm not in a hurry to make in any case
[14:41] <desrt> but also: davidz is no longer with red hat
[14:41] <desrt> so uh...
[14:41] <seb128> oh, he's not? didn't know about that
[14:42] <desrt> ya.  got sucked into the black hole
[14:43] <pitti> desrt: I don't know much more, just that I don't like it at all either
[14:44] <pitti> desrt: oh, I wasn't aware of that either -- did he go to the big G?
[14:44] <desrt> ya
[14:44] <pitti> seb128: FYI, I updated udisks today, but not gdu yet (as this needs libdvdread in universe)
[14:45] <seb128> pitti, I saw, thanks!
[14:45] <seb128> pitti, will make those users, who are afraid about devices not being powered off on eject, happy
[14:46] <pitti> :)
[14:50] <seb128> desrt, subscribed you to https://blueprints.launchpad.net/ubuntu/+spec/foundations-1303-consolekit-logind-migration btw
[14:50] <desrt> oh.  that nightmare :)
[14:50]  * desrt is starting to notice a theme here
[14:51] <seb128> hehe
[14:51] <Laney> stgraber: Yeah, that's basically where I'm at. My upstart skills are crap enough though that I can't make these jobs work; can't get upstart to track the processes right (in the case of im-launch there's a few processes spawned for example)
[14:51] <desrt> pitti: btw: any plans for having a PID1 systemd drop-in?
[14:52] <pitti> desrt: not from my side; this is pretty much foundations team area
[14:53] <stgraber> Laney: right, I'm busy with NM/ofono this morning but I'll poke upstart again this afternoon. 1.7 was now released upstream so I need to get things in shape to land in Ubuntu now.
[14:53] <ogra_> desrt, just aftter we spent massive work on getting upstart into user session management ?
[14:53] <ogra_> verz verz unlikely
[14:53] <ogra_> *very even
[14:54] <desrt> upstart managing the user session is..... annoying
[14:55] <ogra_> desrt, how so ?
[14:55] <ogra_> desrt, systemd managing them is better ?
[14:55] <desrt> ogra_: it's going to cause us problems going forward
[14:55] <desrt> like when dbus-in-the-kernel lands
[14:55] <desrt> will be a systemd-based feature
[14:55] <ogra_> upstart will learn to cope
[14:55] <desrt> and, like usual, we will be playing catch-up
[14:55] <desrt> ogra_: problem is that upstart tends to cope ~2 years after
[14:56] <desrt> look how long logind is taking, as an example
[14:56] <ogra_> upstart is far ahead in other areas
[14:56] <ogra_> you dont really want systemd on a production server for example
[14:56] <desrt> i don't know of any
[14:56] <ogra_> afaik not even RH does that yet
[14:56] <desrt> people keep making hand-wavy arguments around this point, but i don't understand why
[14:57] <ogra_> because systemd is far from being done as a generic init
[14:57] <ogra_> there are many cases where it cant cope atrm
[14:57] <ogra_> and which lennart simply ignores
[14:57] <desrt> i keep hearing lots of statements like 'many cases'
[14:58] <ogra_>  /usr on a foreign machine or different disk is a very typical server setup in datacenters
[14:58] <ogra_> systemd cant handle that
[14:58] <seb128> desrt, ogra_: let's not have another systemd/upstart argument today
[14:58] <ogra_> yeah
[14:58]  * ogra_ stops :) 
[14:58] <seb128> thanks ;-)
[15:01] <pitti> ogra_: (also, it's not true)
[15:01] <desrt> pitti: stop :p
[15:01]  * pitti STFU, sorry
[15:02] <ogra_> we'll go all android anyway ... so that discussion is moot :P
[15:02] <pitti> ah right -- time to dust off my ancient java sk1llz from uni? :-)
[15:02] <ogra_> hehe
[15:16] <seb128> chrisccoulson, bah, I deleted my firefox's corruption screenshot from the other day ... if you have one please add it to https://bugs.launchpad.net/ubuntu/+source/xserver-xorg-video-intel/+bug/1144558
[15:16] <ubot2`> Launchpad bug 1144558 in xserver-xorg-video-intel "Images corruption in firefox when using "sna"" [Undecided,New]
[15:17] <chrisccoulson> seb128, sure. 1 second
[15:17] <seb128> chrisccoulson, thanks
[15:18]  * seb128 is out for some exercice, be back in ~1h
[15:40] <qengho> Hey guys, it turns out we're moving to Android SurfaceFlinger instead of Wayland.
[15:40] <qengho> http://soltesza.wordpress.com/2013/03/03/ubuntu-may-switch-to-android-technologies-to-keep-the-linux-desktop-competitive/
[15:41] <qengho> It's deductions like this that make me scoff at Sherlock Holmes stories. There are so so many places to go wrong.
[15:44] <desrt> seb128: add a 6th package to your list :)
[15:44] <desrt> policykit-desktop-privileges needs an update
[15:46] <desrt> pitti: where is this package maintained?
[15:46] <pitti> desrt: just apt-get source or UDD
[15:46] <pitti> no magic
[15:46] <desrt> huh
[15:46] <desrt> it's maintained via source package uploads?
[15:46] <desrt> neat version control system :)
[15:46] <pitti> desrt: bzr branch ubuntu:policykit-desktop-privileges :)
[15:46] <pitti> (IOW, UDD)
[15:47] <desrt> iow? udd?
[15:47]  * desrt is not hip with the lingo
[15:47] <pitti> "in other words", "ubuntu distributed development", sorry
[15:50] <mterry> seb128, FYI: bug 1137907 should keep GTK+ 3.7 in the desktop PPA for now  (we were planning to move that to raring eventually right?)
[15:50] <ubot2`> Launchpad bug 1137907 in gnome-screenshot (Ubuntu) "After pressing my Print Screen button, screen turns white and stays white" [Undecided,Confirmed] https://launchpad.net/bugs/1137907
[15:56] <larsu> why does ubuntuone-client depend on apport?
[15:57] <desrt> larsu: to help you uninstall things easier when you setup a new ubuntu system =)
[15:58] <larsu> desrt: erm, lol?!
[15:58]  * larsu wants u1, but not apport
[15:58] <desrt> oh ya.  you love your u1 :)
[15:58] <ogra_> just disable it in /etc/default/apport
[15:59] <ogra_> its not like it eats massive amnounts of disk space
[15:59] <larsu> ogra_: ah, didn't know about that one. Thanks!
[15:59] <larsu> still, the dependency is a bit weird...
[16:00] <ogra_> file a bug to downgrade it to a recommends
[16:00] <larsu> doing so right now :)
[16:02] <desrt> polkit is smart.
[16:02]  * desrt is happy
[16:04] <desrt> seb128: do you mind if i do uploads to desktop team ppa?
[16:06] <desrt> oh.  he's doing that weird being outside thing
[16:07] <Laney> you won't be able to - you're not in the ubuntu-desktop team
[16:07] <desrt> interesting
[16:08] <desrt> seb128: http://fpaste.org/QK5O/ for when you get back :)
[16:08] <Laney> yeah, and it's a team that grants upload rights
[16:08] <Laney> so ... there's A Procedure
[16:09] <desrt> my Normal Procedure is 'ping seb' :)
[16:09]  * desrt will keep doing that
[16:09] <Laney> if you make your own PPA then we can copy from that into the desktop one
[16:09] <Laney> let me upload that thing now
[16:09] <desrt> nah... there's no point in that
[16:10] <desrt> we're using the desktop team PPA to stage this changeset before tossing it in the distro already
[16:10] <desrt> no need to add yet another layer of indirection
[16:13] <Laney> ok done
[16:13] <Laney> I gave it a PPA version
[16:15] <tkamppeter> I have some questions to the online UDS. Can I participate without G+ account? Or without webcam (audio-only)?
[16:16] <ogra_> i guess to run a session you need G+
[16:16] <ogra_> for partitcipating it should work via IRC and audio (guessing here though)
[16:16] <Laney> there's some thing to get audio out of hangouts?
[16:17] <ogra_> oh, that i dont know ... but i would expect the old ways to still work
[16:18] <ogra_> worst case through an analog speaker jack with some volume levelling into the input jack i bet
[16:18] <ogra_> (to then forward it to icecast)
[16:18] <qengho> Laney: you could "pacat ... >capture".
[16:18] <Laney> yes I am aware there are ways
[16:18] <Laney> someone will need to set this up though
[16:19] <Laney> so that third parties can listen in on the streams
[16:19] <ogra_> i would have expected such a setup to be in the default infrastructure though
[16:19] <Laney> I suspect this is work that hasn't been done
[16:25] <desrt> Laney: thanks for the upload
[16:25] <Laney> np
[16:29] <larsu> pitti: I'm getting undeterministic errors when running my test suite on armhf.  Using dbusmock and pygi, two of your specialties ;)
[16:30] <pitti> larsu: great timing! five seconds ago I pushed enter on ftpadmin install pygobject-3.7.91.tar.xz :)
[16:30] <larsu> haha
[16:31] <larsu> pitti: any clue what could be wrong?  The errors range from endless loops to segvs to "unsupported ancillary data" to "assertion `GI_IS_ARG_INFO (info)' failed"
[16:31] <pitti> larsu: not really; pygobject's test suite is happy on armhf (it would FTBFS if it wasn't)
[16:32] <pitti> larsu: so I guess I'd need some reproducer which I could run on the n7
[16:32] <pitti> or the porter box
[16:32] <larsu> pitti: I'm running it in that qemu pbuilder thing...
[16:33] <pitti> ooh
[16:33] <larsu> pitti: same as the one on jenkins (at least I think so)
[16:33] <pitti> there are lots of bugs in those; better run it on real iron first to confirm
[16:34] <larsu> pitti: ah!  Good to know.  /me updates his galaxy nexus
[16:34] <larsu> thanks
[16:41] <seb128> re
[16:42] <seb128> desrt, thanks ... did Laney sponsor it?
[16:42] <Laney> yeah
[16:42] <seb128> mterry, that's GTK? weird
[16:42] <seb128> mterry, I plan to keep it in the ppa until 3.8 release, there are dnd issues as well
[16:42] <seb128> Laney, thanks
[16:54] <desrt> seb128: looks like we're at 6 uploads now :)
[16:55] <seb128> desrt, 3 done on the 6 right?
[16:55] <desrt> yes.  i suppose so.
[16:55] <desrt> indicator-datetime will be the interesting one
[16:55] <seb128> shim, ubuntu-system-services, privileges
[16:55] <desrt> we have to negotiate what you want to do there
[16:55] <seb128> datetime, g-c-c, g-s-s to do?
[16:56] <seb128> oh? why?
[16:56] <desrt> yup
[16:56] <desrt> well
[16:56] <desrt> (waiting for LP)
[16:56] <desrt> https://code.launchpad.net/~desrt/indicator-datetime/timedated
[16:56] <desrt> this is what you want to ship
[16:56] <desrt> but i didn't MR it yet because it doesn't make sense until the g-s-d/g-c-c changes are ready to land
[16:57] <desrt> i don't want it to autoland/autoupload
[16:57] <larsu> desrt: feel free to MR if the code is done.  With a note not to merge it yet
[16:57] <seb128> well, there is no "negotiation", it's just "synchronization"
[16:57] <larsu> this will get some more eyeballs on the code
[16:57] <desrt> okay
[16:57] <desrt> wtf
[16:58] <desrt> i have to 2fa to MR?
[16:58] <desrt> arghgghhg
[16:58]  * desrt looks for his keys
[16:58] <seb128> desrt, it will be easy to sync g-c-c/g-s-d/indicator for upload once systemd-service is promoted and the shim is in as well
[17:00] <desrt> okay.  merge proposed
[17:00] <desrt> seb128: meanwhile, though, i'd like to get the work tested in the desktop team PPA
[17:00] <desrt> which is what i'm asking about
[17:00] <desrt> should we just vendor-patch it, or what?
[17:02] <larsu> yes
[17:02] <seb128> desrt, I will build/upload the 3 remaining ones
[17:02] <desrt> ok
[17:02] <desrt> seb128: i'm negotiating with upstream about the hostname thing
[17:03] <desrt> since they changed stuff around here for 3.8
[17:03] <seb128> what is needed to change with your uploads n6?
[17:03] <desrt> i think the way forward looks like: we remove the entry from the info/details panel
[17:03] <seb128> oh, ok
[17:03] <desrt> then it's accessible in the new sharing panel
[17:03] <desrt> which will (hopefully) get an unlock button
[17:03] <seb128> great
[17:04] <desrt> btw: did i mention that polkit is really nice?
[17:04] <seb128> why is hostname part of sharing?
[17:04] <desrt> seb128: what is a hostname if not "that thing that other computers call you"?
[17:04] <desrt> definitely related to sharing imho
[17:04] <seb128> "the name displayed on the login screen"
[17:04] <desrt> lol
[17:04] <seb128> ;-)
[17:04] <desrt> so let's put hostname in lightdm settings :)
[17:05] <seb128> would be better than in sharing :p
[17:05] <seb128> it would be somewhat "infos about this computer"
[17:05] <desrt> which is where it is now
[17:05] <seb128> right, which makes sense
[17:05] <seb128> no?
[17:05] <desrt> the other thing we could do is to make this problem go away entirely
[17:05] <desrt> we allow admins to change the timezone without authentication
[17:05] <desrt> why not hostname?
[17:06] <jbicha_> the sharing panel is designed for a Red Hat system not a Debian one
[17:06] <desrt> jbicha_: what does that mean?
[17:07] <jbicha_> desrt: for instance, the design of the ssh section assumes that openssh-server is pre-installed & upstream isn't interested in packagekit integration for installing it; they think distros should include all those pieces
[17:08] <jbicha> we could probably still get it to work and it definitely is an improvement from how we do things now
[17:11] <jbicha> mpt isn't a fan of the listbox either
[18:08] <Sweetshark> seb128, bdrung: trying to debug/fix https://launchpad.net/ubuntu/+source/libcmis/0.3.1-1ubuntu1/+build/4337070. So far I found: the test crashes with a corrupted stack on raring. Now recompiling on precise to find its a regression.
[18:08] <Sweetshark> smells like a toolchain f^*&up though ...
[18:15] <BigWhale> Just reading about Mir. Wow.
[18:18]  * didrocks waves good evening
[18:18] <didrocks> Mir opened up, time for rest! :)
[18:29] <seb128> Sweetshark, stopping looking in direction of chrisccoulson while mentioning toolchain issues
[18:29] <seb128> ;-)
[18:29] <Sweetshark> chrisccoulson: ....
[18:30] <Sweetshark> chrisccoulson: O^O
[18:30] <Sweetshark> chrisccoulson: TOOLCHAIN!
[18:30] <Sweetshark> seb128: and yes, its a toolchain issue. The package compiles and tests just fine on precise.
[18:31] <seb128> Sweetshark, try bugging doko ;-)
[18:31] <chrisccoulson> my ears are burning
[18:31] <Sweetshark> seb128: well it could also be a dep being broken, but with a stack all messed up Im rather confident its gcc and friends here ...
[18:32]  * chrisccoulson cuts off ears
[18:32] <seb128> chrisccoulson, :-(
[18:32] <chrisccoulson> heh :)
[18:33]  * Sweetshark writes 'toolchain' on a big piece of paper and walks around waving it like a box ring girl.
[18:33]  * mlankhorst puts a I heart libreoffice sticker on Sweetshark's back
[18:34] <seb128> cyphermox, mterry: can one of you review/ack https://code.launchpad.net/~larsu/ubuntu-themes/fix-separators/+merge/151330 ?
[18:34] <seb128> it's one of the blockers for the new gtk landing
[18:37] <mterry> seb128, OK
[18:37] <mterry> seb128, do you know if this works fine on 3.6 too?
[18:37] <seb128> mterry, I think it does
[18:37] <seb128> larsu, ^
[18:42] <jbicha> mir? you should name your systemd shim 'sru' then
[18:43] <Sweetshark> mlankhorsts "I heart libreoffice" soothes Sweetshark and he stops teasing chrisccoulson
[18:43] <mdeslaur> jbicha: the mir MIR may get a bit confusing :)
[18:43] <mterry> jbicha, :)  the Mir team insists strongly on the Mir spelling (i.e. a proper noun like the space station), to avoid confusion with things like MIRs
[18:43] <chrisccoulson> mdeslaur, which mir team do you subscribe?
[18:44] <Sweetshark> and that consolekit replacement should be renamed ppa asap.
[18:44] <jbicha> mterry: oh it's case sensitive? lol
[18:45] <mterry> jbicha, I didn't name it, I'm not defending it.  Just saying  :)
[18:45] <sarnold> chrisccoulson: you laugh but I damn near assigned the display server people to handle the next step of Sweetshark's libmspub MIR...
[18:45] <mdeslaur> hahaha
[18:45] <chrisccoulson> lol
[18:45] <jbicha> wouldn't the clear choice have been UDS? Unity Display Server
[18:46] <Sweetshark> jbicha: true dat
[18:47] <jbicha> we'll have UDS every day! or something...
[18:47] <Sweetshark> jbicha: we need virus scanner on linux then though ...
[18:48] <Sweetshark> jbicha: I dont want to get ubuflu from by display server
[18:48] <sarnold> you've got to catch it from _something_, might as well be your display server.. :)
[18:53] <seb128> desrt, still around?
[18:53] <seb128> desrt, should the indicator changes be enough to get the indicator-datetime's panel to work by itself?
[19:00] <Chucrute301> canonical killed wayland?
[19:03] <dobey> no?
[19:04] <bryce> no.
[19:07] <xnox> Chucrute301: we didn't use it yet, and are not planning to use it.
[19:07] <xnox> Chucrute301: wayland can happily continue developing =)
[19:23] <desrt> seb128: yes.
[19:23] <desrt> seb128: (sorry -- lunch)
[19:24] <desrt> seb128: of course, you need systemd-shim as well
[19:24] <desrt> seb128: and probably also need new g-c-c for the polkit file changes
[19:30] <larsu> mterry, seb128: I tesed in on 3.6 as well, seems t owork. Would appreciate someone else testing it again though
[19:31] <mterry> larsu, OK.  Works fine in 3.7 for me, will test in 3.6 in a bit
[19:37] <jbicha> desrt: I plan to look at it tonight, do we need to install systemd-services too?
[19:38] <larsu> mterry: thanks :)
[19:40] <desrt> jbicha: yes.
[19:40] <desrt> there are 6 things you need to get this working
[19:40] <desrt> well, 7, including systemd-services -- but 6 "patched" packages you need
[19:40] <seb128> desrt, back from dinner
[19:41] <seb128> desrt, ok, I just updated indicator-datetime and I've the lock button in the panel ... there is an auth file in g-c-c?
[19:41] <desrt> seb128: yes.
[19:41] <desrt> and our revert-mechanism patch gets rid of it
[19:41] <seb128> desrt, k, I overlooked that piece, I though g-c-c would be only for the upstream panel
[19:41] <seb128> desrt, thanks, building that next ;-)
[19:41] <desrt> i'm using the policy file from the upstream panel
[19:41] <desrt> we could add our own, i guess
[19:42] <desrt> but i didn't see the point since it's exactly the same as (for now) we depend on g-c-c for the panel to work anyway
[19:42] <seb128> I don't see an issue sharing
[19:42] <desrt> good
[19:43] <desrt> the reason polkit is awesome:
[19:43] <desrt> g-c-c installs this policy file that contains no real policy of its own
[19:43] <desrt> but a metapolicy that "implies" all the necessary systemd policies for everything you would want to do with datetime
[19:43] <desrt> set ntp, set timezone, set time, etc.
[19:43] <desrt> (each of which is a different action for systemd)
[19:43] <desrt> this g-c-c file unifies them so that we can have a single unlock button
[19:43] <desrt> okay... so far so good
[19:44] <desrt> but what about in the case that we don't specifically have the unlock button on the meta-policy?  like for selecting the timezone by clicking on it in the indicator?
[19:44] <desrt> turns out polkit is smart enough to deal with this case
[19:44] <desrt> it searches not just for the requested permission but any permission that may imply it
[19:44] <desrt> and notices that, as an admin user, i have this permission enabled
[19:44] <desrt> -> no dialog
[19:44] <desrt> total win
[19:45] <seb128> nice
[19:47] <desrt> seb128: btw: systemd-shim is going to need a security audit
[19:47] <seb128> desrt, yeah, anything going through main promotion gets one
[19:47] <desrt> okay.  good.
[19:56] <mterry> larsu, set a commit message on the style branch, and you can set it to Approved
[19:56] <mterry> larsu, I commented that it works for me
[19:57] <larsu> mterry: will do, thanks a lot
[19:59] <larsu> mterry: there's another blocker for 3.8: https://code.launchpad.net/~larsu/unity/panel-service-menu-style-fix-again/+merge/151329
[20:00] <larsu> if you are bored ;)
[20:00] <mterry> hm
[20:01] <mterry> larsu, ah... I wondered why I had lost highlight-on-hover
[20:02] <mterry> larsu, still, I'll let cimi do that review
[20:02] <larsu> mterry: yeah, I guess that makes sense.  I hope he'll get around to it soonish
[20:03] <seb128> larsu, mterry: I will ping him this week if needed, I didn't ping mterry/cyphermox on this one on purpose, seems like somebody from the unity team should review
[20:03] <seb128> mterry, thanks for getting the other one sorted ;-)
[20:04] <seb128> mterry, oh, and thanks for the gtk bug, I hit it the other day while taking a screenshot and ranted about compiz :p
[20:04] <larsu> seb128: yeah - I figured that mterry was on a roll ;)
[20:06] <mterry> seb128, :)  it finally annoyed me enough to investigate
[20:27] <mterry> seb128, where is that update-manager bug you filed?  I was going to work on it
[20:32] <seb128> mterry, https://bugs.launchpad.net/ubuntu/+source/update-manager/+bug/1145157
[20:32] <ubot2`> Launchpad bug 1145157 in update-manager "should use icons in /usr/share/app-install/icons" [Undecided,New]
[20:34] <mterry> seb128, thanks
[20:34] <seb128> mterry, thank *you*! ;-)
[20:42] <desrt> seb128: are you going to push the changed panel to desktop-team ppa despite the problem, or will you wait?
[20:43] <seb128> desrt, will push anyway
[20:43] <seb128> the issue is minor enough for that ppa
[20:43] <desrt> i have a patch.... testing it now, locally
[20:44] <seb128> limited usecase and easy to workaround
[20:44] <seb128> desrt, I will do the uploads tomorrow morning
[20:44] <seb128> so feel free to just email mail the patch
[20:44] <seb128> I'm about to call it a day
[20:44] <desrt> k
[20:44] <desrt> i figured as much
[20:49] <notgary> Does anyone in here know how to build egg-list-box using jhbuild, and would be willing to give me a hand? I'm trying it myself but am getting this error (http://paste.ubuntu.com/5586215/) and don't know what it means.
[20:58] <jbicha> notgary: install autotools-dev?
[21:00] <notgary> jbicha, already installed
[21:21] <desrt> seb128: got the fix now...
[21:23] <seb128> desrt, k, email still, I'm out of work mode (laptop with IRC running next to TV) ;-)
[21:23] <desrt> seb128: it's on the same branch
[21:25] <seb128> k, good, I will git it tomorrow
[21:25] <desrt> i guess you didn't do the upload after all?
[21:27] <seb128> desrt, no, I didn't get to g-s-d, will do the set tomorrow morning
[21:27] <desrt> k
[21:28]  * desrt selects the overnight delivery option =)
[22:08] <desrt> seb128: need a patched systemd upload too now :)
[22:09]  * desrt just added the CanNTP property at charles' request (to implement the insensitivity of the ntp option in the UI)
[22:10] <charles> \o/
[22:10] <desrt> charles: it means i need a new patch for you, though.... that comes next.
[22:19] <desrt> Laney: hey.  still around?
[22:19]  * desrt could use an upload :)
[22:23] <desrt> http://fpaste.org/8LOk/ if anyone is still around...
[22:27] <desrt> seb128: are you sure you don't want to do more work tonight? :)
[22:27] <seb128> desrt, what do you need? ;-)
[22:27] <desrt> systemd upload
[22:27] <desrt> with http://fpaste.org/8LOk/
[22:28] <seb128> to the ppa or the archive?
[22:28] <desrt> ppa first please
[22:28] <desrt> i sent the patch upstream as well and lennart said on IRC that he wants the feature
[22:28] <desrt> and it will make charles happier to merge my indicator-datetime work
[22:30] <seb128_> bah, another dsl disconnect
[22:31]  * desrt was under the impression that DSL in france is awesome
[22:31] <sarnold> seb128_: < desrt> ppa first please < desrt> i sent the patch upstream as well and lennart said on IRC that he wants the feature < desrt> and it will make charles happier to merge my indicator-datetime work
[22:31] <charles> desrt: well it's not a showstopper, the regression is minor. But for sure I appreciate you cooking up the revised patch, it's better to avoid the regression
[22:32] <desrt> charles: patching indicator-datetime now...
[22:32] <desrt> then i'm done for the day
[22:32] <desrt> since we're up to _8_ modified packages needing to land for this change
[22:33] <charles> the only other issue I had with your patch isn't the patch's fault at all... it's i-datetime's lack of test coverage. Even if you were adding  timedated tests, there are no previous ntp tests for you to modify :(
[22:34] <charles> that's one thing I'm going to try & pick up during the i-datetime GMenuification
[22:34] <desrt> cool.  thanks :)
[22:34] <seb128__> desrt, sorry, got disconnected again ... cool what?
[22:35] <desrt> seb128__: charles said he will add some testcases to i-datetime when he does GMenuModelification
[22:35] <seb128__> oh, nice
[22:35] <seb128> desrt, https://launchpad.net/~ubuntu-desktop/+archive/ppa/+sourcepub/3010579/+listing-archive-extra
[22:36] <charles> seb128: > the only other issue I had with your patch isn't the patch's fault at all... it's i-datetime's lack of test coverage. Even if you were adding timedated tests, there are no previous ntp tests for you to modify :(
[22:36] <desrt> seb128: nice!
[22:36] <seb128> charles, the indicators stack could do with some extra testing indeed
[22:37]  * desrt tests his CanNTP changes
[22:37] <charles> seb128: > that's one thing I'm going to try & pick up during the i-datetime GMenuification
[22:37] <charles> :)
[22:37] <desrt> charles: any idea what's up with this?
[22:37] <desrt>  /usr/bin/ld.bfd.real: libgtest.a(gtest-all.o): undefined reference to symbol 'pthread_key_delete@@GLIBC_2.2.5'
[22:38] <desrt> this happens whenever i try to build the i-datetime package -- i end up having to disable the tests :p
[22:40] <charles> desrt: alesage was experiencing that earlier today with a handful of indicators, too
[22:40] <charles> desrt: my first guess is one of the .pc files changed s.t. -lpthread isn't in LDFLAGS anymore?
[22:41] <alesage> cyphermox is working on a fix, bug here https://bugs.launchpad.net/indicator-datetime/+bug/1126362
[22:41] <ubot2`> Launchpad bug 1126362 in indicator-session "Jenkins build failure on gtest "undefined reference to symbol 'pthread_key_delete@@GLIBC_2.2.5'"" [Critical,New]
[22:41] <alesage> desrt ^^
[22:41] <cyphermox> aye
[22:41] <charles> I was kind of surprised it's in glib's pc's .priv ldflags but not in the public one
[22:41] <charles> *ones
[22:41] <cyphermox> unrelated to glib
[22:42] <desrt> charles: that makes a lot of sense...
[22:42] <jbicha> desrt: you're learning packaging?
[22:42] <cyphermox> charles: alesage: it should be in raring-proposed now, so it will be able to build successfully tonight
[22:42] <desrt> if you use the pthread API directly then you _must_ explicitly link to -lpthread
[22:42] <desrt> instead of relying on another library to pull it in
[22:42] <desrt> jbicha: no... not really
[22:45] <cyphermox> desrt: pull xorg-gtest from raring-proposed
[22:46] <desrt> seb128: have a new systemd-shim to upload as well :)
[22:48] <desrt> http://people.gnome.org/~ryanl/systemd-shim-0.0.tar.xz again
[22:49] <desrt> charles: i pushed the CanNTP patch to the same branch again if you want to re-review
[22:49] <charles> desrt, will do
[22:49] <seb128> desrt, k
[22:50]  * desrt is gonna head out early.  thanks very much for all the uploading/reviewing :)
[22:55] <charles> desrt, did you mean to add that second call to toggle_ntp?
[22:56] <charles> desrt, it looks like the one on line 261 is a copy-paste side effect
[22:56] <charles> ah, you've headed out early & I'm talking to an idling account. I'll ask in the MP instead...
[22:58] <mterry> attente, sorry to keep harping on UBUNTU_MENUPROXY, but an empty UBUNTU_MENUPROXY should be treated as disabled too.   (in appmenu-gtk world, it is normally defined to "libappmenu.so", so anything that isn't that exact string had the effect of disabling the menu code
[22:59] <attente> mterry: i just spent today realizing that the gtk 2 and 3 which are in the desktop ppa still have the menu proxy patch applied
[23:00] <mterry> attente, are we currently running both then?   :)
[23:00] <attente> apparently so :(
[23:01] <attente> anyways, i did a fix to look for UBUNTU_MENUPROXY instead of NO_UNITY_GTK_MODULE
[23:01] <attente> i'm not sure when it will get into the ppa
[23:01] <attente> by empty i assume you mean "" and not NULL
[23:02] <attente> on NULL UBUNTU_MENUPROXY, we'll treat it as if UBUNTU_MENUPROXY is set to something non-empty
[23:02] <attente> let me just fix this now
[23:02] <mterry> attente, right, sorry.  NULL should probably be the normal case of enabled.   But if someone is going to the trouble of defining it to "", they likely want it off
[23:02] <desrt> charles: okay.  fixed.
[23:03] <mterry> attente, by "we'll treat it as if UBUNTU_MENUPROXY is set to something non-empty", I understand you (and the code) to mean, "we'll treat it as a request to enable the menu proxy"
[23:03] <desrt> really leaving now :)
[23:04] <mterry> attente, that is, I just want you to add a check for "" in your is_true function
[23:04] <attente> mterry: sorry, yes, exactly
[23:05] <mterry> attente, I like the variety of ways you allow people to say "no please".  You should add some translations in there too  ;)
[23:06] <attente> mterry: was considering adding "awhellnaw" :)
[23:27] <mterry> :)
[23:28] <chrisccoulson> ah, gdb, stop screwing with me