[00:27] <ion> 17 hours and Apport retracing service still hasn’t retraced the crash report. :-P
[00:30] <ion> Ah, missed seb128’s message.
[05:31] <ion> My issue had already been reported. LP #686698.
[06:51] <kvalo> morning
[06:52] <ion> that.
[07:06] <kvalo> ion: moro
[07:06] <ion> hi
[07:31] <didrocks> good morning
[07:44] <smspillaz> mornin
[07:45] <smspillaz> didrocks: could you check something for me?
[07:45] <didrocks> hey smspillaz
[07:45] <didrocks> smspillaz: sure, not right now, but I can
[07:45] <smspillaz> didrocks: have you got two screens ?
[07:46] <didrocks> smspillaz: no, only one there…
[07:46] <smspillaz> hmm ok
[07:46] <smspillaz> didrocks: I'm wondering whether this thing that gnome-panel does is a feature or a bug
[07:47] <smspillaz> basically 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] <smspillaz> err, the panel needs to be on the right hand side, rather
[07:47] <smspillaz> actually, on the left (slightly dyslexic)
[07:47] <smspillaz> anyways
[07:47] <smspillaz> if you have that, and then maximize a window on screen 1, it will go underneath the panel
[07:47] <smspillaz> this happens with metacity and compiz
[07:48] <smspillaz> I checked with xprop, it seems that the panel does not set the _NET_WM_STRUT for that part of the screen
[07:48] <smspillaz> it 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 behaviour
[07:52] <didrocks> smspillaz: ok, so if you have two screens
[07:52] <didrocks> and gnome-panel is on the left
[07:53] <didrocks> you mean that it's doesn't set WM_STRUT and so, apps go underneath?
[07:53] <smspillaz> didrocks: yes
[07:53] <smspillaz> but only on panels that are adjacent to other screens
[07:54] <smspillaz> it would seem like this is not the desired behaviour ...
[07:54] <didrocks> smspillaz: weird, WM_STRUT is set screen by screen, isn't it?
[07:54] <didrocks> so there is no reason to not restrain the area on the one with the panel
[07:58] <smspillaz> didrocks: it should be set by the output
[07:59] <smspillaz> I'm just compiling docky now to see what the behaviour should be
[08:27] <MacSlow> greetings everybody
[08:29] <didrocks> morning MacSlow
[08:30] <MacSlow> didrocks, salut here too :)
[08:30] <didrocks> :)
[08:33] <smspillaz> reasons I hate autotools #24533452: It only tells you which dependencies you are missing one dependency at a time
[08:34] <MacSlow> smspillaz, :)
[08:35] <didrocks> smspillaz: reasons I hate cmake #${num_smspillaz}+1: cmake ignores it on some case and tell "let's take a chance" :-)
[08:35] <MacSlow> smspillaz, annoying indeed
[08:38] <RAOF> Heh.  Reason to hate cmake #whatever: CFLAGS?  What's that? ☺
[08:40] <didrocks> hey RAOF!
[08:40] <smspillaz> RAOF: you mean CMAKE_CXX_FLAGS_DEBUG ?
[08:40] <RAOF> Hey didrocks!  Looks like that segfault in mipmapping on r600 might be fixed in git.
[08:41]  * smspillaz w00ts
[08:41] <didrocks> RAOF: oh nice!
[08:41] <RAOF> smspillaz: One of the rather endeering parts of autotools is that setting the CFLAGS/CXXFLAGS environment variable DTRT.
[08:42] <smspillaz> I guess. Although cmake was really designed to be multiplatform
[08:42] <smspillaz> it is not a full buildsystem, it only generates native makefiles
[08:42] <smspillaz> so it doesn't do things like setting env vars
[08:43] <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] <smspillaz> really someone at debian needs to sit down and write proper macros for cmake
[08:43] <didrocks> RAOF: shell + cp ftw :)
[08:43] <didrocks> why?
[08:44] <smspillaz> because there are none?
[08:44] <didrocks> we have our packaging system :)
[08:44] <didrocks> so, no interest in writing something specific to cmake
[08:45] <didrocks> grrr, too much flickering when switching ws in compiz :/
[08:45] <smspillaz> really?
[08:45] <didrocks> smspillaz: yeah, the thing we discussed the other day
[08:45] <didrocks> smspillaz: like if you had a "magnet", not sure what's the best way to describe it
[08:46] <smspillaz> intel ?
[08:46] <didrocks> nvidia
[08:46] <smspillaz> so it's like a vsync issue ?
[08:46] <didrocks> exactly
[08:46] <didrocks> quite disturbing
[08:47] <smspillaz> that's nvidia being stupid. You need to go into nvidia-settings, -> opengl settings -> sync to vblank, then ccsm -> opengl -> sync to vblank and restart compioz
[08:47] <smspillaz> that will fix it
[08:47] <didrocks> and I have X damage not triggered when scrolling sometimes
[08:47] <didrocks> smspillaz: hum, let me see if it fixes it
[08:47] <smspillaz> didrocks: ccsm -> workarounds -> force X to GLX sync
[08:47] <didrocks> smspillaz: the thing is that we should do a distro choice there
[08:48] <smspillaz> perhaps
[08:48] <didrocks> we can't ask our users to do that :)
[08:48] <didrocks> not perhaps "certain" :)
[08:48] <smspillaz> didrocks: it should be enabled by default
[08:48] <didrocks> smspillaz: even for non nvidia drivers?
[08:49] <smspillaz> didrocks: in the detection module we can distro patch workarounds  to enable "force x to glx sync" if an nvidia card is detected
[08:49] <didrocks> smspillaz: agreed
[08:49] <didrocks> ccsm -> opengl -> sync to vblank is already checked
[08:49] <didrocks> let me see the workarounds plugin
[08:49] <smspillaz> didrocks: check nvidia settings
[08:49] <smspillaz> that's where it counts
[08:49] <didrocks> smspillaz: yeah, I checked it
[08:50] <didrocks> what's the difference between the workarund setting and the opengl one?
[08:50] <smspillaz> workaround setting fixes damage issues
[08:50] <smspillaz> because the nvidia driver is stupid
[08:50] <smspillaz> so we have to slow down compiz a bit
[08:51] <didrocks> smspillaz: heh, you are telling intel is stupid, nvidia is stupid, which next? :p
[08:51] <didrocks> smspillaz: all checked, we'll see
[08:52] <didrocks> smspillaz: should the workaround plugin be enabled by default?
[08:52] <didrocks> smspillaz: i mean, I enabled it, and so, I have other settings now that I didn't have before
[08:54] <smspillaz> didrocks: sure
[08:54] <smspillaz> didrocks: disable by default the "show previews of minimized windows" though, that has random bugs that I'd rather not hit for no reason right now
[08:55] <smspillaz> (since it tells all kinds of lies to X11 to work correctly)
[08:56] <smspillaz> didrocks: hm, ok with docky I don't see the maximization issue
[08:56] <didrocks> smspillaz: it's not activated by default
[08:56] <smspillaz> didrocks: ah ok good :)
[08:56] <didrocks> smspillaz: what I have are:
[08:56] <didrocks> OOo Menu Fix
[08:56] <didrocks> Java Window Fix
[08:56] <didrocks> Java Taskbar Fix
[08:56] <didrocks> AIGLX Fragment Parameter Fix
[08:57] <didrocks> and of course, the Synx X and GLX :)
[08:57] <smspillaz> you can disable AIGLX Fragment Parameter fix
[08:57] <smspillaz> it's not necessary any more
[08:57] <smspillaz> IMO I should write a patch for workarounds where we will enable Sync X and GLX only if there is and NVIDIA card
[08:58] <didrocks> smspillaz: right
[08:58] <didrocks> smspillaz: and the others?
[08:58] <smspillaz> didrocks: the others are find
[08:58] <smspillaz> *fine
[08:58] <didrocks> what's the OO menu fix?
[08:58] <smspillaz> didrocks: OO menus don't register as type "menu"
[08:59] <didrocks> smspillaz: so, why OO is enabled and not firefox/thunderbird?
[08:59] <smspillaz> didrocks: *shrug*
[09:00] <didrocks> which means? ;)
[09:00] <smspillaz> didrocks: maybe the person who wrote them didn't enable them by default
[09:00] <didrocks> smspillaz: so it should as well, I guess?
[09:00] <smspillaz> didrocks: only if you see problems
[09:00] <didrocks> I didn't see any issue with OOo before enabling the plugin
[09:01] <smspillaz> ok :)
[09:03] <didrocks> grrr, tomboy crashes… hard to take a note this way :)
[09:04] <smspillaz> didrocks: hmm, have you been getting random crashes with an error like "call to empty boost::function ?"
[09:04] <didrocks> smspillaz: no, I've been quite lucky like that, all is pretty stable to me (just had a crash this morning)
[09:05] <didrocks> s/like that/on that side/
[09:08] <didrocks> smspillaz: it didn't fix the sync issue
[09:08] <didrocks> hum… maybe I should restart X to take the nvidia settings into account
[09:08] <smspillaz> didrocks: perhaps
[09:08] <smspillaz> didrocks: it fixed it here at least
[09:08] <smspillaz> didrocks: although it doesn't work on multiple monitors
[09:09] <didrocks> well, not my case there :)
[09:09] <kvalo> arghhhh. 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 _SCRIPTS
[09:15] <smspillaz> didrocks: have you got this commit in the packages ? http://git.compiz.org/~dbo/compiz-with-glib-mainloop/commit/?h=glibmm-experimental&id=3e1c09f13eb99af88bf0bd8d73be8bbcf533ded7
[09:15] <RAOF> didrocks: You shouldn't need to restart X; in fact, restarting X will probably revert the changes :)
[09:16] <didrocks> RAOF: smspillaz: ok, in that case, it doesn't fix it :)
[09:17] <didrocks> RAOF: 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:18] <didrocks> smspillaz: I think I have, let me check
[09:22] <kvalo> didrocks: 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 tricky
[09:23] <didrocks> kvalo: depends if you only care about the package or also the upstream tarball + make && make install
[09:24] <kvalo> didrocks: good point
[09:24] <didrocks> smspillaz: I'm still opening a bug for that issue (nvidia/sync), so that we can get it tracked
[09:24] <kvalo> didrocks: I'll try to make the autofoo magic work. thanks
[09:30] <didrocks> smspillaz: the commit is in
[09:45] <smspillaz> didrocks: huh, ok
[09:45] <smspillaz> didrocks: I thought that commit fixed it :/
[09:46] <didrocks> smspillaz: did you check the bug report mentionned the last ubuntu compiz package?
[09:46] <didrocks> smspillaz: alpha1 version has it (1:0.9.2.1+glibmainloop2-0ubuntu4) maybe even ubuntu3
[09:46] <lamalex> jcastro, didn't njpatel present a diagram like the one you ask for in ayatana-dev at uds?
[09:47] <lamalex> or ted at gnome summit
[09:47] <smspillaz> didrocks: no not yet, but I am still getting that crash
[09:47] <smspillaz> let me dist upgrade
[09:47] <seb128> lamalex, well then to start this diagram should be on the unity.ubuntu.com webpage
[09:48] <lamalex> ping ted when he's awake
[09:48] <seb128> lamalex, the diagram is also not enough, we need things like "if you want to use appindicator you should check those apis"
[09:48] <seb128> with the list of libraries that you might have to use to write an indicator
[09:48] <seb128> and a short explanation of what they do
[09:49] <lamalex> seb128, right
[09:49] <lamalex> we need awesome developer docs
[09:49] <RAOF> Hm.  While I wait for mesa to build, is there any particular reason why unity-panel-service should take up 200MB RSS?
[09:50] <seb128> that's being worked but will take a bit
[09:50] <seb128> (documentation)
[09:50] <seb128> until now we could do a better job to at least point the sources used and what they do
[09:50] <seb128> RAOF, out of leaks you mean? ;-)
[09:51] <RAOF> seb128: That thought had crossed my mind, yes :)
[09:58] <didrocks> lamalex: hey, seeing your comment on the merge, what's the difference between LAUNCHER_ICON_QUICK_STARTING and LAUNCHER_ICON_QUICK_LAUNCHING ?
[10:02] <lamalex> didrocks, there is no LAUNCHING
[10:02] <lamalex> didrocks, I just updated that comment, thanks
[10:02] <lamalex> I'm sick :\
[10:03] <didrocks> lamalex: oh? the grey weather of London hit you?
[10:13] <dbarth> salut seb128
[10:13] <dbarth> seb128: about https://bugs.launchpad.net/unity/+bug/686603
[10:13] <seb128> lut dbarth
[10:13] <dbarth> seb128: you get that on the console, i mean it's a debug log?
[10:14] <seb128> I get that by running test-panel under valgrind
[10:14] <dbarth> ah right
[10:18] <dbarth> smspillaz: hey sam, have you seen mvo bug with the link.py testcase?
[10:19] <dbarth> smspillaz: that sounds like a scary regression
[10:20] <smspillaz> dbarth: no, could you link it to me ?
[10:20] <smspillaz> dbarth: also, could you post your config profile somewhere? I'm trying to track down the crash in windowInitPlugins
[10:55] <didrocks> kamstrup: "Why can't I ever get the email API for LP working!?!?11one" <- same here :)
[10:55] <didrocks> kamstrup: thanks, will remove the "there" :)
[10:56] <didrocks> kamstrup: exec didn't want if you want to embeeded in gdb probably
[10:56] <kamstrup> didrocks: ah, right
[10:56] <kamstrup> didrocks: I found the answer on https://help.launchpad.net/Code/Review I think
[10:56] <kamstrup> "review approve" needs a whitespace prefix
[10:56] <didrocks> kamstrup: oh?
[10:56] <kamstrup> " review approve"
[10:56] <kamstrup> :-S
[10:56] <kvalo> kamstrup: sorry dude, but I have a long merge request again: https://code.launchpad.net/~kvalo/indicator-network/settings-merge/+merge/43183
[10:56] <didrocks> kamstrup: yeah, that's what I'm doing
[10:57] <didrocks> kamstrup: all commands need a whitespace prefix
[10:57] <didrocks> kamstrup: but for unknown reason, it hates me :)
[10:57] <kvalo> kamstrup: but this is just a start, so I think you don't need to review it in so detail
[10:58] <kamstrup> kvalo: I think I may have to push it to monday
[10:58] <kamstrup> I really need to get the dee gdbus port out of the door
[10:58] <kamstrup> and friday is sprint
[10:58] <kvalo> kamstrup: monday is ok for me :)
[10:59]  * kamstrup rolls up sleeves to get those sodded unit tests running again
[12:12] <lamalex> when someone has a minute? https://code.launchpad.net/~alexlauni/unity/xids-introspection/+merge/43193
[12:17] <lamalex> njpatel_, didrocks, with CMake can we have unityshell.xml.in have variables in it that get set on build?
[12:18] <didrocks> lamalex: yes, look at what I've done some commits ago with unity.cmake in tools/ (it's the binary)
[12:18] <lamalex> didrocks, sweet thanks
[12:19] <htorque> hello everyone! don't know when it started, but window buttons now have tooltips again - should this be filed as bug?
[12:19] <didrocks> lamalex: so that to replace VERSION and tools/CMakeFile*
[12:19] <didrocks> htorque: it's already filed again compiz, I dropped my patch and I'm just working on it again :)
[12:19] <didrocks> htorque: I dropped it when going to 0.9
[12:19] <htorque> didrocks, nice, thanks!
[12:19] <didrocks> htorque: thanks for your excellent reporting and testing job :)
[12:20] <htorque> yw! thanks for all the fixes. ;-)
[12:25] <seb128> njpatel_, didrocks: so my launcher is acting weird
[12:25] <didrocks> seb128: oh weird?
[12:25] <seb128> like the launchers do a start animation when clicked
[12:25] <seb128> but they don't start anything
[12:26] <seb128> if I run firefox manually it doesn't match the pinned icon
[12:26] <seb128> but add a new one in the launcher
[12:26] <didrocks> seb128: is it an application you already started?
[12:26] <seb128> same for nautilus or gedit
[12:26] <seb128> well firefox and gedit have no running process
[12:26] <seb128> but I used them today yes
[12:26] <didrocks> like, you started/closed them 4 times?
[12:26] <seb128> could be, I didn't count
[12:26] <didrocks> seb128: I think you're striken by the ubuntu starter edition :)
[12:27] <didrocks> seb128: should be fixed in trunk btw :)
[12:27] <seb128> is there a way to reset the counter?
[12:27] <didrocks> seb128: killing the bamf service and restarting unity
[12:27] <didrocks> we didn't merge the fix last week because it added others issues IIRC
[12:28] <seb128> ok, compiz --replace did it
[12:28] <seb128> is that known that compiz crashes on --replaec?
[12:28] <seb128> well apport triggers
[12:28] <didrocks> seb128: bug #683623
[12:28] <seb128> yeah, I've read this one last week
[12:29] <didrocks> seb128: yeah, compiz isn't good when you give unknown args and unknown plugins
[12:29] <didrocks> IMHO, if it can't find a plugin, it should just ignore it
[12:29] <seb128> I just though it was when clicked 4 times in a row
[12:29] <didrocks> seb128: not in a row :)
[12:29] <seb128> didrocks, thanks
[12:29] <didrocks> seb128: yw
[12:30] <didrocks> seb128: btw, we will have the unity binary from now on
[12:30] <seb128> yeah, I noticed, great work
[12:30] <didrocks> so, will make things easier to reset the config and such
[12:31] <didrocks> well, not very difficult, just that I hate python with making some kind of tee-like behavior :)
[12:36] <didrocks> smspillaz: are you changing gtk/window-decorator/gtk-window-decorator.c with your decorator work?
[12:49] <smspillaz> didrocks: no, I am writing my own decorator
[12:49] <smspillaz> didrocks: well actually, I forked gtk-window-decorator (temporarily) and I am adding inactive/active shadow work
[12:49] <didrocks> smspillaz: ok, so I should maybe wait or set it to you the task of the option to hide tooltip?
[12:50] <didrocks> smspillaz: it was a distro-patch we had (that I submitted upstream as well some month ago).
[12:50] <smspillaz> didrocks: Our unity-window-decorator can neuter tooltips
[12:50] <didrocks> smspillaz: nice! so yeah, please do that :)
[12:50]  * didrocks drops his task then ;)
[12:50] <smspillaz> could you link me to the patch so I don't have to hunt around in the code ? ;-)
[12:51] <didrocks> smspillaz: well, you removed the gconf part in 0.9 so you will have to add it again
[12:51] <smspillaz> didrocks: did we want it as an option? I thought design said no tooltips ever
[12:51] <didrocks> smspillaz: but the distro-patch was http://paste.ubuntu.com/541431/ for 0.8
[12:52] <didrocks> smspillaz: well, I added an option to get it accepted upstream
[12:52] <didrocks> smspillaz: but you never answered!
[12:52] <didrocks> :)
[12:52] <smspillaz> didrocks: must have missed it
[12:52] <didrocks> smspillaz: so if it's our own, yeah, no worry, no option
[12:52] <smspillaz> didrocks: yeah, that's what I was thinking
[12:52] <didrocks> smspillaz: http://bugs.opencompositing.org/show_bug.cgi?id=1264
[12:53] <smspillaz> didrocks: ah right. I never checked the opencompositing bugs, I was too busy porting plugins :p
[12:53] <didrocks> smspillaz: no worry, patch tagging is useful to refind bugs even if the bugtracker is down :p
[12:53] <didrocks> smspillaz: so I'm assign you the relevant compiz bug, one sec
[12:54] <smspillaz> didrocks: yeah, the opencompositing bugtracker is (still) down
[12:54] <didrocks> smspillaz: you should use launchpad for upstream! ;)
[12:54] <smspillaz> we probably could
[12:54] <smspillaz> bugzilla is a PITA
[12:55] <klattimer> smspillaz: +1
[12:55] <didrocks> smspillaz: so, activating the workaround plugin with upstream defaults for now minus AIGLX one
[13:04] <lamalex> njpatel_, how does the launcher's autohide get set initially?
[13:04] <lamalex> does optionChanged get called once at start?
[13:06] <njpatel_> lamalex, I'm not sure about the compiz options stuff, sorry
[13:07] <lamalex> njpatel_, who is that? Jason?
[13:07] <njpatel_> yep
[13:07] <njpatel_> or smspillaz
[13:07] <lamalex> ok
[13:07] <lamalex> DBO, wake up
[13:18] <lamalex> njpatel_, could you please review https://code.launchpad.net/~alexlauni/unity/xids-introspection/+merge/43193
[13:19] <kvalo> ronoc: hi. time again for a review: https://code.launchpad.net/~kvalo/indicator-network/gdbus-cellular-crash/+merge/43204
[13:26] <jcastro> ok guys, added C++ to the style guide: https://wiki.ubuntu.com/Unity/CodingStyle
[13:28] <njpatel_> lamalex, will take a look in a bit
[13:30] <njpatel_> jcastro, you can remove everything from "Be Verbose" to before "Commit Messages", as it's all stated at the start of the C++ bit
[13:31] <jcastro> ok
[13:32] <jcastro> done
[13:32] <jcastro> njpatel_: what else is C++ after nux and compiz?
[13:47] <didrocks> njpatel_: 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> jcastro, that's it for now
[13:51] <njpatel> lamalex, approved with a comment
[13:51] <njpatel> please fix before merging
[13:52] <jcastro> lamalex: njpatel: any new people in merges yet?
[13:52] <lamalex> jcastro, waiting for one to sign CA
[13:52] <jcastro> I see 2 pending merges from people I don't know, so crosses fingers
[13:52] <njpatel> yes a few, but I'm not sure their success rate, looking now
[13:52] <jcastro> rock
[13:52] <lamalex> er, waiting for one to fix his branch and sign ca
[13:52] <jcastro> lamalex: 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 photo
[13:52] <jcastro> so I can put them on planet
[13:53] <jcastro> lamalex: just a fwd or CC is fine, I can handle introducing myself, etc.
[13:53] <lamalex> jcastro,  sure
[13:55] <didrocks> nooooooo, I'm starting to google "cmake" when I want to google "autotools"
[13:55] <didrocks> that's the end /o\
[13:57] <didrocks> njpatel: did you see my question about unity_support_test location and nux?
[13:59] <ssj6akshat> jcastro, I am always getting interrupted when I am about to write a post on the Bitesize bugs
[14:00] <ssj6akshat> I guess someone doesn't want to it to appear :P
[14:01] <jcastro> ssj6akshat: it's ok, it's a marathon not a sprint
[14:02] <kenvandine> didrocks, 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] <kenvandine> and imho it is a better use of space
[14:02] <jcastro> dbarth: speaking of, getting branches on these, I will need more! https://bugs.launchpad.net/unity/+bugs?field.tag=bitesize
[14:02] <didrocks> kenvandine: it will do that later the cycle, but currently compiz doesn't listen to gconf gnome keys
[14:02] <njpatel> didrocks, no, sorry. Erm, I don't know what that does so feel free to move it where you feel right :)
[14:03] <kenvandine> didrocks, humm... i have mine set to do that now
[14:03] <didrocks> njpatel: ok, it's the detection module at start, so definitively not an example :)
[14:03] <njpatel> right
[14:03] <didrocks> kenvandine: right, but it's not the GNOME keys, that means, it's not share with your desktop classic session
[14:03] <didrocks> njpatel: ok, doing and commiting
[14:05] <jcastro> njpatel: didrocks: while I have you here, post release I need more bugs kthx. :)
[14:06] <didrocks> jcastro: well, I'm tagging everything I can… i've reviewed all the bugs as you already know and add them :)
[14:06] <jcastro> I can't believe I am disappointed that there aren't enough bugs
[14:06] <jcastro> yesterday 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:07] <didrocks> jcastro: I guess once the dash/places will be there, we can have more of them
[14:07] <jcastro> yeah
[14:07] <didrocks> how many are still there?
[14:08] <jcastro> ~10
[14:09] <jcastro> but with these 2 pending ones we've had one per day since I announced the report on tuesday
[14:09] <kenvandine> didrocks, i see
[14:09] <jcastro> so I think it's off to a good start considering we're still just around A1
[14:09] <kenvandine> didrocks, ok, thats fine... glad it is planned
[14:09] <kenvandine> :)
[14:10] <didrocks> jcastro: right, little issues will come later, when it doesn't need refactoring everything :)
[14:12] <lamalex> njpatel, I don't think I should free the list from bamf
[14:12] <lamalex> I'm pretty sure bamf owns that list (granted bamf should probably be duping the list and then returning it)
[14:17] <njpatel> lamalex, the docs say it owns the contents but not the list, that's what I was going on
[14:18] <njpatel> lamalex, the code seems to suggest the same
[14:18] <ronoc> kvalo, done
[14:20] <kvalo> ronoc: thanks!
[14:21] <ronoc> kvalo, np
[14:21] <ronoc> agateau, got a sec?
[14:22] <agateau> ronoc: give me 5 minutes
[14:22] <ronoc> agateau, no rush
[14:29] <agateau> ronoc: ok, I am back
[14:30] <ronoc> agateau, one sec
[14:34] <ronoc> agateau, can see anything obvious why the submenu in this dbusmenu is not appearing on the indicator
[14:34] <ronoc> http://pastebin.ubuntu.com/541475/
[14:35] <lamalex> haha so which is worse, potential race conditions, or making a member variable public
[14:35]  * agateau looks
[14:35] <ronoc> lamalex, race condition any day of the week
[14:36] <lamalex> man but making a member variable public feels so wrong
[14:37] <agateau> ronoc: menu looks good
[14:37] <DBO> lamalex, i dont like waking up
[14:37] <ronoc> lamalex, I know, but sure isn't every python obj wide open
[14:37] <agateau> lamalex: making a member variable public can be a good way to get race conditions :)
[14:37] <tedg> DBO, I think the same thing about you waking up, but I didn't want to say anything ;)
[14:37] <lamalex> ronoc, yeah and it's one of the things I hate about python
[14:38] <DBO> tedg, can you turn that into a policy please
[14:38] <klattimer> lamalex: did you get any further with atk?
[14:38] <ronoc> lamalex, I take ruby over python any day of the week
[14:38] <agateau> ronoc: I think tedg built a tool to load a menu from json, you should try if your dump shows there
[14:38] <klattimer> I'm wondering about the event redirection issues i'm facing, but they may be the result of offscreen rendering
[14:38] <klattimer> still trying to work it out :/
[14:38]  * ronoc must stop saying 'any day of the week'
[14:39] <ronoc> tedg, good morning -> http://pastebin.ubuntu.com/541475/
[14:39] <seb128> ok
[14:39] <seb128> so is appmenu maintained and by who?
[14:40] <seb128> dbarth_, ^
[14:40] <ronoc> tedg, the submenu is not being rendered in the indicator
[14:40] <ronoc> tedg, can you see any problems there
[14:40] <seb128> doh! ;-)
[14:40] <ronoc> agateau, thx - might try this
[14:41] <tedg> seb128, indicator-appmenu is me, appmenu-gtk is bratsche
[14:41] <tedg> ronoc, looking
[14:41] <seb128> tedg, I guess with unity none is yours? ;-)
[14:41] <seb128> tedg, things like clicking on a menu activating another item than the one selected
[14:41] <tedg> seb128, ?  indicator-appmenu is used by unity
[14:42] <seb128> tedg, I confused with indicator-applet-appmenu
[14:42] <seb128> ignore me ;-)
[14:42] <tedg> seb128, If it's an item on the panel, that's njpatel, if it's in the actual menu that's me.
[14:42] <seb128> it's an actual menu item
[14:42] <seb128> I select "show conversations history" and it sends a "share desktop request"
[14:42] <seb128> it's quite embarassing
[14:43] <seb128> I just shared my desktop with someone I didn't want to talk to
[14:43] <seb128> twice
[14:43] <tedg> Hmm, that's odd.
[14:43] <tedg> Consistently?
[14:43] <seb128> when it happens yes
[14:44]  * tedg can't find anyone to share his desktop with :(
[14:44] <seb128> now the menu item just doesn't work
[14:44] <seb128> tedg, well, open a buddy list
[14:44] <seb128> double click on someone
[14:45] <seb128> go to contact, and select conversation history
[14:45] <tedg> Yeah, share my desktop is consistently greyed out.
[14:45] <seb128> does it work?
[14:45] <seb128> or does it do nothing?
[14:46] <tedg> It did nothing, but now I've got a menu from somewhere else?
[14:46] <seb128> I had that before as well
[14:46] <seb128> I got the nautilus menu at some point
[14:46] <seb128> but otherwise it does nothing
[14:46] <seb128> then I picked "info"
[14:46] <seb128> which send a share desktop request to dholbach
[14:47] <tedg> seb128, And you didn't want to talk to dholbach more than once? ;)
[14:47] <seb128> no, I'm just testing on dholbach now :p
[14:47] <tedg> Hmm, okay.  I'm not sure what's up there.
[14:47] <seb128> I had the issue first with someone else in my list
[14:48] <seb128> ok
[14:49] <tedg> We'll have to grab a dbus capture to see if who's wrong....  I of course think it's appmenu-gtk ;)
[14:50] <tedg> ronoc, 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] <tedg> ronoc, Can you try switching it to a standard item real quick and see if that fixes things?
[14:50] <ronoc> tedg, will do
[14:59] <ronoc> tedg, yup i was doing something (or not doing something) with the custom item, works with a normal item
[15:00] <ronoc> tedg, 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 menuitem
[15:01] <tedg> ronoc, 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:03] <njpatel> +1
[15:05] <ronoc> tedg, 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:06] <ssj6akshat> jcastro, http://www.omgubuntu.co.uk/2010/12/take-a-bite-out-of-unitys-bitesize-bugs/
[15:07] <jcastro> NICE!
[15:07]  * jcastro clicks Like
[15:07]  * ssj6akshat thought it was crappy
[15:09] <ssj6akshat> The top 3 posts on OMG! Ubuntu! are about.......
[15:09] <ssj6akshat> Unity!
[15:10] <Devil505> :)
[15:19] <cando> hi! running unity from source (last revision) the launcher panel becomes insensitive and i get these g_critical:http://pastebin.com/XixGHxby
[15:19] <cando> yesterday it worked fine...
[15:19] <kvalo> ronoc: another one: https://code.launchpad.net/~kvalo/indicator-network/bug-687414/+merge/43228
[15:21] <ronoc> tedg, 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's
[15:21] <ronoc> kvalo on it
[15:22] <kvalo> kenvandine: I try to get indicator-network 0.3.1 ready "soonish". do you have time to upload it today?
[15:22] <lamalex> njpatel, did you ever get back to me about g_list_free?
[15:22] <lamalex> I can't find me asking you, or you replying in xchat
[15:22] <cyphermox> didrocks, I just added a manpage for your unity wrapper on bug 684896.
[15:22] <kvalo> cyphermox: hi. any luck with bug #683302?
[15:23] <njpatel> i did
[15:23] <cyphermox> kvalo, err, yes, it's pretty much done, hold on :)
[15:23] <njpatel> lamalex, the docs say it owns the contents but not the list, that's what I was going on
[15:23] <njpatel> lamalex, the code seems to suggest the same
[15:23] <kvalo> cyphermox: cool
[15:24] <cyphermox> kvalo, ah, I need to ping micahg and let him know there was an update :)
[15:24] <lamalex> njpatel, well then there's another leak in Launcher.cpp
[15:24] <lamalex> of the same
[15:24] <kvalo> cyphermox: yeah, I got worried because nothing was happening :)
[15:24] <lamalex> want me to fix them both?
[15:24] <cyphermox> kvalo, unless kenvandine or seb128  or somebody else want to review the merge and take it over from there ?
[15:25] <kvalo> cyphermox: I'm fine with all options as long as the end result is the new (working) ofono in natty ;)
[15:25] <ronoc> kvalo, done
[15:26] <cyphermox> kvalo, I think we just need to be patient really... I'll ping micahg as soon as I see him
[15:26] <kvalo> ronoc: thanks again
[15:27] <njpatel> lamalex, fix it?
[15:28] <njpatel> lamalex, lp:bamf/lib/libbamf/bamf-view.c::141, please double check to make sure
[15:31] <lamalex> njpatel, yah you're right
[15:31] <lamalex> so should I fix both?
[15:31] <njpatel> yep
[15:32] <jcastro> tedg: are you talking to this guy? http://lizards.opensuse.org/2010/12/09/indicator-sessionnetwork/
[15:33] <jcastro> cyphermox: where's your branch for your nm-applet port? I seem to only be able to find the distro package
[15:33] <jcastro> port to indicators I mean
[15:37] <cyphermox> jcastro, https://code.edge.launchpad.net/~network-manager/network-manager-applet/ubuntu.head contains it.
[15:37] <jcastro> ta
[15:48] <lamalex> smspillaz, DBO, is optionChanged called automatically once at load, or just when the check box is clicked?
[15:49] <DBO> I dont know
[15:49] <DBO> ask smspillaz
[15:49] <lamalex> I pinged you both doofus
[15:49] <DBO> or add a printf to check
[15:49] <DBO> I imagine its called no matter what
[15:50] <boulabiar_> join #wayland
[15:50] <boulabiar_> oops
[15:51] <lamalex> oh yeha, i could just add a printf and check
[15:51] <lamalex> duh
[15:56] <smspillaz> lamalex: optionChanged gets called whenever the option gets changed
[16:01] <tedg> jcastro, No I haven't talked to him at all.
[16:01] <tedg> jcastro, Very cool though!
[16:02] <jcastro> tedg: ok, if he asks for questions I will send him your way
[16:25] <lamalex> didrocks, what was that file you told me to look at for variables in .in files via cmake?
[16:25] <didrocks> lamalex: in tools/, look at the CMakeLis… and unity.cmake
[16:27] <lamalex> thanks
[16:33] <lamalex> someone done broke my launcher
[16:34] <lamalex> brb
[16:43] <rbnswartz> Anabody know how get the number of workspaces using the unity source code?
[16:46] <seb128> ok
[16:46] <seb128> didrocks, commited unity r329
[16:47] <seb128> in the packaging
[16:47] <njpatel> rbnswartz, there is a merge proposal doing trying to do the same thing, sam added some comments on how it should be done
[16:47] <njpatel> one sec
[16:47] <didrocks> seb128: thanks
[16:48] <njpatel> rbnswartz,  https://code.launchpad.net/~ruben-verweij/unity/fix-677594-workspaces/+merge/43060
[16:50] <rbnswartz> 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] <rbnswartz> njpatel Okay thanks I was trying to fix one of the bite sized bugs I guess I'll switch over to fixing another one.
[16:53] <jono> rbnswartz, working on bitesize bugs? sweet :-)
[16:54] <lamalex> didrocks, is there a difference between SET and set in cmake?
[16:54] <rbnswartz> jono, Trying too.
[16:54] <didrocks> lamalex: not that I know of
[16:55] <jono> rbnswartz, :-)
[16:55] <rbnswartz> Thanks for all the help njpatel. bye.
[16:56] <njpatel> lamalex, they used to use capitals and now they don't
[16:56] <njpatel> lamalex, welcome to CMake
[17:10] <lamalex> https://code.launchpad.net/~alexlauni/unity/optional-debugging/+merge/43244 if someone would be so kind
[17:14] <njpatel> dbarth_, why shouldnt' introspection be on in final release?
[17:14] <njpatel> dbarth_, it has no operational cost during runtime
[17:30] <lamalex> njpatel, I still can't arrow through menus
[17:31] <lamalex> and also my launcher has since become totally f'd up http://imgur.com/aQAM8
[17:31] <cando_> i confirm this, same for me..
[17:31] <njpatel> lamalex, did you restart the panel service?
[17:31] <njpatel> lamalex, I didn't touch the launcher, ask DBO
[17:32] <DBO> lamalex, I am getting the same issue after merging trunk
[17:32] <DBO> into my branch
[17:32] <DBO> someone else f'd it up
[17:33] <njpatel> DBO, urgh, take a look in trunk please
[17:33] <njpatel> ffs
[17:33] <DBO> i am already doing it
[17:33] <lamalex> njpatel, I did restart panel service
[17:33] <lamalex> I rebooted, and installed your debs
[17:33] <lamalex> not in that order
[17:34] <cando_> lamalex, maybe revision 685?
[17:34] <njpatel> didrocks, https://launchpad.net/bamf/0.2/0.2.66
[17:34] <njpatel> lamalex, debs were for something else. Let me check with trunk, something might have screwed it up
[17:34] <didrocks> njpatel: thanks
[17:38] <njpatel> lamalex, 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] <njpatel> but it works fine
[17:38] <njpatel> launcher is fully f*cked, though
[17:38] <lamalex> njpatel, how do you start the right service? I usually just do kill `pidof unity-panel-service` and then run /usr/lib/unity/unity-panel-service
[17:39] <lamalex> ah
[17:39] <njpatel> lamalex, are you installing into /usr?
[17:39] <lamalex> no, I just realized why it's not working
[17:39] <lamalex> because I'm running /usr/...
[17:39] <lamalex> and not /usr/local
[17:39] <njpatel> right
[17:39] <lamalex> njpatel, how do you get things on your panel with panel-service from trunk
[17:45] <njpatel> lamalex, killall unity-panel-service; /usr/local/lib/unity/unity-panel-service &
[17:45] <njpatel> then things should start working
[17:47] <njpatel> didrocks, unity release is delayed as trunk is messed up
[17:47] <didrocks> njpatel: well, I guess you can imagine what is still building in my pbuilder…
[17:48] <didrocks> (hint: begin with n, finished by ux)
[17:48] <lamalex> njpatel, well, now unity is just crashing
[17:51] <didrocks> argh FTBFS…
[17:51]  * didrocks will cry :)
[17:52] <lamalex> I'm getting a crash from nux::Object now
[17:52] <lamalex> looks like 149 may have f'ed things up for me
[17:54] <lamalex> http://pastebin.com/92E1WspP
[17:54] <DBO> wtf is up with the double launcher
[17:54] <DBO> I dont get it
[17:54] <njpatel> neither do i
[17:55] <DBO> oh duh
[17:55] <njpatel> and lamalex's error is fucked
[17:55] <DBO> I see it
[17:55] <DBO> merge fail
[17:55] <njpatel> it's like we're being called twice
[17:55] <lamalex> was it my fault?
[17:55] <lamalex> I bet it was..
[17:55] <DBO> I think so
[17:57] <DBO> double launcher fixed
[17:57] <DBO> njpatel, sorry for the delay, was on the phone with design
[17:57] <njpatel> np
[17:58] <DBO> I have a lunch date in 3 minutes
[17:58] <DBO> gotta run
[17:58] <DBO> back in an hour
[17:59] <njpatel> DBO, thanks
[17:59] <lamalex> sorry guys :\
[18:01] <lamalex> I don't really know how that happened, what's weird is that there's no matchin + initLauncher call elsewhere
[18:01] <njpatel> no biggie, just make sure to check the system before pushing
[18:01] <njpatel> didrocks, I'm rolling unity now then
[18:03] <didrocks> njpatel: ok, in any case, I can't upload anymore this evening
[18:03] <didrocks> njpatel: there is no more archive admin around and I have two new binary packages to NEW
[18:04] <didrocks> oh, seb128 is back?
[18:04] <didrocks> so yeah, let's check that, I *can* :)
[18:07] <lamalex> so nux is still broken for me..
[18:08] <didrocks> lamalex: there is an abi break, ensure you rebuild unity against it
[18:08] <lamalex> ah
[18:09] <njpatel> jaytaoko, did you actually merge your event faker class?
[18:09] <nmarques> kenvandine, fixed ;) awesome stuff
[18:09] <didrocks> njpatel: there is one in nux
[18:09] <nmarques> kenvandine, the indicators are building now.... thx for your great help :)
[18:10] <njpatel> didrocks, yeah, the one in nux
[18:10] <jaytaoko> njpatel: I added the support in nux
[18:10] <njpatel> okay, so it's a local problem, sorry
[18:10] <jaytaoko> njpatel: but the event faker itself is in unity
[18:10] <seb128> didrocks, ?
[18:11] <didrocks> seb128: I'll have some bin packages to NEW for you :)
[18:11] <kenvandine> nmarques, anytime!
[18:11] <seb128> didrocks, I'm on my way off to sport now and I'm not sure I will come back tonight
[18:11] <kenvandine> seb128, one sec
[18:11] <seb128> so maybe ping cjwatson or pitti
[18:11] <didrocks> seb128: ok, then, it will be for tomorrow I guess :)
[18:11] <lamalex> didrocks, I've rebuild unity a number of times
[18:11] <didrocks> (pitti yeah)
[18:11] <seb128> didrocks, or that
[18:12] <didrocks> seb128: enjoy sport and you week-end! :)
[18:12] <kenvandine> to get indicator-datetime to build... we need geoclue to get promoted to main... the MIR was approved
[18:12] <seb128> didrocks, thanks
[18:12] <kenvandine> should i ping an archive admin?
[18:12] <seb128> kenvandine, did you upload already?
[18:12] <njpatel> jaytaoko, thanks, it's builds now, was just a weird error
[18:12] <kenvandine> yes
[18:12] <kenvandine> it's been sitting for a couple days waiting
[18:12] <seb128> kenvandine, ok, just let a comment on #ubuntu-devel
[18:12] <kenvandine> now i just uploaded again to make sure it builds with the new dbusmenu
[18:13] <seb128> if nobody pick it up I will promote it later
[18:13] <kenvandine> thx
[18:13] <seb128> but I need to run now
[18:13] <kenvandine> have fun!
[18:13] <kenvandine> all the indicators are uploaded
[18:13] <kenvandine> :)
[18:13] <didrocks> gord: wooots bug #687403! Where is the syntax guide?
[18:14] <tareth> Who should I send my Canonical contributer agreement to for unity?
[18:14] <didrocks> jcastro: ^^
[18:15] <tareth> Ok thanks :)
[18:15] <didrocks> tareth: no no, I'm asking jcastro to answer :)
[18:15] <tareth> oh nvm that wasn't an answer to my question
[18:15] <gord> didrocks, its the same as the indicator stuff
[18:15] <tareth> my brain completely ignored that colon
[18:15] <didrocks> gord: can we chat about it tomorrow? :)
[18:15] <jcastro> tareth: one sec.
[18:15] <didrocks> gord: just for a refresh ;)
[18:15] <gord> didrocks, just make sure that ShowOnlyIn isn't set to MessagingMenu ;) just needs to be Unity
[18:15] <gord> didrocks, sure
[18:16] <jcastro> http://www.canonical.com/contributors
[18:16] <didrocks> gord: thanks!
[18:16] <jcastro> tareth: step 2, the person you want to CC to is david.barth@canonical
[18:16] <lamalex> tareth, if it's a unity fix you send it to david barth
[18:16] <lamalex> his email is on that page jcastro just linked you to
[18:16] <jcastro> tareth: out of curiosity what bug are you working on?
[18:17] <tareth> https://bugs.launchpad.net/unity/+bug/686182 and https://bugs.launchpad.net/unity/+bug/681871
[18:20] <njpatel> didrocks, https://launchpad.net/unity/3.0/3.2.6
[18:21] <kvalo> wtf, adding debug messages fixes the bug :-O
[18:23] <njpatel> kvalo, yay for memory errors
[18:23] <njpatel> didrocks, bugs updated too
[18:23] <kvalo> njpatel: or a nasty race somewhere :/
[18:24] <njpatel> true, true
[18:25] <didrocks> njpatel: thanks!
[18:25] <didrocks> njpatel: did you change something on the bug list in the last 15 minutes?
[18:25] <njpatel> er, nope
[18:25] <didrocks> njpatel: perfect! :)
[18:25] <njpatel> I might go collapse for a little while now
[18:25] <didrocks> njpatel: sure should be good
[18:26] <njpatel> didrocks, mail me if there are issues, otherwise jaytaoko/DBO can help :)
[18:26] <didrocks> njpatel: enjoy your week-end dude! :)
[18:26] <njpatel> didrocks, you too :)
[18:26] <kvalo> weekend?
[18:26] <didrocks> njpatel: I'll bother them, no worry :p
[18:26] <didrocks> njpatel: thanks )
[18:26] <didrocks> ;)
[18:26] <kvalo> is it friday already?
[18:26] <njpatel> kvalo, taking tomorrow off
[18:26] <njpatel> unless something goes drastically wrong, that is ;)
[18:26] <kvalo> njpatel: oh, enjoy then!
[18:26] <kvalo> njpatel: you should have an accident with your router to prevent that ;)
[18:38] <njpatel> kvalo, heh, thanks
[18:38] <coz_> howdy doody
[18:41] <cyphermox> kvalo, ping, do you know why geoclue depends on ofono?
[18:42] <kvalo> cyphermox: depends? not recommends or suggests?
[18:42] <cyphermox> I don't know, but I would guess it's a depends
[18:42] <kvalo> cyphermox: no idea
[18:42] <cyphermox> hrm, maybe not
[18:42] <kvalo> cyphermox: some people have been working for gps interface to ofono, but I doubt that's the reason
[18:44] <cyphermox> mkay. well, I'll check it out. I can't find something looking quickly over apt-cache show...
[18:47] <didrocks> jcastro: do you have the trash icon bug # handy?
[18:48] <jcastro> didrocks: https://bugs.launchpad.net/unity/+bug/683241
[18:48] <jcastro> jamalta: I added the C++ style stuff to that page
[18:49] <didrocks> jcastro: thanks :)
[18:49] <jamalta> jcastro: ah awesome, thanks!
[18:50] <jamalta> well, i already have some stuff to fix
[18:52] <jcastro> Anyone who keeps DBO busy gets my vote. :p
[18:56] <kvalo> anyone have time for a quick review? https://code.launchpad.net/~kvalo/indicator-network/bug-686415/+merge/43269
[18:59] <didrocks> ok, unity and nux pushed
[18:59] <didrocks> (bamf was aready done)
[19:00] <jamalta> first conflicts :X
[20:21] <didrocks> DBO: btw, just before going away, I just noticed your minimize effect on the launcher icon. You rock!
[20:21] <DBO> oh
[20:21] <DBO> thanks :)
[20:21] <didrocks> DBO: I reconnected for that ;)
[20:21] <didrocks> now, time to go
[20:21] <DBO> :)
[20:21] <didrocks> bye bye!
[20:21] <jamalta> didrocks: holy crap it is awesome
[20:21] <jamalta> and my user list is broken... silly empathy
[20:21] <didrocks> jamalta: yeah, really ;)
[20:21] <jamalta> DBO: hey
[20:22] <jamalta> so i think i've got the issue pretty much worked out except i have a minor bug that i can't figure out
[20:22] <DBO> hey jamalta :)
[20:22] <jamalta> can i bug you sometime today about it?
[20:22] <DBO> sure
[20:22] <DBO> hit me with it now
[20:22] <jamalta> well, basically i think i'm missing something regarding input grabbing
[20:22] <jamalta> because the quicklist doesn't handle hovers correctly
[20:22] <jamalta> until you click on a quicklist, then it will work fine for all quicklists for the rest of the session
[20:23] <DBO> can you describe that better?
[20:23] <DBO> IRC is very unclear :)
[20:23] <jamalta> DBO: Let me try...
[20:23] <jamalta> So, 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 state
[20:24] <jamalta> When you click on the Quicklist menu, the action you clicked on will run correctly.
[20:24] <jamalta> But then, all quicklist menus opened after that work fine...
[20:25] <jamalta> Actually that last part is a lie... it doesn't always work after clicking on it
[20:27] <DBO> okay
[20:27] <jamalta> Does that make any sense?
[20:27] <DBO> yeah for the most part
[20:27] <DBO> I need to check your branch out
[20:27] <DBO> where is it?
[20:28] <jamalta> lp:~jamalta/unity/683261-autohide-quicklist
[20:29] <jamalta> but i think, other than that issue and some minor cleanup everything should be set
[20:29] <DBO> working on it
[20:30] <DBO> give me a couple minutes to figure it out
[20:30] <jamalta> DBO: no problem, thanks so much
[20:30] <DBO> thank you
[20:30] <DBO> you are super :)
[20:30] <DBO> so you work for flickr?
[20:30] <jamalta> yeha
[20:31] <jamalta> yeah*
[20:31] <DBO> where are they based?
[20:32] <jamalta> san francisco
[20:32] <DBO> (i hope you dont mind the small talk, I like chatting up contributors while i review)
[20:32] <jamalta> oh no problem, i don't mind
[20:32] <jamalta> i think you're EST, right?
[20:32] <DBO> I am
[20:32] <DBO> located in Michigan
[20:32] <jamalta> cool! where from?
[20:32] <DBO> Kalamazoo if you have heard of it
[20:32] <jamalta> oh awesome :)
[20:32] <jamalta> I haven't, but my mom's side of the family is from that area
[20:33] <jamalta> but to be honest, i'm not quite sure where they live :X
[20:33] <DBO> haha, it happens
[20:33] <DBO> its the lower left part of michigan
[20:33] <DBO> 3 hours from chicago, 2 hours from detroit
[20:33] <jamalta> man, i bet its cold there
[20:33] <DBO> yeah
[20:33] <DBO> we get the lake effect pretty strong
[20:33] <jamalta> wow
[20:34] <DBO> still, I prefer snow to heat
[20:34] <jamalta> heh, i wouldn't know really..
[20:34] <jamalta> but i'm not built for heat at all
[20:34] <jamalta> i mean, cold...
[20:35] <jamalta> i grew up in tropical weather
[20:36] <DBO> hmmm
[20:36] <DBO> this is interesting
[20:36] <DBO> it keeps crashing for me
[20:36] <jamalta> uh oh
[20:36] <DBO> /home/jason/staging//bin/compiz: symbol lookup error: /home/jason/staging/lib/compiz/libunityshell.so: undefined symbol: _ZN16QuicklistManager7DefaultEv
[20:36] <DBO> it looks like a failed compile or something
[20:36] <DBO> but I did a make clean to check
[20:36] <jamalta> i think you have to run cmake again
[20:36] <jamalta> so it adds the new files
[20:36] <DBO> I'll retry :)
[20:36] <DBO> oh you know what
[20:37] <DBO> I did the cmake after I merged trunk
[20:37] <DBO> not your stuff
[20:37] <DBO> thanks :)
[20:37] <DBO> beautiful
[20:37] <DBO> yep
[20:37] <DBO> I see there is a hover issue
[20:37] <DBO> looking at it now
[20:37] <jamalta> yeap, and its frustrating :(
[20:38] <jamalta> thanks!
[20:38] <DBO> hey you cleaned up the dup code :)
[20:39] <jamalta> DBO: yeah :)
[20:39] <DBO> in QuicklistView::Hide you check IsVisible twice, I think you meant to check it and _enable_quicklist_for_testing
[20:40] <jamalta> oh oops! yeah that's what I meant to do
[20:42] <DBO> rather than nested if's I would use &&
[20:42] <DBO> we dont liek nesting :)
[20:42] <jamalta> in fact, i'll move those to the same
[20:42] <jamalta> err
[20:42] <jamalta> yeah
[20:42] <jamalta> what you just said :)
[20:42] <DBO> :)
[20:43] <DBO> interesting bug
[20:43] <DBO> you did a nicer implementation than I expected
[20:44] <jamalta> thanks :) glad you're liking it
[20:44] <jamalta> and 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 doing
[20:44] <jamalta> i just made sure those calls were being made just as they had been
[20:44] <jamalta> anyways, if you do a pull you should have the fix for QuicklistView::Hide
[20:48] <DBO> jamalta, figured it out
[20:48] <DBO> in your Show () method
[20:48] <DBO> make ShowWindow (true) the first thing done in the block, not the last
[20:49] <DBO> also mark that with a fixme comment
[20:49] <DBO> I need to fix that in nux since it really shouldn't matter
[20:50] <DBO> oh man it works great jamalta, super work
[20:50] <jamalta> thanks! :)
[20:50] <DBO> make those fixes
[20:50] <DBO> propose merge
[20:50] <DBO> and I'll merge it :)
[20:53] <jamalta> DBO: awesome it wokrs :D
[20:53] <jamalta> i would've never figured that out on my own, thanks!
[20:55] <DBO> jamalta, no biggie :)
[20:58] <jamalta> DBO: ok i've submitted a MP
[20:58] <DBO> awesome
[20:58]  * DBO goes to review
[21:01] <jamalta> DBO: just noticed that i left a ton of commented out stuff in LauncherIcon::RecvMouseEnter
[21:02] <DBO> jamalta, I'll delete it on merge
[21:02] <jamalta> thanks
[21:03] <jamalta> also, what do you think the block starting at line 137?
[21:03] <jamalta> i commented "We probably should let QuicklistManager deal with this case..." and meant to ask about it
[21:03] <jamalta> quicklist 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 up
[21:03] <DBO> jamalta, I dont mind either way really
[21:04] <DBO> you could indeed let the manager handle it
[21:04] <jamalta> (the line number in reference was from the MP diff)
[21:04] <DBO> but being explicit is fine too
[21:04] <DBO> you are merged my friend
[21:04] <jamalta> DBO: oh sweet! :)
[21:04] <jamalta> ty
[21:05] <DBO> kudos are in order, a cake maybe
[21:05] <jamalta> haha
[21:06]  * RAOF resists the obvious comment
[21:06] <jamalta> i'd rather replicate that functionality
[21:07] <jamalta> mind if i push to the branch with that removed?
[21:07] <jamalta> just tested it and it works fine when it is removed
[21:07] <jamalta> the manager is hiding the quicklist correctly before displaying a new one
[21:11] <jamalta> DBO: if i make that change, should i push to a new branch, or is the same branch ok?
[21:12] <DBO> new branch since you are merged
[21:12] <DBO> is the change just to delete those lines?
[21:12] <jamalta> DBO: yes, just removing that whole block
[21:12] <DBO> okay, I'll do it really quick
[21:12] <jamalta> DBO: ok thanks!
[21:14] <DBO> :) done
[21:14] <jamalta> DBO: sweet, awesome.. thanks so much
[21:14] <DBO> yep
[21:16] <jamalta> DBO: is there another bug you'd like to throw my way? i can probably work on it over the weekend
[21:16] <DBO> uhhhhhm
[21:16] <jcastro> I have some!
[21:16] <jcastro> https://bugs.launchpad.net/unity/+bugs?field.tag=bitesize
[21:16] <jcastro> any of those!
[21:17] <jcastro> jamalta: ^^
[21:17] <jcastro> but if they're too easy I'm sure DBO has more
[21:17] <DBO> jamalta, this one could be fun and easy
[21:17] <DBO> https://bugs.launchpad.net/unity/+bug/687956
[21:18] <DBO> there are a lot of got-yas in there
[21:18] <DBO> it will take a bit to get the corner cases right
[21:19] <DBO> basically there should be a way for the launcher to say "okay, it's now okay to show tooltips"
[21:19] <DBO> and on enter it sends that signal 1 second after enter
[21:19] <jamalta> jcastro: heh, i forgot about that.. ty :)
[21:20] <DBO> yeah some of the bite size bugs are not so bite sized
[21:20] <DBO> but that tend to happen
[21:21] <DBO> if its truly bite sized it means we tend to eat it for lunch :P
[21:21] <jamalta> haha
[21:21] <jamalta> yeah i can take a look at this bug
[21:22] <jamalta> DBO: 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] <jamalta> then, when they mouse out, it would be set back to false
[21:22] <jamalta> (this is regarding mark's comment on that bug)
[21:23] <DBO> jamalta, ideally it would be a static method in the LauncherIcon class
[21:24] <DBO> static void LauncherIcon::SetTooltipsAllowed (bool allowed)
[21:24] <jamalta> DBO: ah ok
[21:24] <DBO> that would then fire off a signal or something internally into the LauncherIcon instances
[21:24] <DBO> the one that is hovered could then hide/show its tooltip
[21:25] <jamalta> DBO: should i be using g_timeout_add for the timer?
[21:25] <jamalta> i haven't worked with timers in C++ yet
[21:25] <DBO> yep
[21:25] <jamalta> ok cool
[21:25] <DBO> g_timeout_add is a C call
[21:25] <DBO> so the callback you give it must be static
[21:26] <DBO> you'll see I use it a lot of places
[21:26] <jamalta> ah, ok
[21:26] <jamalta> that makes sense
[21:26] <jamalta> then i'll start digging into this tonight and bring up any questions i come up with tomorrow
[21:26] <DBO> awesome
[21:26] <DBO> you're a hero
[21:28] <jamalta> DBO: thanks :).. i'm just happy to be helping, and learning a ton too
[21:28] <DBO> what do you do at flickr?
[21:28] <jamalta> i haven't had a chance to work with c++/c much before this
[21:29] <jamalta> i'm a backend developer
[21:29] <DBO> doing frontends can be fun too :)
[21:30] <jamalta> DBO: yeah hehe :)
[21:30] <DBO> I cant imagine going to work for a closed shop after working for Canonical
[21:30] <jamalta> DBO: heh i know what you mean
[21:30] <jamalta> i love the idea of working on an open source project
[21:30] <jamalta> although i also enjoy what i do here
[21:31] <jamalta> it has been a great experience for me so far.. i've learned a lot working on such a high traffic website.
[21:32] <DBO> I just keep thinking about these image hosting websites
[21:32] <DBO> "somewhere in that company, it is someones job to view all the questionable material and delete it"
[21:32] <DBO> he must not sleep at night...
[21:33] <jamalta> DBO: tell me about it
[21:34] <jamalta> i don't know how they do it
[21:34] <jamalta> working on our moderation tools is not fun :(
[21:34] <DBO> haha
[21:34] <DBO> "This is a pineapple... this however... is not"
[21:35] <jamalta> haha
[21:36] <DBO> I was reading on reddit the other day
[21:36] <DBO> the dude who does imgur (still a one man show I guess), does all the moderation by hand...
[21:37] <jamalta> DBO: wow... that's insane
[21:38] <DBO> jamalta, I fear for his soul
[21:38] <DBO> it must be crushed
[21:39] <jamalta> i didn't know they even did moderation
[21:39] <jamalta> but yes, i completely agree
[21:40] <DBO> its limited, I think it comes down mostly to "things the FBI might arrest me for"
[21:41] <jamalta> that's the scariest thing about running a site like that
[21:41] <jamalta> there is so much liability
[21:42] <DBO> I have to go now, girlfirend needs a ride home
[21:42] <jamalta> DBO: np, have a good one
[21:49] <kvalo> tedg: hi, do you have time to review this: https://code.launchpad.net/~kvalo/indicator-network/bug-686415/+merge/43269
[21:51] <tedg> kvalo, Sorry, not really.  Turns out I broke natty :)
[21:51] <tedg> bratsche, ^ ?
[21:51] <kvalo> tedg: heh, can't argue with that :)
[21:54]  * bratsche takes a look
[21:54] <kvalo> tedg: I hope you get natty working :)
[21:55] <kvalo> bratsche: thanks
[21:56] <bratsche> kvalo: Is it necessary to g_object_ref (priv->default_service) ?
[21:57] <bratsche> Err, nm.. I get it.
[21:57] <kvalo> bratsche: I guess it would be possible to avoid the ref, but it would get too complicated for me ;)
[21:58] <bratsche> No I was thinking that it was creating an object there, but it looks like it's not.
[21:58] <bratsche> So nevermind, it looks fine.
[21:58] <kvalo> ah, ok
[21:58] <bratsche> I thought it was creating an object and then it would have a refcount of 2.. and then the unref wouldn't free it.
[21:59] <bratsche> kvalo: I didn't actually test it, but the code looks good.  Approved. :)
[21:59] <kvalo> bratsche: thanks!
[22:11] <kvalo> kenvandine: I know you are extremely busy, but still fyi: https://launchpad.net/indicator-network/trunk/0.3.1
[22:11] <kenvandine> kvalo, cool
[22:17] <kenvandine> kvalo, that requires the newer connman doesn't it?
[22:17] <kenvandine> oh... nm
[22:20] <kvalo> kenvandine: new connman is already in natty, but ofono upload is still pending
[22:20] <kvalo> kenvandine: but a versioned dependency to connman would be nice
[22:21] <kenvandine> or maybe it does
[22:22] <kvalo> kenvandine: connman 0.64 is the version which is needed
[23:18] <kvalo> kenvandine: thanks for the upload
[23:18] <kvalo> kenvandine: why does geoclue build depend on ofono? that's just weird