User1: hello, i updated bug 527061
User2: Launchpad bug 527061 in indicator-application "python bindings for libappindicator don't allow to change icon-theme-path (affects: 2)" [Undecided,Confirmed] https://launchpad.net/bugs/527061
User1: i'm still unsure on how to test if this actually works, make&&make install the bindings doesn't produce any difference..
User3: morning seb128
User4: hey jcastro
User4: how are you today?
User3: good
User3: did you see Kalle ported Tracker to a-i?
User4: no I didn't
User3: https://bugs.edge.launchpad.net/ubuntu/+source/tracker/+bug/497912
User4: nice ;-)
User3: !!!
User2: Launchpad bug 497912 in tracker (Ubuntu) "Support Application Indicators (affects: 1)" [Low,Triaged]
User3: User4 so what's the plan for g-s-d?
User3: polkit-1-gnome is also ready!
User4: g-s-d relies on those libgnomekbd refactoring which neither pitti or I like to get in distro
User4: I guess I will have to sit down and look at this one this week
User3: ok
User5: The code could basically be moved from libgnomekbd to g-s-d
User6: kklimonda: you're afk, but you'll probably read your pings: the text field in the Me Menu also sets your IM status after all. I was wrong.
User3: User5 whichever makes it easier to put it in the distro I would guess. seb128?
User4: jpetersen, I've not looked at the details but your change is change the way the code is structured a bit to make easier to share code between appindicator and notification?
User5: for upstream it should be in libgnomekbd but for lucid just patching g-s-d would maybe be easier
User4: right that's what I was thinking
User4: could you work on that change?
User4: I would prefer a code copy rather than a redesign
User4: and better to not make those api public if not required
User5: seb128, yes ok i will do
User4: thanks
User4: we also need to make sure to not break the screensaver lock screen
User7: User8 would Bug #391414 be addressed for Lucid?  seems to be an a11y problem
User2: Launchpad bug 391414 in hundredpapercuts "Unable to shutdown system from GNOME using keyboard (affects: 1)" [Undecided,New] https://launchpad.net/bugs/391414
 * vish thinking of assigning milestone for Lucid
User8: User7 Yes, please do.
User7: User8 shall i assign it to you?
User8: User7 No, all bugs should always be assigned to kenvandine, it makes him feel loved ;)
User8: User7 Yeah, to me :)
User7: hehe ;)
 * vish first searches for dupes
User9: hi jpetersen, I've been talking to jcastro about a translation issue from the Rhythmbox indicator and he told me you'd be the person to ask.
User9: Here's the question:
User5: hi dpm
User9: hi :)
User9: The "Show Rhythmbox" message on http://imagebin.ca/view/2Vubpv2T.html appears untranslated, and I'd like to find out why it's not translatable. The string does not seem to come from Rhythmbox itself. Any idea where it comes from?
User5: dpm, it is from plugins/status-icon/rb-status-icon-plugin.c
User9: ah, thanks jpetersen, let me have a look...
User5: dpm, hm the app-indicator patch changed it from "Show Music Player" to "Show Rhythmbox", that was in the original patch
User5: dpm, I am not sure why it was changed
User9: I'm just looking at why the Show Rhythmbox message does not appear in the Ubuntu translation template, perhaps it was not marked as translatable in the patch
User5: dpm, it is marked translatable
User9: weird, the only similar message is "_Show Rhythmbox" in the Ubuntu template -> https://translations.launchpad.net/ubuntu/lucid/+source/rhythmbox/+pots/rhythmbox/ca/+translate?batch=10&show=all&search=show+rhythmbox
User5: dpm, yes that is the right one
User9: it's not, the one shown does not have the accelerator -> "Show Rhythmbox" <> "_Show Rhythmbox"
User5: dpm, it seems there seems to be a problem with accelerators in the indicator applet
User5: dpm, but it is that string
User9: jpetersen, (sorry, I was on the phone). Ah, ok. Do you know if the problem with accelerators a known bug? If so, I'll subscribe to it, otherwise I'll file a new one, I just want to make sure that that string's translations are loaded
User5: dpm, I have not found any bug
User9: jpetersen, no worries, if it's actually that string, and if it's discarding the accelerator, I'll file a new one. Which source package would be the best to file it against?
User5: indicator-application
User9: thanks a lot for your help jpetersen!
User5: dpm, your wlecome
User10: Hi guys
User10: Back from my appointments
User3: User10 I believe pitti had some questions outstanding on the vino work
User10: Ok
User3: User10 how's brasero coming along?
User10: No comments yet from upstream
User10: Fine, got one last issue if I don't figure it out soon I'll reach out for some help
User3: User10 oh right, you needed a GTK guru right?
User10: in general, yes, but for the specific problem now for brasero it is more of figuring out the right callback
User3: perhaps jpetersen?
User3: we need a box with glass and jpetersend in it ... "Break in case of App Indicator Emergency"
User10: Heh
User10: Give me a few minutes and I'll see if I need help this time
 * jcastro nods
User5: jcastro, Nafai what is the problem?
User10: User5 not an immediate problem at the moment, but are you going to be around for a while?
User5: Nafai, yes I will have dinner soon, but I will continue to work for a bit after that
User10: Ok, I'll try to ping you quickly if I need help today
User10: thanks for being available
User5: Nafai, ok :)
User5: your welcome
User3: User6 hey, did you know we had a karmic app-indicator ppa already?
User6: User3 no :P
User6: Where?
User3: it's listed on the wiki page
User6: lol
User3: heh, it's been there since like December. :D
User3: for upstreams who want to play but don't want to run Lucid
User1: tedg, (sorry for the pinging) about bug 527061 , i posted a new diff. however, today i managed to correctly build the whole stuff, and seems like the 001 diff works correctly while the new one (002) doesn't. unfortunately there's little (read: no) documentation on bindings creations so i'm walking in the dark (i don't know how duplicate a constructor). any chance some expert can take a look at this? anyway, the 001 seems fine to me, if you wanna test it
User2: Launchpad bug 527061 in indicator-application "python bindings for libappindicator don't allow to change icon-theme-path (affects: 2)" [Undecided,Confirmed] https://launchpad.net/bugs/527061
User8: User1 I'm not sure how to do that in Python either.  kenvandine, do you know how to overload a constructor?
User1: well, shouldn't be safe to use *_new_from_model? with plain *_new that path is set to Null, so there's little difference i think
User10: You don't overload the constructor in Python
User10: What I would do is add an optional parameter to __init__
User1: Nafai, but the defs file allows to specify optional parameters for the method from what i've read, and that's what i used
User1: it's just i'm using _new_from_model instead of plain _new when creating a new appindicator object, and i think there should be no difference (even if i didn't read the source, i admit)
 * Nafai nods
User1: using _new sets that parameter to Null already
User1: sorry if i'm bothering too much, but i really need that upstreamed in order to hax icons in the indicator :p
User8: User1 I don't think it's an issue, just no one is quite sure how it should work.
User1: that sounds very reassuring
User1: heh
User1: anyway, here's my reference http://live.gnome.org/PyGTK/WhatsNew28#Hand-written_constructors_need_to_be_updated
User1: notice that appindicator seems using some going-for-deprecation syntax for bindings creation, but that's another story
User1: anyway, if there's something i can do to speed up the process let me know, i fear about python bindings be not ready for lucid..
User10: bbiab, lunch
User3: User6 that was awesome!
 * qense blushes
User11: I am going to leave for the day
User11: bye bye
User10: User3 brasero patch attached and ready for review!
User10: User3 should I go back and separate the vino patch into two patches as suggested by pitti, or work on gnome-bluetooth first?
User3: User10 pitti's work first
User10: okay, figured as much
User3: good job on brasero btw!
User10: thanks
User10: Is the opportunistic developer week going well?  I've forgotten to pay attention today :(
User3: yeah, qense ran an app indicator session
User10: awesome
User10: He rocks
User12: seg|ars, no, it's a practical joke
User12: Feel free to play along or dismiss it at your choice
User12: seg|ars, I'm sure you could think of some earnest pledge to support all #twitterpremium features in Gwibber 3.0
User12: seg|ars, beautiful
User12: seg|ars, misspelled the hashtag :-(
User12: Yeah, isn't it cool?
ayatana
Ubuntu IRC