/srv/irclogs.ubuntu.com/2010/12/09/#ayatana.txt

ion17 hours and Apport retracing service still hasn’t retraced the crash report. :-P00:27
ionAh, missed seb128’s message.00:30
ionMy issue had already been reported. LP #686698.05:31
ubot5Launchpad bug 686698 in Nux "compiz crashed with SIGSEGV in nux::IOpenGLSurface::UnlockRect()" [Undecided,Triaged] https://launchpad.net/bugs/68669805:31
=== ssj6akshat is now known as akshat|school
kvalomorning06:51
ionthat.06:52
kvaloion: moro07:06
ionhi07:06
didrocksgood morning07:31
=== smorar__ is now known as smorar_
smspillazmornin07:44
smspillazdidrocks: could you check something for me?07:45
didrockshey smspillaz07:45
didrockssmspillaz: sure, not right now, but I can07:45
smspillazdidrocks: have you got two screens ?07:45
didrockssmspillaz: no, only one there…07:46
smspillazhmm ok07:46
smspillazdidrocks: I'm wondering whether this thing that gnome-panel does is a feature or a bug07:46
smspillazbasically if you have a panel on the left side of screen 1, and then there is another screen directly to the left of that panel (eg screen 2 is left of screen 1)07:47
smspillazerr, the panel needs to be on the right hand side, rather07:47
smspillazactually, on the left (slightly dyslexic)07:47
smspillazanyways07:47
smspillazif you have that, and then maximize a window on screen 1, it will go underneath the panel07:47
smspillazthis happens with metacity and compiz07:47
smspillazI checked with xprop, it seems that the panel does not set the _NET_WM_STRUT for that part of the screen07:48
smspillazit seems like a bug, but there'd be some specific code path that would make it not set that strut property, so I'm wondering if it is the intentional behaviour07:48
=== smorar_ is now known as smorar__
didrockssmspillaz: ok, so if you have two screens07:52
didrocksand gnome-panel is on the left07:52
didrocksyou mean that it's doesn't set WM_STRUT and so, apps go underneath?07:53
smspillazdidrocks: yes07:53
smspillazbut only on panels that are adjacent to other screens07:53
smspillazit would seem like this is not the desired behaviour ...07:54
didrockssmspillaz: weird, WM_STRUT is set screen by screen, isn't it?07:54
didrocksso there is no reason to not restrain the area on the one with the panel07:54
smspillazdidrocks: it should be set by the output07:58
smspillazI'm just compiling docky now to see what the behaviour should be07:59
MacSlowgreetings everybody08:27
didrocksmorning MacSlow08:29
MacSlowdidrocks, salut here too :)08:30
didrocks:)08:30
smspillazreasons I hate autotools #24533452: It only tells you which dependencies you are missing one dependency at a time08:33
MacSlowsmspillaz, :)08:34
didrockssmspillaz: reasons I hate cmake #${num_smspillaz}+1: cmake ignores it on some case and tell "let's take a chance" :-)08:35
MacSlowsmspillaz, annoying indeed08:35
RAOFHeh.  Reason to hate cmake #whatever: CFLAGS?  What's that? ☺08:38
didrockshey RAOF!08:40
smspillazRAOF: you mean CMAKE_CXX_FLAGS_DEBUG ?08:40
RAOFHey didrocks!  Looks like that segfault in mipmapping on r600 might be fixed in git.08:40
* smspillaz w00ts08:41
didrocksRAOF: oh nice!08:41
RAOFsmspillaz: One of the rather endeering parts of autotools is that setting the CFLAGS/CXXFLAGS environment variable DTRT.08:41
smspillazI guess. Although cmake was really designed to be multiplatform08:42
smspillazit is not a full buildsystem, it only generates native makefiles08:42
smspillazso it doesn't do things like setting env vars08:42
didrocks(and you have to do a COMPIZ_DESTDIR which conflicts with the env one :p)08:43
* RAOF is not going to get diverted into a build system discussion. They all suck!08:43
smspillazreally someone at debian needs to sit down and write proper macros for cmake08:43
didrocksRAOF: shell + cp ftw :)08:43
didrockswhy?08:43
smspillazbecause there are none?08:44
didrockswe have our packaging system :)08:44
didrocksso, no interest in writing something specific to cmake08:44
didrocksgrrr, too much flickering when switching ws in compiz :/08:45
smspillazreally?08:45
didrockssmspillaz: yeah, the thing we discussed the other day08:45
didrockssmspillaz: like if you had a "magnet", not sure what's the best way to describe it08:45
smspillazintel ?08:46
didrocksnvidia08:46
smspillazso it's like a vsync issue ?08:46
didrocksexactly08:46
didrocksquite disturbing08:46
smspillazthat's nvidia being stupid. You need to go into nvidia-settings, -> opengl settings -> sync to vblank, then ccsm -> opengl -> sync to vblank and restart compioz08:47
smspillazthat will fix it08:47
didrocksand I have X damage not triggered when scrolling sometimes08:47
didrockssmspillaz: hum, let me see if it fixes it08:47
smspillazdidrocks: ccsm -> workarounds -> force X to GLX sync08:47
didrockssmspillaz: the thing is that we should do a distro choice there08:47
smspillazperhaps08:48
didrockswe can't ask our users to do that :)08:48
didrocksnot perhaps "certain" :)08:48
smspillazdidrocks: it should be enabled by default08:48
didrockssmspillaz: even for non nvidia drivers?08:48
smspillazdidrocks: in the detection module we can distro patch workarounds  to enable "force x to glx sync" if an nvidia card is detected08:49
didrockssmspillaz: agreed08:49
didrocksccsm -> opengl -> sync to vblank is already checked08:49
didrockslet me see the workarounds plugin08:49
smspillazdidrocks: check nvidia settings08:49
smspillazthat's where it counts08:49
didrockssmspillaz: yeah, I checked it08:49
didrockswhat's the difference between the workarund setting and the opengl one?08:50
smspillazworkaround setting fixes damage issues08:50
smspillazbecause the nvidia driver is stupid08:50
smspillazso we have to slow down compiz a bit08:50
didrockssmspillaz: heh, you are telling intel is stupid, nvidia is stupid, which next? :p08:51
didrockssmspillaz: all checked, we'll see08:51
didrockssmspillaz: should the workaround plugin be enabled by default?08:52
didrockssmspillaz: i mean, I enabled it, and so, I have other settings now that I didn't have before08:52
smspillazdidrocks: sure08:54
smspillazdidrocks: disable by default the "show previews of minimized windows" though, that has random bugs that I'd rather not hit for no reason right now08:54
smspillaz(since it tells all kinds of lies to X11 to work correctly)08:55
smspillazdidrocks: hm, ok with docky I don't see the maximization issue08:56
didrockssmspillaz: it's not activated by default08:56
smspillazdidrocks: ah ok good :)08:56
didrockssmspillaz: what I have are:08:56
didrocksOOo Menu Fix08:56
didrocksJava Window Fix08:56
didrocksJava Taskbar Fix08:56
didrocksAIGLX Fragment Parameter Fix08:56
didrocksand of course, the Synx X and GLX :)08:57
smspillazyou can disable AIGLX Fragment Parameter fix08:57
smspillazit's not necessary any more08:57
smspillazIMO I should write a patch for workarounds where we will enable Sync X and GLX only if there is and NVIDIA card08:57
didrockssmspillaz: right08:58
didrockssmspillaz: and the others?08:58
smspillazdidrocks: the others are find08:58
smspillaz*fine08:58
didrockswhat's the OO menu fix?08:58
smspillazdidrocks: OO menus don't register as type "menu"08:58
didrockssmspillaz: so, why OO is enabled and not firefox/thunderbird?08:59
smspillazdidrocks: *shrug*08:59
didrockswhich means? ;)09:00
smspillazdidrocks: maybe the person who wrote them didn't enable them by default09:00
didrockssmspillaz: so it should as well, I guess?09:00
smspillazdidrocks: only if you see problems09:00
didrocksI didn't see any issue with OOo before enabling the plugin09:00
smspillazok :)09:01
didrocksgrrr, tomboy crashes… hard to take a note this way :)09:03
smspillazdidrocks: hmm, have you been getting random crashes with an error like "call to empty boost::function ?"09:04
didrockssmspillaz: no, I've been quite lucky like that, all is pretty stable to me (just had a crash this morning)09:04
didrockss/like that/on that side/09:05
didrockssmspillaz: it didn't fix the sync issue09:08
didrockshum… maybe I should restart X to take the nvidia settings into account09:08
smspillazdidrocks: perhaps09:08
smspillazdidrocks: it fixed it here at least09:08
smspillazdidrocks: although it doesn't work on multiple monitors09:08
didrockswell, not my case there :)09:09
kvaloarghhhh. spent over 30 mins trying to find out why automake doesn't install a script to pkgdatadir and it was because I had _SCRIPT, not _SCRIPTS09:09
smspillazdidrocks: have you got this commit in the packages ? http://git.compiz.org/~dbo/compiz-with-glib-mainloop/commit/?h=glibmm-experimental&id=3e1c09f13eb99af88bf0bd8d73be8bbcf533ded709:15
RAOFdidrocks: You shouldn't need to restart X; in fact, restarting X will probably revert the changes :)09:15
didrocksRAOF: smspillaz: ok, in that case, it doesn't fix it :)09:16
didrocksRAOF: smspillaz: for once, I'll bring my heavy laptop at the Rally, maybe we can sit and look at it if it can help you?09:17
didrockssmspillaz: I think I have, let me check09:18
kvalodidrocks: about the symlink from /usr/bin/ to the python script in /usr/share/app. is it better to create the symlink with autotools or with dh_link? with autotools it's a bit tricky09:22
didrockskvalo: depends if you only care about the package or also the upstream tarball + make && make install09:23
kvalodidrocks: good point09:24
didrockssmspillaz: I'm still opening a bug for that issue (nvidia/sync), so that we can get it tracked09:24
kvalodidrocks: I'll try to make the autofoo magic work. thanks09:24
didrockssmspillaz: the commit is in09:30
smspillazdidrocks: huh, ok09:45
smspillazdidrocks: I thought that commit fixed it :/09:45
didrockssmspillaz: did you check the bug report mentionned the last ubuntu compiz package?09:46
didrockssmspillaz: alpha1 version has it (1:0.9.2.1+glibmainloop2-0ubuntu4) maybe even ubuntu309:46
lamalexjcastro, didn't njpatel present a diagram like the one you ask for in ayatana-dev at uds?09:46
lamalexor ted at gnome summit09:47
smspillazdidrocks: no not yet, but I am still getting that crash09:47
smspillazlet me dist upgrade09:47
seb128lamalex, well then to start this diagram should be on the unity.ubuntu.com webpage09:47
lamalexping ted when he's awake09:48
seb128lamalex, the diagram is also not enough, we need things like "if you want to use appindicator you should check those apis"09:48
seb128with the list of libraries that you might have to use to write an indicator09:48
seb128and a short explanation of what they do09:48
lamalexseb128, right09:49
lamalexwe need awesome developer docs09:49
RAOFHm.  While I wait for mesa to build, is there any particular reason why unity-panel-service should take up 200MB RSS?09:49
seb128that's being worked but will take a bit09:50
seb128(documentation)09:50
seb128until now we could do a better job to at least point the sources used and what they do09:50
seb128RAOF, out of leaks you mean? ;-)09:50
RAOFseb128: That thought had crossed my mind, yes :)09:51
=== smorar__ is now known as smorar_
didrockslamalex: hey, seeing your comment on the merge, what's the difference between LAUNCHER_ICON_QUICK_STARTING and LAUNCHER_ICON_QUICK_LAUNCHING ?09:58
lamalexdidrocks, there is no LAUNCHING10:02
lamalexdidrocks, I just updated that comment, thanks10:02
lamalexI'm sick :\10:02
didrockslamalex: oh? the grey weather of London hit you?10:03
dbarthsalut seb12810:13
dbarthseb128: about https://bugs.launchpad.net/unity/+bug/68660310:13
ubot5Ubuntu bug 686603 in Unity "valgrind "Conditional jump or move depends on uninitialised value"" [Undecided,New]10:13
seb128lut dbarth10:13
dbarthseb128: you get that on the console, i mean it's a debug log?10:13
seb128I get that by running test-panel under valgrind10:14
dbarthah right10:14
dbarthsmspillaz: hey sam, have you seen mvo bug with the link.py testcase?10:18
dbarthsmspillaz: that sounds like a scary regression10:19
smspillazdbarth: no, could you link it to me ?10:20
smspillazdbarth: also, could you post your config profile somewhere? I'm trying to track down the crash in windowInitPlugins10:20
didrockskamstrup: "Why can't I ever get the email API for LP working!?!?11one" <- same here :)10:55
didrockskamstrup: thanks, will remove the "there" :)10:55
didrockskamstrup: exec didn't want if you want to embeeded in gdb probably10:56
kamstrupdidrocks: ah, right10:56
kamstrupdidrocks: I found the answer on https://help.launchpad.net/Code/Review I think10:56
kamstrup"review approve" needs a whitespace prefix10:56
didrockskamstrup: oh?10:56
kamstrup" review approve"10:56
kamstrup:-S10:56
kvalokamstrup: sorry dude, but I have a long merge request again: https://code.launchpad.net/~kvalo/indicator-network/settings-merge/+merge/4318310:56
didrockskamstrup: yeah, that's what I'm doing10:56
didrockskamstrup: all commands need a whitespace prefix10:57
didrockskamstrup: but for unknown reason, it hates me :)10:57
kvalokamstrup: but this is just a start, so I think you don't need to review it in so detail10:57
kamstrupkvalo: I think I may have to push it to monday10:58
kamstrupI really need to get the dee gdbus port out of the door10:58
kamstrupand friday is sprint10:58
kvalokamstrup: monday is ok for me :)10:58
* kamstrup rolls up sleeves to get those sodded unit tests running again10:59
=== njpatel is now known as njpatel_
=== akshat|school is now known as ssj6akshat
lamalexwhen someone has a minute? https://code.launchpad.net/~alexlauni/unity/xids-introspection/+merge/4319312:12
=== smorar_ is now known as smorar__
=== MacSlow is now known as MacSlow|lunch
lamalexnjpatel_, didrocks, with CMake can we have unityshell.xml.in have variables in it that get set on build?12:17
didrockslamalex: yes, look at what I've done some commits ago with unity.cmake in tools/ (it's the binary)12:18
lamalexdidrocks, sweet thanks12:18
htorquehello everyone! don't know when it started, but window buttons now have tooltips again - should this be filed as bug?12:19
didrockslamalex: so that to replace VERSION and tools/CMakeFile*12:19
didrockshtorque: it's already filed again compiz, I dropped my patch and I'm just working on it again :)12:19
didrockshtorque: I dropped it when going to 0.912:19
htorquedidrocks, nice, thanks!12:19
didrockshtorque: thanks for your excellent reporting and testing job :)12:19
htorqueyw! thanks for all the fixes. ;-)12:20
seb128njpatel_, didrocks: so my launcher is acting weird12:25
didrocksseb128: oh weird?12:25
seb128like the launchers do a start animation when clicked12:25
seb128but they don't start anything12:25
seb128if I run firefox manually it doesn't match the pinned icon12:26
seb128but add a new one in the launcher12:26
didrocksseb128: is it an application you already started?12:26
seb128same for nautilus or gedit12:26
seb128well firefox and gedit have no running process12:26
seb128but I used them today yes12:26
didrockslike, you started/closed them 4 times?12:26
seb128could be, I didn't count12:26
didrocksseb128: I think you're striken by the ubuntu starter edition :)12:26
didrocksseb128: should be fixed in trunk btw :)12:27
seb128is there a way to reset the counter?12:27
didrocksseb128: killing the bamf service and restarting unity12:27
didrockswe didn't merge the fix last week because it added others issues IIRC12:27
seb128ok, compiz --replace did it12:28
seb128is that known that compiz crashes on --replaec?12:28
seb128well apport triggers12:28
didrocksseb128: bug #68362312:28
ubot5Launchpad bug 683623 in unity (Ubuntu) "Starting a launcher only works four times" [High,Triaged] https://launchpad.net/bugs/68362312:28
seb128yeah, I've read this one last week12:28
didrocksseb128: yeah, compiz isn't good when you give unknown args and unknown plugins12:29
didrocksIMHO, if it can't find a plugin, it should just ignore it12:29
seb128I just though it was when clicked 4 times in a row12:29
didrocksseb128: not in a row :)12:29
seb128didrocks, thanks12:29
didrocksseb128: yw12:29
didrocksseb128: btw, we will have the unity binary from now on12:30
seb128yeah, I noticed, great work12:30
didrocksso, will make things easier to reset the config and such12:30
didrockswell, not very difficult, just that I hate python with making some kind of tee-like behavior :)12:31
didrockssmspillaz: are you changing gtk/window-decorator/gtk-window-decorator.c with your decorator work?12:36
smspillazdidrocks: no, I am writing my own decorator12:49
smspillazdidrocks: well actually, I forked gtk-window-decorator (temporarily) and I am adding inactive/active shadow work12:49
didrockssmspillaz: ok, so I should maybe wait or set it to you the task of the option to hide tooltip?12:49
didrockssmspillaz: it was a distro-patch we had (that I submitted upstream as well some month ago).12:50
smspillazdidrocks: Our unity-window-decorator can neuter tooltips12:50
didrockssmspillaz: nice! so yeah, please do that :)12:50
* didrocks drops his task then ;)12:50
smspillazcould you link me to the patch so I don't have to hunt around in the code ? ;-)12:50
didrockssmspillaz: well, you removed the gconf part in 0.9 so you will have to add it again12:51
smspillazdidrocks: did we want it as an option? I thought design said no tooltips ever12:51
didrockssmspillaz: but the distro-patch was http://paste.ubuntu.com/541431/ for 0.812:51
didrockssmspillaz: well, I added an option to get it accepted upstream12:52
didrockssmspillaz: but you never answered!12:52
didrocks:)12:52
smspillazdidrocks: must have missed it12:52
didrockssmspillaz: so if it's our own, yeah, no worry, no option12:52
smspillazdidrocks: yeah, that's what I was thinking12:52
didrockssmspillaz: http://bugs.opencompositing.org/show_bug.cgi?id=126412:52
smspillazdidrocks: ah right. I never checked the opencompositing bugs, I was too busy porting plugins :p12:53
didrockssmspillaz: no worry, patch tagging is useful to refind bugs even if the bugtracker is down :p12:53
didrockssmspillaz: so I'm assign you the relevant compiz bug, one sec12:53
smspillazdidrocks: yeah, the opencompositing bugtracker is (still) down12:54
didrockssmspillaz: you should use launchpad for upstream! ;)12:54
smspillazwe probably could12:54
smspillazbugzilla is a PITA12:54
klattimersmspillaz: +112:55
didrockssmspillaz: so, activating the workaround plugin with upstream defaults for now minus AIGLX one12:55
=== MacSlow|lunch is now known as MacSlow
lamalexnjpatel_, how does the launcher's autohide get set initially?13:04
lamalexdoes optionChanged get called once at start?13:04
njpatel_lamalex, I'm not sure about the compiz options stuff, sorry13:06
lamalexnjpatel_, who is that? Jason?13:07
njpatel_yep13:07
njpatel_or smspillaz13:07
lamalexok13:07
lamalexDBO, wake up13:07
=== oubiwann is now known as oubiwann_
lamalexnjpatel_, could you please review https://code.launchpad.net/~alexlauni/unity/xids-introspection/+merge/4319313:18
kvaloronoc: hi. time again for a review: https://code.launchpad.net/~kvalo/indicator-network/gdbus-cellular-crash/+merge/4320413:19
jcastrook guys, added C++ to the style guide: https://wiki.ubuntu.com/Unity/CodingStyle13:26
njpatel_lamalex, will take a look in a bit13:28
njpatel_jcastro, you can remove everything from "Be Verbose" to before "Commit Messages", as it's all stated at the start of the C++ bit13:30
jcastrook13:31
jcastrodone13:32
jcastronjpatel_: what else is C++ after nux and compiz?13:32
didrocksnjpatel_: don't you mind if I move unity_support_test from examples to tools (I'll add the directory and such) as it's not really an example :)13:47
=== njpatel_ is now known as njpatel
njpateljcastro, that's it for now13:47
njpatellamalex, approved with a comment13:51
njpatelplease fix before merging13:51
jcastrolamalex: njpatel: any new people in merges yet?13:52
lamalexjcastro, waiting for one to sign CA13:52
jcastroI see 2 pending merges from people I don't know, so crosses fingers13:52
njpatelyes a few, but I'm not sure their success rate, looking now13:52
jcastrorock13:52
lamalexer, waiting for one to fix his branch and sign ca13:52
jcastrolamalex: hey, let's do this, when they're new and asking you for CA when you reply to them CC me, I like to get a little bio on each person and a photo13:52
jcastroso I can put them on planet13:52
jcastrolamalex: just a fwd or CC is fine, I can handle introducing myself, etc.13:53
lamalexjcastro,  sure13:53
didrocksnooooooo, I'm starting to google "cmake" when I want to google "autotools"13:55
didrocksthat's the end /o\13:55
didrocksnjpatel: did you see my question about unity_support_test location and nux?13:57
=== oubiwann_ is now known as oubiwann
ssj6akshatjcastro, I am always getting interrupted when I am about to write a post on the Bitesize bugs13:59
ssj6akshatI guess someone doesn't want to it to appear :P14:00
jcastrossj6akshat: it's ok, it's a marathon not a sprint14:01
kenvandinedidrocks, has anyone considered changing the workspaces settings to be 2x2, so expose would look more like it did in the mutter based unity?14:02
kenvandineand imho it is a better use of space14:02
jcastrodbarth: speaking of, getting branches on these, I will need more! https://bugs.launchpad.net/unity/+bugs?field.tag=bitesize14:02
didrockskenvandine: it will do that later the cycle, but currently compiz doesn't listen to gconf gnome keys14:02
njpateldidrocks, no, sorry. Erm, I don't know what that does so feel free to move it where you feel right :)14:02
kenvandinedidrocks, humm... i have mine set to do that now14:03
didrocksnjpatel: ok, it's the detection module at start, so definitively not an example :)14:03
njpatelright14:03
didrockskenvandine: right, but it's not the GNOME keys, that means, it's not share with your desktop classic session14:03
didrocksnjpatel: ok, doing and commiting14:03
jcastronjpatel: didrocks: while I have you here, post release I need more bugs kthx. :)14:05
didrocksjcastro: well, I'm tagging everything I can… i've reviewed all the bugs as you already know and add them :)14:06
jcastroI can't believe I am disappointed that there aren't enough bugs14:06
jcastroyesterday or so neil fixed like three of them and I was like "no! Don't take them, let someone else do them!" then I came to my senses.14:06
didrocksjcastro: I guess once the dash/places will be there, we can have more of them14:07
jcastroyeah14:07
didrockshow many are still there?14:07
jcastro~1014:08
jcastrobut with these 2 pending ones we've had one per day since I announced the report on tuesday14:09
kenvandinedidrocks, i see14:09
jcastroso I think it's off to a good start considering we're still just around A114:09
kenvandinedidrocks, ok, thats fine... glad it is planned14:09
kenvandine:)14:09
didrocksjcastro: right, little issues will come later, when it doesn't need refactoring everything :)14:10
lamalexnjpatel, I don't think I should free the list from bamf14:12
lamalexI'm pretty sure bamf owns that list (granted bamf should probably be duping the list and then returning it)14:12
njpatellamalex, the docs say it owns the contents but not the list, that's what I was going on14:17
njpatellamalex, the code seems to suggest the same14:18
ronockvalo, done14:18
kvaloronoc: thanks!14:20
ronockvalo, np14:21
ronocagateau, got a sec?14:21
agateauronoc: give me 5 minutes14:22
ronocagateau, no rush14:22
agateauronoc: ok, I am back14:29
ronocagateau, one sec14:30
ronocagateau, can see anything obvious why the submenu in this dbusmenu is not appearing on the indicator14:34
ronochttp://pastebin.ubuntu.com/541475/14:34
lamalexhaha so which is worse, potential race conditions, or making a member variable public14:35
* agateau looks14:35
ronoclamalex, race condition any day of the week14:35
lamalexman but making a member variable public feels so wrong14:36
agateauronoc: menu looks good14:37
DBOlamalex, i dont like waking up14:37
ronoclamalex, I know, but sure isn't every python obj wide open14:37
agateaulamalex: making a member variable public can be a good way to get race conditions :)14:37
tedgDBO, I think the same thing about you waking up, but I didn't want to say anything ;)14:37
lamalexronoc, yeah and it's one of the things I hate about python14:37
DBOtedg, can you turn that into a policy please14:38
klattimerlamalex: did you get any further with atk?14:38
ronoclamalex, I take ruby over python any day of the week14:38
agateauronoc: I think tedg built a tool to load a menu from json, you should try if your dump shows there14:38
klattimerI'm wondering about the event redirection issues i'm facing, but they may be the result of offscreen rendering14:38
klattimerstill trying to work it out :/14:38
* ronoc must stop saying 'any day of the week'14:38
ronoctedg, good morning -> http://pastebin.ubuntu.com/541475/14:39
seb128ok14:39
seb128so is appmenu maintained and by who?14:39
seb128dbarth_, ^14:40
ronoctedg, the submenu is not being rendered in the indicator14:40
ronoctedg, can you see any problems there14:40
seb128doh! ;-)14:40
ronocagateau, thx - might try this14:40
tedgseb128, indicator-appmenu is me, appmenu-gtk is bratsche14:41
tedgronoc, looking14:41
seb128tedg, I guess with unity none is yours? ;-)14:41
seb128tedg, things like clicking on a menu activating another item than the one selected14:41
tedgseb128, ?  indicator-appmenu is used by unity14:41
seb128tedg, I confused with indicator-applet-appmenu14:42
seb128ignore me ;-)14:42
tedgseb128, If it's an item on the panel, that's njpatel, if it's in the actual menu that's me.14:42
seb128it's an actual menu item14:42
seb128I select "show conversations history" and it sends a "share desktop request"14:42
seb128it's quite embarassing14:42
seb128I just shared my desktop with someone I didn't want to talk to14:43
seb128twice14:43
tedgHmm, that's odd.14:43
tedgConsistently?14:43
seb128when it happens yes14:43
* tedg can't find anyone to share his desktop with :(14:44
seb128now the menu item just doesn't work14:44
seb128tedg, well, open a buddy list14:44
seb128double click on someone14:44
seb128go to contact, and select conversation history14:45
tedgYeah, share my desktop is consistently greyed out.14:45
seb128does it work?14:45
seb128or does it do nothing?14:45
tedgIt did nothing, but now I've got a menu from somewhere else?14:46
seb128I had that before as well14:46
seb128I got the nautilus menu at some point14:46
seb128but otherwise it does nothing14:46
seb128then I picked "info"14:46
seb128which send a share desktop request to dholbach14:46
tedgseb128, And you didn't want to talk to dholbach more than once? ;)14:47
seb128no, I'm just testing on dholbach now :p14:47
tedgHmm, okay.  I'm not sure what's up there.14:47
seb128I had the issue first with someone else in my list14:47
seb128ok14:48
tedgWe'll have to grab a dbus capture to see if who's wrong....  I of course think it's appmenu-gtk ;)14:49
tedgronoc, I don't see anything odd.  The only thing that I can think of is that we're not handling something right with a custom type as the base item for the submenu.  I don't think I've done that before.14:50
tedgronoc, Can you try switching it to a standard item real quick and see if that fixes things?14:50
ronoctedg, will do14:50
ronoctedg, yup i was doing something (or not doing something) with the custom item, works with a normal item14:59
ronoctedg, to set the icon should I use the icon_data property, does it work well ? up until now I have used custom items, I think though the design for this can be catered for with a normal menuitem15:00
tedgronoc, That's fine if you want bitmaps.  But if you can use icon names you should do that, as it's much less data over the bus.15:01
njpatel+115:03
ronoctedg, sure, the name though returned for the playlists does not seem to load on the other side of the bus. I could do a custom widget again and send over the name and load the icon myself in the indicator (similar to the title menu item as it is now)15:05
ssj6akshatjcastro, http://www.omgubuntu.co.uk/2010/12/take-a-bite-out-of-unitys-bitesize-bugs/15:06
jcastroNICE!15:07
* jcastro clicks Like15:07
* ssj6akshat thought it was crappy15:07
ssj6akshatThe top 3 posts on OMG! Ubuntu! are about.......15:09
ssj6akshatUnity!15:09
Devil505:)15:10
candohi! running unity from source (last revision) the launcher panel becomes insensitive and i get these g_critical:http://pastebin.com/XixGHxby15:19
candoyesterday it worked fine...15:19
kvaloronoc: another one: https://code.launchpad.net/~kvalo/indicator-network/bug-687414/+merge/4322815:19
ronoctedg, I suppose what I am asking is if there is a way to set the image path on a normal dbusmenuitem and for it to work. right now banshee sends me something like this for an icon (file:///usr/share/banshee-1/icons/hicolor/22x22/categories/source-smart-playlist.png). I have tried to set the icon_name with this and with 'source-smart-playlist' both have come up with missing png's15:21
ronockvalo on it15:21
kvalokenvandine: I try to get indicator-network 0.3.1 ready "soonish". do you have time to upload it today?15:22
lamalexnjpatel, did you ever get back to me about g_list_free?15:22
lamalexI can't find me asking you, or you replying in xchat15:22
cyphermoxdidrocks, I just added a manpage for your unity wrapper on bug 684896.15:22
ubot5Launchpad bug 684896 in unity (Ubuntu) "no unity manpage" [Low,Triaged] https://launchpad.net/bugs/68489615:22
kvalocyphermox: hi. any luck with bug #683302?15:22
ubot5Launchpad bug 683302 in ofono (Ubuntu) "Please merge ofono 0.36-1 from Debian unstable (main)" [Wishlist,In progress] https://launchpad.net/bugs/68330215:23
njpateli did15:23
cyphermoxkvalo, err, yes, it's pretty much done, hold on :)15:23
njpatellamalex, the docs say it owns the contents but not the list, that's what I was going on15:23
njpatellamalex, the code seems to suggest the same15:23
kvalocyphermox: cool15:23
cyphermoxkvalo, ah, I need to ping micahg and let him know there was an update :)15:24
lamalexnjpatel, well then there's another leak in Launcher.cpp15:24
lamalexof the same15:24
kvalocyphermox: yeah, I got worried because nothing was happening :)15:24
lamalexwant me to fix them both?15:24
cyphermoxkvalo, unless kenvandine or seb128  or somebody else want to review the merge and take it over from there ?15:24
kvalocyphermox: I'm fine with all options as long as the end result is the new (working) ofono in natty ;)15:25
ronockvalo, done15:25
cyphermoxkvalo, I think we just need to be patient really... I'll ping micahg as soon as I see him15:26
kvaloronoc: thanks again15:26
njpatellamalex, fix it?15:27
njpatellamalex, lp:bamf/lib/libbamf/bamf-view.c::141, please double check to make sure15:28
lamalexnjpatel, yah you're right15:31
lamalexso should I fix both?15:31
njpatelyep15:31
jcastrotedg: are you talking to this guy? http://lizards.opensuse.org/2010/12/09/indicator-sessionnetwork/15:32
jcastrocyphermox: where's your branch for your nm-applet port? I seem to only be able to find the distro package15:33
jcastroport to indicators I mean15:33
cyphermoxjcastro, https://code.edge.launchpad.net/~network-manager/network-manager-applet/ubuntu.head contains it.15:37
jcastrota15:37
lamalexsmspillaz, DBO, is optionChanged called automatically once at load, or just when the check box is clicked?15:48
DBOI dont know15:49
DBOask smspillaz15:49
lamalexI pinged you both doofus15:49
DBOor add a printf to check15:49
DBOI imagine its called no matter what15:49
boulabiar_join #wayland15:50
boulabiar_oops15:50
lamalexoh yeha, i could just add a printf and check15:51
lamalexduh15:51
=== boulabiar_ is now known as boulabiar
smspillazlamalex: optionChanged gets called whenever the option gets changed15:56
tedgjcastro, No I haven't talked to him at all.16:01
tedgjcastro, Very cool though!16:01
jcastrotedg: ok, if he asks for questions I will send him your way16:02
=== Cimi_ is now known as Cimi
lamalexdidrocks, what was that file you told me to look at for variables in .in files via cmake?16:25
didrockslamalex: in tools/, look at the CMakeLis… and unity.cmake16:25
lamalexthanks16:27
lamalexsomeone done broke my launcher16:33
lamalexbrb16:34
=== bregma is now known as bregma|away
rbnswartzAnabody know how get the number of workspaces using the unity source code?16:43
seb128ok16:46
seb128didrocks, commited unity r32916:46
seb128in the packaging16:47
njpatelrbnswartz, there is a merge proposal doing trying to do the same thing, sam added some comments on how it should be done16:47
njpatelone sec16:47
didrocksseb128: thanks16:47
njpatelrbnswartz,  https://code.launchpad.net/~ruben-verweij/unity/fix-677594-workspaces/+merge/4306016:48
rbnswartzOkay thanks I was trying to fix one of the bite sized bugs I guess I'll switch over to fixing another one.16:50
rbnswartznjpatel Okay thanks I was trying to fix one of the bite sized bugs I guess I'll switch over to fixing another one.16:50
jonorbnswartz, working on bitesize bugs? sweet :-)16:53
lamalexdidrocks, is there a difference between SET and set in cmake?16:54
rbnswartzjono, Trying too.16:54
didrockslamalex: not that I know of16:54
jonorbnswartz, :-)16:55
rbnswartzThanks for all the help njpatel. bye.16:55
njpatellamalex, they used to use capitals and now they don't16:56
njpatellamalex, welcome to CMake16:56
lamalexhttps://code.launchpad.net/~alexlauni/unity/optional-debugging/+merge/43244 if someone would be so kind17:10
njpateldbarth_, why shouldnt' introspection be on in final release?17:14
njpateldbarth_, it has no operational cost during runtime17:14
lamalexnjpatel, I still can't arrow through menus17:30
lamalexand also my launcher has since become totally f'd up http://imgur.com/aQAM817:31
cando_i confirm this, same for me..17:31
njpatellamalex, did you restart the panel service?17:31
njpatellamalex, I didn't touch the launcher, ask DBO17:31
DBOlamalex, I am getting the same issue after merging trunk17:32
DBOinto my branch17:32
DBOsomeone else f'd it up17:32
njpatelDBO, urgh, take a look in trunk please17:33
njpatelffs17:33
DBOi am already doing it17:33
lamalexnjpatel, I did restart panel service17:33
lamalexI rebooted, and installed your debs17:33
lamalexnot in that order17:33
cando_lamalex, maybe revision 685?17:34
njpateldidrocks, https://launchpad.net/bamf/0.2/0.2.6617:34
njpatellamalex, debs were for something else. Let me check with trunk, something might have screwed it up17:34
didrocksnjpatel: thanks17:34
njpatellamalex, it works for me. I need to start the correct service from the install location before unity starts to make sure it's not using the system one (dbus activation)17:38
njpatelbut it works fine17:38
njpatellauncher is fully f*cked, though17:38
lamalexnjpatel, how do you start the right service? I usually just do kill `pidof unity-panel-service` and then run /usr/lib/unity/unity-panel-service17:38
lamalexah17:39
njpatellamalex, are you installing into /usr?17:39
lamalexno, I just realized why it's not working17:39
lamalexbecause I'm running /usr/...17:39
lamalexand not /usr/local17:39
njpatelright17:39
lamalexnjpatel, how do you get things on your panel with panel-service from trunk17:39
njpatellamalex, killall unity-panel-service; /usr/local/lib/unity/unity-panel-service &17:45
njpatelthen things should start working17:45
njpateldidrocks, unity release is delayed as trunk is messed up17:47
didrocksnjpatel: well, I guess you can imagine what is still building in my pbuilder…17:47
didrocks(hint: begin with n, finished by ux)17:48
lamalexnjpatel, well, now unity is just crashing17:48
didrocksargh FTBFS…17:51
* didrocks will cry :)17:51
lamalexI'm getting a crash from nux::Object now17:52
lamalexlooks like 149 may have f'ed things up for me17:52
lamalexhttp://pastebin.com/92E1WspP17:54
DBOwtf is up with the double launcher17:54
DBOI dont get it17:54
njpatelneither do i17:54
DBOoh duh17:55
njpateland lamalex's error is fucked17:55
DBOI see it17:55
DBOmerge fail17:55
njpatelit's like we're being called twice17:55
lamalexwas it my fault?17:55
lamalexI bet it was..17:55
DBOI think so17:55
DBOdouble launcher fixed17:57
DBOnjpatel, sorry for the delay, was on the phone with design17:57
njpatelnp17:57
DBOI have a lunch date in 3 minutes17:58
DBOgotta run17:58
DBOback in an hour17:58
njpatelDBO, thanks17:59
lamalexsorry guys :\17:59
lamalexI don't really know how that happened, what's weird is that there's no matchin + initLauncher call elsewhere18:01
njpatelno biggie, just make sure to check the system before pushing18:01
njpateldidrocks, I'm rolling unity now then18:01
didrocksnjpatel: ok, in any case, I can't upload anymore this evening18:03
didrocksnjpatel: there is no more archive admin around and I have two new binary packages to NEW18:03
didrocksoh, seb128 is back?18:04
didrocksso yeah, let's check that, I *can* :)18:04
=== ssj6akshat is now known as ssj6akshat|sleep
lamalexso nux is still broken for me..18:07
didrockslamalex: there is an abi break, ensure you rebuild unity against it18:08
=== ssj6akshat|sleep is now known as ssj6akshat
lamalexah18:08
njpateljaytaoko, did you actually merge your event faker class?18:09
nmarqueskenvandine, fixed ;) awesome stuff18:09
didrocksnjpatel: there is one in nux18:09
nmarqueskenvandine, the indicators are building now.... thx for your great help :)18:09
njpateldidrocks, yeah, the one in nux18:10
jaytaokonjpatel: I added the support in nux18:10
njpatelokay, so it's a local problem, sorry18:10
jaytaokonjpatel: but the event faker itself is in unity18:10
seb128didrocks, ?18:10
didrocksseb128: I'll have some bin packages to NEW for you :)18:11
kenvandinenmarques, anytime!18:11
seb128didrocks, I'm on my way off to sport now and I'm not sure I will come back tonight18:11
kenvandineseb128, one sec18:11
seb128so maybe ping cjwatson or pitti18:11
didrocksseb128: ok, then, it will be for tomorrow I guess :)18:11
lamalexdidrocks, I've rebuild unity a number of times18:11
didrocks(pitti yeah)18:11
seb128didrocks, or that18:11
didrocksseb128: enjoy sport and you week-end! :)18:12
kenvandineto get indicator-datetime to build... we need geoclue to get promoted to main... the MIR was approved18:12
seb128didrocks, thanks18:12
kenvandineshould i ping an archive admin?18:12
seb128kenvandine, did you upload already?18:12
njpateljaytaoko, thanks, it's builds now, was just a weird error18:12
kenvandineyes18:12
kenvandineit's been sitting for a couple days waiting18:12
seb128kenvandine, ok, just let a comment on #ubuntu-devel18:12
kenvandinenow i just uploaded again to make sure it builds with the new dbusmenu18:12
seb128if nobody pick it up I will promote it later18:13
kenvandinethx18:13
seb128but I need to run now18:13
kenvandinehave fun!18:13
kenvandineall the indicators are uploaded18:13
kenvandine:)18:13
didrocksgord: wooots bug #687403! Where is the syntax guide?18:13
ubot5Launchpad bug 687403 in unity (Ubuntu) "Add desktop action support to launcher quicklists" [Undecided,New] https://launchpad.net/bugs/68740318:13
tarethWho should I send my Canonical contributer agreement to for unity?18:14
didrocksjcastro: ^^18:14
tarethOk thanks :)18:15
didrockstareth: no no, I'm asking jcastro to answer :)18:15
tarethoh nvm that wasn't an answer to my question18:15
gorddidrocks, its the same as the indicator stuff18:15
tarethmy brain completely ignored that colon18:15
didrocksgord: can we chat about it tomorrow? :)18:15
jcastrotareth: one sec.18:15
didrocksgord: just for a refresh ;)18:15
gorddidrocks, just make sure that ShowOnlyIn isn't set to MessagingMenu ;) just needs to be Unity18:15
gorddidrocks, sure18:15
jcastrohttp://www.canonical.com/contributors18:16
didrocksgord: thanks!18:16
jcastrotareth: step 2, the person you want to CC to is david.barth@canonical18:16
lamalextareth, if it's a unity fix you send it to david barth18:16
lamalexhis email is on that page jcastro just linked you to18:16
jcastrotareth: out of curiosity what bug are you working on?18:16
tarethhttps://bugs.launchpad.net/unity/+bug/686182 and https://bugs.launchpad.net/unity/+bug/68187118:17
ubot5Ubuntu bug 686182 in unity (Ubuntu) "Unity launchers run multiple copies of program if clicked multiple times before the program loads" [Low,Triaged]18:17
njpateldidrocks, https://launchpad.net/unity/3.0/3.2.618:20
kvalowtf, adding debug messages fixes the bug :-O18:21
njpatelkvalo, yay for memory errors18:23
njpateldidrocks, bugs updated too18:23
kvalonjpatel: or a nasty race somewhere :/18:23
njpateltrue, true18:24
=== gord is now known as gord|afk
didrocksnjpatel: thanks!18:25
didrocksnjpatel: did you change something on the bug list in the last 15 minutes?18:25
njpateler, nope18:25
didrocksnjpatel: perfect! :)18:25
njpatelI might go collapse for a little while now18:25
didrocksnjpatel: sure should be good18:25
njpateldidrocks, mail me if there are issues, otherwise jaytaoko/DBO can help :)18:26
didrocksnjpatel: enjoy your week-end dude! :)18:26
njpateldidrocks, you too :)18:26
kvaloweekend?18:26
didrocksnjpatel: I'll bother them, no worry :p18:26
didrocksnjpatel: thanks )18:26
didrocks;)18:26
kvalois it friday already?18:26
njpatelkvalo, taking tomorrow off18:26
njpatelunless something goes drastically wrong, that is ;)18:26
kvalonjpatel: oh, enjoy then!18:26
kvalonjpatel: you should have an accident with your router to prevent that ;)18:26
njpatelkvalo, heh, thanks18:38
coz_howdy doody18:38
cyphermoxkvalo, ping, do you know why geoclue depends on ofono?18:41
kvalocyphermox: depends? not recommends or suggests?18:42
cyphermoxI don't know, but I would guess it's a depends18:42
kvalocyphermox: no idea18:42
cyphermoxhrm, maybe not18:42
kvalocyphermox: some people have been working for gps interface to ofono, but I doubt that's the reason18:42
cyphermoxmkay. well, I'll check it out. I can't find something looking quickly over apt-cache show...18:44
didrocksjcastro: do you have the trash icon bug # handy?18:47
jcastrodidrocks: https://bugs.launchpad.net/unity/+bug/68324118:48
ubot5Ubuntu bug 683241 in unity (Ubuntu) "Recycle bin icon is empty when there are items in the bin" [Low,Triaged]18:48
jcastrojamalta: I added the C++ style stuff to that page18:48
didrocksjcastro: thanks :)18:49
jamaltajcastro: ah awesome, thanks!18:49
jamaltawell, i already have some stuff to fix18:50
=== ssj6akshat is now known as ssj6akshat|sleep
jcastroAnyone who keeps DBO busy gets my vote. :p18:52
kvaloanyone have time for a quick review? https://code.launchpad.net/~kvalo/indicator-network/bug-686415/+merge/4326918:56
didrocksok, unity and nux pushed18:59
didrocks(bamf was aready done)18:59
jamaltafirst conflicts :X19:00
=== bregma|away is now known as bregma
didrocksDBO: btw, just before going away, I just noticed your minimize effect on the launcher icon. You rock!20:21
DBOoh20:21
DBOthanks :)20:21
didrocksDBO: I reconnected for that ;)20:21
didrocksnow, time to go20:21
DBO:)20:21
didrocksbye bye!20:21
jamaltadidrocks: holy crap it is awesome20:21
jamaltaand my user list is broken... silly empathy20:21
didrocksjamalta: yeah, really ;)20:21
jamaltaDBO: hey20:21
jamaltaso i think i've got the issue pretty much worked out except i have a minor bug that i can't figure out20:22
DBOhey jamalta :)20:22
jamaltacan i bug you sometime today about it?20:22
DBOsure20:22
DBOhit me with it now20:22
jamaltawell, basically i think i'm missing something regarding input grabbing20:22
jamaltabecause the quicklist doesn't handle hovers correctly20:22
jamaltauntil you click on a quicklist, then it will work fine for all quicklists for the rest of the session20:22
DBOcan you describe that better?20:23
DBOIRC is very unclear :)20:23
jamaltaDBO: Let me try...20:23
jamaltaSo, basically.. when you first launch Unity (compiz) with my code, and right click on an icon on Launcher, the quicklist items won't show a hover state20:23
jamaltaWhen you click on the Quicklist menu, the action you clicked on will run correctly.20:24
jamaltaBut then, all quicklist menus opened after that work fine...20:24
jamaltaActually that last part is a lie... it doesn't always work after clicking on it20:25
DBOokay20:27
jamaltaDoes that make any sense?20:27
DBOyeah for the most part20:27
DBOI need to check your branch out20:27
DBOwhere is it?20:27
jamaltalp:~jamalta/unity/683261-autohide-quicklist20:28
jamaltabut i think, other than that issue and some minor cleanup everything should be set20:29
DBOworking on it20:29
DBOgive me a couple minutes to figure it out20:30
jamaltaDBO: no problem, thanks so much20:30
DBOthank you20:30
DBOyou are super :)20:30
DBOso you work for flickr?20:30
jamaltayeha20:30
jamaltayeah*20:31
DBOwhere are they based?20:31
jamaltasan francisco20:32
DBO(i hope you dont mind the small talk, I like chatting up contributors while i review)20:32
jamaltaoh no problem, i don't mind20:32
jamaltai think you're EST, right?20:32
DBOI am20:32
DBOlocated in Michigan20:32
jamaltacool! where from?20:32
DBOKalamazoo if you have heard of it20:32
jamaltaoh awesome :)20:32
jamaltaI haven't, but my mom's side of the family is from that area20:32
jamaltabut to be honest, i'm not quite sure where they live :X20:33
DBOhaha, it happens20:33
DBOits the lower left part of michigan20:33
DBO3 hours from chicago, 2 hours from detroit20:33
jamaltaman, i bet its cold there20:33
DBOyeah20:33
DBOwe get the lake effect pretty strong20:33
jamaltawow20:33
DBOstill, I prefer snow to heat20:34
jamaltaheh, i wouldn't know really..20:34
jamaltabut i'm not built for heat at all20:34
jamaltai mean, cold...20:34
jamaltai grew up in tropical weather20:35
DBOhmmm20:36
DBOthis is interesting20:36
DBOit keeps crashing for me20:36
jamaltauh oh20:36
DBO/home/jason/staging//bin/compiz: symbol lookup error: /home/jason/staging/lib/compiz/libunityshell.so: undefined symbol: _ZN16QuicklistManager7DefaultEv20:36
DBOit looks like a failed compile or something20:36
DBObut I did a make clean to check20:36
jamaltai think you have to run cmake again20:36
jamaltaso it adds the new files20:36
DBOI'll retry :)20:36
DBOoh you know what20:36
DBOI did the cmake after I merged trunk20:37
DBOnot your stuff20:37
DBOthanks :)20:37
DBObeautiful20:37
DBOyep20:37
DBOI see there is a hover issue20:37
DBOlooking at it now20:37
jamaltayeap, and its frustrating :(20:37
jamaltathanks!20:38
DBOhey you cleaned up the dup code :)20:38
jamaltaDBO: yeah :)20:39
DBOin QuicklistView::Hide you check IsVisible twice, I think you meant to check it and _enable_quicklist_for_testing20:39
jamaltaoh oops! yeah that's what I meant to do20:40
DBOrather than nested if's I would use &&20:42
DBOwe dont liek nesting :)20:42
jamaltain fact, i'll move those to the same20:42
jamaltaerr20:42
jamaltayeah20:42
jamaltawhat you just said :)20:42
DBO:)20:42
DBOinteresting bug20:43
DBOyou did a nicer implementation than I expected20:43
jamaltathanks :) glad you're liking it20:44
jamaltaand yeah, i've been wrapping my head around this for almost two nights now... specially confusing since i don't completely understand what all the input stuff is doing20:44
jamaltai just made sure those calls were being made just as they had been20:44
jamaltaanyways, if you do a pull you should have the fix for QuicklistView::Hide20:44
DBOjamalta, figured it out20:48
DBOin your Show () method20:48
DBOmake ShowWindow (true) the first thing done in the block, not the last20:48
DBOalso mark that with a fixme comment20:49
DBOI need to fix that in nux since it really shouldn't matter20:49
DBOoh man it works great jamalta, super work20:50
jamaltathanks! :)20:50
DBOmake those fixes20:50
DBOpropose merge20:50
DBOand I'll merge it :)20:50
=== jamalta_ is now known as jamalta
jamaltaDBO: awesome it wokrs :D20:53
jamaltai would've never figured that out on my own, thanks!20:53
DBOjamalta, no biggie :)20:55
jamaltaDBO: ok i've submitted a MP20:58
DBOawesome20:58
* DBO goes to review20:58
jamaltaDBO: just noticed that i left a ton of commented out stuff in LauncherIcon::RecvMouseEnter21:01
DBOjamalta, I'll delete it on merge21:02
jamaltathanks21:02
jamaltaalso, what do you think the block starting at line 137?21:03
jamaltai commented "We probably should let QuicklistManager deal with this case..." and meant to ask about it21:03
jamaltaquicklist manager already takes care of hiding the quicklist showing, so do we even need to do that?21:03
* jamalta goes test to make sure he's not making stuff up21:03
DBOjamalta, I dont mind either way really21:03
DBOyou could indeed let the manager handle it21:04
jamalta(the line number in reference was from the MP diff)21:04
DBObut being explicit is fine too21:04
DBOyou are merged my friend21:04
jamaltaDBO: oh sweet! :)21:04
jamaltaty21:04
DBOkudos are in order, a cake maybe21:05
jamaltahaha21:05
* RAOF resists the obvious comment21:06
jamaltai'd rather replicate that functionality21:06
jamaltamind if i push to the branch with that removed?21:07
jamaltajust tested it and it works fine when it is removed21:07
jamaltathe manager is hiding the quicklist correctly before displaying a new one21:07
jamaltaDBO: if i make that change, should i push to a new branch, or is the same branch ok?21:11
DBOnew branch since you are merged21:12
DBOis the change just to delete those lines?21:12
jamaltaDBO: yes, just removing that whole block21:12
DBOokay, I'll do it really quick21:12
jamaltaDBO: ok thanks!21:12
DBO:) done21:14
jamaltaDBO: sweet, awesome.. thanks so much21:14
DBOyep21:14
jamaltaDBO: is there another bug you'd like to throw my way? i can probably work on it over the weekend21:16
DBOuhhhhhm21:16
jcastroI have some!21:16
jcastrohttps://bugs.launchpad.net/unity/+bugs?field.tag=bitesize21:16
jcastroany of those!21:16
jcastrojamalta: ^^21:17
jcastrobut if they're too easy I'm sure DBO has more21:17
DBOjamalta, this one could be fun and easy21:17
DBOhttps://bugs.launchpad.net/unity/+bug/68795621:17
ubot5Ubuntu bug 687956 in unity (Ubuntu) "should display the launcher tooltips after a delay" [Medium,Triaged]21:17
DBOthere are a lot of got-yas in there21:18
DBOit will take a bit to get the corner cases right21:18
DBObasically there should be a way for the launcher to say "okay, it's now okay to show tooltips"21:19
DBOand on enter it sends that signal 1 second after enter21:19
jamaltajcastro: heh, i forgot about that.. ty :)21:19
DBOyeah some of the bite size bugs are not so bite sized21:20
DBObut that tend to happen21:20
DBOif its truly bite sized it means we tend to eat it for lunch :P21:21
jamaltahaha21:21
jamaltayeah i can take a look at this bug21:21
jamaltaDBO: so would it be ok to set a bool flag when the signal fires off, so that the next mouseover knows to show the tooltip right away?21:22
jamaltathen, when they mouse out, it would be set back to false21:22
jamalta(this is regarding mark's comment on that bug)21:22
DBOjamalta, ideally it would be a static method in the LauncherIcon class21:23
DBOstatic void LauncherIcon::SetTooltipsAllowed (bool allowed)21:24
jamaltaDBO: ah ok21:24
DBOthat would then fire off a signal or something internally into the LauncherIcon instances21:24
DBOthe one that is hovered could then hide/show its tooltip21:24
jamaltaDBO: should i be using g_timeout_add for the timer?21:25
jamaltai haven't worked with timers in C++ yet21:25
DBOyep21:25
jamaltaok cool21:25
DBOg_timeout_add is a C call21:25
DBOso the callback you give it must be static21:25
DBOyou'll see I use it a lot of places21:26
jamaltaah, ok21:26
jamaltathat makes sense21:26
jamaltathen i'll start digging into this tonight and bring up any questions i come up with tomorrow21:26
DBOawesome21:26
DBOyou're a hero21:26
jamaltaDBO: thanks :).. i'm just happy to be helping, and learning a ton too21:28
DBOwhat do you do at flickr?21:28
jamaltai haven't had a chance to work with c++/c much before this21:28
jamaltai'm a backend developer21:29
DBOdoing frontends can be fun too :)21:29
jamaltaDBO: yeah hehe :)21:30
DBOI cant imagine going to work for a closed shop after working for Canonical21:30
jamaltaDBO: heh i know what you mean21:30
jamaltai love the idea of working on an open source project21:30
jamaltaalthough i also enjoy what i do here21:30
jamaltait has been a great experience for me so far.. i've learned a lot working on such a high traffic website.21:31
DBOI just keep thinking about these image hosting websites21:32
DBO"somewhere in that company, it is someones job to view all the questionable material and delete it"21:32
DBOhe must not sleep at night...21:32
jamaltaDBO: tell me about it21:33
jamaltai don't know how they do it21:34
jamaltaworking on our moderation tools is not fun :(21:34
DBOhaha21:34
DBO"This is a pineapple... this however... is not"21:34
jamaltahaha21:35
DBOI was reading on reddit the other day21:36
DBOthe dude who does imgur (still a one man show I guess), does all the moderation by hand...21:36
jamaltaDBO: wow... that's insane21:37
DBOjamalta, I fear for his soul21:38
DBOit must be crushed21:38
jamaltai didn't know they even did moderation21:39
jamaltabut yes, i completely agree21:39
DBOits limited, I think it comes down mostly to "things the FBI might arrest me for"21:40
jamaltathat's the scariest thing about running a site like that21:41
jamaltathere is so much liability21:41
DBOI have to go now, girlfirend needs a ride home21:42
jamaltaDBO: np, have a good one21:42
kvalotedg: hi, do you have time to review this: https://code.launchpad.net/~kvalo/indicator-network/bug-686415/+merge/4326921:49
tedgkvalo, Sorry, not really.  Turns out I broke natty :)21:51
tedgbratsche, ^ ?21:51
kvalotedg: heh, can't argue with that :)21:51
* bratsche takes a look21:54
kvalotedg: I hope you get natty working :)21:54
kvalobratsche: thanks21:55
bratschekvalo: Is it necessary to g_object_ref (priv->default_service) ?21:56
bratscheErr, nm.. I get it.21:57
kvalobratsche: I guess it would be possible to avoid the ref, but it would get too complicated for me ;)21:57
bratscheNo I was thinking that it was creating an object there, but it looks like it's not.21:58
bratscheSo nevermind, it looks fine.21:58
kvaloah, ok21:58
bratscheI thought it was creating an object and then it would have a refcount of 2.. and then the unref wouldn't free it.21:58
bratschekvalo: I didn't actually test it, but the code looks good.  Approved. :)21:59
kvalobratsche: thanks!21:59
kvalokenvandine: I know you are extremely busy, but still fyi: https://launchpad.net/indicator-network/trunk/0.3.122:11
kenvandinekvalo, cool22:11
kenvandinekvalo, that requires the newer connman doesn't it?22:17
kenvandineoh... nm22:17
kvalokenvandine: new connman is already in natty, but ofono upload is still pending22:20
kvalokenvandine: but a versioned dependency to connman would be nice22:20
kenvandineor maybe it does22:21
kvalokenvandine: connman 0.64 is the version which is needed22:22
=== Schendje-2 is now known as schendje
kvalokenvandine: thanks for the upload23:18
kvalokenvandine: why does geoclue build depend on ofono? that's just weird23:18

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!