/srv/irclogs.ubuntu.com/2013/12/13/#ubuntu-unity.txt

=== duflu_ is now known as duflu
didrocksSaviq: hey! I saw robru released unity8 without qmenumodel (both were listed in the landing ask), is there any risk if I kick an image without qmenumodel?07:26
Saviqdidrocks, the bug won't be fixed07:39
didrocksgrrr, I wonder how robru confirmed the fix then…07:39
* didrocks *sigh*07:39
Saviqdidrocks, well, it's a race07:40
Saviqdidrocks, so he might not have encountered it07:40
didrocksok07:41
didrocksSaviq: I'm building qmenumodel right now07:41
didrockswill deliver and quick an image with it07:41
Saviqdidrocks, cool, thanks07:41
Saviqdidrocks, I found the issue with bug #1256061 - it's unity8 and the indicator in the end - have a fix for unity8 ready, need yet to write a test07:42
ubot5bug 1256061 in Unity 8 "clock forcefully switches from 24h to 12h AM/PM format once the panel clock is loaded" [High,In progress] https://launchpad.net/bugs/125606107:42
didrocksSaviq: argh, ok. I'll have to explain that to management then :/07:42
didrockson the "why did we release unity8 without the fix"07:43
didrocksSaviq: I'll try to put it nicely ;)07:43
Saviqdidrocks, well, that bug wasn't blocking07:44
didrocksSaviq: for the unity8 release yeah, it was07:44
Saviqdidrocks, ah right, damn07:44
didrocksSaviq: do you think this is going to land soon?07:44
Saviqbut was attributed to qtubuntu07:44
didrocksif so, we can bundle it to the same image :)07:44
Saviqdidrocks, well, today07:44
didrocksSaviq: if it's in the incoming hour, we can07:45
Saviqdidrocks, but some hours ahead for sure07:45
didrocksbut ok, so will be too late07:45
didrocksnevermind, keep getting things landing :)07:45
dandraderSaviq, for getting https://bugs.launchpad.net/unity-api/+bug/125805708:40
ubot5Ubuntu bug 1258057 in Unity API "unity-api fails to build against Qt 5.2" [Critical,In progress]08:40
dandraderSaviq, sorry, I meant this http://paste.ubuntu.com/6557929/08:41
dandraderSaviq, did you just "bt" or did you also add some arguments08:41
dandraderfor printing out variables on each frame08:41
Saviqdandrader, that was `bt full`08:42
dandraderah08:42
dandraderdidn't know that one, always used just "bt"08:42
dandraderSaviq, is that a normal output for testLauncher from unity-api? http://pastebin.com/d4EKTZ9k (btw, Ubuntu's pastebin refused this text)09:11
dandraderI wonder if I'm missing something in my setup...09:12
Saviqdandrader, looks that way09:12
dandraderthe test didn't fail though...09:13
Saviqdandrader, yeah, that's just warnings09:13
Saviqdandrader, should be cleaned up anyway, true09:13
dandraderSaviq, so you don't get those warnings?09:14
Saviqdandrader, yes I do09:14
dandraderah, ok. then my setup is fine09:15
Saviqdandrader, sorry, the "looks that way" was to your previous message09:15
dandraderSaviq, I didn't get any crash in the tests. but testApplication failed, but without crashing09:15
Saviqdandrader, under 5.2 on i386?09:16
dandraderSaviq,  that's with the "stable" qt5 branch on a i386 chroot09:16
Saviqdandrader, ah, stable is 5.109:16
Saviqerm09:16
Saviq5.2.109:16
Saviqdandrader, maybe it was fixed between 5.2 and 5.2.109:16
dandraderso that's good news I suppose09:16
Saviqdandrader, yeah, would be awesome to bisect and get the commit into our packages09:17
dandraderwill check why testApplication is failing...09:17
Cimimzanetti, ping :)09:30
mzanettihi Cimi :)09:30
Cimimzanetti, did you go any further with the exploration? is there anything you suggest ding?09:31
Cimi*doing09:31
mzanettiCimi: did you find out if its related to that hack?09:32
Ciminope, didn't try actually09:32
mzanettiCimi: iirc there was some discussion lately regarding dynamically adding/removing tabs. I think the Tabs component gained support for that recently. So I'd say way to go would be to check out if that hack is actually needed any more and if removing it fixes the issue09:33
mzanettias that's the only thing I can think of what's different in apps and unity809:33
mzanettimaybe dednick can explain what exactly is this hack for09:34
mzanettiI didn't fully understand the bug report tbh09:34
tsdgeoshttps://code.launchpad.net/~aacid/unity8/verticalJournal/+merge/198897 is up if anyone wants to do some view reviewing09:49
tsdgeosSaviq: i'll do hJournal next and try to share as much code as possible with the vJournal09:49
Cimimzanetti, it does not fix the issue09:50
mzanettiCimi: strange09:53
Saviqtsdgeos, yup, great09:54
mzanettiCimi: I guess at this point I would start with a new file and Tabs, and starting to build the same thing (with repeater etc) until the issue is reproducable outside of unity09:54
Cimimzanetti, I think it might be for the loaders?09:55
tsdgeosdandrader: totally right09:55
tsdgeosdandrader: and something like what listviewwithpageheader.cpp has on the top09:55
tsdgeosexplaining a bit the code won't hurt either09:55
mzanettiCimi: Don't know... but yeah. the fact that the width is set multiple times suggests something like this09:56
xnoxSaviq: tsdgeos: can we leave out policy changes from cross-compilation fixes? i.e. at the moment unity8 generates dbus code at build time, and I simply fixed it to succeed when cross-compiling.09:58
xnoxSaviq: tsdgeos: changing when dbus code is generated, or stored. is not my proposal at all.09:58
tsdgeosxnox: as said it looks good to me, but i'll let Saviq be the one to decide09:59
xnoxtsdgeos: cause i have tasty cmake upload which will cross-compile unity8 .... if above is merged.09:59
xnoxtsdgeos: ah, looks like Saviq agrees to keep regenerating dbus code at build time.10:00
Saviqxnox, tsdgeos, yeah10:01
Saviqdidrocks, good news10:01
Saviqdidrocks, https://bugs.launchpad.net/ubuntu/+source/qtubuntu/+bug/125606110:02
ubot5Ubuntu bug 1256061 in Unity 8 "clock forcefully switches from 24h to 12h AM/PM format once the panel clock is loaded" [High,Opinion]10:02
Saviqdidrocks, that bug is actually invalid...10:02
didrocksSaviq: oh, it's magically fixed?10:03
didrocksI'll flash latest10:03
didrocksand switch to French10:03
xnoxtsdgeos: hm, jenkins gave "Needs Fixing" review, is there actually something I need to fix?10:04
tsdgeosnah10:04
tsdgeosit's some crap we need to fix10:05
Saviqxnox, some flakiness still10:05
tsdgeosreally really10:05
xnoxtsdgeos: ok, thanks =)10:05
mhr3_tsdgeos, something already using the vertical journal?10:07
tsdgeosmhr3_: the test :D10:07
mhr3_tsdgeos, maybe you want to hook it up with lp:~unity-team/unity8/new-scopes? ;)10:08
tsdgeosmhr3_: i just hooked it up where Saviq told me to10:08
Saviqtsdgeos, *hook* as in actually make use of it10:09
Saviqtsdgeos, mhr3_, let's get it merged into unity8 trunk10:09
Saviqtsdgeos, mhr3_ then merge trunk into new scopes, and make use of it10:09
mhr3_sure, i'm thinking step 2 already :)10:09
Saviqtsdgeos, mhr3_ oh yeah, we can go with new-scopes + vjournal in a branch already10:10
Saviqmhr3_, only we don't have variable height cards yet ;)10:10
mhr3_:(10:10
Saviqmhr3_, or a scope that gives us data that could be used for that, for that matter :P10:10
mhr3_Saviq, details :P10:10
Saviqmhr3_, I *know*10:11
mhr3_Saviq, btw json-defaults was looking for you, and there's one more branch built on top of it10:12
mhr3_fwiw those are really simple branches :)10:12
=== dandrader is now known as dandrader|afk
=== anpok|afk is now known as anpok
Saviqmhr3_, yeah, on my list for today10:26
Saviqdidrocks, news?10:34
=== dandrader|afk is now known as dandrader
didrocksSaviq: flashing right now, under a tons of things :p10:36
Saviqdidrocks, k10:40
didrocksSaviq: not fixed here10:47
didrocksSaviq: I move to French10:47
Saviqdidrocks, rebooted?10:47
didrocksset to 19h in the system settings ui10:47
didrocks(after a reboot)10:47
didrocksand it's written 7:4510:47
didrocksfor instance10:47
Saviqdidrocks, but no AM/PM?10:47
didrocksno AM/PM10:47
didrocksjust 7:4510:47
Saviqdidrocks, let me check10:47
Saviqdidrocks, ok I think I know what's happening then10:48
Saviqdidrocks, the default setting is 12hr10:49
didrocksSaviq: this default setting is in unity8?10:50
Saviqdidrocks, no, it's for the datetime indicator10:50
Saviqdidrocks, that string comes directly from the indicator now10:50
Saviqdidrocks, preformatted10:50
Saviqgsettings get com.canonical.indicator.datetime custom-time-format10:50
nic-doffaydidrocks, ping10:51
didrocksSaviq: can you ensure the indicators guys are in the loop then?10:51
didrocksnic-doffay: pong10:51
nic-doffaydidrocks, going to look into this: https://bugs.launchpad.net/unity8/+bug/126037910:51
ubot5Ubuntu bug 1260379 in Unity 8 "A unity8.override file should be shipped to allow apport completion during test runs" [High,Triaged]10:51
didrocksno need to ping me when I'm talking on that channel, just ask :p10:51
nic-doffaywhere should I start.10:51
didrocksnic-doffay: ah great!10:51
Saviqlarsu, around?10:51
Cimimzanetti, found10:52
Cimimzanetti, line 141 of MenuContent.qml10:52
mzanettiCimi: \o/10:52
Cimithat binding does not work10:52
didrocksnic-doffay: I would say, look at the unity8 file, ship an upstart override file with the extra shutdown period10:52
Cimimzanetti, if you put title: "Example" at line 103 and you remove that binding, the indicator icon is rightly placed10:53
mzanettiinteresting10:53
Cimimzanetti, obviously the reason why this is causing the style to break is obscure10:53
Cimito me10:53
Cimimzanetti, because you get the title (the text)10:54
mzanettiCimi: so I guess the binding does work, but because it waits for the loader.item to be loaded it overwrites width a second time which breaks the tab10:54
Cimibut breaks other bindings10:54
mhr3_Saviq, you were trying to build libunity-scopes on saucy and it failed?10:54
Cimimzanetti, good spot10:54
Saviqmhr3_, yes10:54
larsuSaviq: yep10:54
mhr3_Saviq, do we need to fix that?10:54
Saviqmhr3_, exceptions in tests10:54
=== mhr3_ is now known as mhr3
Saviqlarsu, are you maintaining indicator-datetime?10:55
mzanettiCimi: ok.. but in this case the proper fix is indeed inside the style (I'd say)10:55
mhr3Saviq, fwiw i'm running it fine on saucy10:55
mhr3but i'm building the pkg locally...10:55
mzanettiCimi: a tab should not break if you change title at runtime10:55
Saviqmhr3, https://launchpad.net/~phablet-team/+archive/desktop-deps/10:56
larsuSaviq: no, charles is10:56
Saviqlarsu, I've tracked down bug #1256061 to be, in fact, an indicator-datetime issue10:56
ubot5bug 1256061 in Indicator Date and Time "clock forcefully switches from 24h to 12h AM/PM format once the panel clock is loaded" [Undecided,New] https://launchpad.net/bugs/125606110:56
mzanettiSaviq: see, told ya :) (re: OSK swipe down from everywhere)10:56
Saviqmzanetti, ;P10:56
Saviqlarsu, ok, will assign to him then10:57
larsuSaviq: yep, looks like it. We should fix this for real and send a time stamp...10:57
Saviqlarsu, yeah, another bug10:57
Saviqlarsu, I actually already have a branch that "fixed" it by ignoring what the indicator sent up completely10:57
Saviqlarsu, but wasn't real happy with it, and it didn't take all the other settings into account etc.10:58
larsuSaviq: right, sending a time stamp doesn't even make sense for the current time10:58
Saviqlarsu, exactly10:58
larsuwhat settings are those?10:58
larsuwhether to show the year, 24h, etc?10:58
Saviqlarsu, https://wiki.ubuntu.com/TimeAndDate?action=AttachFile&do=view&target=settings-clock.png10:59
larsuright10:59
dednickCimi: ping11:00
Cimidednick, pong11:00
dednickCimi: hey, can i get you to do a review for me? https://code.launchpad.net/~nick-dedekind/ubuntu-settings-components/visual-updates/+merge/19330111:00
dednickwhen you have some time.11:00
nic-doffaydidrocks, what am I looking for in this crash file?11:00
Cimidednick, sure, now11:00
dednickCimi: thanks11:01
dandraderMirv, what's the plan for trusty regarding Qt? is our target to endup shipping 5.2 or there's no fixed target yet, meaning that it could be 5.2.1, etc?11:01
Saviqdandrader, 5.2 for sure, if 5.2.1 gets there in time - hopefully11:03
didrocksnic-doffay: what you want is to have a stop 30 in this override to let the coredump to be collected before upstart is killing it11:03
Saviqdidrocks, "kill timeout 30", no?11:04
didrocksyep11:04
Saviqnic-doffay, just have a unity8.override file with that line ↑↑11:04
Saviqnic-doffay, put it in data/11:04
Saviqnic-doffay, and add a line to debian/unity8-autopilot.install11:04
dandraderit's interesting that there's already an enormous diff between qtdeclarative release (5.2) and stable (5.2.1)11:05
Saviqnic-doffay, to install that .override file into /usr/share/upstart/sessions11:05
Saviqdandrader, yeah indeed11:05
Saviqdidrocks, unless there's a better place to put it in ↑↑?11:05
didrocksSaviq: no, thats perfect11:06
nic-doffaySaviq, right will get going with that then.11:09
Saviqdoes anyone else's right-mouse-button get crazy sometimes and does a click as soon as you move the mouse, without you releasing it?11:10
Saviqlike it creates a new folder on-right-click for me, instead of opening the menu :/11:11
Saviqit does that for a minute at a time and then goes back to normal11:11
Saviqmhr3, with https://code.launchpad.net/~michihenning/unity-scopes-api/what-for-exceptions/+merge/198872 there will be some actual useful output in the build log11:11
mhr3Saviq, yea, just got a mail from michi about that branch11:12
dandraderSaviq, working fine on my desktop11:12
Saviqdandrader, yeah, I'm not sure if it's my mouse, or the touchpad playing games...11:12
=== iahmad is now known as iahmad|afk
=== dandrader is now known as dandrader|lunch
Mirvdandrader|lunch: as saviq said, 5.2.x is the target for sure. the focus is on 5.2 + backported patches for now.11:43
=== dandrader|lunch is now known as dandrader
=== _salem is now known as salem_
=== iahmad|afk is now known as iahmad
nic-doffaySaviq, how should I test for this now?11:55
Cimiguys, we should fix the scrolling...12:00
Cimiof flickable12:00
Cimivelocity and deceleration12:01
Cimiit's broken on the desktop and on the devices12:01
SaviqCimi, filed a bug yet?12:09
Mirvphew, so much to do... (with Qt 5.2)12:10
SaviqMirv, between rc1 and release?12:10
Saviqnic-doffay, with unity running, go:12:11
Saviqkill -11 `pidof unity8`; stop unity812:11
Saviqnic-doffay, stop unity8 should return after more than 5s12:12
MirvSaviq: not really between those, but doing this correctly instead of ad-hoc. symbols updates for all archs (multiple builds), syncing with Debian, rebuilding everything after noticing Debian did now do the qt5core lib rename, patches coming from all directions for enabling tests etc (which should be then contributed back to Debian also at some point)12:12
SaviqMirv, right, so basically the real packaging12:13
MirvSaviq: so I thought today I would make progress, instead I find myself being back to near beginning with 5.2 final build :)12:13
CimiSaviq, I need to test again on my galaxy nexus, but it's definitely broken in the desktop12:14
Mirvbut of course progress made reducing delta, going through changes etc12:14
CimiSaviq, try flicking on the sdk components gallery12:14
CimiSaviq, it's like crazy :)12:14
Cimiactually now works12:15
Cimi:-\12:15
SaviqCimi, yeah, seems fine in the pickers12:16
CimiSaviq, it's fine now for me12:16
CimiSaviq, was badly broken yesterday12:16
CimiSaviq, like over scrolling with no limits12:16
CimiSaviq, can I test qt 5.2 on maguro?12:18
SaviqCimi, you could try12:18
SaviqCimi, ppa:canonical-qt5-edgers/qt5-beta212:18
SaviqCimi, but it's probably better to wait a bit12:19
CimiSaviq, I wanted to try the scrolling12:19
SaviqCimi, it's all in flux now that 5.2 was released finally12:19
SaviqCimi, I don't think anything changed tehre12:19
CimiSaviq, was broken in all our previous versions12:19
CimiSaviq, try seeing the default velocity of the flickable12:20
SaviqCimi, and it all basically deserves a QTBUG that the default values are a) pixel-dependent and b) not customizable without rebuilding Qt12:20
CimiSaviq, it was 1500 iirc, should be much higher with high dpi12:20
mzanettiI have a weird issue right now... I cannot drag (flick) any scope upwards unless I pull it down a little beforehand12:20
mzanettiis that known?12:21
SaviqCimi, https://qt.gitorious.org/qt/qtdeclarative/source/e895bc315f9959a5dad80a8f0bf7b73bced0bde1:src/quick/items/qquickflickable.cpp#L6112:21
Saviqmzanetti, no12:21
Saviqmzanetti, phone?12:21
mzanettiyeah12:22
Saviqnic-doffay, so the real test there is to see that the package built and the file gets installed into /usr/share/upstart/sessions/12:23
Saviqnic-doffay, if it is - that's all we can do TBH12:23
nic-doffaySaviq, killing it from a separate console has strange results.12:23
nic-doffayUnity8 jus segfaults.12:24
Saviqnic-doffay, that's what -11 does12:25
Saviqnic-doffay, that's SIGSEGV12:25
Saviqnic-doffay, but anyway - more real-life test12:26
Saviqhmm crap12:26
nic-doffaySaviq, getting this too: stop: Unknown job: unity812:26
Saviqnic-doffay, as phablet12:26
nic-doffaySaviq, so run on device then run as phablet user.12:27
Saviqnic-doffay, man, run on device won't do you any good12:27
Saviqnic-doffay, that file will only get installed where it needs to when packaged12:28
Saviqnic-doffay, run_on_device doesn't install anything12:28
nic-doffaySaviq, what's the deal with stop: Unknown job: unity812:28
Saviqnic-doffay, you're not logged in as phablet, most likely12:28
Saviqnic-doffay, or you managed to uninstall unity812:29
nic-doffaySaviq, the former then.12:29
CimiSaviq, that's bad12:29
SaviqCimi, you've been complaining about this for what, half a year now?12:30
SaviqCimi, did you file a bug?12:30
Ciminope12:30
CimiI'm a lazy guy (my main flaw) and not registered yet to qt bugs12:31
CimiI'm going to to this today and I will be digia's worst nightmare12:31
Cimiperfectly in time for Friday the 13th12:31
Saviqnic-doffay, just file the merge proposal12:33
tsdgeosSaviq: ok i think i know why that test_show_scope_on_load is failing12:33
Saviqnic-doffay, we'll know soon enough whether it's correct12:33
Saviqtsdgeos, pray tell12:33
tsdgeosSaviq: now i want to know if i change the test of the code :D12:33
nic-doffaySaviq, yeah was trying to avoid that though!12:34
tsdgeoslet me to a pastebin explaining12:34
Saviqnic-doffay, it's this kind of change12:34
Saviqnic-doffay, to test for real you'd have to make unity8 crash on exit12:34
Saviqnic-doffay, which isn't difficult, btw, if you want - it's just a case of tweaking the main() to not destroy the view or the application, but return application->exec() straight away12:35
Saviqnic-doffay, then, without your .override file, upstart will kill unity8 before apport's managed to collect the full crash report12:35
Saviqnic-doffay, with it, upstart will wait for up to 30s (as opposed to 5s) before killing12:35
Saviqnic-doffay, giving time for apport to collect the crash file12:36
nic-doffaySaviq, I'll give that a go while waiting for jenkins.12:36
Saviqnic-doffay, you can do 'stop unity8; start unity8 BINARY=$PWD/builddir/unity8' to launch the locally built unity8 under upstart12:37
nic-doffaySaviq, cool thanks12:37
Saviqnic-doffay, having built it first with run_on_device, and logging in as phablet, unity8 is built in /home/phablet/shell/builddir/unity812:37
Saviqnic-doffay, and BINARY=... just tells upstart to launch that binary instead of the system-installed one12:38
tsdgeosdamn it's hard to write the thoughts down12:38
tsdgeosSaviq: actually no, scrap that, can't tell :-/12:41
tsdgeosbut i'm going to add a hell lot of console.log and will find out12:41
Saviqtsdgeos, maan :/12:41
tsdgeosi'm tired of all those errors12:41
Saviqtsdgeos, please do12:41
Saviqtsdgeos, +112:41
dednickSaviq: did we decide we would pass https://code.launchpad.net/~nick-dedekind/unity8/StrFTimeFormatter/+merge/192343 ?12:57
Saviqdednick, I'm still not really sure what it's supposed to fix ;D12:57
Saviqdednick, but then we decided that the indicators should just put up timestamps (in case of non-current time) or nothing at all (in case of current time), and users' preferences should be applied globally by applied globally by new SDK component(s) - TimeFormatter and RealTime or so12:59
Saviqdednick, so unless that branch actually fixes something that I'm not aware of, I'd reject it12:59
dednickSaviq: right, but thats a todo later right...12:59
dednickSaviq: it's for the datetime alarm menu items.12:59
Saviqdednick, ah, ok12:59
Saviqdednick, can we see them already?13:00
Saviqdednick, like if I add an event in my calendar, will they show up?13:00
dednick"you can see them", but there are no times.13:00
Saviqdednick, which that branch is supposed to fix, correct?13:00
mhr3Saviq, ok with exposing the raw json in the category models purely for the dev-tool usage?13:01
dednickSaviq: um. it will, when we have an event item :)  It's in ubuntu-settings-components. Currently we're just using the standard item (no time).13:02
=== alan_g is now known as alan_g|lunch
Saviqdednick, so still no way to see the fix, eh? ;)13:02
dednicknope. but this branch also moves the formatter into Utils :)13:03
dednickSaviq: oh, and my indicator tests branch is done. :) https://code.launchpad.net/~nick-dedekind/unity8/indicator-page-factory-tests/+merge/19878513:13
Saviqdednick, yup, saw that13:13
Saviqdednick, and ohkay, if you twist my arm... can you just add a TODO/FIXME somewhere where it makes sense, and refer to bug #126072813:21
ubot5bug 1260728 in Unity 8 "Indicators should send timestamps, not pre-formatted strings" [Medium,Triaged] https://launchpad.net/bugs/126072813:21
dednickSaviq: :) ok13:21
Saviqxnox, so does that mean that with the new cmake upload and the unity8 branch, we'll be able to cross-build directly? :)13:21
xnoxSaviq: yes, along with pleora of other packages. =))) one time setup: mk-sbuild --target armhf, and after that it's just $ sbuild -A --host armhf unity8*.dsc13:23
Saviqxnox, YUMMY13:23
Saviqxnox, do you know if distcc should work fine when cross-building? ;)13:23
xnoxSaviq: once qt*-dev stack is co-installable it will be possible to even do it without a chroot with: dpkg-architecture -aarmhf -c cmake ../ && make13:24
Saviqxnox, I know ccache does work fine13:24
xnoxSaviq: why do I need distcc, or ccache if it takes 2m20s to cross-compile unity8 clean from start to finish?13:24
Saviqxnox, nice one, we'll have to transition our run_on_device script to that :)13:25
Saviqxnox, right, it's much simpler recently ;)13:25
Saviqxnox, but then I'm thinking of moving to an ultrabook at some time in the near future, will not get the quad-core thing I have now ;)13:25
xnoxSaviq: .... cross-building happens on amd64/i386 which is fast and multi-core (typically) so simply DEB_BUILD_OPTIONS=parallel=1213:25
Saviqxnox, yeah -j9 is my default13:25
dednickSaviq: done. i just put it in the Timeformatter header.13:25
Saviqdednick, cheers13:26
Saviqxnox, and well, it's not just about unity8, I actually compile other things as well ;)13:26
xnoxSaviq: you do want to set it via DEB_BUILD_OPTIONS or even specifically in ~/.sbuildrc $build_environment {'DEB_BUILD_OPTIONS' => 'parallel=9'}13:26
Saviqxnox, I barely do anything else than bzr bd, which does have sbuild -j9 as the builder13:27
Saviqxnox, when building packages, that is13:27
xnoxSaviq: right. so the only requirement is that distcc / ccache generates / replaces $DEB_HOST_GNU_TYPE-[gcc|g++]13:27
xnoxSaviq: cause that's the compiler that cmake will pick when cross-compiling.13:27
Saviqxnox, yeah, I thought it *should* work in theory13:27
Saviqxnox, will do some testing13:28
* xnox has no need though =)13:28
xnoxI have 8 core, 32GB of RAM, and I compile in tmpfs with no iowait ;-)13:28
* dandrader makes a mental note that "good" and "bad" in git bisect mean the other way round when you're looking for the commit that fixes something13:32
Saviqxnox, well, I'm 4 core HT, 8GB RAM and compile in shm, too (and SSD to boot)13:35
Saviqxnox, but as I said, am thinking of moving to an ultrabook, at which point I'd like to offload some of that umph ;)13:35
davmor2xnox: man that is an awesome phone :D13:35
seb128Saviq, larsu: so the timestamp issue is back on the indicator side?13:35
Saviqseb128, yeah13:36
xnoxdavmor2: =))))) not very portable, and battary UPS only lasts 10m13:36
Saviqseb128, it's a string straight from indicator-datetime13:36
Saviqseb128, I did fix it unity8-side, by ignoring what that string says at all, but didn't like the solution for now13:36
Saviqseb128, bug #1260728 to fix it long-term13:36
ubot5bug 1260728 in Unity 8 "Indicators should send timestamps, not pre-formatted strings" [Medium,Triaged] https://launchpad.net/bugs/126072813:36
seb128Saviq, ok, thanks13:36
xnoxSaviq: right. I made my machine ssh accessible and my terminal on my ultrabook has gnome-terminal with the following shell set $ ssh -X mydesktop.IP13:37
seb128Saviq, that's different from the messaging menu one, right?13:37
Saviqseb128, yeah, that's fixed in u8 trunk13:37
seb128cool13:37
Saviqseb128, and released already, btw13:37
seb128Saviq, good to read ;-)13:37
* seb128 didn't keep up with what was going on this week, just too much stuff changing ;-)13:38
Saviqseb128, yeah, but it's a good problem to be had :)13:38
seb128indeed13:38
dednicklarsu: howdy. Do you know why are all the messaging menu items "not sensitive"13:39
Saviqmpt, hey, q: looking at https://wiki.ubuntu.com/TimeAndDate, there are settings for the clock on the desktop, but not on phone - is that on purpose? would kind-of mean that you can change those when your phone is docked, and it should affect your phone, but you don't get to change those settings on the phone?13:40
Saviqmpt, or the other way - you can change it on the desktop, but that doesn't affect your phone clock?13:41
Saviqmpt, also, those settings are *only* meant to affect the indicator? no way for users to select 24h clock under British English, for example?13:41
Saviqglobally, that is13:41
Saviqmpt, it'd be pretty weird if you ask me, if you chose your clock to be 24h, but it would only affect the indicator and not the rest of the system...13:42
Saviqmhr3, for https://code.launchpad.net/~mhr3/unity-scopes-shell/json-defaults/+merge/19859113:43
Saviqmhr3, what I did in JavaScript, and was thinking you'd do as well, is work the other way - parse a JSON file with defaults, and update the values based on data coming from the scope13:44
Saviqmhr3, your way involves adding lotsa code just to add a new key with a default for any component/template13:45
Saviqmarcusto_, hey, did you have a chance to look at http://json-schema.org/ ?13:47
Saviqmhr3, oh, and on that note... should those defaults not be applied server-side?13:48
marcusto_Saviq: hey, yeah I did, and it looks like this lib fits our needs quite well: http://avro.apache.org/docs/1.6.3/api/cpp/html/13:49
Saviqmarcusto_, oh, awesome13:49
marcusto_Saviq: problem is, until I get something running I'm holding up the web guys on SSS13:50
Saviqmarcusto_, I'm not pushing - was just wondering if json-schema is still something we can plan for13:50
Saviqmarcusto_, hmm, avro has it's own schema does it?13:50
Saviqmhr3, hmm did I answer your question about exposing raw json for dev-tool? if not - I'd *rather* not in the long run, since we'll have a side-channel for talking to the scope backend directly, could we not get it through there?13:52
marcusto_Saviq: I've looked at this briefly, but it looks like standard schema. Its def on the todo list13:53
Saviqmarcusto_, it doesn't seem to mention any relation to json-schema.org13:54
Saviqmarcusto_, and looking briefly it does differ slightly (that's not to say it's worse, or bad, for that matter)13:54
Saviqmarcusto_, but it definitely seems less standardized13:55
Saviqif something that's non-standardized can be even less standardized...13:55
marcusto_Saviq: hmmm, ok, clearly I have not thought about this enough :p13:55
Saviqmarcusto_, ;)13:55
Saviqmarcusto_, no worries13:56
marcusto_Saviq: yeah, resources are few13:56
Saviqmarcusto_, we're not blocked on it, it's only a it-would-be-nice-to-have-when-possible type of situation ;)13:56
Saviqmarcusto_, but no worries, we have plenty to do regardless13:56
marcusto_Saviq: well we are planning on schema validation, just a matter of time13:57
Saviqmarcusto_, yup, great13:57
=== alan_g|lunch is now known as alan_g
mptSaviq, it’s somewhat on purpose, because there is much more screen real estate for variations on the PC.14:07
mptSaviq, but I haven’t thought through what happens when you dock a phone.14:07
mhr3Saviq, i'm not sure what side-channel you mean14:08
mhr3Saviq, ultimately it's just exposing it to qml, i don't see any harm in that14:08
mptSaviq, I talked about having a locale-independent 24-hour option on Tuesday with … with you, actually.14:08
Saviqmpt, yeah, that's why I'm asking - 'cause there isn't a setting for it :)14:08
Saviqmhr3, the one that will let us "upload" a modified JSON string for a category14:09
mhr3Saviq, that will be internal to the shell, it won't talk to the scope14:09
Saviqmhr3, otp, back in 0.5h or so14:09
mhr3Saviq, right, let's hangout afterwards14:09
Saviqmhr3, https://code.launchpad.net/~mhr3/unity-scopes-shell/json-defaults/+merge/198591/comments/461511 in the mean time14:09
mhr3thx14:10
mptSaviq, I think the overall setting would go in “Language & Text” (along with settings for decimal places, thousand separators, etc), but with a cross-reference from “Time & Date”.14:11
Saviqmpt, yup, sounds good14:11
Saviqmpt, if you didn't see me saying yesterday, MeeGo I think got it right, by allowing you to select:14:12
Saviqlanguage, format-locale, and 12 vs. 24hr clock14:12
Saviqformat-locale including number format, currency format etc.14:13
Saviqmpt, feels like a well-balanced solution14:13
larsudednick: no, I don't. When did that start?14:17
tsdgeosdandrader: can you read http://bazaar.launchpad.net/~aacid/unity8/verticalJournal/revision/596 and see if it gives you enough info on what it is supposed to do?14:19
dandradertsdgeos, sure14:20
tsdgeosdandrader: it gives it to me, but i know what it's supposed to do so i don't obviously count :D14:20
nic-doffaySaviq, I haven't gotten around to testing the branch yet.14:22
nic-doffayWhat was wrong with it?14:22
Saviqnic-doffay, it doesn't actually package the file14:23
dandradertsdgeos, "The number of rules is calculated[...]" <- number of *rules*?14:23
tsdgeoslol14:23
tsdgeosrows14:23
dandradertsdgeos, columns?14:23
tsdgeosthat14:24
* tsdgeos slaps himself14:24
dandraderhehehe14:24
tsdgeosdandrader: ok r597 fixes the typo and adds a bit more of info14:25
dandradertsdgeos, also, why is the documentation in the class instead of in the header?14:25
dandraders/class/cpp14:25
tsdgeosdandrader: because it''s what e had for listviewwithpageheader.cpp14:25
tsdgeosbut yeah it's weird14:25
tsdgeosalso lvwph.cpp is more about internal14:25
tsdgeosand the first part of this is more about external14:26
tsdgeosmaybe i can split this one into two?14:26
tsdgeosso "The implementation is centered around m_columnVisibleItems" and after is on the .cpp and the rest in the .h?14:26
dandraderyeah, in the .h I would say "what it is" and in the .cpp I would put the comments about "how it's done"14:27
dednicklarsu: no idea. I just added enable item by sensitivity and it stopped responding to clicks.14:28
tsdgeosdandrader: splitted14:29
tsdgeosSaviq: mzanetti: greyback: standup?14:31
Saviqtsdgeos, otp again14:32
greybacktsdgeos: same for me14:32
Cimimzanetti, notes14:32
larsudednick: is the service reporting disabled actions? Or is it something in qmenumodel?14:33
dednicklarsu: looks like it's in qmenumodel. the can_activate is false for the menu item actions.14:33
dednicklarsu: because action_target is null, but parameter_type is not.14:35
dednicklarsu: same old story14:35
dednick!14:35
larsudednick: ah right. Didn't we fix this back when we first found it?14:36
larsuapparently not :)14:36
dednicklarsu: i thought so :)14:36
dednicklarsu: been fixed in a few places14:36
larsudednick: I'll make the service put a target on there, that will be easiest14:37
larsubecause we want to keep the logic for "normal" menu items14:37
=== iahmad is now known as iahmad|afk
codebrainzhi. is there a way to have a program added to a blacklist or something to opt-out of the GTK+ menu stealing thing?14:43
codebrainzthe global menu or libdbusmenu or whatever it's called14:43
tsdgeosyes there is14:44
tsdgeosat least there's an envvar14:44
tsdgeosbut i don't remember it :D14:44
tsdgeos<--- not useful14:44
codebrainzyeah, that's what we tell all the bug reporters, but it'd be cool to opt out at the distro level14:45
=== dandrader is now known as dandrader|afk
tsdgeosSaviq: obviously all the runs wiht more logs are now passing :D14:45
codebrainzto avoid the continuous stream of bug reports about our main menu, which isn't really our main menu :)14:45
Saviqtsdgeos, hit it again!14:45
tsdgeosi am14:45
tsdgeosre-running only the qmluitests instead of the whole thing14:45
tsdgeosfor faster turnaround14:45
Saviqtsdgeos, yup14:46
codebrainzmaybe on ubuntu we could install a script along with the binary and have the script clear UBUNTU_MENUPROXY before calling the real binary?14:47
Saviqcodebrainz, is there a reason for this particular app to opt-out from globalmenu?14:47
codebrainzSaviq, tons of bug reports to our project not caused by our project14:48
Saviqcodebrainz, caused by the global menu integration?14:48
codebrainzyes14:49
Saviqcodebrainz, can you forward the bugs to the project that causes them, so that it's fixed instead of disabling it?14:49
codebrainzwe do, but quite frankly it's super annoying and not very cool to mess with an app that others have to support :)14:49
codebrainznone of our devs use unity, we only test/support normal GTK+14:50
tsdgeosSaviq: yeah the code is not really awesome, for example in kate if you do "Alt" it shows the menu but then if you press the accelerator (i.e. Alt+F) you get the F written in the text area instead of popping-up the file menu14:50
tsdgeosi fixed that already last year14:50
tsdgeosand somehow it broke again14:50
tsdgeosnot cool :D14:50
tsdgeosand last year is was less busy than now14:50
tsdgeosthat helped finding a time to fix it14:51
Saviqcodebrainz, dunno, don't see that as a reason to disable it, rather a reason to make it more obvious to people reporting bugs against the upstream project that they should report it against ubuntu instead14:52
codebrainzSaviq, that's highly optimistic14:53
codebrainzI wonder if we could put the main menu bar inside another GtkBox to break the heuristics used to steal the menu from the window?14:54
Saviqcodebrainz, who does the packaging for ubuntu, btw, and - what's the project, btw?14:54
tsdgeoscodebrainz: i guess the "easiest" for you is setting that envvar as the first thing in your main.cpp?14:54
tsdgeosif you really really really really want to disable it14:54
codebrainzI'm not sure setting the envvar from main() would do it, would it?14:54
codebrainzSaviq, the app is Geany14:54
codebrainzhyperair is the packager AFAIK14:55
tsdgeoscodebrainz: i'd say it would, but not sure tbh14:55
codebrainztsdgeos, it might, if set before gtk_init() actually14:56
Saviqcodebrainz, I dunno, if people use the apport to report bugs on ubuntu, they will get to the ubuntu project first14:57
codebrainzthey use our bug tracker14:57
codebrainzsome do anyway14:57
Saviqcodebrainz, I really don't feel like that's the right solution to the problem14:57
Saviqcodebrainz, feels short-sighted14:57
codebrainzSaviq, stealing the menu from an app with a hack, is *definitively* not the right solution :)14:58
Saviqcodebrainz, yeah, I know Ubuntu/Unity/Canonical is evil14:58
Saviqthat's established14:58
codebrainzno no, I rather like Ubuntu, I use Xubuntu myself14:58
codebrainzI just hate this hack because it does something evil14:58
Saviqcodebrainz, it's a design decision, and at that time it was the only way to apply it14:59
Saviqcodebrainz, with gtk2 it still is, afaik, and with gtk3 is better, iirc?14:59
codebrainzI can only hope it is14:59
codebrainzwe don't have full/proper gtk3 support yet14:59
Saviqcodebrainz, I think it's basically a first-class concept that the menu is exported14:59
Saviqcodebrainz, to support MacOS, for example15:00
mhr3Saviq, got time for the hangout?15:00
Saviqmhr3, right, forgot to ping you15:00
Saviqmhr3, sure15:00
mhr3let's do it then15:00
codebrainzSaviq, the gtk3 way seems OK, but the dbusmenu hack thing is just plain wrong IMO15:00
Saviqcodebrainz, again, only way to apply the design decision that was made15:01
Saviqcodebrainz, we'd have done it otherwise if was possible15:01
Saviqand we did, with gtk3, with qt etc.15:01
codebrainzSaviq, just because you did it the only way possible doesn't make it any cooler :)15:01
=== dandrader|afk is now known as dandrader
Saviqcodebrainz, sure it does, because *we did it* ;)15:02
Saviqcodebrainz, instead of saying it's impossible15:02
codebrainzSaviq, it was impossible to do it right, hence all the bug reports :)15:02
tsdgeosE_NOT_CONSTRUCTIVE_DISCUSSION15:02
seb128codebrainz, it's working pretty well, there is like 10 apps that are known to not be working correctly and they are mostly doing weird stuff15:03
codebrainzour app doesn't do "weird" stuff, but it does load stuff dynamically into the menus15:04
seb128codebrainz, what app are we talking about?15:05
codebrainzI wonder is there a way we could detect the UI changes taking place with a widget signal and pop up a one-time dialog warning the user what is happening and where to report bugs?15:05
codebrainzseb128, geany15:05
seb128codebrainz, ok, we have https://bugs.launchpad.net/ubuntu/+source/indicator-appmenu/+bug/1072109 about that it seems15:06
ubot5Ubuntu bug 1072109 in indicator-appmenu (Ubuntu) "appmenu shows wrong content" [Undecided,Confirmed]15:06
codebrainzseb128, there's more than that (especially on our tracker)15:10
larsudednick: that should fix it: https://code.launchpad.net/~larsu/indicator-messages/set-targets/+merge/19895715:11
dednicklarsu: cool. will give it a quick test15:11
seb128codebrainz, you should open a bug with the specific, and maybe talk to "attente" in #ubuntu-desktop to see if he can help you to get those fixed (if not there is a blacklist, he can add geany to it)15:11
codebrainzseb128, it's hard enough to support own bugs though, we can't be proxying all of appmenu's bugs because it proxies our menu :)15:13
codebrainzbut if the gtk3 way is less crazy, I could try and convince the team to start supporting that :)15:14
seb128codebrainz, well, the gtk3 way works with less crazyness/hacks yes15:16
codebrainzhmmm, maybe we could put a dialog message on our "Report a Bug" help menu item if ubuntu is detected, asking them to use ubuntu's builtin bug reporter?15:16
seb128codebrainz, with traditional menus we need to do hacks to export parse the menus/export them, where gtk3 gmenus does it by itself15:17
nic-doffaySaviq, do I specifically need to install the override in autopilot install or is adding the path sufficient?15:17
dednicklarsu: cheers, worked.15:17
larsudednick: cool15:18
codebrainzseb128, yeah I remember looking at the code last year when I submitted a patch to fix an issue on one of the bugs (which AFAIK still hasn't been applied)15:18
seb128codebrainz, do you have a link to the patch?15:18
codebrainzi can probably find it, one sec15:18
codebrainzhttps://bugs.launchpad.net/ubuntu/+source/libdbusmenu/+bug/90763515:19
ubot5Ubuntu bug 907635 in DBus Menu "lidbusmenu-GTK crash with Geany IDE using Python" [Undecided,Expired]15:19
codebrainz(the patch just fixes a harmless GTK-critical message, but it's still useful to avoid console spam, if it still applies)15:20
codebrainzwell, it's harmless as long as the program isn't run without assertions :)15:21
seb128codebrainz, oh, the bug has expired (and it's usually better to file merge proposal) ... in any case we don't use libdbusmenu anymore for the appmenu stuff (only in some specific cases and we are cleaning those out)15:22
codebrainzseems like users' bug reports showing the console output still shows this critical warning15:22
codebrainz(often for unrelated bugs)15:22
seb128they might still run the lts?15:23
codebrainzit's possible15:23
dednickSaviq: do we still need to deal with seeds for phone for unity8 dependencies, or is it sorted by the deps?15:23
seb128tedg, do you think you could review the null check patch on this bug ^15:23
seb128tedg, https://launchpadlibrarian.net/91512767/libdbusmenu-null-submenu.patch15:23
dednickSaviq: oh, i think we only needed seeds if there wasn't a dep right?15:23
tedgseb128, Sure, can you give me the bug number?  (can't seem to get back to there)15:24
tedgAh, I see.15:24
seb128tedg, 90763515:25
codebrainzfor rationale, see: https://developer.gnome.org/gtk3/stable/GtkMenuItem.html#gtk-menu-item-set-submenu15:25
Saviqdednick, yeah, debian/control is the way to go15:27
Saviqdednick, it's like: nothing else would depend on unity8, so ubuntu-phone seed depends on it15:28
=== alan_g is now known as alan_g|tea
Saviqdednick, but for anything else, standard deps are good15:28
didrocksSaviq: we still have 2 flacky unity8 tests I guess15:28
didrockshttp://ci.ubuntu.com/smokeng/trusty/touch/maguro/64:20131213.1:20131211.2/5443/unity8-autopilot/15:28
dednickSaviq: cool15:28
codebrainzseb128, here's the latest bug, not sure it'll get reported to the proper place: https://sourceforge.net/p/geany/bugs/1013/15:28
codebrainzit's parsing underscores in filenames as mnemonics15:29
Saviqnic-doffay, how is dpkg supposed to know where to install that file?15:30
nic-doffaySaviq, what's the purpose of this? usr/lib/python*/*/unity*15:30
nic-doffaySaviq, I don't know enough about how all this fits in the bigger picture to properly tackle this.15:31
nic-doffayIt's that simple really.15:31
seb128codebrainz, could you open a bug about that on https://launchpad.net/ubuntu/+source/unity-gtk-module/+filebug ?15:33
alesagehi I'm getting a ./build error on "Could not determine plugin installation dir.", can someone suggest a remedy? http://paste.ubuntu.com/6567242/15:33
Saviqalesage, ./build -s15:33
alesageSaviq, thx15:33
Saviqalesage, you don't have new libunity-api-dev installed15:33
Saviqalesage, or ./build -c, for that matter15:33
alesageSaviq, ok will investigate15:34
codebrainzseb128, hopefully the bug reporter will15:34
seb128codebrainz, you could also do it if you want to help your users... ;-)15:34
codebrainzI don't remember my launchpad login, that patch I submitted was likely the first/last time I used it :)15:35
codebrainzI can try though, and just link to the source forge bug15:35
codebrainz(I'm not affected as I use xfce panel)15:35
seb128codebrainz, I've pinged the maintainer in #ubuntu-desktop, I'm sure a bug would still be useful though15:36
codebrainzseb128, can you tell me what email address is associated with my account?15:36
codebrainzit won't show me unless I'm logged in15:37
codebrainz(I have like 50 email addresses)15:37
seb128codebrainz, @geany.org15:37
Saviqnic-doffay, http://www.debian.org/doc/manuals/maint-guide/dother.en.html#install15:37
codebrainzseb128, thanks15:37
seb128codebrainz, yw15:37
Saviqnic-doffay, so you need:15:38
Saviqdata/unity8.override usr/share/upstart/sessions15:38
Saviqor so15:38
nic-doffaySaviq, yeah that's what the manual seems to say, ta.15:38
codebrainzseb128, what project/component would that bug go against?15:40
codebrainzappmenu-gtk?15:40
seb128codebrainz, could you open a bug about that on https://launchpad.net/ubuntu/+source/unity-gtk-module/+filebug ?15:41
codebrainzheh, sorry, i missed that the first time :)15:41
codebrainzseb128, https://bugs.launchpad.net/ubuntu/+source/unity-gtk-module/+bug/126077715:57
ubot5Ubuntu bug 1260777 in unity-gtk-module (Ubuntu) "Underscores parsed as mnemonics in menu items" [Undecided,New]15:57
codebrainzheh15:57
codebrainzthanks bot15:57
seb128codebrainz, thanks15:57
codebrainznp, it actually sounds like an unsolvable issue15:58
Cimipeople don't like my branch? https://code.launchpad.net/~cimi/unity8/searchIndicator-swipe/+merge/19825816:03
seb128codebrainz, why?16:03
codebrainzseb128, how could the module know whether an item happened to be created by the constructor that parses mnemonics?16:05
codebrainzI don't think it can tell, and it can't assume all menu items were16:05
codebrainzunless there is a flag/property on the menu items or something, not sure16:05
seb128codebrainz, I don't know that code well, but I think those stuff were working with libdbusmenu, so it should be doable16:06
seb128codebrainz, let's trust the guys working on that code and see what they come with ;-)16:06
codebrainzah yeah, there is: https://developer.gnome.org/gtk3/stable/GtkMenuItem.html#GtkMenuItem--use-underline16:07
codebrainzso it's totally doable16:08
Saviqnic-doffay, I *think* you need absolute path in the .install file there16:10
Saviqor, didrocks, can you look at the .install file change in https://code.launchpad.net/~nicolas-doffay/unity8/upstart-kill-fix/+merge/19893116:10
Saviqnic-doffay, ah it's still missing data/unity8.override in the .install file16:11
Saviqdidrocks, unping16:11
nic-doffaySaviq, just pushed it now.16:11
nic-doffayI was wondering about an abs path though, but we'll see.16:11
Saviqnic-doffay, relative is correct16:12
Saviqnic-doffay, see debian/unity8.install16:12
dednickCimi: fixed review comments16:30
seb128codebrainz, I guess you commented on the wrong bug (https://bugs.launchpad.net/ubuntu/+source/libdbusmenu/+bug/1260779)?16:36
ubot5Ubuntu bug 1260779 in libdbusmenu (Ubuntu) "Test issues in trusty (g_source_remove invalid use warnings), fails to build" [High,Confirmed]16:36
codebrainzseb128, totally, I had too many bugs open16:40
codebrainzcan i delete it?16:40
seb128codebrainz, no, but don't worry about it (or just post a new comment saying "sorry, wrong bug")16:41
CimiSaviq, my first qt bug report <3 https://bugreports.qt-project.org/browse/QTBUG-3560816:45
SaviqCimi, ;)16:45
SaviqCimi, file another one that it's not easily customizable16:45
SaviqCimi, separatet16:45
Saviq-t16:45
=== alan_g|tea is now known as alan_g
CimiSaviq, ok16:46
Cimithe voice "create issue" is fun16:46
Cimishould be file bug imho :D16:46
Cimibut let's create an issue :D16:46
CimiSaviq, but how would you address the latter?16:48
CimiSaviq, why would you want to change *all* values?16:48
CimiSaviq, if the default value is sane you don't want to customise it16:48
CimiSaviq, I'm waiting your input16:52
Cimiwell filed the second https://bugreports.qt-project.org/browse/QTBUG-3560917:00
Cimialthough I'm not 100% convinced if the first gets fixed17:00
=== jhodapp is now known as jhodapp|bbiab
Cimidednick, hey dude17:28
Cimidednick, the comment i wrote17:28
Cimidednick, I have qt 5.217:29
CimiI believe it's thrown because of the new JS engine17:29
dednickCimi: hm. give me a sec17:31
Cimidednick, or let's do this next week :P17:31
Cimidednick,             tryCompare(function() { signalSpyDismiss.count > 0; }, true);17:31
Cimi:-\17:31
dednickCimi: no return maybe...17:32
* Cimi tries17:32
dednickbut succeeds for me, which is a bit odd17:32
Cimidednick, no luck for me17:33
Cimidednick, which qt?17:33
dednick5.0.217:33
Cimidednick, I'm on 5.217:33
Cimidednick, as said, they changed Js17:33
dednickyeah, but still... it's the same in other places17:33
Cimidunno then17:33
Cimilet me rebase17:34
dednickCimi: does 'make testSimpleTextMessageMenu' work?17:34
Cimidednick, dismiss fails here17:35
CimiFAIL!  : qmltestrunner::SimpleTextMessageMenu::test_dismiss() A value is required for tryCompare17:35
Cimi   Loc: [/home/cimi/Development/usc/visual-updates/tests/qmltests/Menus/tst_SimpleTextMessageMenu.qml(170)]17:35
Cimimzanetti, you should be eod - in case you aren't, any idea on fixing tryCompare with qt 5.2? ^17:36
dednickCimi: i think there may have been checks for 5.2 in unity8 which i dont have in usc17:36
Cimidednick, that's why I asked mzanetti17:36
mzanettihmmm...17:37
mzanettican you paste me the line that fails?17:37
mzanettiCimi: ^17:37
Cimimzanetti,  tryCompare(function() { signalSpyDismiss.count > 0; }, true);17:37
mzanettiCimi: I don't think this works with 5.0 either17:37
Cimior             tryCompare(function() { return signalSpyDismiss.count > 0; }, true);17:37
Cimimzanetti, they both fail17:38
mzanettiCimi: with 5.0 too?17:38
Cimimzanetti, not with 5.017:38
Cimimzanetti, dednick said it's working for him17:38
dednickCimi: eh...17:38
mzanettiCimi: yeah. what I'm saying is that this is wrong17:38
mzanettiand I don't think the test was actually working17:38
dednickCimi: it's supposed to be tryCompareFunction17:38
mzanettiyeah17:38
Cimidednick, so why was not failing for you?17:38
dednickno idea17:38
mzanettibut tryCompareFunction is something we have in UnityTestCase only17:39
mzanettiso you'd need to copy that (or upstream into the SDK ideally)17:39
dednickmzanetti: yeah, i already have it i think17:39
Cimidednick, with function works here17:39
Cimidednick, but next to fail is17:39
Cimicompare(signalSpyTriggered.count > 0, true, "should have been triggered");17:39
Cimiwhich is false here17:40
dednickCimi: yeah, looks like the tests were just bombing on mine, i get that as well.17:41
dednicknow17:41
Cimidednick, ok no problem, let me know when you fix them17:41
Cimidon't worry for today17:41
=== jhodapp|bbiab is now known as jhodapp
sil2100charles: hi! I saw that you put up a branch for fixing the 24/etc in indicator-datetime - it seems to be still failing on CI17:47
sil2100charles: can you take a look what's up?17:47
charlessil2100, yes I'm looking at it right now17:47
charlessil2100: looks like the unit tests are failing because it doesn't have locales configured s.t. we can test in a 12h and 24h locale17:47
charlessil2100: I'm currently adding hooks to debian/ for this17:47
sil2100charles: ah, makes sense - thanks for taking care of this!17:48
dednickCimi: fixed and pushed17:51
xnoxSaviq: if you are around, can you please do a quick review of https://code.launchpad.net/~xnox/unity-action-api/fix-cross/+merge/198847 ?18:11
* greyback eow18:33
mhr3_same18:37
* mhr3_ out18:37
Saviqxnox, hey, so no need to tweak the Qt .cmake modules in the end?19:58
kdubSaviq, would know where I could find QDBusConnectionPrivate19:59
Saviqkdub, ESYNTAXERROR20:00
kdubokay  :)20:00
Saviqkdub, lookin'20:00
xnoxSaviq: what do you mean? =)20:01
xnoxSaviq: cmake modules are all correct and installed into multiarch paths.20:01
Saviqxnox, but I meant to actually find moc and such20:01
xnoxSaviq: automoc works correctly out of the box, i fixed that in cmake.20:01
Saviqxnox, ah, awesome20:01
xnoxSaviq: same with rcc & qmake, but custom utils are need to be handled by hand.20:02
xnoxSaviq: i think the cmake modules should provide standard FOO_EXECUTABLE for all qt tools, e.g. qdoc, dbuscpp, etc.20:02
Saviqkdub, https://qt.gitorious.org/qt/qt/source/33a34960328cce7a6994d2ea771c82da7bfdb598:src/dbus/qdbusconnection.cpp#L6720:03
kdubthanks Saviq!20:04
Saviqxnox, you mean that's the current state after your recent cmake upload or that's the desired state?20:04
xnoxSaviq: rcc/qmake/moc - work out of the box; other tools to be defined by default is a desirable.20:11
sil2100bregma: hello! I noticed a FTBFS for lp:unity trunk in daily-build - did you guys see that?20:11
xnox(but low priority, upstream should start exposing those)20:11
Saviqxnox, ok cool, q: you said "dpkg-architecture -aarmhf -c cmake .." should cross-build locally, doesn't seem to work here?20:12
Saviqxnox, i.e. I still get a build for amd6420:12
xnoxSaviq: well, that works out of the box, if there are cross-dependencies and cross-compilers installed locally.20:13
xnoxand all the libfoo-dev:armhf packages20:13
Saviqxnox, ok, /me goes apt-get build-dep -aarmhf unity820:13
xnoxin practice not many libfoo-dev are multiarch:same, thus one would need to remove & install to change arches.20:13
xnoxSaviq: and that implies to did dpkg --add-foreign-arch armhf & added arch qualified sources in /etc/apt/sources.list20:14
xnoxand ports archive20:14
Saviqxnox, aah right, haven't done that yet apparently20:14
xnoxyeah, that's what mk-sbuild --target armhf does in a chroot =)20:14
xnoxso dpkg-architecture -aarmhf cmake ../20:14
xnoxis intended to be run inside a suitable chroot =)20:14
xnoxor e.g. under click chroot.20:15
Saviqxnox, yeah, what I'm thinking about now is to transition ./run_on_device to cross-build in a chroot and rsync the results to the device and run from there20:15
xnox(well click chroots export vars set by dpkg-architecture -aarmhf already, so actually, it's just cmake ../ under those)20:15
Saviqxnox, instead of building on the device20:15
xnoxSaviq: well sbuild --host armhf, should cross-build local dir and spit out ../*_armhf.deb20:16
xnox(without specifying path to .dsc, it should work with building unpacked debian package tree, which unity8 folder is)20:16
Saviqxnox, yeah, I know, but that's still too long for quickly iterating during development20:17
Saviqxnox, and requires a writable device20:19
Saviqxnox, which we shouldn't need for unity8-only dev20:19
xnox.... dude, that cross-compiles.20:20
xnoxbzr branch lp:unity8; cd unity8; sbuild -A -d trusty --host armhf; adb push ../*_armhf.deb /userdata/20:20
xnoxoh, right20:21
xnoxSaviq: but you can $ dpkg-deb -R *.deb unpack/20:21
xnoxto unpack them20:21
xnoxwithout installing in a home dir.20:21
Saviqxnox, sure, that I can do20:21
xnoxand exectue them, or like make an override in the upstart job to run a custom one20:21
Saviqxnox, but still builds the whole thing every time20:21
Saviqxnox, usually with installing deps first20:22
Saviqxnox, unless you have a pre-made specific sbuild chroot20:22
Saviqxnox, it just does more than we generally need for quick iterations20:22
xnoxe.g. $ echo "exec ~/bin/unity8" > ~/.config/upstart/unity8.override20:22
Saviqxnox, no need20:22
xnoxSaviq: well full clean build here is 2m20s20:22
Saviqxnox, yeah exactly20:22
Saviqxnox, too long20:22
Saviqxnox, we're talking about during-development-deployment20:23
xnoxSaviq: you can optimise, so for things that I do often i have $ mk-sbuild --target armhf --name unity820:23
Saviqxnox, yeah, I know20:23
xnoxpre-installing deps into that, I think you can bring the time lower.20:23
xnoxso i'm not sure what you are asking about?20:23
xnoxyou want dirty builds?20:23
Saviqxnox, I'm not *asking* about anything, really :D20:23
Saviqxnox, yes20:23
xnoxwhile qt-dev is not co-installable?20:24
Saviqxnox, no, permanently20:24
xnoxwell, I could point cmake at /var/lib/schroot/unity8-amd64-armhf/* to link/compile against20:24
Saviqxnox, I just don't want the overhead of building packages when all we're after is as-quick-as-possible possibly-dirty quick deployments onto device20:25
xnox(such that we can re-use the same chroot, for dirty builds, without chrooting into it)20:25
xnox(well pre-installed packages)20:25
Saviqxnox, yeah, we might use the sbuild chroot indeeed20:25
Saviq-e20:25
Saviqxnox, even chrooting into it20:25
Saviqxnox, but bind-mouting20:26
xnoxto be honest this calls out to simply prioritise making all -dev packages multiarch-same which unity8 depends on.20:26
xnoxi never use schroots like them.20:26
xnoxlet me see how it can be used in bind-mounting mode.20:26
Saviqxnox, no worries, unless you want to :)20:27
Saviqxnox, I'm all-happy with what you enabled for us already20:27
xnoxhm i ponder how save it is to bind-mount /home/ into schroot.20:28
Saviqxnox, I do it all the time20:29
xnoxso if the named chroot has all the right dependencies one should be able to do: schroot -c unity8-amd64-armhf "cmake .; make"20:31
Saviqdidrocks, seems indeed the retry helped on maguro, awesome20:32
Saviq"Failed to unlock greeter, trying again..." yay20:32
didrocksSaviq: yeah… but still flaky then :p20:38
Saviqdidrocks, well, not *really*20:38
Saviqdidrocks, it's just that we don't know when the device is not busy on startup any more20:38
Saviqdidrocks, to deliver the input in a timely manner20:38
Saviqdidrocks, aactually20:39
Saviqdidrocks, we should move to pressing the BFB20:39
didrocksSaviq: actually, we start the test 5 minutes after the boot20:39
Saviqdidrocks, instead of swiping the greeter20:39
didrocksso the device isn't busy20:39
Saviqdidrocks, but unity8 is starting20:39
Saviqdidrocks, being restarted with testability20:39
didrocksah yeah20:39
didrocksyep on that one20:39
Saviqdidrocks, and loading dash and everything20:39
didrocksneed to either send an signal20:39
didrocksto be more reliable20:39
didrocksor use qmltestrunner (shhh) ;)20:40
Saviqdidrocks, we don't want to open a "ok, signal me and I'll unlock" security hole ;)20:40
Saviqdidrocks, qmltestrunner might not really help in that case20:40
Saviqdidrocks, what would help would be having control of the time20:40
Saviqdidrocks, so with each input movement you'd tell QML how much time passed20:41
Saviqdidrocks, so it's never jerky in that sense, which it is now, 'cause input just doesn't get timely to the gesture recognizer, which rejects it if it's not smooth enough20:41
Saviqdidrocks, but yeah, maybe we should move to the BFB - but still would need to swipe the launcher in, so not *really* more reliable20:42
didrocksSaviq: totally make sense, should be more reliable20:45
didrocksor20:46
didrockshum20:46
didrockswe are not really testing unlocking the greeter20:46
Saviqdidrocks, yeah, that's tested regardless20:46
didrocksshould be one test20:46
didrocksmaybe for the other20:46
didrockswe should ship a flag/a way only with this AP test20:46
didrocksto get the greeter always unlocked?20:46
Saviqdidrocks, what you called out as flaky in your email... must not have included the latest release of unity8: http://ci.ubuntu.com/smokeng/trusty/touch/maguro/64:20131213.1:20131211.2/5443/unity8-autopilot/570019/20:49
Saviqdidrocks, there would have to be a "greeter unlocked, continuing" message, or two "failed to unlock greeter"20:50
didrocksSaviq: well, the result if flaky20:50
didrockslike, I rerun the test, it pass20:50
Saviqdidrocks, even with the latest unity8?20:50
didrocksyeah, this is with latest20:50
Saviqdidrocks, the log suggests it's not :/20:51
didrocksSaviq: http://people.canonical.com/~ogra/touch-image-stats/20131213.1.changes20:51
didrocksimage 6420:51
Saviqdidrocks, I need camera on the devices then :/, I'm done guessing20:52
didrocksSaviq: I think the CI team has one20:52
Saviqdidrocks, yeah, sorry for bothering you20:52
Saviqdidrocks, go have your weekend20:52
didrockswell, trying to promote the image first :)20:52
Saviqdidrocks, ah, those tests don't use the helpers20:54
didrocksahah!20:54
didrocks:)20:54
Saviqdidrocks, where the retrying is done20:54
Saviqdidrocks, will have to fix that20:54
didrockssee, everything has an explanation!20:54
didrocks:)20:54
didrocksSaviq: great ;)20:54
=== salem_ is now known as _salem

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