=== chaotic_ is now known as chaotic [16:06] * hyperair just spotted the forcefully-shutdown-notify-osd block of code to work around memory leaks [16:06] interesting workaround [19:13] qense: hmm i saw your blog post about libappindicator0-cil not being in the dependencies. that would be a bug in appindicator's packaging [19:13] hyperair: really? [19:14] hyperair: how? [19:14] The extension does depend on indicator-applet and indicator-application, but as far as I'm aware that's not necessary. [19:14] qense: just like how shared libs are supposed to put themselves into ${shlibs:Depends}, cli libs are supposed to put themselves in ${cli:Depends} [19:14] ${cli:Depends} is there, as evidenced by notify-sharp being pulled in [19:15] yes [19:15] but what's missing is libappindicator0-cil from that list [19:15] which is why it's a bug in appindicator's side of things [19:15] ok [19:15] it's not generating the clilibs file [19:15] what's the package name? [19:15] appindicator-sharp-.01 [19:15] appindicator-sharp-0.1 [19:15] ah okay thanks [19:16] i'll go fix it [19:16] it's probably missing some dh_ calls [19:16] dh_makeclilibs or something [19:16] hyperair: great, thank you [19:17] np [19:17] i should have caught this earlier [19:17] hyperair: but the source package is just indicator-application [19:17] what [19:17] say that first >_> [19:17] I thought you were asking for the pkg-config name [19:18] oh [19:18] * hyperair kicks pull-lp-source [19:18] stupid thing [19:19] sorry! [19:19] nevermind [19:19] that wasn't addressed to you [19:19] hyperair: lp:~ubuntu-desktop/indicator-application/ubuntu [19:19] ok [19:19] eh i hate bzr. nevermind =p [19:20] i'll just throw patches around [19:20] I knew I wasn't the stupid thing. :) [19:20] okay good =) [19:20] but it's so easy! [19:20] i like git. [19:20] bzr trips me up [19:20] it gets in my way [19:20] they're both not bad [19:21] bzr branch lp:branch -> edit -> bzr commit -m "message" --fixes lp:123 -> go to launchpad and file merge request [19:21] done! [19:21] well, when you've experienced wielding great power, stepping back to something less powerful really sucks [19:21] that's true [19:21] but not necessarily for Bazaar ;) [19:21] according to me, of coruse [19:21] heh [19:21] i'm not going back to bzr. period. =p [19:23] tedg needs a good kicking from the CLI team. [19:24] I agree that Bazaar lacks the power of format-patch and am, and the distinction between author and committer, but I do think that Bazaar is better for (quick) distributed development. [19:24] Bazaar is for opportunistic developers, GIT for kernel hackers. ;) [19:25] i'm no kernel hacker [19:25] i'm an opportunistic developer, but i appreciate my git. [19:25] GIT is probably more suitable for large code bases like GNOME and Banshee, but I do think that Bazaar is easier for small projects. [19:26] jcastro: hi.. got a sec? [19:26] git has pristine-tar. [19:26] qense: do me a favour and dpkg -L libappindicator0-cil for me please? thanks [19:26] pastebin that too [19:27] hyperair: [19:27] http://paste.ubuntu.com/412226/ [19:27] thanks [19:27] * hyperair groans [19:27] what? [19:28] The non-confirmation to assembly standards again you spoke about earlier? [19:29] qense: yeah, that. [19:29] the package throws things into the gac directly [19:29] and installs files in the middle of nowhere [19:29] it's supposed to be in /usr/lib/cli/blah [19:29] and use dh_installcligac [19:30] hyperair: bzr branch and file a merge request! ;) [19:30] see, this is why many seasoned ubuntu developers suck. they don't read policies anymore and just whack up some half-baked packaging. [19:30] qense: i'll throw a debdiff. [19:30] ok [19:30] That'll do [19:31] if i can even figure out how to begin fixing this mess. [19:31] hyperair: I think it's because the policies don't have such a central place in Ubuntu as they do in Debian. We don't have them on our wiki and linked to from everywhere. [19:32] qense: it is the ubuntu developer's job to *LOOK* for the policies [19:32] qense: either way there's a package for it. [19:32] containing all the policies [19:33] and if they can't find the policy, then it's their job to ask the people who know where it is [19:33] ah yes, the package is cli-common [19:33] that's where the policy resides [19:35] plus: ubuntu-policy and debian-policy [19:35] right. [19:36] hyperair: Do you want me to mark the bug as In Progress and assign it to you? [19:36] qense: please do. [19:36] ok [19:36] er what bug? [19:36] bug 560095 [19:36] Launchpad bug 560095 in banshee-community-extensions (Ubuntu) "banshee-extension-appindicator doesn't depend on libappindicator0-cil, but it has to (affects: 4) (heat: 20)" [Undecided,New] https://launchpad.net/bugs/560095 [19:36] * hyperair didn't see it [19:36] oh please point it to indicator-application [19:36] thanks [19:37] yw [19:45] qense: could you pastebin the output of ls /var/lib/dpkg/info/libappindicator0-cil.*? [19:45] * hyperair really needs a lucid environment [19:46] I'll be your Lucid environment :) [19:46] hyperair: /var/lib/dpkg/info/libappindicator0-cil.list /var/lib/dpkg/info/libappindicator0-cil.md5sums [19:46] hmm [19:47] thanks [19:50] heh. i thought it was weird. looks like dh_makeclilibs was being called in the wrong place. [19:52] make: Circular binary/libappindicator0-cil <- binary-arch dependency dropped. [19:52] this is what happened. [19:52] ah [19:52] vish: yo yo [19:52] I think this happened because there is not much C# experience in the Ubuntu/Canonical Desktop team. [19:52] They know what they're doing, but less when it's about Mono. ;) [19:52] along with cdbs stupidity. [19:52] that was before we had RAOF around. :D [19:53] jcastro: heya > pm [19:53] jcastro: tedg still doesn't know what he's doing regarding cli libs packaging >_> [19:54] jcastro: which is why b-e-appind has missing dependencies, and needs to be rebuilt. [19:54] hyperair: please file a bug on ted. [19:54] jcastro: I filed a bug on hyperair :) [19:54] jcastro: the bug that caused this wasn't policy breakage, but a failure in writing a proper debian/rules. [19:55] ok [19:56] what's up with the icon messed up for non-app-indicator tray? [19:56] ALSO, I found a great tip [19:56] since I now want my banshee there all the time I launch it at startup with "banshee-1 --quiet", FOR THE WIN. [19:56] jcastro: that's a regression in gtk, the panel, the notification area, or the engine. [19:56] hah, so situation "normal" [19:56] jcastro: what's --quiet? [19:57] it launches it minimized [19:57] in the app indicator [19:57] ah [19:57] heh [19:57] Like Rhythmbox does by default. [19:57] i thought it was banshee-1 --hide [19:57] oh dude, yeah, --hide, my bad [19:57] I just checked [19:57] qense: does it? [19:57] I've not been able to find out how to do that [19:57] jcastro: yes, it's a bug, supposedly reported but not solved [19:57] lol [19:57] hyperair: I thought it was a regression in all the things you named + the themes [19:57] hey so if it's totally broken in the tray case then clearly we should ship with the app indicator on by default in ubuntu! [19:57] qense: er right. that too, maybe [19:58] jcastro: maybe it's an option in gconf-editor [19:58] nooooooooooooooo [19:58] hyperair: come on dude, it looks freaking great up there [19:58] with the shiny new icon [19:58] jcastro: the menu looks bad in my eyes. [19:58] it's lovely [19:58] jcastro: gimme back my scrolling superpowers and my tooltip and i'll gladly worship the appindicators. [19:58] hyperair: the notification problems are quite severe and deep and nasty. There have been released fixes for that bug several times in several packages, but it still persists. [19:58] hyperair: scrolling is back, at least in libindicator [19:58] hmm yeah, in hindsight maybe it should have seperators like the rb one [19:59] hyperair: we got the scrolling back for the system volume [19:59] that was ugly [19:59] :) [19:59] maybe I can fix ted at UDS wrt. the music player volume [19:59] changing tracks! [19:59] that's inconsistency! [19:59] it's rhythmbox being inconsistent [20:00] if the volume indicator changes the volume, then it's only right that the media player changes the currently playing track [20:00] oh you mean scrolling on the icon to change tracks [20:00] it's logical. [20:00] yes, that thing you hate so much [20:00] do you know the history behind that feature? [20:00] jimmac got one of those usb scrolly wheel things [20:00] you told me once, but i don't really care much [20:00] and wanted to use it, so they added that [20:00] well. i want to use it with my mouse. [20:01] it works perfectly in conjunction to my fat tooltip that shows me what song is currently playing [20:01] * jcastro has a keyboard key for that [20:01] scroll once, the current song updates [20:01] don't like? scroll more [20:01] hmmm [20:01] hey [20:01] keep in mind that not all keyboards have media keys [20:01] notifications and keyboard shortcuts are broken for me [20:01] my current usb keyboard doesn't. [20:01] hyperair: yeah it's why I'm not strongly for or against it [20:01] notifications? blame that on qense =p [20:01] I still think it's crack [20:02] notification cannot be broken [20:02] i don't see an enable-notifications/disable-notifications option there [20:02] qense: where's it on your menu? [20:02] We removed that because it was redundant. [20:02] oh wait [20:02] too much clutter [20:02] it works [20:02] see! [20:02] it's my keyboard shortcuts that are broken [20:02] qense: but i want to *disable* it! [20:02] It only doesn't work when you have the main window focussed. [20:02] qense: because notify-osd leaks memory like hell when it's enabled. [20:03] hyperair: there is a preference for in the extension, but I'm not sure where you can edit it. gconf? [20:03] hyperair: use a different notification daemon! [20:03] qense: no! notify-osd rocks! [20:03] at least, UI-wise [20:03] it sucks in terms of memory leakage [20:03] and implementation wise, i think it was pretty messed up >_> [20:03] In Banshee? [20:04] didn't we just upload a new version? [20:04] jcastro: the new version *still* sucks. [20:04] qense: what? [20:04] nvm [20:04] jcastro: i compiled my notify-osd and now my volume notifications don't show up >_> [20:04] We should reevaluate Notify OSD during the UDS. Especially considering the fact that GNOME Shell is persuing something very similar,. [20:05] let's start with keyboard shortcuts [20:05] are yours working? [20:06] What shortcuts, for play and pause? [20:06] What are they? [20:07] The menu should be using the same GtkAction for PlayPause as the GtkStatusIcon. [20:07] jcastro: yes they do, with karmic's notify-osd. everything. [20:09] mine are busted in lucid [20:09] jcastro: i think that's handled by gnome-settings-daemon [20:10] jcastro: global media keys, right? [20:10] That isn't something for the menu, iirc. [20:10] the AppInd menu nor the GtkStatusIcon menu [20:11] okay, for some strange reason, notify-osd is adding "notification-" in front of my volume icon names [20:11] loading icon 'notification-notification-audio-volume-high' caused error: 'Icon 'notification-notification-audio-volume-high' not present in theme'loading icon 'notification-audio-volume-high' caused error: 'Icon 'notification-audio-volume-high' not present in theme'** (notify-osd:15339): DEBUG: [2010-04-11T03:11:51-00:00, gnome-settings-daemon, id:0, icon:notification-audio-volume-high (synchronous)] [20:12] see? [20:12] hyperair: did you install the new notify-osd-icons? [20:13] qense: nonono, look at it properly. [20:13] qense: look at the message. [20:13] ah [20:13] "loading Icon 'notification-notification-audio-volume-high'" [20:13] what sound applet do you have? [20:13] gnome-settings-daemon [20:13] this is karmic [20:14] maybe that's where the bug is. Maybe the notification- prefix was stripped in Lucid. [20:17] perhaps it was [20:17] but i can't find where it's prepended [20:17] but in other news, i think i found where notify-osd is leaking!! [20:17] =D [20:17] yay! [20:18] first it does g_memdup [20:18] after that it passes the data to gdk_pixbuf_new_from_data [20:18] but it doesn't give it a function to free the data [20:19] so it's never freed [20:19] which explains why my hi-res album art always screws around with notify-osd [20:22] qense: #ifdef TEMPORARY_ICON_PREFIX_WORKAROUND [20:22] I FOUND IT [20:28] yay! [20:28] hyperair: report, diff, report! [20:28] heh [20:28] hyperair: You want to speak to MacSlow, but he's in my timezone and mostly unavailable outside working hours. [20:29] wait, this might not be it =\ [20:30] anyway, I've been online for far too long already. I'm shutting down! [20:37] heh, i keep getting highlighted because of people talking about gnome-settings-daemon ;) [20:38] can you talk about something else? ;) [20:51] we've stopped, i think. [23:18] yay appind now works as expected, clilibs and all.