=== DrPepperKid is now known as MacSlow [11:32] is this the right channel to find graphics designer? [11:34] can someone convert the svg icon of audacious into a monochrome one that integrates well with ubuntu's default themes (bug #563043)? [11:34] Launchpad bug 563043 in audacious (Ubuntu) "audacious2.png alpha blending is wrong (affects: 1) (dups: 1)" [Medium,Fix released] https://launchpad.net/bugs/563043 [11:36] bdrung, you could try #ubuntu-artwork [11:36] k, will ask there === MacSlow is now known as MacSlow|lunch === MacSlow|lunch is now known as MacSlow [15:57] jcastro: There are building issues and all kinds of nasty installation errors and crashes with Banshee.AppIndicator -- including the version in the Lucid repo -- because the much needed fix for libappindicator0-cil hasn't been released yet. Am I right hyperair? [15:57] I was wondering if you could bug anyone about this issue. [15:58] oh look seb128_ is here too [15:58] maybe he can help us [15:59] jcastro, seb128_: it's this bug: bug #564506 (Critical) [15:59] Launchpad bug 564506 in indicator-application (Ubuntu) (and 1 other project) "libappindicator-cil-dev's .pc file points to the wrong place (affects: 12) (dups: 4) (heat: 96)" [Critical,Triaged] https://launchpad.net/bugs/564506 [15:59] qense, is it fixed upstream and just needs to get into the distro? [15:59] jcastro: there is a merge request for the Ubuntu packaging, but it needs to be reviewed and uploaded. [16:00] jcastro: The banshee-community-extensions part has been fixed, but it will continue to FTBS until the appind fix is uploaded. [16:00] I see [16:00] ok so we need to hug seb128_ [16:01] tedg, Hi, you are the one working on libappindicator integration in rhythmbox? [16:01] Zdra: I thought that was Nafai [16:02] Zdra, it was initially bratsche and then jpetersen finished it off [16:02] no, bratsche wrote the rhythmbox change [16:02] hyperair: the diff renames libappindicator0-cil to libappindicator0.0-cil, but libappindicator-cil-dev to libappindicator0.1-cil-dev? Why the different number? [16:03] Zdra, it's sitting on bgo but the upstream maintainer wants to discuss it so we'll probably chat about it at UDS. [16:04] bratsche: So when I don't have the indicator applet, the status icon goes to the notification area. But its behaviors isn't reverted to upstream's. For example I don't have tooltip, popup menu is on left-click instead of right-click, no way to play/pause with middle-click, not changing volume on scrolling, no way to show/hide the main window [16:05] jcastro, tedg Nafai (who ever is doing that) ^ [16:05] I would like to fix that, but I don't know how I can know in the code if the indicator applet is present [16:05] I was wrong about Nafai, sorry. [16:05] Zdra: I think we're providing it as a patch [16:06] yep, there is a patch in the rb package [16:06] Zdra: I think they're using preprocessor #ifdef structures. [16:07] but that should be run-time since I can add/remove the indicator applet at runtime [16:07] that's what I would like to change, but I don't see how [16:07] Zdra: You can override the fallback functions. [16:07] Zdra: So we have a default way to create the GtkStatusIcon [16:07] Zdra: And you can build it anyway you want if you subclass and override. [16:08] jcastro, seb128_: To add to the pain of the problems with Banshee.AppIndicator due to libappindicator0-cil's packaging: I can't work on the two bug reports since libappindicator0-cil doesn't seem to be working at all now. [16:08] it's not used anywhere in lucid anyway [16:09] I fail to see that as a real issue, we will get it working next cycle [16:09] seb128_: You mean AppInd-sharp? [16:10] There is a fix for it already. The bug is blocking the building of banshee-community-extensions. [16:10] If the fix won't land we'd have to remove banshee-extension-appindicator from the archives because it's not workable. [16:10] right, but we are frozen for rc now [16:10] is this SRUable? [16:10] well it's universe software, after rc doesn't seem time to rename binaries from a source in main to accomodate universe software which never worked [16:10] Not due to a complicated programming bug, but due to a packaging mistake in a library that is used by exactly one package. [16:11] plus: the packaging is a violation of the Debian CLI Policy. [16:11] right, we have people every day arguing that $random_universe_software not working is end of the world [16:11] we ship on time, it that change is late for lucid we will get it working next cycle [16:11] and in its current state we actually should not only remove banshee-extension-appindicator from the archives, but also libappindicator0-cil and libappindicator-cil-dev because they are not working at all. [16:12] they are not breaking anything either [16:12] we limit changes to what we really need between rc and lucid [16:12] I understand that, but those three packages aren't even on the CD, nor required or even suggested by any other package at all. [16:12] can we fix it in a PPA, test it for a bit and then propose it as an SRU post-lucid? [16:13] no but they require an upload for a chance which is on the CD [16:13] tedg, is all the code in 82_rhythmbox-indicators.patch ? [16:13] Zdra, yes [16:13] seb128_: what change? [16:13] chance -> source [16:13] qense, indicator-application has binaries on the CD [16:13] if we update it we need to respin CD [16:13] ah, you mean the fact that you can't build the C# bindings apart. [16:14] yes [16:14] it needs an indicator-application upload [16:14] which will mean new binaries for all the thing it builds [16:14] including the C library [16:14] I forgot to look at it from the source package perspective, you're completely right here. [16:14] In that case, would it be possible to do the update post-release? [16:16] qense, I will sponsor the pending change and let slangasek decide for after rc and with eventual respins [16:16] qense, otherwise yes we will be able to SRU that [16:16] seb128_: great, thank you [16:17] np [16:17] tedg, for what I understand, the function rb_tray_icon_new() is implemented in 2 places: in the indicator patch and in status-icon plugin === seb128_ is now known as seb128 [16:18] Zdra: I'm not as familiar with the RB patch, I'm sorry. I'm more familiar with the library. [16:18] I'm not sure who did the RB patch [16:18] brastche and jpertersen working on it [16:19] I see, it is in makefile it replace rb-tray-icon-gtk.c by rb-indicator.c [16:20] tedg, so do you know how in runtime I can know if tray icon will be embeded inside the notification area or the indicator applet? [16:22] I wrote the original RB patch, but I don't understand what the issue is. [16:24] bratsche, I would like to modify the patch, so at runtime, if the indicator applet is not in the panel, it revert to upstream behavior [16:24] Zdra: You can either listen to the signal or wait for the function to be called. [16:24] Zdra: I think in general it's best to just implement fallback/unfallback and they'll be called if needed. [17:25] seb128: slangasek already gave his approval [17:26] seb128: directhex and i discussed it with him yesterday [17:26] after which i pinged you in #ubuntu-release [17:26] I was probably away for lunch [17:26] I didn't read that backlog [17:26] ah [17:27] seb128: basically he acked it as long as there would be *no* further changes to indicator-application [17:27] ok [17:27] I will just sponsor what is waiting if you confirm what should be uploaded [17:28] seb128: the changes that were in the merge request. that's it. [17:28] seb128: tested by directhex to build, install, and work fine [17:29] ok [17:29] seb128: if tedg doesn't have any more changes then i think it can be uploaded. [17:29] seb128: I think the only other possible change would be if Nafai can figure out the gnome BT one. [17:30] yeah :) [17:30] seb128: The Mono updates work for me though. I was able to rebuild my local Tomboy happy happy now. [17:30] ok [17:30] will sponsor that after the meeting in ubuntu-desktop [17:31] tedg: I'll be restarting my work on the C# bindings in the Maverick cycle, it is too late now to push such a large change to the Mono bindings that isn't needed by anything at the moment, actually, and I don't really have time to work on it. I'll await the roadmap for AppInd. [17:32] qense: Cool, that'd be great! [17:32] qense: I don't think there'll be many changes in the Maverick cycle. Hopefully just fixes. [17:32] tedg: We already now what the exact problem is, but I need to make time to write a proper fix. And i haven't made time yet. ;) [17:32] tedg: No exciting new stuff like categorising? :D [17:33] * tedg hates exciting and new as they all create bugs :) [17:33] qense: thanks for looking at the merge > https://code.launchpad.net/~ubunt-u-markbenjamin/indicator-applet/reorient/+merge/23524 :) [17:33] vish: yw [17:34] tedg: I like exciting and new because I make myself forget all the bugs1 [17:34] qense: its now awaiting tedg to merge it to main? [17:34] vish: I have no idea, but considering the RC freeze: only after the release [17:34] seb128: We also have another merge request, this time for indicator-applet. It adds support for vertical panels. Definitely something for post-release updates? [17:37] Yeah, I can't imagine we can do that one at this point in the release cycle. [17:37] It seems reasonably for an SRU though to me. [17:37] \o/ [17:38] I should say too, qense thanks for that :) [17:38] MarkieMark1: thank you for the fix [17:39] it's a pleasure :) [17:39] and apologises for making you create a new branch because I can't figure out the difference between indicator-application and indicator-applet. :P [17:40] no worries, I learnt some of how debian packaging works [17:40] good! === jono_ is now known as jono [17:42] qense, yes [17:42] ok [17:43] lets not make your life harder now :) === doctormo_ is now known as doctormo [23:47] Can someone help me get Pidgin working with the messenger applet please?