htorque | mpt, i have the 'menus_have_icons' gconf key enabled on my installation and am now seeing some appmenus with "missing-icon" icons. after reading your comment on bug 608584, i'm no longer sure if this is something to report? | 14:23 |
---|---|---|
ubot5 | Launchpad bug 608584 in indicator-appmenu (Ubuntu) "Should outdent icons into the left gutter (affected: 2, heat: 14)" [Undecided,Incomplete] https://launchpad.net/bugs/608584 | 14:23 |
mpt | htorque, sure, as long as that option exists, it shouldn't return missing-icon images | 14:23 |
htorque | mpt, ok, thanks! | 14:24 |
mpt | np | 14:24 |
sense | good afternoon | 14:29 |
bilalakhtar | good evening sense ! | 14:30 |
sense | hi bilalakhtar! | 14:31 |
kenvandine | klattimer, ping | 15:25 |
klattimer | hey sup? | 15:25 |
kenvandine | hey klattimer, i have a vino bug for you to look at, just talked to jcastro about it | 15:25 |
kenvandine | https://bugs.launchpad.net/ubuntu/+source/vino/+bug/628967 | 15:26 |
ubot5 | Launchpad bug 628967 in vino (Ubuntu Maverick) "vino crashes on desktop sharing with Empathy (affected: 2, heat: 16)" [High,Triaged] | 15:26 |
klattimer | oh was meaning to ask you to get this fix pushed out https://bugs.launchpad.net/ubuntu/+source/indicator-application/+bug/558841 | 15:26 |
ubot5 | Launchpad bug 558841 in indicator-application (Ubuntu Lucid) "bluetooth "devices" menu item not working in bluetooth indicator (affected: 21, heat: 132)" [Low,Triaged] | 15:26 |
kenvandine | i'll look | 15:26 |
kenvandine | lp:~ubuntu-desktop/vino/ubuntu | 15:26 |
kenvandine | klattimer, i pushed enough of a fix that fixes the crasher there | 15:26 |
kenvandine | but there is one piece i didn't get to yet | 15:27 |
klattimer | right | 15:27 |
klattimer | what's that? | 15:27 |
kenvandine | when it builds the menu for appmenu | 15:27 |
kenvandine | it does a lookup for the contact alias | 15:27 |
kenvandine | but the server seems to be null at that point | 15:27 |
kenvandine | so can't do that lookup | 15:27 |
kenvandine | the statusicon code isn't helpful there as a reference, because it looks up the alias when you activate the menu | 15:28 |
kenvandine | which is much later | 15:28 |
kenvandine | so for now, if it is NULL, the menu just says "Disconnect" | 15:28 |
kenvandine | instead of "Disconnect Jorge" | 15:28 |
kenvandine | so we either need to refactor a bit so we build the menu later in the process, or figure out a way to get that server property | 15:29 |
kenvandine | note that his worked as is in 2.28 | 15:29 |
kenvandine | but there has been some refactoring in vino since then.. | 15:29 |
kenvandine | so i mostly ported to the new vino, just didn't get this part finished | 15:29 |
kenvandine | klattimer, completely unrelated, in my testing vino or vinagre is broken as well... even without our appindicator patch i can't get a client to talk to a server in maverick | 15:31 |
kenvandine | so don't be surprised if that part fails :) | 15:31 |
klattimer | kenvandine: I found a bunch of race conditions in vino which I didn't want to get too involved with | 15:32 |
klattimer | this causes all kinds of difficulties | 15:32 |
klattimer | I filed one bug upstream for it | 15:32 |
klattimer | but that was only one of a few potential problems, some of which I couldn't actually get to prove while the first was in the way | 15:32 |
kenvandine | klattimer, so what about bug 558841 ? | 15:34 |
ubot5 | Launchpad bug 558841 in indicator-application (Ubuntu Lucid) "bluetooth "devices" menu item not working in bluetooth indicator (affected: 21, heat: 132)" [Low,Triaged] https://launchpad.net/bugs/558841 | 15:34 |
klattimer | it just needs this simple fix pushing up | 15:34 |
kenvandine | that patch has never been pushed? | 15:34 |
kenvandine | wow | 15:34 |
klattimer | again with bluetooth we have some race conditions, so hiding the menu item separately causes trouble, instead I just desensitise/sensitise it when appropriate | 15:34 |
klattimer | and it worked much better in testing | 15:34 |
klattimer | the item is still hidden by something else though | 15:35 |
klattimer | as far as I could see | 15:35 |
klattimer | it's a bit sketchy inside of bluetooth :/ | 15:35 |
kenvandine | klattimer, ok, gnome-bluetooth was just updated yesterday | 15:37 |
kenvandine | i am building it to test against latest now | 15:37 |
klattimer | gah :/ | 15:37 |
kenvandine | will upload after i test it | 15:37 |
kenvandine | thx! | 15:37 |
klattimer | np | 15:37 |
kenvandine | yuck... patch failed | 15:37 |
klattimer | :/ | 15:37 |
klattimer | is it bad? | 15:37 |
kenvandine | nah... i think i have it cleaned up | 15:43 |
kenvandine | klattimer, although it might be good for you to look at my updated patch | 15:45 |
klattimer | link me | 15:45 |
kenvandine | specifically, in update_discoverability | 15:45 |
kenvandine | they added this | 15:45 |
kenvandine | if (iter == NULL) { | 15:45 |
kenvandine | discover_lock = FALSE; | 15:45 |
kenvandine | the discover_lock = FALSE | 15:46 |
* kenvandine pushes the patch | 15:46 | |
klattimer | hmm, that seems ok | 15:48 |
htorque | tedg, i think we're talking past each other at bug 63321. all i'm saying is, that you'll end up with two menus when running an application with sudo: the local one and the global fallback menu. | 15:48 |
ubot5 | Error: Could not parse data returned by Launchpad: list.index(x): x not in list (https://launchpad.net/bugs/63321) | 15:48 |
htorque | bug 633211 | 15:48 |
ubot5 | Launchpad bug 633211 in Application Menu Indicator "Fallback menus shown for windows on the blacklist (affected: 1, heat: 8)" [High,In progress] https://launchpad.net/bugs/633211 | 15:48 |
htorque | if that's how it's supposed to be, i'll keep quiet. ;-) | 15:48 |
kenvandine | lp:~ken-vandine/gnome-bluetooth/ubuntu | 15:48 |
kenvandine | klattimer, i think it is | 15:48 |
tedg | htorque, It's not ideal, but it's not easily fixable. And honestly, sudo should die, so I'm not willing to invest much in it. | 15:49 |
technoviking | iainfarrell: ping | 15:50 |
iainfarrell | hi technoviking | 15:51 |
technoviking | iainfarrell: I was wondering if I could get those graphic I need for the forums theme. the dotted background and the pictographs (to make icons and ranks) | 15:51 |
technoviking | iainfarrell: also made a couple of test theme at http://www.mikesplanet.net/forums/, try out ubuntu2 and dark Auguine in the bottom left hand corner | 15:53 |
technoviking | would love some design team feedback:) | 15:53 |
htorque | tedg, ok, that's fine by me. :-) i just thought the sudo double menus could be fixed the same way as those blacklisted applications. | 15:53 |
kenvandine | klattimer, so how do i test this bluetooth patch? | 15:54 |
klattimer | kenvandine: turn bluetooth off and on | 15:54 |
klattimer | then see if visible is still shown in the menu | 15:54 |
klattimer | it should hide when off, and show when on | 15:54 |
* kenvandine wonders how to do that | 15:55 | |
kenvandine | used to be able to do that from that menu | 15:55 |
klattimer | there should be turn bluetooth on/off in the menu | 15:55 |
kenvandine | there isn't | 15:55 |
klattimer | gah :/ | 15:55 |
klattimer | this is a different bluetooth bug that you're seeing now | 15:55 |
kenvandine | hasn't been for a while | 15:55 |
klattimer | this happens to me sometimes | 15:55 |
klattimer | :/ | 15:55 |
* kenvandine uses rfkill | 15:55 | |
kenvandine | ok, that works | 15:56 |
kenvandine | well | 15:56 |
kenvandine | blocking it makes it go away... | 15:56 |
iainfarrell | technoviking: cool, thanks for sharing | 15:56 |
kenvandine | i guess that isn't the right test | 15:56 |
iainfarrell | I'm going to find a suitable person to talk to you | 15:56 |
iainfarrell | as I'm stacked | 15:57 |
iainfarrell | and we can find someone who can provide proper feedback | 15:57 |
technoviking | iainfarrell: thanks, sorry to bug you | 15:57 |
iainfarrell | technoviking: nw | 15:57 |
kenvandine | klattimer, can you build the package and test it? | 16:02 |
kenvandine | verify it still works as expected, and i'll upload it | 16:02 |
klattimer | kenvandine: hang on I'll see if my bluetooth is working | 16:02 |
kenvandine | ok | 16:02 |
kenvandine | mine is working... i just can't turn it off in the menu :) | 16:03 |
klattimer | well not being able to turn it on/off in the menu is a problem | 16:04 |
klattimer | because it appears that's what the decision to show the visible/invisible is based on | 16:04 |
klattimer | kenvandine: where can I get your updated patch from? | 16:04 |
klattimer | and the current source package? | 16:04 |
klattimer | I'm worried how badly my bluetooth is working overall | 16:05 |
klattimer | I only tested this patch once weeks ago | 16:05 |
klattimer | :/ | 16:05 |
kenvandine | lp:~ken-vandine/gnome-bluetooth/ubuntu | 16:05 |
klattimer | kenvandine: building now | 16:08 |
klattimer | kenvandine: it says my computer has no bluetooth adapters present | 16:15 |
klattimer | :/ | 16:15 |
kenvandine | :/ | 16:15 |
vish | kenvandine: are you pushing a ubuntu-mono update anytime soon? | 18:25 |
kenvandine | vish, i don't think so | 18:25 |
vish | u-m probably for the nm icons ivanka was mentioning.. | 18:25 |
kenvandine | oh, i hadn't heard | 18:26 |
vish | ah.. k :) | 18:26 |
ivanka | DanRabbit: can you give us an update on this ^ | 18:26 |
vish | kenvandine: well , i guess you'd hear pretty soon.. but when you do , humanity would also need a similar update.. :D | 18:26 |
DanRabbit | ivanka: ah yes I posted a mail to our thread before saying I pushed the icons to ubuntu-mono :p | 18:27 |
kenvandine | actually i think i will need to do an ubuntu-mono anyway, i think we are renaming sound_icon to sound-icon | 18:28 |
vish | kenvandine: cool! usually seb-128 does humanity updates.. i guessed it would be easier for you , since it would be the same update ;) | 18:29 |
ivanka | kenvandine: I forwarded you the email thread | 18:30 |
ivanka | kenvandine: apologies if proper processes haven't been followed. | 18:30 |
kenvandine | np | 18:31 |
kenvandine | ivanka, is there a bug filed for that? | 18:33 |
ivanka | kenvandine: I emailed the team to ask them to deal with it as I found it on the train. DanRabbit can you file a bug if there isn't one already please? | 18:34 |
DanRabbit | ivanka: yes m'am | 18:34 |
ivanka | DanRabbit: thanks! | 18:34 |
kenvandine | i am mostly wondering if this is something we consider a bug/regression or if it we need a UIF exception | 18:35 |
ivanka | kenvandine: this is a bug | 18:35 |
kenvandine | i think currently it is a regression | 18:35 |
kenvandine | yeah | 18:35 |
kenvandine | and are the icon we need ready? | 18:35 |
DanRabbit | kenvandine: a work-around has been pushed to ubuntu-mono | 18:37 |
DanRabbit | ivanka: what should I call this bug? "broadband icon shows ugly tower and little letters" ? :p | 18:39 |
ivanka | DanRabbit: how about: broadband icons lacks clarity | 18:40 |
DanRabbit | ivanka: ty | 18:40 |
kenvandine | DanRabbit, so should i upload the current work around? or are proper icons coming in time? | 18:41 |
DanRabbit | kenvandine: https://bugs.launchpad.net/ubuntu/+source/ubuntu-mono/+bug/638276 to be marked fix released when you upload :) | 18:42 |
ubot5 | Launchpad bug 638276 in ubuntu-mono (Ubuntu) "broadband icon lacks clarity (affected: 1, heat: 6)" [Undecided,Fix committed] | 18:42 |
DanRabbit | ivanka, kenvandine: I'm not sure if we are going to take further action to change the code in nm or.. ? | 18:42 |
kenvandine | DanRabbit, where did you push that branch? | 18:58 |
DanRabbit | kenvandine: lp:~darklight-team/darklight/ubuntu-mono | 18:59 |
kenvandine | oh... hummm | 18:59 |
kenvandine | is that a related branch? | 18:59 |
kenvandine | different project | 18:59 |
DanRabbit | kenvandine: that's the branch we had been working from for Lucid | 19:00 |
kenvandine | bzr says it isn't a branch | 19:00 |
kenvandine | bzr: ERROR: Not a branch: "bzr+ssh://bazaar.launchpad.net/~darklight-team/darklight/ubuntu-mono/". | 19:00 |
kenvandine | lp:ubuntu-mono | 19:00 |
kenvandine | is what we are using in maverick | 19:00 |
DanRabbit | oh, I see, I wonder if I have perms for that branch | 19:01 |
kenvandine | you can just create your own, branch from lp:ubuntu-mono and push to your own branch | 19:01 |
kenvandine | then i can merge | 19:02 |
DanRabbit | okay, give me a second | 19:02 |
DanRabbit | kenvandine: it looks like someone already pushed it to lp:ubuntu-mono | 19:08 |
kenvandine | humm | 19:10 |
kenvandine | then it is in maverick already :) | 19:10 |
DanRabbit | lol | 19:11 |
vish | DanRabbit: already in lp:u-m? i cant see it .. | 19:20 |
DanRabbit | vish: check 22px status | 19:20 |
DanRabbit | the tower icon is blank | 19:21 |
vish | odd , i dont have that icon 0.o | 19:22 |
vish | DanRabbit: are you sure you pushed it? | 19:22 |
DanRabbit | vish: when I pulled it was already there :p | 19:23 |
vish | DanRabbit: werid , if it was in already then ivanka shouldnt have had the bug.. | 19:24 |
vish | weird* | 19:24 |
DanRabbit | right? | 19:24 |
DanRabbit | well let me double check | 19:24 |
DanRabbit | vish: try pulling ubuntu-mono and check for nm-wwan-tower | 19:26 |
vish | DanRabbit: nope.. i dont see it anywhere.. are you sure it is lp:ubuntu-mono ? | 19:28 |
vish | there are atleast 2-3 branches.. | 19:28 |
DanRabbit | lol | 19:28 |
DanRabbit | okay nvm.. branched again and they magically weren't there | 19:30 |
kenvandine | :) | 19:31 |
DanRabbit | kenvandine: okay should be pushed now :p | 19:31 |
kenvandine | got it | 19:33 |
kenvandine | thx | 19:33 |
vish | kenvandine: similarly for humanity lp:~ubuntu-art-pkg/humanity/release :) | 19:54 |
kenvandine | vish, can that fall under the same bug? | 19:55 |
vish | yup.. | 19:55 |
kenvandine | mind added a "affects" for that? | 19:55 |
vish | already done :) | 19:55 |
kenvandine | thx | 19:55 |
vish | kenvandine: thanks.. :) | 19:55 |
vish | kenvandine: incase you had pulled immediately after i gave you the branch.. i had the day wrong in the changelog , fixed that in the next commit a few mins later.. but nevermind if you havent pulled yet:) | 20:24 |
kenvandine | i got the day right one :) | 20:24 |
kenvandine | but not the mail_id | 20:24 |
kenvandine | :) | 20:24 |
vish | ah! :) | 20:24 |
=== bilalakhtar_ is now known as bilalakhtar | ||
vish | .. who will be uploading the example content? | 20:40 |
=== ajmitch_ is now known as ajmitch | ||
=== mdeslaur_ is now known as mdeslaur | ||
vish | kenvandine: sheesh, people and their "sources"! ;p | 22:13 |
kenvandine | :) | 22:14 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!