/srv/irclogs.ubuntu.com/2011/11/16/#ayatana.txt

thumperTrevinho: I'm just relocating home again... back soon00:20
andyrockTrevinho, still here?00:28
andyrockbtw make check doesn't work for me00:39
andyrockit never ends00:39
andyrockand make test gives me an error00:39
Andy80andyrock: hey, what are you testing?01:12
andyrocki'm testing the tests :)01:12
andyrockAndy80, i'm going to write a test for a my branch, but make check doesn't work for me01:13
Trevinhoandyrock: here I am01:22
andyrockTrevinho, make check doesn't work for me :/01:22
andyrockit never ends01:22
Trevinhothumper: I've just synced this old patch to the new code: https://code.launchpad.net/~3v1n0/unity/panel-opacity-toggle/+merge/77219 It was accepted, but it doesn't include any test yet. Also because there's not a test framework for the panel yet, so I didn't know if there's something in progress01:23
Trevinhoandyrock: I'm sorry for you01:23
Trevinhoit works here01:23
* thumper is back01:23
andyrockTrevinho, i'm getting crazy01:23
* thumper looks at the merge01:24
thumperandyrock: where does it get stuck?01:24
andyrockthumper, it's not a probelm with my branch01:24
andyrockmake check doesn't work for unity trunk too01:25
thumperandyrock: right, but where is it stopping?01:25
andyrocktask-1: [       OK ] TestFilter.TestConnect (1 ms)01:25
andyrocktask-1: [ RUN      ] TestFilter.TestChanged01:25
andyrocktask-1: [       OK ] TestFilter.TestChanged (0 ms)01:25
andyrocktask-1: [ RUN      ] TestFilter.TestRemoved01:25
andyrockand it never ends...01:25
thumperhmm...01:38
andyrockremoving TestFilter.TestRemoved01:40
andyrockmakes that the test goes on01:41
thumperhmm... works fine for me here01:43
thumperwhat is it doing?01:43
andyrockthumper, i'm giving a look to it...01:57
andyrockthumper, have you given a look to test_filter.cpp?02:05
andyrockso about TestRemoved02:07
andyrockit try to remove an item...02:07
andyrockto be sure that the item has been removed02:08
andyrockit uses a variable02:08
andyrockthis variable is setted in the callback of the remove signal02:08
andyrockbut what happens if we need variable value but remove signal callbacks has not yet been called?02:10
thumperhmm...02:21
thumperideally it should only wait for a short time for the result02:22
thumperand if it gets a timeout, the test should fail02:22
thumperI wonder why you aren't getting a signal callback02:22
andyrockthumper, so i've added some printf functions in the test02:33
andyrockto debug02:33
andyrockand...02:33
* thumper waits02:33
andyrockthe removed callback is called02:34
thumperha.02:34
thumperbut?02:34
andyrocknow we have02:34
andyrock  FilterRecorder recorder(model_, iter1_);02:34
andyrock  dee_model_remove(model_, iter2_);02:34
andyrock  EXPECT_EQ(recorder.removed_, false);02:34
andyrock  dee_model_remove(model_, iter0_);02:34
andyrock  EXPECT_EQ(recorder.removed_, false);02:34
andyrock  02:34
andyrock  dee_model_remove(model_, iter1_);02:34
andyrock  EXPECT_EQ(recorder.removed_, true);02:34
andyrocki've added a printf at the end02:35
andyrockand the string is printed02:35
andyrockbut the test never ends...02:35
andyrockexpect_eq works well02:35
thumperwhere does it wait?02:35
thumperactually02:36
thumperwhile you are there02:36
andyrocki don't know... maybe a gtest bug02:36
thumpercan you change EXPECT_EQ to EXPECT_TRUE and EXPECT_FALSE?02:36
andyrockyeah02:36
* thumper goes to look at the source too02:36
thumperandyrock: have you tried stepping through with gdb?02:41
andyrocki'm doing it right now...02:42
andyrockthumper, debug with gdb of a gtest test is evil03:07
andyrockthere are too many macro03:07
thumperdebug macros are awful agreed03:07
thumpercan you step over though?03:07
andyrockbut it's really really weird, the dtor of TestFilter is called03:08
andyrockstep by step? right now no...03:09
andyrockbecause i cannot add a break03:09
thumperbecause?03:09
andyrockbecause (maybe) tests are not build with debug info...03:09
andyrockat least i guess03:09
thumperthey should be if you are normally building with debug info03:10
thumperhow do you run cmake?03:11
andyrockcmake .. -DCMAKE_BUILD_TYPE=Debug -DCOMPIZ_PLUGIN_INSTALL_TYPE=local -DCMAKE_INSTALL_PREFIX=/usr03:13
andyrockso to run in on gdb03:14
andyrocki do03:14
andyrockgdb dbus-test-runner03:14
andyrockthan...03:14
andyrockrun --task ./test-gtest-service --task ./test-gtest03:15
* andyrock is going to bed (4:17 AM in Italy)03:17
andyrockthumper, mail me if something is wrong on my cmake, etc.03:18
andyrockplease :)03:19
thumperok03:19
thumperit is very weird03:19
smspillazthumper: macros are why I am skeptical of google-test :)03:38
thumpersmspillaz: all testing frameworks use macros03:38
thumpersad but true03:38
smspillazthey can't use inlines ?03:39
thumperno03:39
nolavizanyone there?06:16
smspillazthumper: interesting fact. vanvugt is in perth07:35
* smspillaz did not know that :)07:35
thumpersmspillaz: and vanvugt is?07:41
smspillazhe proposed a whole bunch of vsync fixes07:42
hrwhi09:42
hrwcan someone tell me how to disable F10 going to indicators?09:42
andyrockhrw, from ccs09:51
andyrock*ccsm09:51
hrwok09:51
hrwother question: how to disable keyboard shortcut in gnome apps? Clicking on shortcut gives me way to change key but not to disable09:52
hrwok, found - backspace10:01
hrwis this a known problem then when I switch to Thunderbird (start+3 on 3rd desktop) I have to switch between windows to be able to type in one of them?10:17
hrwnot always happens but quite often10:17
hrwsame happened with terminal (start+1, 1st desktop)10:21
hrwbug 891077 anyone?10:34
ubot5Launchpad bug 891077 in unity-2d (Ubuntu) "Add vertical/horizontal maximize on RMB/MMB" [Undecided,New] https://launchpad.net/bugs/89107710:34
Andy80hi11:00
hrwis most of ayatana team based in US?12:16
=== MacSlow is now known as MacSlow|lunch
hrwbug 891116 is irritating12:48
ubot5Launchpad bug 891116 in unity (Ubuntu) "After switching desktops I am unable to enter data from keyboard" [Undecided,New] https://launchpad.net/bugs/89111612:48
smspillazhrw: I'll look into that next week when I'm off exams and we're based all over the world pretty much, thumper workdays starting at UTC+11, me for workdays starting UTC+8, most of the others come on at around UTC+1 and the US people come on at about UTC-812:50
smspillaz(starting at 9AM at each of those times)12:51
hrwthx smspillaz12:51
smspillazso there is roughly 24 hour coverage, except for when I'm at university and part time, in which case there is a slight 3 hour gap between when thumper logs off and I log on12:51
smspillaz(since I switch to Europe then)12:51
hrwok12:53
=== davidcalle_ is now known as davidcalle
ockham_mhr3: nag nag...14:21
=== MacSlow|lunch is now known as MacSlow
ockham_mhr3: ... about my mp ...14:21
mhr3ockham_, it looks good, the only thing i'm not sure about are the BindFlags you're using14:32
mhr3i'd suggest using just GET14:33
kamstrupmhr3: ockham_ signed the ca - so I think you can approve https://code.launchpad.net/~ockham-razor/unity-lens-applications/lp785101/+merge/8228014:49
mhr3kamstrup, yea, i got the mail, what do you think about the flags? ^^14:51
ockham_mhr3: sry, been afk15:08
kamstrupmhr3: which "flags"?15:09
kamstrupmhr3: if it's about having the conf keys in general, I am ok with it. Not that I like adding options, but there has been more than enough user requests to justify it I guess15:11
mhr3kamstrup, GSettings' BindFlags when binding the properties15:11
kamstrupah, that15:11
ockham_this one, right: GLib.SettingsBindFlags.DEFAULT15:11
mhr3yea, i'm thinking we want just .GET15:11
mhr3kamstrup, ^^15:11
mhr3ockham_, also drop the GLib. pls15:12
ockham_mhr3: one sec...15:13
kamstrupmhr3: I don't have any particular opinion. DEFAULT definitely works. GET will too, but has a slim chance of subtly breaking in the future...15:13
mhr3kamstrup, what breaking do you have in mind?15:14
ockham_kamstrup: ^^ i'm also curious, as i'm waiting for what i should commit...15:45
ockham_mhr3: if i'd change it to GET, should i also change the properties to { get; default = true; } instead of { get; set; default = true; } ?15:46
mhr3ockham_, no15:49
=== nuthinking__ is now known as nuthinking
om26er_kenvandine, Hello! do you have some insights on bug 854666 ? Could it be related to telepathy-indicator?16:05
ubot5Launchpad bug 854666 in empathy (Ubuntu) "Starting empathy from messaging indicator makes window disappear" [Medium,Confirmed] https://launchpad.net/bugs/85466616:05
kenvandineom26er_, maybe16:06
om26er_also kenvandine I suppose bug 886056 is on the TODO for LTS ?16:07
ubot5Launchpad bug 886056 in telepathy-indicator (Ubuntu) "Cannot respond to friendship requests from Empathy" [High,Confirmed] https://launchpad.net/bugs/88605616:07
om26er_sounds also related to the telepathy-indicator16:07
om26er_ronoc, I found you :-)16:09
ockham_mhr3: i've pushed the .DEFAULT -> .GET change now16:09
om26er_ronoc, there is a problem with the sound menu controls, ref: bug 86440516:10
ubot5Launchpad bug 864405 in indicator-sound (Ubuntu) "Clicking on left 25% of "Pause/Play" circle clicks "Previous"" [Medium,Confirmed] https://launchpad.net/bugs/86440516:10
kenvandineom26er_, 886056  is  tp-indicator16:11
kenvandinenot sure about 85466616:11
kenvandinethat sounds more like focus related stuff16:12
om26er_kenvandine, I will be tested a few versions of telepathy-indicator and maybe downgrade empathy16:12
om26er_I just made a clean install and I think I dont have the issue here16:13
kenvandineom26er_, i couldn't reproduce it16:13
om26er_my two fully updated systems are affected16:13
om26er_I believe it happens the first time just after login (so could be compiz)16:13
kenvandineom26er_,  tp-indicator really just calls g_app_info_launch on empathy.desktop16:14
kenvandineso the first time the contacts list window opens?16:15
om26er_kenvandine, yep, first time16:15
kenvandineand when it goes away, is the window really gone?16:15
kenvandineor just not on top?16:15
om26er_it hides, you have to invoke from the messaging menu again16:16
om26er_just appears for few miliseconds and gone16:16
om26er_next time it starts fine though.16:16
mgedminunity launcher keeps losing track of applications: now it doesn't see that I have xchat-gnome running any more...16:20
ronocom26er, hey, checking now16:23
om26erronoc, can you reproduce it as well ?16:27
ronocom26er, yep i can16:28
ronoci thought i fixed that16:28
om26erronoc, I might do an SRU when its fixed :-)16:30
=== zyga is now known as zyga-afk
ockham_mhr3: ping17:20
ronocom26er, fixed on trunk18:49
ronocom26er, if you want a release you will need to hassle one of the packager18:49
mhr3ockham_, sorry it took a while, approved19:25
ockham_mhr3: great, thx, no problem!19:25
mgedminhey, wait a sec -- unity now thinks my xchat-gnome window belongs to Chromium20:07
mgedminhow exactly does it assign windows to applications, again?20:08
thumpermorning20:11
thumpermgedmin: it uses a library called bamf20:13
thumperas I understand it there are a number of different rules it follows20:13
mgedminthanks for the pointer20:15
mgedminhm, and there's a bamfdaemon20:17
mgedminwoohoo, somehow my xchat window now has _NET_WM_DESKTOP_FILE(STRING) pointing to chromium-browser.desktop20:18
ockham_mhr3: one more question, as i'm not really familiar with launchpad workflows -- what happens after your approval? who performs the actual merge?20:34
mhr3we don't have automatic merging yet, so either me or kamstrup20:35
Andy80mgedmin: who develop xchat-gnome? I'm using it since yesterday and there would be some things I'd like to be changed to improve it....20:35
ockham_mhr3: ok. but i don't need any more reviews, or do i?20:36
om26erfork it :p20:36
mgedmindunno, https://launchpad.net/xchat-gnome ?20:36
mhr3ockham_, not unless someone else will have comments to the merge20:37
Andy80om26er: if it was not coded in C/Gtk I would be glad :P I'm not good with C/Gtk, sorry :\20:37
ockham_mhr3: hm, sry, i still don't quite understand. are you going to merge it? or are we waiting for kamstrup to approve?21:20
mhr3ockham_, we'll wait if anyone else will have comments, if not someone will merge it in a couple of hours/days21:22
ockham_mhr3: alright, thanks for clearing that up. and sorry for nagging21:23

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