ion | 17 hours and Apport retracing service still hasn’t retraced the crash report. :-P | 00:27 |
---|---|---|
ion | Ah, missed seb128’s message. | 00:30 |
ion | My issue had already been reported. LP #686698. | 05:31 |
ubot5 | Launchpad bug 686698 in Nux "compiz crashed with SIGSEGV in nux::IOpenGLSurface::UnlockRect()" [Undecided,Triaged] https://launchpad.net/bugs/686698 | 05:31 |
=== ssj6akshat is now known as akshat|school | ||
kvalo | morning | 06:51 |
ion | that. | 06:52 |
kvalo | ion: moro | 07:06 |
ion | hi | 07:06 |
didrocks | good morning | 07:31 |
=== smorar__ is now known as smorar_ | ||
smspillaz | mornin | 07:44 |
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:45 |
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:46 |
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:47 |
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:48 |
=== smorar_ is now known as smorar__ | ||
didrocks | smspillaz: ok, so if you have two screens | 07:52 |
didrocks | and gnome-panel is on the left | 07:52 |
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:53 |
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:54 |
smspillaz | didrocks: it should be set by the output | 07:58 |
smspillaz | I'm just compiling docky now to see what the behaviour should be | 07:59 |
MacSlow | greetings everybody | 08:27 |
didrocks | morning MacSlow | 08:29 |
MacSlow | didrocks, salut here too :) | 08:30 |
didrocks | :) | 08:30 |
smspillaz | reasons I hate autotools #24533452: It only tells you which dependencies you are missing one dependency at a time | 08:33 |
MacSlow | smspillaz, :) | 08:34 |
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:35 |
RAOF | Heh. Reason to hate cmake #whatever: CFLAGS? What's that? ☺ | 08:38 |
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:40 |
* 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:41 |
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: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 | |
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:43 |
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:44 |
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:45 |
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:46 |
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:47 |
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:48 |
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:49 |
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:50 |
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:51 |
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:52 |
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:54 |
smspillaz | (since it tells all kinds of lies to X11 to work correctly) | 08:55 |
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:56 |
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:57 |
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:58 |
didrocks | smspillaz: so, why OO is enabled and not firefox/thunderbird? | 08:59 |
smspillaz | didrocks: *shrug* | 08:59 |
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:00 |
smspillaz | ok :) | 09:01 |
didrocks | grrr, tomboy crashes… hard to take a note this way :) | 09:03 |
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:04 |
didrocks | s/like that/on that side/ | 09:05 |
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:08 |
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:09 |
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:15 |
didrocks | RAOF: smspillaz: ok, in that case, it doesn't fix it :) | 09:16 |
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:17 |
didrocks | smspillaz: I think I have, let me check | 09:18 |
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:22 |
didrocks | kvalo: depends if you only care about the package or also the upstream tarball + make && make install | 09:23 |
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:24 |
didrocks | smspillaz: the commit is in | 09:30 |
smspillaz | didrocks: huh, ok | 09:45 |
smspillaz | didrocks: I thought that commit fixed it :/ | 09:45 |
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:46 |
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:47 |
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:48 |
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:49 |
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:50 |
RAOF | seb128: That thought had crossed my mind, yes :) | 09:51 |
=== smorar__ is now known as smorar_ | ||
didrocks | lamalex: hey, seeing your comment on the merge, what's the difference between LAUNCHER_ICON_QUICK_STARTING and LAUNCHER_ICON_QUICK_LAUNCHING ? | 09:58 |
lamalex | didrocks, there is no LAUNCHING | 10:02 |
lamalex | didrocks, I just updated that comment, thanks | 10:02 |
lamalex | I'm sick :\ | 10:02 |
didrocks | lamalex: oh? the grey weather of London hit you? | 10:03 |
dbarth | salut seb128 | 10:13 |
dbarth | seb128: about https://bugs.launchpad.net/unity/+bug/686603 | 10:13 |
ubot5 | Ubuntu bug 686603 in Unity "valgrind "Conditional jump or move depends on uninitialised value"" [Undecided,New] | 10:13 |
seb128 | lut dbarth | 10:13 |
dbarth | seb128: you get that on the console, i mean it's a debug log? | 10:13 |
seb128 | I get that by running test-panel under valgrind | 10:14 |
dbarth | ah right | 10:14 |
dbarth | smspillaz: hey sam, have you seen mvo bug with the link.py testcase? | 10:18 |
dbarth | smspillaz: that sounds like a scary regression | 10:19 |
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:20 |
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:55 |
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:56 |
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:57 |
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:58 |
* kamstrup rolls up sleeves to get those sodded unit tests running again | 10:59 | |
=== njpatel is now known as njpatel_ | ||
=== akshat|school is now known as ssj6akshat | ||
lamalex | when someone has a minute? https://code.launchpad.net/~alexlauni/unity/xids-introspection/+merge/43193 | 12:12 |
=== smorar_ is now known as smorar__ | ||
=== MacSlow is now known as MacSlow|lunch | ||
lamalex | njpatel_, didrocks, with CMake can we have unityshell.xml.in have variables in it that get set on build? | 12:17 |
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:18 |
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:19 |
htorque | yw! thanks for all the fixes. ;-) | 12:20 |
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:25 |
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:26 |
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:27 |
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 |
ubot5 | Launchpad bug 683623 in unity (Ubuntu) "Starting a launcher only works four times" [High,Triaged] https://launchpad.net/bugs/683623 | 12:28 |
seb128 | yeah, I've read this one last week | 12:28 |
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:29 |
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:30 |
didrocks | well, not very difficult, just that I hate python with making some kind of tee-like behavior :) | 12:31 |
didrocks | smspillaz: are you changing gtk/window-decorator/gtk-window-decorator.c with your decorator work? | 12:36 |
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:49 |
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:50 |
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:51 |
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:52 |
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:53 |
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:54 |
klattimer | smspillaz: +1 | 12:55 |
didrocks | smspillaz: so, activating the workaround plugin with upstream defaults for now minus AIGLX one | 12:55 |
=== MacSlow|lunch is now known as MacSlow | ||
lamalex | njpatel_, how does the launcher's autohide get set initially? | 13:04 |
lamalex | does optionChanged get called once at start? | 13:04 |
njpatel_ | lamalex, I'm not sure about the compiz options stuff, sorry | 13:06 |
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:07 |
=== oubiwann is now known as oubiwann_ | ||
lamalex | njpatel_, could you please review https://code.launchpad.net/~alexlauni/unity/xids-introspection/+merge/43193 | 13:18 |
kvalo | ronoc: hi. time again for a review: https://code.launchpad.net/~kvalo/indicator-network/gdbus-cellular-crash/+merge/43204 | 13:19 |
jcastro | ok guys, added C++ to the style guide: https://wiki.ubuntu.com/Unity/CodingStyle | 13:26 |
njpatel_ | lamalex, will take a look in a bit | 13: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++ bit | 13:30 |
jcastro | ok | 13:31 |
jcastro | done | 13:32 |
jcastro | njpatel_: what else is C++ after nux and compiz? | 13:32 |
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_ is now known as njpatel | ||
njpatel | jcastro, that's it for now | 13:47 |
njpatel | lamalex, approved with a comment | 13:51 |
njpatel | please fix before merging | 13:51 |
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:52 |
jcastro | lamalex: just a fwd or CC is fine, I can handle introducing myself, etc. | 13:53 |
lamalex | jcastro, sure | 13:53 |
didrocks | nooooooo, I'm starting to google "cmake" when I want to google "autotools" | 13:55 |
didrocks | that's the end /o\ | 13:55 |
didrocks | njpatel: did you see my question about unity_support_test location and nux? | 13:57 |
=== oubiwann_ is now known as oubiwann | ||
ssj6akshat | jcastro, I am always getting interrupted when I am about to write a post on the Bitesize bugs | 13:59 |
ssj6akshat | I guess someone doesn't want to it to appear :P | 14:00 |
jcastro | ssj6akshat: it's ok, it's a marathon not a sprint | 14:01 |
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:02 |
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:03 |
jcastro | njpatel: didrocks: while I have you here, post release I need more bugs kthx. :) | 14:05 |
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:06 |
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:07 |
jcastro | ~10 | 14:08 |
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:09 |
didrocks | jcastro: right, little issues will come later, when it doesn't need refactoring everything :) | 14:10 |
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:12 |
njpatel | lamalex, the docs say it owns the contents but not the list, that's what I was going on | 14:17 |
njpatel | lamalex, the code seems to suggest the same | 14:18 |
ronoc | kvalo, done | 14:18 |
kvalo | ronoc: thanks! | 14:20 |
ronoc | kvalo, np | 14:21 |
ronoc | agateau, got a sec? | 14:21 |
agateau | ronoc: give me 5 minutes | 14:22 |
ronoc | agateau, no rush | 14:22 |
agateau | ronoc: ok, I am back | 14:29 |
ronoc | agateau, one sec | 14:30 |
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:34 |
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:35 |
lamalex | man but making a member variable public feels so wrong | 14:36 |
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:37 |
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:38 | |
ronoc | tedg, good morning -> http://pastebin.ubuntu.com/541475/ | 14:39 |
seb128 | ok | 14:39 |
seb128 | so is appmenu maintained and by who? | 14:39 |
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:40 |
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:41 |
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:42 |
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:43 |
* 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:44 |
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:45 |
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:46 |
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:47 |
seb128 | ok | 14:48 |
tedg | We'll have to grab a dbus capture to see if who's wrong.... I of course think it's appmenu-gtk ;) | 14:49 |
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:50 |
ronoc | tedg, yup i was doing something (or not doing something) with the custom item, works with a normal item | 14:59 |
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:00 |
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:01 |
njpatel | +1 | 15:03 |
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:05 |
ssj6akshat | jcastro, http://www.omgubuntu.co.uk/2010/12/take-a-bite-out-of-unitys-bitesize-bugs/ | 15:06 |
jcastro | NICE! | 15:07 |
* jcastro clicks Like | 15:07 | |
* ssj6akshat thought it was crappy | 15:07 | |
ssj6akshat | The top 3 posts on OMG! Ubuntu! are about....... | 15:09 |
ssj6akshat | Unity! | 15:09 |
Devil505 | :) | 15:10 |
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:19 |
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:21 |
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 |
ubot5 | Launchpad bug 684896 in unity (Ubuntu) "no unity manpage" [Low,Triaged] https://launchpad.net/bugs/684896 | 15:22 |
kvalo | cyphermox: hi. any luck with bug #683302? | 15:22 |
ubot5 | Launchpad bug 683302 in ofono (Ubuntu) "Please merge ofono 0.36-1 from Debian unstable (main)" [Wishlist,In progress] https://launchpad.net/bugs/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:23 |
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:24 |
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:25 |
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:26 |
njpatel | lamalex, fix it? | 15:27 |
njpatel | lamalex, lp:bamf/lib/libbamf/bamf-view.c::141, please double check to make sure | 15:28 |
lamalex | njpatel, yah you're right | 15:31 |
lamalex | so should I fix both? | 15:31 |
njpatel | yep | 15:31 |
jcastro | tedg: are you talking to this guy? http://lizards.opensuse.org/2010/12/09/indicator-sessionnetwork/ | 15:32 |
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:33 |
cyphermox | jcastro, https://code.edge.launchpad.net/~network-manager/network-manager-applet/ubuntu.head contains it. | 15:37 |
jcastro | ta | 15:37 |
lamalex | smspillaz, DBO, is optionChanged called automatically once at load, or just when the check box is clicked? | 15:48 |
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:49 |
boulabiar_ | join #wayland | 15:50 |
boulabiar_ | oops | 15:50 |
lamalex | oh yeha, i could just add a printf and check | 15:51 |
lamalex | duh | 15:51 |
=== boulabiar_ is now known as boulabiar | ||
smspillaz | lamalex: optionChanged gets called whenever the option gets changed | 15:56 |
tedg | jcastro, No I haven't talked to him at all. | 16:01 |
tedg | jcastro, Very cool though! | 16:01 |
jcastro | tedg: ok, if he asks for questions I will send him your way | 16:02 |
=== Cimi_ is now known as Cimi | ||
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:25 |
lamalex | thanks | 16:27 |
lamalex | someone done broke my launcher | 16:33 |
lamalex | brb | 16:34 |
=== bregma is now known as bregma|away | ||
rbnswartz | Anabody know how get the number of workspaces using the unity source code? | 16:43 |
seb128 | ok | 16:46 |
seb128 | didrocks, commited unity r329 | 16:46 |
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:47 |
njpatel | rbnswartz, https://code.launchpad.net/~ruben-verweij/unity/fix-677594-workspaces/+merge/43060 | 16:48 |
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:50 |
jono | rbnswartz, working on bitesize bugs? sweet :-) | 16:53 |
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:54 |
jono | rbnswartz, :-) | 16:55 |
rbnswartz | Thanks for all the help njpatel. bye. | 16:55 |
njpatel | lamalex, they used to use capitals and now they don't | 16:56 |
njpatel | lamalex, welcome to CMake | 16:56 |
lamalex | https://code.launchpad.net/~alexlauni/unity/optional-debugging/+merge/43244 if someone would be so kind | 17:10 |
njpatel | dbarth_, why shouldnt' introspection be on in final release? | 17:14 |
njpatel | dbarth_, it has no operational cost during runtime | 17:14 |
lamalex | njpatel, I still can't arrow through menus | 17:30 |
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:31 |
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:32 |
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:33 |
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:34 |
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:38 |
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:39 |
njpatel | lamalex, killall unity-panel-service; /usr/local/lib/unity/unity-panel-service & | 17:45 |
njpatel | then things should start working | 17:45 |
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:47 |
didrocks | (hint: begin with n, finished by ux) | 17:48 |
lamalex | njpatel, well, now unity is just crashing | 17:48 |
didrocks | argh FTBFS… | 17:51 |
* didrocks will cry :) | 17:51 | |
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:52 |
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:54 |
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:55 |
DBO | double launcher fixed | 17:57 |
DBO | njpatel, sorry for the delay, was on the phone with design | 17:57 |
njpatel | np | 17:57 |
DBO | I have a lunch date in 3 minutes | 17:58 |
DBO | gotta run | 17:58 |
DBO | back in an hour | 17:58 |
njpatel | DBO, thanks | 17:59 |
lamalex | sorry guys :\ | 17:59 |
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:01 |
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:03 |
didrocks | oh, seb128 is back? | 18:04 |
didrocks | so yeah, let's check that, I *can* :) | 18:04 |
=== ssj6akshat is now known as ssj6akshat|sleep | ||
lamalex | so nux is still broken for me.. | 18:07 |
didrocks | lamalex: there is an abi break, ensure you rebuild unity against it | 18:08 |
=== ssj6akshat|sleep is now known as ssj6akshat | ||
lamalex | ah | 18:08 |
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:09 |
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:10 |
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:11 |
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:12 |
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:13 |
ubot5 | Launchpad bug 687403 in unity (Ubuntu) "Add desktop action support to launcher quicklists" [Undecided,New] https://launchpad.net/bugs/687403 | 18:13 |
tareth | Who should I send my Canonical contributer agreement to for unity? | 18:14 |
didrocks | jcastro: ^^ | 18:14 |
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:15 |
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:16 |
tareth | https://bugs.launchpad.net/unity/+bug/686182 and https://bugs.launchpad.net/unity/+bug/681871 | 18:17 |
ubot5 | Ubuntu bug 686182 in unity (Ubuntu) "Unity launchers run multiple copies of program if clicked multiple times before the program loads" [Low,Triaged] | 18:17 |
njpatel | didrocks, https://launchpad.net/unity/3.0/3.2.6 | 18:20 |
kvalo | wtf, adding debug messages fixes the bug :-O | 18:21 |
njpatel | kvalo, yay for memory errors | 18:23 |
njpatel | didrocks, bugs updated too | 18:23 |
kvalo | njpatel: or a nasty race somewhere :/ | 18:23 |
njpatel | true, true | 18:24 |
=== gord is now known as gord|afk | ||
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:25 |
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:26 |
njpatel | kvalo, heh, thanks | 18:38 |
coz_ | howdy doody | 18:38 |
cyphermox | kvalo, ping, do you know why geoclue depends on ofono? | 18:41 |
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:42 |
cyphermox | mkay. well, I'll check it out. I can't find something looking quickly over apt-cache show... | 18:44 |
didrocks | jcastro: do you have the trash icon bug # handy? | 18:47 |
jcastro | didrocks: https://bugs.launchpad.net/unity/+bug/683241 | 18:48 |
ubot5 | Ubuntu bug 683241 in unity (Ubuntu) "Recycle bin icon is empty when there are items in the bin" [Low,Triaged] | 18:48 |
jcastro | jamalta: I added the C++ style stuff to that page | 18:48 |
didrocks | jcastro: thanks :) | 18:49 |
jamalta | jcastro: ah awesome, thanks! | 18:49 |
jamalta | well, i already have some stuff to fix | 18:50 |
=== ssj6akshat is now known as ssj6akshat|sleep | ||
jcastro | Anyone who keeps DBO busy gets my vote. :p | 18:52 |
kvalo | anyone have time for a quick review? https://code.launchpad.net/~kvalo/indicator-network/bug-686415/+merge/43269 | 18:56 |
didrocks | ok, unity and nux pushed | 18:59 |
didrocks | (bamf was aready done) | 18:59 |
jamalta | first conflicts :X | 19:00 |
=== bregma|away is now known as bregma | ||
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:21 |
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:22 |
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:23 |
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:24 |
jamalta | Actually that last part is a lie... it doesn't always work after clicking on it | 20:25 |
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:27 |
jamalta | lp:~jamalta/unity/683261-autohide-quicklist | 20:28 |
jamalta | but i think, other than that issue and some minor cleanup everything should be set | 20:29 |
DBO | working on it | 20:29 |
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:30 |
jamalta | yeah* | 20:31 |
DBO | where are they based? | 20:31 |
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:32 |
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:33 |
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:34 |
jamalta | i grew up in tropical weather | 20:35 |
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:36 |
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:37 |
jamalta | thanks! | 20:38 |
DBO | hey you cleaned up the dup code :) | 20:38 |
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:39 |
jamalta | oh oops! yeah that's what I meant to do | 20:40 |
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:42 |
DBO | interesting bug | 20:43 |
DBO | you did a nicer implementation than I expected | 20:43 |
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:44 |
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:48 |
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:49 |
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:50 |
=== jamalta_ is now known as jamalta | ||
jamalta | DBO: awesome it wokrs :D | 20:53 |
jamalta | i would've never figured that out on my own, thanks! | 20:53 |
DBO | jamalta, no biggie :) | 20:55 |
jamalta | DBO: ok i've submitted a MP | 20:58 |
DBO | awesome | 20:58 |
* DBO goes to review | 20:58 | |
jamalta | DBO: just noticed that i left a ton of commented out stuff in LauncherIcon::RecvMouseEnter | 21:01 |
DBO | jamalta, I'll delete it on merge | 21:02 |
jamalta | thanks | 21:02 |
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:03 |
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:04 |
DBO | kudos are in order, a cake maybe | 21:05 |
jamalta | haha | 21:05 |
* RAOF resists the obvious comment | 21:06 | |
jamalta | i'd rather replicate that functionality | 21:06 |
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:07 |
jamalta | DBO: if i make that change, should i push to a new branch, or is the same branch ok? | 21:11 |
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:12 |
DBO | :) done | 21:14 |
jamalta | DBO: sweet, awesome.. thanks so much | 21:14 |
DBO | yep | 21:14 |
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:16 |
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:17 |
ubot5 | Ubuntu bug 687956 in unity (Ubuntu) "should display the launcher tooltips after a delay" [Medium,Triaged] | 21:17 |
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:18 |
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:19 |
DBO | yeah some of the bite size bugs are not so bite sized | 21:20 |
DBO | but that tend to happen | 21:20 |
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:21 |
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:22 |
DBO | jamalta, ideally it would be a static method in the LauncherIcon class | 21:23 |
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:24 |
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:25 |
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:26 |
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:28 |
jamalta | i'm a backend developer | 21:29 |
DBO | doing frontends can be fun too :) | 21:29 |
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:30 |
jamalta | it has been a great experience for me so far.. i've learned a lot working on such a high traffic website. | 21:31 |
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:32 |
jamalta | DBO: tell me about it | 21:33 |
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:34 |
jamalta | haha | 21:35 |
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:36 |
jamalta | DBO: wow... that's insane | 21:37 |
DBO | jamalta, I fear for his soul | 21:38 |
DBO | it must be crushed | 21:38 |
jamalta | i didn't know they even did moderation | 21:39 |
jamalta | but yes, i completely agree | 21:39 |
DBO | its limited, I think it comes down mostly to "things the FBI might arrest me for" | 21:40 |
jamalta | that's the scariest thing about running a site like that | 21:41 |
jamalta | there is so much liability | 21:41 |
DBO | I have to go now, girlfirend needs a ride home | 21:42 |
jamalta | DBO: np, have a good one | 21:42 |
kvalo | tedg: hi, do you have time to review this: https://code.launchpad.net/~kvalo/indicator-network/bug-686415/+merge/43269 | 21:49 |
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:51 |
* bratsche takes a look | 21:54 | |
kvalo | tedg: I hope you get natty working :) | 21:54 |
kvalo | bratsche: thanks | 21:55 |
bratsche | kvalo: Is it necessary to g_object_ref (priv->default_service) ? | 21:56 |
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:57 |
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:58 |
bratsche | kvalo: I didn't actually test it, but the code looks good. Approved. :) | 21:59 |
kvalo | bratsche: thanks! | 21:59 |
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:11 |
kenvandine | kvalo, that requires the newer connman doesn't it? | 22:17 |
kenvandine | oh... nm | 22:17 |
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:20 |
kenvandine | or maybe it does | 22:21 |
kvalo | kenvandine: connman 0.64 is the version which is needed | 22:22 |
=== Schendje-2 is now known as schendje | ||
kvalo | kenvandine: thanks for the upload | 23:18 |
kvalo | kenvandine: why does geoclue build depend on ofono? that's just weird | 23:18 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!