=== duflu_ is now known as duflu [07:26] Saviq: 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:39] didrocks, the bug won't be fixed [07:39] grrr, I wonder how robru confirmed the fix then… [07:39] * didrocks *sigh* [07:40] didrocks, well, it's a race [07:40] didrocks, so he might not have encountered it [07:41] ok [07:41] Saviq: I'm building qmenumodel right now [07:41] will deliver and quick an image with it [07:41] didrocks, cool, thanks [07:42] didrocks, 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 test [07:42] bug 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/1256061 [07:42] Saviq: argh, ok. I'll have to explain that to management then :/ [07:43] on the "why did we release unity8 without the fix" [07:43] Saviq: I'll try to put it nicely ;) [07:44] didrocks, well, that bug wasn't blocking [07:44] Saviq: for the unity8 release yeah, it was [07:44] didrocks, ah right, damn [07:44] Saviq: do you think this is going to land soon? [07:44] but was attributed to qtubuntu [07:44] if so, we can bundle it to the same image :) [07:44] didrocks, well, today [07:45] Saviq: if it's in the incoming hour, we can [07:45] didrocks, but some hours ahead for sure [07:45] but ok, so will be too late [07:45] nevermind, keep getting things landing :) [08:40] Saviq, for getting https://bugs.launchpad.net/unity-api/+bug/1258057 [08:40] Ubuntu bug 1258057 in Unity API "unity-api fails to build against Qt 5.2" [Critical,In progress] [08:41] Saviq, sorry, I meant this http://paste.ubuntu.com/6557929/ [08:41] Saviq, did you just "bt" or did you also add some arguments [08:41] for printing out variables on each frame [08:42] dandrader, that was `bt full` [08:42] ah [08:42] didn't know that one, always used just "bt" [09:11] Saviq, is that a normal output for testLauncher from unity-api? http://pastebin.com/d4EKTZ9k (btw, Ubuntu's pastebin refused this text) [09:12] I wonder if I'm missing something in my setup... [09:12] dandrader, looks that way [09:13] the test didn't fail though... [09:13] dandrader, yeah, that's just warnings [09:13] dandrader, should be cleaned up anyway, true [09:14] Saviq, so you don't get those warnings? [09:14] dandrader, yes I do [09:15] ah, ok. then my setup is fine [09:15] dandrader, sorry, the "looks that way" was to your previous message [09:15] Saviq, I didn't get any crash in the tests. but testApplication failed, but without crashing [09:16] dandrader, under 5.2 on i386? [09:16] Saviq, that's with the "stable" qt5 branch on a i386 chroot [09:16] dandrader, ah, stable is 5.1 [09:16] erm [09:16] 5.2.1 [09:16] dandrader, maybe it was fixed between 5.2 and 5.2.1 [09:16] so that's good news I suppose [09:17] dandrader, yeah, would be awesome to bisect and get the commit into our packages [09:17] will check why testApplication is failing... [09:30] mzanetti, ping :) [09:30] hi Cimi :) [09:31] mzanetti, did you go any further with the exploration? is there anything you suggest ding? [09:31] *doing [09:32] Cimi: did you find out if its related to that hack? [09:32] nope, didn't try actually [09:33] Cimi: 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 issue [09:33] as that's the only thing I can think of what's different in apps and unity8 [09:34] maybe dednick can explain what exactly is this hack for [09:34] I didn't fully understand the bug report tbh [09:49] https://code.launchpad.net/~aacid/unity8/verticalJournal/+merge/198897 is up if anyone wants to do some view reviewing [09:49] Saviq: i'll do hJournal next and try to share as much code as possible with the vJournal [09:50] mzanetti, it does not fix the issue [09:53] Cimi: strange [09:54] tsdgeos, yup, great [09:54] Cimi: 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 unity [09:55] mzanetti, I think it might be for the loaders? [09:55] dandrader: totally right [09:55] dandrader: and something like what listviewwithpageheader.cpp has on the top [09:55] explaining a bit the code won't hurt either [09:56] Cimi: Don't know... but yeah. the fact that the width is set multiple times suggests something like this [09:58] Saviq: 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] Saviq: tsdgeos: changing when dbus code is generated, or stored. is not my proposal at all. [09:59] xnox: as said it looks good to me, but i'll let Saviq be the one to decide [09:59] tsdgeos: cause i have tasty cmake upload which will cross-compile unity8 .... if above is merged. [10:00] tsdgeos: ah, looks like Saviq agrees to keep regenerating dbus code at build time. [10:01] xnox, tsdgeos, yeah [10:01] didrocks, good news [10:02] didrocks, https://bugs.launchpad.net/ubuntu/+source/qtubuntu/+bug/1256061 [10:02] Ubuntu 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] didrocks, that bug is actually invalid... [10:03] Saviq: oh, it's magically fixed? [10:03] I'll flash latest [10:03] and switch to French [10:04] tsdgeos: hm, jenkins gave "Needs Fixing" review, is there actually something I need to fix? [10:04] nah [10:05] it's some crap we need to fix [10:05] xnox, some flakiness still [10:05] really really [10:05] tsdgeos: ok, thanks =) [10:07] tsdgeos, something already using the vertical journal? [10:07] mhr3_: the test :D [10:08] tsdgeos, maybe you want to hook it up with lp:~unity-team/unity8/new-scopes? ;) [10:08] mhr3_: i just hooked it up where Saviq told me to [10:09] tsdgeos, *hook* as in actually make use of it [10:09] tsdgeos, mhr3_, let's get it merged into unity8 trunk [10:09] tsdgeos, mhr3_ then merge trunk into new scopes, and make use of it [10:09] sure, i'm thinking step 2 already :) [10:10] tsdgeos, mhr3_ oh yeah, we can go with new-scopes + vjournal in a branch already [10:10] mhr3_, only we don't have variable height cards yet ;) [10:10] :( [10:10] mhr3_, or a scope that gives us data that could be used for that, for that matter :P [10:10] Saviq, details :P [10:11] mhr3_, I *know* [10:12] Saviq, btw json-defaults was looking for you, and there's one more branch built on top of it [10:12] fwiw those are really simple branches :) === dandrader is now known as dandrader|afk === anpok|afk is now known as anpok [10:26] mhr3_, yeah, on my list for today [10:34] didrocks, news? === dandrader|afk is now known as dandrader [10:36] Saviq: flashing right now, under a tons of things :p [10:40] didrocks, k [10:47] Saviq: not fixed here [10:47] Saviq: I move to French [10:47] didrocks, rebooted? [10:47] set to 19h in the system settings ui [10:47] (after a reboot) [10:47] and it's written 7:45 [10:47] for instance [10:47] didrocks, but no AM/PM? [10:47] no AM/PM [10:47] just 7:45 [10:47] didrocks, let me check [10:48] didrocks, ok I think I know what's happening then [10:49] didrocks, the default setting is 12hr [10:50] Saviq: this default setting is in unity8? [10:50] didrocks, no, it's for the datetime indicator [10:50] didrocks, that string comes directly from the indicator now [10:50] didrocks, preformatted [10:50] gsettings get com.canonical.indicator.datetime custom-time-format [10:51] didrocks, ping [10:51] Saviq: can you ensure the indicators guys are in the loop then? [10:51] nic-doffay: pong [10:51] didrocks, going to look into this: https://bugs.launchpad.net/unity8/+bug/1260379 [10:51] Ubuntu bug 1260379 in Unity 8 "A unity8.override file should be shipped to allow apport completion during test runs" [High,Triaged] [10:51] no need to ping me when I'm talking on that channel, just ask :p [10:51] where should I start. [10:51] nic-doffay: ah great! [10:51] larsu, around? [10:52] mzanetti, found [10:52] mzanetti, line 141 of MenuContent.qml [10:52] Cimi: \o/ [10:52] that binding does not work [10:52] nic-doffay: I would say, look at the unity8 file, ship an upstart override file with the extra shutdown period [10:53] mzanetti, if you put title: "Example" at line 103 and you remove that binding, the indicator icon is rightly placed [10:53] interesting [10:53] mzanetti, obviously the reason why this is causing the style to break is obscure [10:53] to me [10:54] mzanetti, because you get the title (the text) [10:54] Cimi: 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 tab [10:54] but breaks other bindings [10:54] Saviq, you were trying to build libunity-scopes on saucy and it failed? [10:54] mzanetti, good spot [10:54] mhr3_, yes [10:54] Saviq: yep [10:54] Saviq, do we need to fix that? [10:54] mhr3_, exceptions in tests === mhr3_ is now known as mhr3 [10:55] larsu, are you maintaining indicator-datetime? [10:55] Cimi: ok.. but in this case the proper fix is indeed inside the style (I'd say) [10:55] Saviq, fwiw i'm running it fine on saucy [10:55] but i'm building the pkg locally... [10:55] Cimi: a tab should not break if you change title at runtime [10:56] mhr3, https://launchpad.net/~phablet-team/+archive/desktop-deps/ [10:56] Saviq: no, charles is [10:56] larsu, I've tracked down bug #1256061 to be, in fact, an indicator-datetime issue [10:56] bug 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/1256061 [10:56] Saviq: see, told ya :) (re: OSK swipe down from everywhere) [10:56] mzanetti, ;P [10:57] larsu, ok, will assign to him then [10:57] Saviq: yep, looks like it. We should fix this for real and send a time stamp... [10:57] larsu, yeah, another bug [10:57] larsu, I actually already have a branch that "fixed" it by ignoring what the indicator sent up completely [10:58] larsu, but wasn't real happy with it, and it didn't take all the other settings into account etc. [10:58] Saviq: right, sending a time stamp doesn't even make sense for the current time [10:58] larsu, exactly [10:58] what settings are those? [10:58] whether to show the year, 24h, etc? [10:59] larsu, https://wiki.ubuntu.com/TimeAndDate?action=AttachFile&do=view&target=settings-clock.png [10:59] right [11:00] Cimi: ping [11:00] dednick, pong [11:00] Cimi: hey, can i get you to do a review for me? https://code.launchpad.net/~nick-dedekind/ubuntu-settings-components/visual-updates/+merge/193301 [11:00] when you have some time. [11:00] didrocks, what am I looking for in this crash file? [11:00] dednick, sure, now [11:01] Cimi: thanks [11:01] Mirv, 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:03] dandrader, 5.2 for sure, if 5.2.1 gets there in time - hopefully [11:03] nic-doffay: what you want is to have a stop 30 in this override to let the coredump to be collected before upstart is killing it [11:04] didrocks, "kill timeout 30", no? [11:04] yep [11:04] nic-doffay, just have a unity8.override file with that line ↑↑ [11:04] nic-doffay, put it in data/ [11:04] nic-doffay, and add a line to debian/unity8-autopilot.install [11:05] it's interesting that there's already an enormous diff between qtdeclarative release (5.2) and stable (5.2.1) [11:05] nic-doffay, to install that .override file into /usr/share/upstart/sessions [11:05] dandrader, yeah indeed [11:05] didrocks, unless there's a better place to put it in ↑↑? [11:06] Saviq: no, thats perfect [11:09] Saviq, right will get going with that then. [11:10] does 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:11] like it creates a new folder on-right-click for me, instead of opening the menu :/ [11:11] it does that for a minute at a time and then goes back to normal [11:11] mhr3, with https://code.launchpad.net/~michihenning/unity-scopes-api/what-for-exceptions/+merge/198872 there will be some actual useful output in the build log [11:12] Saviq, yea, just got a mail from michi about that branch [11:12] Saviq, working fine on my desktop [11:12] dandrader, yeah, I'm not sure if it's my mouse, or the touchpad playing games... === iahmad is now known as iahmad|afk === dandrader is now known as dandrader|lunch [11:43] dandrader|lunch: as saviq said, 5.2.x is the target for sure. the focus is on 5.2 + backported patches for now. === dandrader|lunch is now known as dandrader === _salem is now known as salem_ === iahmad|afk is now known as iahmad [11:55] Saviq, how should I test for this now? [12:00] guys, we should fix the scrolling... [12:00] of flickable [12:01] velocity and deceleration [12:01] it's broken on the desktop and on the devices [12:09] Cimi, filed a bug yet? [12:10] phew, so much to do... (with Qt 5.2) [12:10] Mirv, between rc1 and release? [12:11] nic-doffay, with unity running, go: [12:11] kill -11 `pidof unity8`; stop unity8 [12:12] nic-doffay, stop unity8 should return after more than 5s [12:12] Saviq: 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:13] Mirv, right, so basically the real packaging [12:13] Saviq: so I thought today I would make progress, instead I find myself being back to near beginning with 5.2 final build :) [12:14] Saviq, I need to test again on my galaxy nexus, but it's definitely broken in the desktop [12:14] but of course progress made reducing delta, going through changes etc [12:14] Saviq, try flicking on the sdk components gallery [12:14] Saviq, it's like crazy :) [12:15] actually now works [12:15] :-\ [12:16] Cimi, yeah, seems fine in the pickers [12:16] Saviq, it's fine now for me [12:16] Saviq, was badly broken yesterday [12:16] Saviq, like over scrolling with no limits [12:18] Saviq, can I test qt 5.2 on maguro? [12:18] Cimi, you could try [12:18] Cimi, ppa:canonical-qt5-edgers/qt5-beta2 [12:19] Cimi, but it's probably better to wait a bit [12:19] Saviq, I wanted to try the scrolling [12:19] Cimi, it's all in flux now that 5.2 was released finally [12:19] Cimi, I don't think anything changed tehre [12:19] Saviq, was broken in all our previous versions [12:20] Saviq, try seeing the default velocity of the flickable [12:20] Cimi, and it all basically deserves a QTBUG that the default values are a) pixel-dependent and b) not customizable without rebuilding Qt [12:20] Saviq, it was 1500 iirc, should be much higher with high dpi [12:20] I have a weird issue right now... I cannot drag (flick) any scope upwards unless I pull it down a little beforehand [12:21] is that known? [12:21] Cimi, https://qt.gitorious.org/qt/qtdeclarative/source/e895bc315f9959a5dad80a8f0bf7b73bced0bde1:src/quick/items/qquickflickable.cpp#L61 [12:21] mzanetti, no [12:21] mzanetti, phone? [12:22] yeah [12:23] nic-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] nic-doffay, if it is - that's all we can do TBH [12:23] Saviq, killing it from a separate console has strange results. [12:24] Unity8 jus segfaults. [12:25] nic-doffay, that's what -11 does [12:25] nic-doffay, that's SIGSEGV [12:26] nic-doffay, but anyway - more real-life test [12:26] hmm crap [12:26] Saviq, getting this too: stop: Unknown job: unity8 [12:26] nic-doffay, as phablet [12:27] Saviq, so run on device then run as phablet user. [12:27] nic-doffay, man, run on device won't do you any good [12:28] nic-doffay, that file will only get installed where it needs to when packaged [12:28] nic-doffay, run_on_device doesn't install anything [12:28] Saviq, what's the deal with stop: Unknown job: unity8 [12:28] nic-doffay, you're not logged in as phablet, most likely [12:29] nic-doffay, or you managed to uninstall unity8 [12:29] Saviq, the former then. [12:29] Saviq, that's bad [12:30] Cimi, you've been complaining about this for what, half a year now? [12:30] Cimi, did you file a bug? [12:30] nope [12:31] I'm a lazy guy (my main flaw) and not registered yet to qt bugs [12:31] I'm going to to this today and I will be digia's worst nightmare [12:31] perfectly in time for Friday the 13th [12:33] nic-doffay, just file the merge proposal [12:33] Saviq: ok i think i know why that test_show_scope_on_load is failing [12:33] nic-doffay, we'll know soon enough whether it's correct [12:33] tsdgeos, pray tell [12:33] Saviq: now i want to know if i change the test of the code :D [12:34] Saviq, yeah was trying to avoid that though! [12:34] let me to a pastebin explaining [12:34] nic-doffay, it's this kind of change [12:34] nic-doffay, to test for real you'd have to make unity8 crash on exit [12:35] nic-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 away [12:35] nic-doffay, then, without your .override file, upstart will kill unity8 before apport's managed to collect the full crash report [12:35] nic-doffay, with it, upstart will wait for up to 30s (as opposed to 5s) before killing [12:36] nic-doffay, giving time for apport to collect the crash file [12:36] Saviq, I'll give that a go while waiting for jenkins. [12:37] nic-doffay, you can do 'stop unity8; start unity8 BINARY=$PWD/builddir/unity8' to launch the locally built unity8 under upstart [12:37] Saviq, cool thanks [12:37] nic-doffay, having built it first with run_on_device, and logging in as phablet, unity8 is built in /home/phablet/shell/builddir/unity8 [12:38] nic-doffay, and BINARY=... just tells upstart to launch that binary instead of the system-installed one [12:38] damn it's hard to write the thoughts down [12:41] Saviq: actually no, scrap that, can't tell :-/ [12:41] but i'm going to add a hell lot of console.log and will find out [12:41] tsdgeos, maan :/ [12:41] i'm tired of all those errors [12:41] tsdgeos, please do [12:41] tsdgeos, +1 [12:57] Saviq: did we decide we would pass https://code.launchpad.net/~nick-dedekind/unity8/StrFTimeFormatter/+merge/192343 ? [12:57] dednick, I'm still not really sure what it's supposed to fix ;D [12:59] dednick, 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 so [12:59] dednick, so unless that branch actually fixes something that I'm not aware of, I'd reject it [12:59] Saviq: right, but thats a todo later right... [12:59] Saviq: it's for the datetime alarm menu items. [12:59] dednick, ah, ok [13:00] dednick, can we see them already? [13:00] dednick, like if I add an event in my calendar, will they show up? [13:00] "you can see them", but there are no times. [13:00] dednick, which that branch is supposed to fix, correct? [13:01] Saviq, ok with exposing the raw json in the category models purely for the dev-tool usage? [13:02] Saviq: 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). === alan_g is now known as alan_g|lunch [13:02] dednick, so still no way to see the fix, eh? ;) [13:03] nope. but this branch also moves the formatter into Utils :) [13:13] Saviq: oh, and my indicator tests branch is done. :) https://code.launchpad.net/~nick-dedekind/unity8/indicator-page-factory-tests/+merge/198785 [13:13] dednick, yup, saw that [13:21] dednick, and ohkay, if you twist my arm... can you just add a TODO/FIXME somewhere where it makes sense, and refer to bug #1260728 [13:21] bug 1260728 in Unity 8 "Indicators should send timestamps, not pre-formatted strings" [Medium,Triaged] https://launchpad.net/bugs/1260728 [13:21] Saviq: :) ok [13:21] xnox, so does that mean that with the new cmake upload and the unity8 branch, we'll be able to cross-build directly? :) [13:23] Saviq: 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*.dsc [13:23] xnox, YUMMY [13:23] xnox, do you know if distcc should work fine when cross-building? ;) [13:24] Saviq: once qt*-dev stack is co-installable it will be possible to even do it without a chroot with: dpkg-architecture -aarmhf -c cmake ../ && make [13:24] xnox, I know ccache does work fine [13:24] Saviq: why do I need distcc, or ccache if it takes 2m20s to cross-compile unity8 clean from start to finish? [13:25] xnox, nice one, we'll have to transition our run_on_device script to that :) [13:25] xnox, right, it's much simpler recently ;) [13:25] xnox, 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] Saviq: .... cross-building happens on amd64/i386 which is fast and multi-core (typically) so simply DEB_BUILD_OPTIONS=parallel=12 [13:25] xnox, yeah -j9 is my default [13:25] Saviq: done. i just put it in the Timeformatter header. [13:26] dednick, cheers [13:26] xnox, and well, it's not just about unity8, I actually compile other things as well ;) [13:26] Saviq: you do want to set it via DEB_BUILD_OPTIONS or even specifically in ~/.sbuildrc $build_environment {'DEB_BUILD_OPTIONS' => 'parallel=9'} [13:27] xnox, I barely do anything else than bzr bd, which does have sbuild -j9 as the builder [13:27] xnox, when building packages, that is [13:27] Saviq: right. so the only requirement is that distcc / ccache generates / replaces $DEB_HOST_GNU_TYPE-[gcc|g++] [13:27] Saviq: cause that's the compiler that cmake will pick when cross-compiling. [13:27] xnox, yeah, I thought it *should* work in theory [13:28] xnox, will do some testing [13:28] * xnox has no need though =) [13:28] I have 8 core, 32GB of RAM, and I compile in tmpfs with no iowait ;-) [13:32] * 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 something [13:35] xnox, well, I'm 4 core HT, 8GB RAM and compile in shm, too (and SSD to boot) [13:35] xnox, 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] xnox: man that is an awesome phone :D [13:35] Saviq, larsu: so the timestamp issue is back on the indicator side? [13:36] seb128, yeah [13:36] davmor2: =))))) not very portable, and battary UPS only lasts 10m [13:36] seb128, it's a string straight from indicator-datetime [13:36] seb128, I did fix it unity8-side, by ignoring what that string says at all, but didn't like the solution for now [13:36] seb128, bug #1260728 to fix it long-term [13:36] bug 1260728 in Unity 8 "Indicators should send timestamps, not pre-formatted strings" [Medium,Triaged] https://launchpad.net/bugs/1260728 [13:36] Saviq, ok, thanks [13:37] Saviq: right. I made my machine ssh accessible and my terminal on my ultrabook has gnome-terminal with the following shell set $ ssh -X mydesktop.IP [13:37] Saviq, that's different from the messaging menu one, right? [13:37] seb128, yeah, that's fixed in u8 trunk [13:37] cool [13:37] seb128, and released already, btw [13:37] Saviq, good to read ;-) [13:38] * seb128 didn't keep up with what was going on this week, just too much stuff changing ;-) [13:38] seb128, yeah, but it's a good problem to be had :) [13:38] indeed [13:39] larsu: howdy. Do you know why are all the messaging menu items "not sensitive" [13:40] mpt, 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:41] mpt, or the other way - you can change it on the desktop, but that doesn't affect your phone clock? [13:41] mpt, 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] globally, that is [13:42] mpt, 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:43] mhr3, for https://code.launchpad.net/~mhr3/unity-scopes-shell/json-defaults/+merge/198591 [13:44] mhr3, 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 scope [13:45] mhr3, your way involves adding lotsa code just to add a new key with a default for any component/template [13:47] marcusto_, hey, did you have a chance to look at http://json-schema.org/ ? [13:48] mhr3, oh, and on that note... should those defaults not be applied server-side? [13:49] 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] marcusto_, oh, awesome [13:50] Saviq: problem is, until I get something running I'm holding up the web guys on SSS [13:50] marcusto_, I'm not pushing - was just wondering if json-schema is still something we can plan for [13:50] marcusto_, hmm, avro has it's own schema does it? [13:52] mhr3, 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:53] Saviq: I've looked at this briefly, but it looks like standard schema. Its def on the todo list [13:54] marcusto_, it doesn't seem to mention any relation to json-schema.org [13:54] marcusto_, and looking briefly it does differ slightly (that's not to say it's worse, or bad, for that matter) [13:55] marcusto_, but it definitely seems less standardized [13:55] if something that's non-standardized can be even less standardized... [13:55] Saviq: hmmm, ok, clearly I have not thought about this enough :p [13:55] marcusto_, ;) [13:56] marcusto_, no worries [13:56] Saviq: yeah, resources are few [13:56] marcusto_, we're not blocked on it, it's only a it-would-be-nice-to-have-when-possible type of situation ;) [13:56] marcusto_, but no worries, we have plenty to do regardless [13:57] Saviq: well we are planning on schema validation, just a matter of time [13:57] marcusto_, yup, great === alan_g|lunch is now known as alan_g [14:07] Saviq, it’s somewhat on purpose, because there is much more screen real estate for variations on the PC. [14:07] Saviq, but I haven’t thought through what happens when you dock a phone. [14:08] Saviq, i'm not sure what side-channel you mean [14:08] Saviq, ultimately it's just exposing it to qml, i don't see any harm in that [14:08] Saviq, I talked about having a locale-independent 24-hour option on Tuesday with … with you, actually. [14:08] mpt, yeah, that's why I'm asking - 'cause there isn't a setting for it :) [14:09] mhr3, the one that will let us "upload" a modified JSON string for a category [14:09] Saviq, that will be internal to the shell, it won't talk to the scope [14:09] mhr3, otp, back in 0.5h or so [14:09] Saviq, right, let's hangout afterwards [14:09] mhr3, https://code.launchpad.net/~mhr3/unity-scopes-shell/json-defaults/+merge/198591/comments/461511 in the mean time [14:10] thx [14:11] Saviq, 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] mpt, yup, sounds good [14:12] mpt, if you didn't see me saying yesterday, MeeGo I think got it right, by allowing you to select: [14:12] language, format-locale, and 12 vs. 24hr clock [14:13] format-locale including number format, currency format etc. [14:13] mpt, feels like a well-balanced solution [14:17] dednick: no, I don't. When did that start? [14:19] dandrader: 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:20] tsdgeos, sure [14:20] dandrader: it gives it to me, but i know what it's supposed to do so i don't obviously count :D [14:22] Saviq, I haven't gotten around to testing the branch yet. [14:22] What was wrong with it? [14:23] nic-doffay, it doesn't actually package the file [14:23] tsdgeos, "The number of rules is calculated[...]" <- number of *rules*? [14:23] lol [14:23] rows [14:23] tsdgeos, columns? [14:24] that [14:24] * tsdgeos slaps himself [14:24] hehehe [14:25] dandrader: ok r597 fixes the typo and adds a bit more of info [14:25] tsdgeos, also, why is the documentation in the class instead of in the header? [14:25] s/class/cpp [14:25] dandrader: because it''s what e had for listviewwithpageheader.cpp [14:25] but yeah it's weird [14:25] also lvwph.cpp is more about internal [14:26] and the first part of this is more about external [14:26] maybe i can split this one into two? [14:26] so "The implementation is centered around m_columnVisibleItems" and after is on the .cpp and the rest in the .h? [14:27] yeah, in the .h I would say "what it is" and in the .cpp I would put the comments about "how it's done" [14:28] larsu: no idea. I just added enable item by sensitivity and it stopped responding to clicks. [14:29] dandrader: splitted [14:31] Saviq: mzanetti: greyback: standup? [14:32] tsdgeos, otp again [14:32] tsdgeos: same for me [14:32] mzanetti, notes [14:33] dednick: is the service reporting disabled actions? Or is it something in qmenumodel? [14:33] larsu: looks like it's in qmenumodel. the can_activate is false for the menu item actions. [14:35] larsu: because action_target is null, but parameter_type is not. [14:35] larsu: same old story [14:35] ! [14:36] dednick: ah right. Didn't we fix this back when we first found it? [14:36] apparently not :) [14:36] larsu: i thought so :) [14:36] larsu: been fixed in a few places [14:37] dednick: I'll make the service put a target on there, that will be easiest [14:37] because we want to keep the logic for "normal" menu items === iahmad is now known as iahmad|afk [14:43] hi. 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] the global menu or libdbusmenu or whatever it's called [14:44] yes there is [14:44] at least there's an envvar [14:44] but i don't remember it :D [14:44] <--- not useful [14:45] yeah, that's what we tell all the bug reporters, but it'd be cool to opt out at the distro level === dandrader is now known as dandrader|afk [14:45] Saviq: obviously all the runs wiht more logs are now passing :D [14:45] to avoid the continuous stream of bug reports about our main menu, which isn't really our main menu :) [14:45] tsdgeos, hit it again! [14:45] i am [14:45] re-running only the qmluitests instead of the whole thing [14:45] for faster turnaround [14:46] tsdgeos, yup [14:47] maybe 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] codebrainz, is there a reason for this particular app to opt-out from globalmenu? [14:48] Saviq, tons of bug reports to our project not caused by our project [14:48] codebrainz, caused by the global menu integration? [14:49] yes [14:49] codebrainz, can you forward the bugs to the project that causes them, so that it's fixed instead of disabling it? [14:49] we do, but quite frankly it's super annoying and not very cool to mess with an app that others have to support :) [14:50] none of our devs use unity, we only test/support normal GTK+ [14:50] Saviq: 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 menu [14:50] i fixed that already last year [14:50] and somehow it broke again [14:50] not cool :D [14:50] and last year is was less busy than now [14:51] that helped finding a time to fix it [14:52] codebrainz, 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 instead [14:53] Saviq, that's highly optimistic [14:54] I 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] codebrainz, who does the packaging for ubuntu, btw, and - what's the project, btw? [14:54] codebrainz: i guess the "easiest" for you is setting that envvar as the first thing in your main.cpp? [14:54] if you really really really really want to disable it [14:54] I'm not sure setting the envvar from main() would do it, would it? [14:54] Saviq, the app is Geany [14:55] hyperair is the packager AFAIK [14:55] codebrainz: i'd say it would, but not sure tbh [14:56] tsdgeos, it might, if set before gtk_init() actually [14:57] codebrainz, I dunno, if people use the apport to report bugs on ubuntu, they will get to the ubuntu project first [14:57] they use our bug tracker [14:57] some do anyway [14:57] codebrainz, I really don't feel like that's the right solution to the problem [14:57] codebrainz, feels short-sighted [14:58] Saviq, stealing the menu from an app with a hack, is *definitively* not the right solution :) [14:58] codebrainz, yeah, I know Ubuntu/Unity/Canonical is evil [14:58] that's established [14:58] no no, I rather like Ubuntu, I use Xubuntu myself [14:58] I just hate this hack because it does something evil [14:59] codebrainz, it's a design decision, and at that time it was the only way to apply it [14:59] codebrainz, with gtk2 it still is, afaik, and with gtk3 is better, iirc? [14:59] I can only hope it is [14:59] we don't have full/proper gtk3 support yet [14:59] codebrainz, I think it's basically a first-class concept that the menu is exported [15:00] codebrainz, to support MacOS, for example [15:00] Saviq, got time for the hangout? [15:00] mhr3, right, forgot to ping you [15:00] mhr3, sure [15:00] let's do it then [15:00] Saviq, the gtk3 way seems OK, but the dbusmenu hack thing is just plain wrong IMO [15:01] codebrainz, again, only way to apply the design decision that was made [15:01] codebrainz, we'd have done it otherwise if was possible [15:01] and we did, with gtk3, with qt etc. [15:01] Saviq, just because you did it the only way possible doesn't make it any cooler :) === dandrader|afk is now known as dandrader [15:02] codebrainz, sure it does, because *we did it* ;) [15:02] codebrainz, instead of saying it's impossible [15:02] Saviq, it was impossible to do it right, hence all the bug reports :) [15:02] E_NOT_CONSTRUCTIVE_DISCUSSION [15:03] codebrainz, it's working pretty well, there is like 10 apps that are known to not be working correctly and they are mostly doing weird stuff [15:04] our app doesn't do "weird" stuff, but it does load stuff dynamically into the menus [15:05] codebrainz, what app are we talking about? [15:05] I 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] seb128, geany [15:06] codebrainz, ok, we have https://bugs.launchpad.net/ubuntu/+source/indicator-appmenu/+bug/1072109 about that it seems [15:06] Ubuntu bug 1072109 in indicator-appmenu (Ubuntu) "appmenu shows wrong content" [Undecided,Confirmed] [15:10] seb128, there's more than that (especially on our tracker) [15:11] dednick: that should fix it: https://code.launchpad.net/~larsu/indicator-messages/set-targets/+merge/198957 [15:11] larsu: cool. will give it a quick test [15:11] codebrainz, 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:13] seb128, 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:14] but if the gtk3 way is less crazy, I could try and convince the team to start supporting that :) [15:16] codebrainz, well, the gtk3 way works with less crazyness/hacks yes [15:16] hmmm, 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:17] codebrainz, with traditional menus we need to do hacks to export parse the menus/export them, where gtk3 gmenus does it by itself [15:17] Saviq, do I specifically need to install the override in autopilot install or is adding the path sufficient? [15:17] larsu: cheers, worked. [15:18] dednick: cool [15:18] seb128, 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] codebrainz, do you have a link to the patch? [15:18] i can probably find it, one sec [15:19] https://bugs.launchpad.net/ubuntu/+source/libdbusmenu/+bug/907635 [15:19] Ubuntu bug 907635 in DBus Menu "lidbusmenu-GTK crash with Geany IDE using Python" [Undecided,Expired] [15:20] (the patch just fixes a harmless GTK-critical message, but it's still useful to avoid console spam, if it still applies) [15:21] well, it's harmless as long as the program isn't run without assertions :) [15:22] codebrainz, 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] seems like users' bug reports showing the console output still shows this critical warning [15:22] (often for unrelated bugs) [15:23] they might still run the lts? [15:23] it's possible [15:23] Saviq: do we still need to deal with seeds for phone for unity8 dependencies, or is it sorted by the deps? [15:23] tedg, do you think you could review the null check patch on this bug ^ [15:23] tedg, https://launchpadlibrarian.net/91512767/libdbusmenu-null-submenu.patch [15:23] Saviq: oh, i think we only needed seeds if there wasn't a dep right? [15:24] seb128, Sure, can you give me the bug number? (can't seem to get back to there) [15:24] Ah, I see. [15:25] tedg, 907635 [15:25] for rationale, see: https://developer.gnome.org/gtk3/stable/GtkMenuItem.html#gtk-menu-item-set-submenu [15:27] dednick, yeah, debian/control is the way to go [15:28] dednick, it's like: nothing else would depend on unity8, so ubuntu-phone seed depends on it === alan_g is now known as alan_g|tea [15:28] dednick, but for anything else, standard deps are good [15:28] Saviq: we still have 2 flacky unity8 tests I guess [15:28] http://ci.ubuntu.com/smokeng/trusty/touch/maguro/64:20131213.1:20131211.2/5443/unity8-autopilot/ [15:28] Saviq: cool [15:28] seb128, here's the latest bug, not sure it'll get reported to the proper place: https://sourceforge.net/p/geany/bugs/1013/ [15:29] it's parsing underscores in filenames as mnemonics [15:30] nic-doffay, how is dpkg supposed to know where to install that file? [15:30] Saviq, what's the purpose of this? usr/lib/python*/*/unity* [15:31] Saviq, I don't know enough about how all this fits in the bigger picture to properly tackle this. [15:31] It's that simple really. [15:33] codebrainz, could you open a bug about that on https://launchpad.net/ubuntu/+source/unity-gtk-module/+filebug ? [15:33] hi 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] alesage, ./build -s [15:33] Saviq, thx [15:33] alesage, you don't have new libunity-api-dev installed [15:33] alesage, or ./build -c, for that matter [15:34] Saviq, ok will investigate [15:34] seb128, hopefully the bug reporter will [15:34] codebrainz, you could also do it if you want to help your users... ;-) [15:35] I don't remember my launchpad login, that patch I submitted was likely the first/last time I used it :) [15:35] I can try though, and just link to the source forge bug [15:35] (I'm not affected as I use xfce panel) [15:36] codebrainz, I've pinged the maintainer in #ubuntu-desktop, I'm sure a bug would still be useful though [15:36] seb128, can you tell me what email address is associated with my account? [15:37] it won't show me unless I'm logged in [15:37] (I have like 50 email addresses) [15:37] codebrainz, @geany.org [15:37] nic-doffay, http://www.debian.org/doc/manuals/maint-guide/dother.en.html#install [15:37] seb128, thanks [15:37] codebrainz, yw [15:38] nic-doffay, so you need: [15:38] data/unity8.override usr/share/upstart/sessions [15:38] or so [15:38] Saviq, yeah that's what the manual seems to say, ta. [15:40] seb128, what project/component would that bug go against? [15:40] appmenu-gtk? [15:41] codebrainz, could you open a bug about that on https://launchpad.net/ubuntu/+source/unity-gtk-module/+filebug ? [15:41] heh, sorry, i missed that the first time :) [15:57] seb128, https://bugs.launchpad.net/ubuntu/+source/unity-gtk-module/+bug/1260777 [15:57] Ubuntu bug 1260777 in unity-gtk-module (Ubuntu) "Underscores parsed as mnemonics in menu items" [Undecided,New] [15:57] heh [15:57] thanks bot [15:57] codebrainz, thanks [15:58] np, it actually sounds like an unsolvable issue [16:03] people don't like my branch? https://code.launchpad.net/~cimi/unity8/searchIndicator-swipe/+merge/198258 [16:03] codebrainz, why? [16:05] seb128, how could the module know whether an item happened to be created by the constructor that parses mnemonics? [16:05] I don't think it can tell, and it can't assume all menu items were [16:05] unless there is a flag/property on the menu items or something, not sure [16:06] codebrainz, I don't know that code well, but I think those stuff were working with libdbusmenu, so it should be doable [16:06] codebrainz, let's trust the guys working on that code and see what they come with ;-) [16:07] ah yeah, there is: https://developer.gnome.org/gtk3/stable/GtkMenuItem.html#GtkMenuItem--use-underline [16:08] so it's totally doable [16:10] nic-doffay, I *think* you need absolute path in the .install file there [16:10] or, didrocks, can you look at the .install file change in https://code.launchpad.net/~nicolas-doffay/unity8/upstart-kill-fix/+merge/198931 [16:11] nic-doffay, ah it's still missing data/unity8.override in the .install file [16:11] didrocks, unping [16:11] Saviq, just pushed it now. [16:11] I was wondering about an abs path though, but we'll see. [16:12] nic-doffay, relative is correct [16:12] nic-doffay, see debian/unity8.install [16:30] Cimi: fixed review comments [16:36] codebrainz, I guess you commented on the wrong bug (https://bugs.launchpad.net/ubuntu/+source/libdbusmenu/+bug/1260779)? [16:36] Ubuntu bug 1260779 in libdbusmenu (Ubuntu) "Test issues in trusty (g_source_remove invalid use warnings), fails to build" [High,Confirmed] [16:40] seb128, totally, I had too many bugs open [16:40] can i delete it? [16:41] codebrainz, no, but don't worry about it (or just post a new comment saying "sorry, wrong bug") [16:45] Saviq, my first qt bug report <3 https://bugreports.qt-project.org/browse/QTBUG-35608 [16:45] Cimi, ;) [16:45] Cimi, file another one that it's not easily customizable [16:45] Cimi, separatet [16:45] -t === alan_g|tea is now known as alan_g [16:46] Saviq, ok [16:46] the voice "create issue" is fun [16:46] should be file bug imho :D [16:46] but let's create an issue :D [16:48] Saviq, but how would you address the latter? [16:48] Saviq, why would you want to change *all* values? [16:48] Saviq, if the default value is sane you don't want to customise it [16:52] Saviq, I'm waiting your input [17:00] well filed the second https://bugreports.qt-project.org/browse/QTBUG-35609 [17:00] although I'm not 100% convinced if the first gets fixed === jhodapp is now known as jhodapp|bbiab [17:28] dednick, hey dude [17:28] dednick, the comment i wrote [17:29] dednick, I have qt 5.2 [17:29] I believe it's thrown because of the new JS engine [17:31] Cimi: hm. give me a sec [17:31] dednick, or let's do this next week :P [17:31] dednick, tryCompare(function() { signalSpyDismiss.count > 0; }, true); [17:31] :-\ [17:32] Cimi: no return maybe... [17:32] * Cimi tries [17:32] but succeeds for me, which is a bit odd [17:33] dednick, no luck for me [17:33] dednick, which qt? [17:33] 5.0.2 [17:33] dednick, I'm on 5.2 [17:33] dednick, as said, they changed Js [17:33] yeah, but still... it's the same in other places [17:33] dunno then [17:34] let me rebase [17:34] Cimi: does 'make testSimpleTextMessageMenu' work? [17:35] dednick, dismiss fails here [17:35] FAIL! : qmltestrunner::SimpleTextMessageMenu::test_dismiss() A value is required for tryCompare [17:35] Loc: [/home/cimi/Development/usc/visual-updates/tests/qmltests/Menus/tst_SimpleTextMessageMenu.qml(170)] [17:36] mzanetti, you should be eod - in case you aren't, any idea on fixing tryCompare with qt 5.2? ^ [17:36] Cimi: i think there may have been checks for 5.2 in unity8 which i dont have in usc [17:36] dednick, that's why I asked mzanetti [17:37] hmmm... [17:37] can you paste me the line that fails? [17:37] Cimi: ^ [17:37] mzanetti, tryCompare(function() { signalSpyDismiss.count > 0; }, true); [17:37] Cimi: I don't think this works with 5.0 either [17:37] or tryCompare(function() { return signalSpyDismiss.count > 0; }, true); [17:38] mzanetti, they both fail [17:38] Cimi: with 5.0 too? [17:38] mzanetti, not with 5.0 [17:38] mzanetti, dednick said it's working for him [17:38] Cimi: eh... [17:38] Cimi: yeah. what I'm saying is that this is wrong [17:38] and I don't think the test was actually working [17:38] Cimi: it's supposed to be tryCompareFunction [17:38] yeah [17:38] dednick, so why was not failing for you? [17:38] no idea [17:39] but tryCompareFunction is something we have in UnityTestCase only [17:39] so you'd need to copy that (or upstream into the SDK ideally) [17:39] mzanetti: yeah, i already have it i think [17:39] dednick, with function works here [17:39] dednick, but next to fail is [17:39] compare(signalSpyTriggered.count > 0, true, "should have been triggered"); [17:40] which is false here [17:41] Cimi: yeah, looks like the tests were just bombing on mine, i get that as well. [17:41] now [17:41] dednick, ok no problem, let me know when you fix them [17:41] don't worry for today === jhodapp|bbiab is now known as jhodapp [17:47] charles: hi! I saw that you put up a branch for fixing the 24/etc in indicator-datetime - it seems to be still failing on CI [17:47] charles: can you take a look what's up? [17:47] sil2100, yes I'm looking at it right now [17:47] sil2100: looks like the unit tests are failing because it doesn't have locales configured s.t. we can test in a 12h and 24h locale [17:47] sil2100: I'm currently adding hooks to debian/ for this [17:48] charles: ah, makes sense - thanks for taking care of this! [17:51] Cimi: fixed and pushed [18:11] Saviq: 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:33] * greyback eow [18:37] same [18:37] * mhr3_ out [19:58] xnox, hey, so no need to tweak the Qt .cmake modules in the end? [19:59] Saviq, would know where I could find QDBusConnectionPrivate [20:00] kdub, ESYNTAXERROR [20:00] okay :) [20:00] kdub, lookin' [20:01] Saviq: what do you mean? =) [20:01] Saviq: cmake modules are all correct and installed into multiarch paths. [20:01] xnox, but I meant to actually find moc and such [20:01] Saviq: automoc works correctly out of the box, i fixed that in cmake. [20:01] xnox, ah, awesome [20:02] Saviq: same with rcc & qmake, but custom utils are need to be handled by hand. [20:02] Saviq: i think the cmake modules should provide standard FOO_EXECUTABLE for all qt tools, e.g. qdoc, dbuscpp, etc. [20:03] kdub, https://qt.gitorious.org/qt/qt/source/33a34960328cce7a6994d2ea771c82da7bfdb598:src/dbus/qdbusconnection.cpp#L67 [20:04] thanks Saviq! [20:04] xnox, you mean that's the current state after your recent cmake upload or that's the desired state? [20:11] Saviq: rcc/qmake/moc - work out of the box; other tools to be defined by default is a desirable. [20:11] bregma: hello! I noticed a FTBFS for lp:unity trunk in daily-build - did you guys see that? [20:11] (but low priority, upstream should start exposing those) [20:12] xnox, ok cool, q: you said "dpkg-architecture -aarmhf -c cmake .." should cross-build locally, doesn't seem to work here? [20:12] xnox, i.e. I still get a build for amd64 [20:13] Saviq: well, that works out of the box, if there are cross-dependencies and cross-compilers installed locally. [20:13] and all the libfoo-dev:armhf packages [20:13] xnox, ok, /me goes apt-get build-dep -aarmhf unity8 [20:13] in practice not many libfoo-dev are multiarch:same, thus one would need to remove & install to change arches. [20:14] Saviq: and that implies to did dpkg --add-foreign-arch armhf & added arch qualified sources in /etc/apt/sources.list [20:14] and ports archive [20:14] xnox, aah right, haven't done that yet apparently [20:14] yeah, that's what mk-sbuild --target armhf does in a chroot =) [20:14] so dpkg-architecture -aarmhf cmake ../ [20:14] is intended to be run inside a suitable chroot =) [20:15] or e.g. under click chroot. [20:15] xnox, 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 there [20:15] (well click chroots export vars set by dpkg-architecture -aarmhf already, so actually, it's just cmake ../ under those) [20:15] xnox, instead of building on the device [20:16] Saviq: well sbuild --host armhf, should cross-build local dir and spit out ../*_armhf.deb [20:16] (without specifying path to .dsc, it should work with building unpacked debian package tree, which unity8 folder is) [20:17] xnox, yeah, I know, but that's still too long for quickly iterating during development [20:19] xnox, and requires a writable device [20:19] xnox, which we shouldn't need for unity8-only dev [20:20] .... dude, that cross-compiles. [20:20] bzr branch lp:unity8; cd unity8; sbuild -A -d trusty --host armhf; adb push ../*_armhf.deb /userdata/ [20:21] oh, right [20:21] Saviq: but you can $ dpkg-deb -R *.deb unpack/ [20:21] to unpack them [20:21] without installing in a home dir. [20:21] xnox, sure, that I can do [20:21] and exectue them, or like make an override in the upstart job to run a custom one [20:21] xnox, but still builds the whole thing every time [20:22] xnox, usually with installing deps first [20:22] xnox, unless you have a pre-made specific sbuild chroot [20:22] xnox, it just does more than we generally need for quick iterations [20:22] e.g. $ echo "exec ~/bin/unity8" > ~/.config/upstart/unity8.override [20:22] xnox, no need [20:22] Saviq: well full clean build here is 2m20s [20:22] xnox, yeah exactly [20:22] xnox, too long [20:23] xnox, we're talking about during-development-deployment [20:23] Saviq: you can optimise, so for things that I do often i have $ mk-sbuild --target armhf --name unity8 [20:23] xnox, yeah, I know [20:23] pre-installing deps into that, I think you can bring the time lower. [20:23] so i'm not sure what you are asking about? [20:23] you want dirty builds? [20:23] xnox, I'm not *asking* about anything, really :D [20:23] xnox, yes [20:24] while qt-dev is not co-installable? [20:24] xnox, no, permanently [20:24] well, I could point cmake at /var/lib/schroot/unity8-amd64-armhf/* to link/compile against [20:25] xnox, 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 device [20:25] (such that we can re-use the same chroot, for dirty builds, without chrooting into it) [20:25] (well pre-installed packages) [20:25] xnox, yeah, we might use the sbuild chroot indeeed [20:25] -e [20:25] xnox, even chrooting into it [20:26] xnox, but bind-mouting [20:26] to be honest this calls out to simply prioritise making all -dev packages multiarch-same which unity8 depends on. [20:26] i never use schroots like them. [20:26] let me see how it can be used in bind-mounting mode. [20:27] xnox, no worries, unless you want to :) [20:27] xnox, I'm all-happy with what you enabled for us already [20:28] hm i ponder how save it is to bind-mount /home/ into schroot. [20:29] xnox, I do it all the time [20:31] so if the named chroot has all the right dependencies one should be able to do: schroot -c unity8-amd64-armhf "cmake .; make" [20:32] didrocks, seems indeed the retry helped on maguro, awesome [20:32] "Failed to unlock greeter, trying again..." yay [20:38] Saviq: yeah… but still flaky then :p [20:38] didrocks, well, not *really* [20:38] didrocks, it's just that we don't know when the device is not busy on startup any more [20:38] didrocks, to deliver the input in a timely manner [20:39] didrocks, aactually [20:39] didrocks, we should move to pressing the BFB [20:39] Saviq: actually, we start the test 5 minutes after the boot [20:39] didrocks, instead of swiping the greeter [20:39] so the device isn't busy [20:39] didrocks, but unity8 is starting [20:39] didrocks, being restarted with testability [20:39] ah yeah [20:39] yep on that one [20:39] didrocks, and loading dash and everything [20:39] need to either send an signal [20:39] to be more reliable [20:40] or use qmltestrunner (shhh) ;) [20:40] didrocks, we don't want to open a "ok, signal me and I'll unlock" security hole ;) [20:40] didrocks, qmltestrunner might not really help in that case [20:40] didrocks, what would help would be having control of the time [20:41] didrocks, so with each input movement you'd tell QML how much time passed [20:41] didrocks, 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 enough [20:42] didrocks, but yeah, maybe we should move to the BFB - but still would need to swipe the launcher in, so not *really* more reliable [20:45] Saviq: totally make sense, should be more reliable [20:46] or [20:46] hum [20:46] we are not really testing unlocking the greeter [20:46] didrocks, yeah, that's tested regardless [20:46] should be one test [20:46] maybe for the other [20:46] we should ship a flag/a way only with this AP test [20:46] to get the greeter always unlocked? [20:49] didrocks, 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:50] didrocks, there would have to be a "greeter unlocked, continuing" message, or two "failed to unlock greeter" [20:50] Saviq: well, the result if flaky [20:50] like, I rerun the test, it pass [20:50] didrocks, even with the latest unity8? [20:50] yeah, this is with latest [20:51] didrocks, the log suggests it's not :/ [20:51] Saviq: http://people.canonical.com/~ogra/touch-image-stats/20131213.1.changes [20:51] image 64 [20:52] didrocks, I need camera on the devices then :/, I'm done guessing [20:52] Saviq: I think the CI team has one [20:52] didrocks, yeah, sorry for bothering you [20:52] didrocks, go have your weekend [20:52] well, trying to promote the image first :) [20:54] didrocks, ah, those tests don't use the helpers [20:54] ahah! [20:54] :) [20:54] didrocks, where the retrying is done [20:54] didrocks, will have to fix that [20:54] see, everything has an explanation! [20:54] :) [20:54] Saviq: great ;) === salem_ is now known as _salem