| === jhodapp|afk is now known as jhodapp | ||
| === fginther` is now known as fginther | ||
| === _salem is now known as salem_ | ||
| tsdgeos | Saviq: ping | 07:30 |
|---|---|---|
| Saviq | tsdgeos, pong | 08:11 |
| tsdgeos | Saviq: was going to ask if the test failed standaolne or inside the full test, but i got it to fail in xvfbtest quite reliably, so nothing :D | 08:11 |
| Saviq | tsdgeos, ok, I couldn't get it to fail in isolation... | 08:12 |
| tsdgeos | Saviq: i'll have a branch soon, been runing for 10 min here without a fail, i'll MR it soon | 08:12 |
| Saviq | tsdgeos, awesomes | 08:12 |
| tsdgeos | mzanetti: wops segfault :D | 08:13 |
| tsdgeos | https://launchpadlibrarian.net/176162572/buildlog_ubuntu-utopic-i386.unity-api_1%3A7.81%2B14.10.20140520-0~134%2B201405201819~ubuntu14.10.1%2Bfix-launcher-tests-with-5.3~test1_FAILEDTOBUILD.txt.gz | 08:13 |
| mhr3 | saviq, guess you noticed the unity-api u8 branch landed, but if not you'll need to rebuild the existing u8 silos | 08:15 |
| Saviq | mhr3, yeah, old news | 08:16 |
| * tsdgeos is happy that one landed so quickly :) | 08:17 | |
| tsdgeos | tx guys | 08:17 |
| tsdgeos | Saviq: the fix for me is simply this one ^_^ https://code.launchpad.net/~aacid/unity8/flaky_gsv_test/+merge/220767 | 08:18 |
| Saviq | tsdgeos, coolz, let's see what jenkins says | 08:18 |
| Saviq | I'm worried we multiplied the time to run the tests because of xvfb... | 08:19 |
| Saviq | it takes over 1.5h now :| | 08:19 |
| tsdgeos | yeah :/ | 08:22 |
| tsdgeos | who is my ApplicationManager man? | 08:22 |
| tsdgeos | gerry? | 08:22 |
| * greyback suddenly remembers he has something very important to do somewhere else.... | 08:23 | |
| greyback | tsdgeos: what's up? | 08:23 |
| tsdgeos | greyback: is ApplicationManager.stopApplication blocking? | 08:23 |
| tsdgeos | greyback: i.e. will it wait until the app is gone from the application manager? | 08:24 |
| greyback | tsdgeos: no | 08:24 |
| tsdgeos | greyback: btw where's the ApplicationManager code? | 08:24 |
| tsdgeos | greyback: so the loop in lines 284 here is "wrong" https://code.launchpad.net/~paulliu/unity8/logout/+merge/216373 | 08:25 |
| tsdgeos | no? | 08:25 |
| greyback | tsdgeos: app is removed from the appManager list immediately when you call stopApplication | 08:25 |
| tsdgeos | ah, so it's removed but the app is not stopped immediately | 08:25 |
| greyback | tsdgeos: lp:unity-mir src/modules/Unity/Application/application_manager.cpp | 08:25 |
| tsdgeos | so the loop is not "incorrect" | 08:25 |
| greyback | tsdgeos: right. As from user's point of view, they just want the app to go away | 08:25 |
| greyback | they don't care it takes a second or two for it to stop | 08:26 |
| tsdgeos | greyback: well actually if i''m loging out | 08:27 |
| tsdgeos | i want to make sure the apps have really stopped | 08:27 |
| mzanetti | tsdgeos: greyback: I can fix the crash in unity-api's appman I guess. or is someone on it already? | 08:27 |
| tsdgeos | greyback: i don't want to lose any work because we shutdown stuff too early | 08:28 |
| mzanetti | tsdgeos: I don't think there's a way to wait for that in the current api | 08:28 |
| tsdgeos | greyback: is there any way to make sure all apps are closed? | 08:28 |
| greyback | tsdgeos: the "get" part of loop should not be necessary, as once you call stopApplication, that entry has definitely gone from the list | 08:28 |
| tsdgeos | i.e. something like iterate over all of them, stop them | 08:28 |
| tsdgeos | and then verify they are all really stopped | 08:29 |
| tsdgeos | greyback: well, it needs the get to get the appId, no? | 08:29 |
| greyback | tsdgeos: oh yeah, totally mis-read it. Thought it was checking if app was removed. Duhh | 08:29 |
| greyback | tsdgeos: I've didn't code AppMan to deal with such logging off situations. On phone, I don't want a "these apps have not shut down, kill them?" message though | 08:30 |
| greyback | desktop, sure, different story | 08:31 |
| tsdgeos | greyback: well, it should delay the shut down process enough time to let them close themselves | 08:31 |
| greyback | tsdgeos: upstart is shutting them down | 08:31 |
| greyback | upstart is the thing actually managing the applications, not AppMan | 08:31 |
| tsdgeos | greyback: right, but what happens if unity8 is gone? does the app survive that? | 08:32 |
| greyback | tsdgeos: you'd better ask an upstart person how apps are managed when a user session is ending. I can't say | 08:32 |
| tsdgeos | ok | 08:33 |
| greyback | but AppMan's job is done by then: it told upstart to close all apps | 08:33 |
| mzanetti | greyback: yes, the apps still live on when unity/mir goes away, no? | 08:33 |
| greyback | mzanetti: well they often crash if they try to render, when unity8/mir has gone away. But yes, they live on | 08:34 |
| mzanetti | right... yes. there might be issues with crashes, but assuming they'll be fixed eventually | 08:34 |
| mzanetti | so we shouldn't really have a problem with this shutdown | 08:35 |
| greyback | mzanetti: well you might have a point there, might be good to block mir shutdown until apps have disconnected from mir | 08:36 |
| greyback | otherwise we'll get crash reports for things that weren't really the app's fault | 08:36 |
| mzanetti | greyback: fair enough. But wasn't the ideas that we fix the mirclient qpa plugin in a way that we could even restart mir and the same instance of th app will just reconnect? | 08:37 |
| greyback | mzanetti: eventually yes. That would probably do it | 08:38 |
| greyback | but a more immediate solution might be what tsdgeos was thinking | 08:38 |
| mzanetti | not saying were there yet... | 08:38 |
| tsdgeos | we get random crashes on shutdown of LVPWH in test :/ | 08:38 |
| tsdgeos | http://paste.ubuntu.com/7504702/ | 08:38 |
| tsdgeos | 5.3 at least | 08:38 |
| greyback | tsdgeos: due to paulliu 's change? | 08:39 |
| tsdgeos | greyback: no no | 08:39 |
| tsdgeos | i'm just multi-tasking :D | 08:39 |
| tsdgeos | sorry | 08:39 |
| greyback | heh np | 08:40 |
| mzanetti | Mirv: hey ho, can you give this one a shot in the ppa? https://code.launchpad.net/~mzanetti/unity-api/fix-appman-tests-with-5.3/+merge/220768 | 08:40 |
| Mirv | mzanetti: ok, merging that on top of the other | 08:41 |
| greyback | tsdgeos: for iteration 1, I think paulliu's approach will do nicely. Then if we get a series of bugs on apps shutting down badly, we can try a more blocking approach | 08:41 |
| tsdgeos | greyback: mzanetti: so what's your suggestion for now for that? | 08:41 |
| Saviq | xnox, another problem we noticed: systemtap-sdt-dev is Arch: all, liblttng-ust-dev (Arch: any) depends on the former, so tries to pull :armhf, which doesn't exist | 08:41 |
| tsdgeos | greyback: ok, i'll review it and see what's going on | 08:42 |
| Saviq | xnox, what would be the correct fix there? make systemtap Arch: any? | 08:42 |
| tsdgeos | greyback: note that this is more for the desktop i think | 08:42 |
| Saviq | xnox, or adapting the liblttng B-D? | 08:42 |
| greyback | tsdgeos: well you'd better test it there so ;) | 08:42 |
| tsdgeos | greyback: ok, i'll leave it for you then :D | 08:42 |
| mzanetti | greyback: I would check if there even is an issue with pauls branch in practice, if yes, probably try to fix the crash when mir goes away | 08:42 |
| tsdgeos | i can't get unity8 to run on my desktop | 08:43 |
| mzanetti | greyback: only if that turns out to be too troublesome, add something to allow us to wait for apps to be closed | 08:43 |
| greyback | tsdgeos: oh poor baby | 08:43 |
| mzanetti | :D | 08:43 |
| paulliu | tsdgeos: same here. I got black screen. | 08:46 |
| mzanetti | Mirv: yep, looks better i386 passed | 08:48 |
| paulliu | tsdgeos: what I've been told is you have to shutdown your machine. And login with unity8 at beginning. | 08:48 |
| paulliu | tsdgeos: if you logout and then switch, that will be failed. | 08:48 |
| tsdgeos | paulliu: didn't work here last time i tried | 08:48 |
| Mirv | mzanetti: so it did, awesome! | 08:49 |
| greyback | tsdgeos: paulliu: first check, install "mir-demos", then switch to VT1, run "sudo mir_demo_server_shell" , then switch to VT2 and run "sudo mir_demo_client_egltriangle" | 08:50 |
| greyback | *definitely use sudo or you'll be locked out of your machine* | 08:50 |
| greyback | if you switch back to VT1, you should see a rotating rectangle. That means Mir is working on your machine | 08:52 |
| mzanetti | Cimi: lol, just came by this: https://bugs.launchpad.net/unity8/+bug/1246688/comments/1 | 08:52 |
| ubot5 | Ubuntu bug 1246688 in Unity 8 "[Launcher] Icons don't fill the UbuntuShape" [Low,In progress] | 08:52 |
| Cimi | ahahah | 08:53 |
| Cimi | mzanetti, tempted to write a new comment | 08:53 |
| xnox | Saviq: I'll fix that. | 08:53 |
| Saviq | xnox, awesome, thanks | 08:53 |
| xnox | Saviq: what was example package to test? | 08:53 |
| Cimi | UPDATE: Waiting new ubuntushape rework to land in future centuries | 08:53 |
| Saviq | xnox, unity-scopes-api | 08:53 |
| mzanetti | :D | 08:53 |
| xnox | Saviq: ack. | 08:53 |
| Saviq | Cimi, mzanetti, we still didn't list the features we're missing most from the old shape... | 08:54 |
| mzanetti | Saviq: I sort of did to kgunn | 08:54 |
| Saviq | mzanetti, yeah, we need to put it down on paper | 08:55 |
| Saviq | mzanetti, and send up to Pat | 08:55 |
| mzanetti | Saviq: but I don't think its an issue that they don't know /what/ to do, the issue more they don't know /when/ to do it | 08:55 |
| Saviq | mzanetti, they asked me for a prio list to backport to the old shape | 08:55 |
| mzanetti | uh | 08:56 |
| pete-woods | xnox: if it helps, here is my buildlog of that build failing http://paste.ubuntu.com/7504758/ | 08:56 |
| Saviq | mzanetti, yeah, my reaction as well :| | 08:56 |
| Saviq | I saw the fooking shape working... over a year ago in London | 08:56 |
| Saviq | (new) | 08:56 |
| mzanetti | yeah... I mean... how hard can it be? | 08:56 |
| mzanetti | not to write it, to release it | 08:56 |
| Saviq | when you're a perfectionist like Loïc, it might be really hard... | 08:57 |
| xnox | Saviq: pete-woods: tah. It just needs marking to be M-A:foreign. I'll upload that & forward delta to debian. | 08:57 |
| mzanetti | ok yeah... fair point | 08:57 |
| Saviq | xnox, coolz | 08:57 |
| mzanetti | Saviq: ok, I can write you my list | 08:57 |
| Saviq | Cimi, ↑ please do the same (I can find the log of our discussion about it if you want) | 08:58 |
| Cimi | Saviq, mzanetti just sit down in malta on monday | 08:58 |
| Cimi | Saviq, I will write a post it note today | 08:58 |
| Saviq | Cimi, sure, thinking about it before then is still useful | 08:58 |
| Cimi | Saviq, next week we discuss what has higher priority | 08:58 |
| Saviq | Cimi, remember to bring the note over! ;) | 08:58 |
| mzanetti | or use the reminders app for it :P | 08:59 |
| Saviq | mzanetti, you? recommending sending your private data to Evernote? ;) | 09:00 |
| Saviq | mzanetti, you should've made a backend for ownCloud ;P | 09:00 |
| mzanetti | Saviq: well, given this ends up on Launchpad... | 09:00 |
| mzanetti | Saviq: well, they wanted to drop the notes app and make reminders the default notes app. I vetoed that for this reason, yes | 09:01 |
| Saviq | mzanetti, well, it's probably owned by Evernote before that | 09:01 |
| Saviq | (the data) | 09:01 |
| Saviq | mzanetti, good, I did complain about this, too (I'd gladly have only one app, but being forced to do Evernote to use it... nowai) | 09:02 |
| mzanetti | yep, exactly my words | 09:02 |
| pete-woods | xnox: in addition to the system package, it looks like jsoncpp has an invalid non-multiarch .pc file (http://paste.ubuntu.com/7504825/) | 09:24 |
| pete-woods | (I think) | 09:24 |
| xnox | Saviq: apparmor fix is in https://code.launchpad.net/~xnox/apparmor/xnox-citrain/+merge/220770 | 09:25 |
| xnox | Saviq: systemtap fix in https://launchpad.net/ubuntu/+source/systemtap/2.3-2ubuntu1 | 09:25 |
| Saviq | xnox, awesome | 09:25 |
| xnox | pete-woods: that would not be a problem. | 09:26 |
| Cimi | Saviq, if I want to edit tests/mocks/CMakeLists.txt to add tests/Mocks/Infographics/ (which has Infographics.qml and qmldir) what shall I add? | 09:26 |
| Saviq | Cimi, add_subdirectory(Infographics)? | 09:27 |
| pete-woods | xnox: okay, it's just that cmake isn't finding that library in the sbuild environment | 09:27 |
| xnox | pete-woods: although it is incorrect (locations specified is not where things are) but in practice makes no difference, because it should be in the default location.... | 09:27 |
| xnox | pete-woods: do you have a log? | 09:27 |
| pete-woods | xnox: sure | 09:27 |
| Cimi | Saviq, cointains no cmakelists.txt though | 09:27 |
| Saviq | Cimi, why doesn't it? | 09:28 |
| Cimi | Saviq, because it's just a folder to install | 09:28 |
| Saviq | Cimi, no, it's not "just a folder to install" | 09:28 |
| Saviq | Cimi, it's a plugin | 09:28 |
| Cimi | Saviq, there's nothing to compile | 09:28 |
| Saviq | Cimi, that needs to be copied into the binary dir | 09:28 |
| Saviq | Cimi, just copy what other mocks do, it doesn't matter that there's nothing to compile | 09:28 |
| Cimi | Saviq, indeed, so I thought I would have done this inside tests/mocks/CMakeLists.txt | 09:28 |
| pete-woods | xnox: there's the lttng stuff (but I expected that, I just wanted to see if there'd be other errors) - http://paste.ubuntu.com/7504845/ | 09:29 |
| Saviq | Cimi, just do what other mocks do | 09:29 |
| Cimi | ok | 09:29 |
| tsdgeos | Saviq: top-approve https://code.launchpad.net/~aacid/unity8/nomorecardheader/+merge/220659 ? | 09:31 |
| Saviq | tsdgeos, just remembered, don't we have a PreviewHeader now? are tests for it enabled? | 09:31 |
| tsdgeos | Saviq: they are | 09:32 |
| Saviq | tsdgeos, ok | 09:32 |
| tsdgeos | ./qmltests/CMakeLists.txt:48:add_qml_test(Dash/Previews PreviewHeader IMPORT_PATHS ${qmltest_DEFAULT_IMPORT_PATHS} ${CMAKE_BINARY_DIR}/tests/mocks) | 09:33 |
| Saviq | tsdgeos, done | 09:34 |
| === vrruiz_ is now known as rvr | ||
| tsdgeos | Saviq: since organicgrid test didn't fail in https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/63/? either i guess we can say https://code.launchpad.net/~aacid/unity8/organiccgridcrash/+merge/220651 "is good" ? | 09:57 |
| Saviq | tsdgeos, yeah | 09:58 |
| Saviq | tsdgeos, so looks like the zoomable tests are the last ones to fix | 09:59 |
| Saviq | ah no, greeter, too | 09:59 |
| tsdgeos | yep | 09:59 |
| tsdgeos | and 5.3 regresses some too | 09:59 |
| tsdgeos | there's some "html" that gets converted to text | 10:00 |
| tsdgeos | like <br> to actual newlines | 10:00 |
| tsdgeos | and stuff | 10:00 |
| tsdgeos | so tests fail | 10:00 |
| Saviq | oh | 10:00 |
| tsdgeos | want me to run them all and see what's the diff? | 10:00 |
| Saviq | tsdgeos, I will | 10:00 |
| tsdgeos | ok | 10:00 |
| Saviq | :| gotta reboot again, usb loop once more | 10:02 |
| la_juyis | hi Saviq! I was windering if there were any chances that we could add an easier (shorter and more intuitive) way to close apps in the phone. | 10:20 |
| Saviq | la_juyis, you'll be able to swipe the app away in the right-edge switcher soon | 10:20 |
| la_juyis | Saviq, yay! :D | 10:21 |
| Saviq | la_juyis, the app thumbnails in dash are going away, actually | 10:21 |
| la_juyis | Saviq, cool. I had thought of either that option, or another option in the meny of the launcher | 10:21 |
| la_juyis | (as it is on the computer) | 10:21 |
| la_juyis | launcher, long tap, close | 10:21 |
| la_juyis | Saviq, awesome, that's great news :D | 10:22 |
| la_juyis | Saviq, tnx a lot :) | 10:22 |
| Saviq | la_juyis, cheers | 10:28 |
| karni | Saviq: what's the easiest way to upgrade to U? I saw you threw a sed command replacing "trusty" with "utopic" in sources.list, I'm not sure that's the encouraged way to do it though ;) | 10:38 |
| Saviq | karni, do-release-upgrade -d works now | 10:38 |
| karni | Saviq: thanks! | 10:38 |
| === salem_ is now known as _salem | ||
| Saviq | xnox, liburcu-dev has basically the same issue jsoncpp has | 10:46 |
| xnox | Saviq: no liburcu-dev is not multiarch, and locations in .pc file match actual. | 10:59 |
| xnox | Saviq: even when cross-compiling, it is valid on ubuntu to link against /usr/lib/libfoo.so since we insure we install the correct arch library. | 10:59 |
| Cimi | Saviq, it is not working, I cannot see what's wrong http://paste.ubuntu.com/7505139/ | 11:00 |
| Cimi | Saviq, make tryInfographics cannot find Infographics | 11:00 |
| Saviq | xnox, sure, but cmake doesn't find the .pc files | 11:00 |
| Cimi | but make -n shows -I /home/cimi/Development/unity8/infographics-new-lightdm/builddir/tests/mocks | 11:01 |
| xnox | Saviq: then we need to fix our cmake. | 11:01 |
| Saviq | xnox, it seems to have regressed indeed | 11:01 |
| Cimi | and in /home/cimi/Development/unity8/infographics-new-lightdm/builddir/tests/mocks/Infographics I have the files | 11:01 |
| Saviq | xnox, http://paste.ubuntu.com/7505144/ | 11:01 |
| Saviq | xnox, same problem in mir (for android-properties) | 11:01 |
| Saviq | Cimi, you don' | 11:03 |
| Saviq | Cimi, why did you create a C++ module for infographics? | 11:03 |
| Cimi | I didn't | 11:03 |
| Saviq | Cimi, when you just said it's only qml + qmldir? | 11:03 |
| Saviq | Cimi, for the mock | 11:03 |
| Cimi | I just removed the add_library stuff and see | 11:03 |
| Saviq | Cimi, you're not building a library | 11:03 |
| Cimi | I know | 11:04 |
| Saviq | Cimi, .qml files can't be sources for a library | 11:04 |
| Saviq | Cimi, you only need the add_unity8_mock stanza in that CMakeLists.txt file | 11:04 |
| Cimi | just add_unity8_mock(Infographics 0.1 Infographics) ? | 11:04 |
| Saviq | yes | 11:04 |
| Saviq | AH motherfooker | 11:05 |
| Saviq | it's qtcreator that causes my devices to reconnect all the time | 11:05 |
| xnox | Saviq: | 11:06 |
| xnox | -- checking for module 'liburcu-bp' | 11:06 |
| xnox | -- found liburcu-bp, version 0.7.12 | 11:06 |
| Cimi | Saviq, just remove the freakinc ubuntu plugin from qtcreator | 11:07 |
| Cimi | I use qtcreator vanilla and I am much happier | 11:07 |
| Cimi | and doesn't screw qmljsdebugger | 11:07 |
| Cimi | btw | 11:07 |
| Cimi | https://code.launchpad.net/~cimi/unity8/qmljsdebugger/+merge/217474 | 11:08 |
| Saviq | xnox, hmm http://paste.ubuntu.com/7505169/ :/ | 11:08 |
| Saviq | Cimi, yeah, I know, wanted to give it a chance | 11:08 |
| xnox | Saviq: hm, something fishy is going on. | 11:09 |
| Saviq | Cimi, look at either Ubuntu.Thumbnailer or Unity.Notifications mocks, that's all you need to do | 11:09 |
| Saviq | xnox, indeed | 11:09 |
| Saviq | xnox, that was just "apt-get source" + "sbuild..." in a clean, updated utopic chroot | 11:10 |
| Saviq | Cimi, you can always check if a plugin will import fine with `QML2_IMPORT_PATH=builddir/tests/mocks/ qmlplugindump Infographics 0.1` | 11:12 |
| Cimi | Saviq, the module {} is empty here | 11:15 |
| Saviq | Cimi, well, how about when you point it at tests/mocks directly? | 11:20 |
| Saviq | Cimi, looks like your qml module is just broken is all | 11:20 |
| Cimi | still empty | 11:20 |
| Cimi | http://paste.ubuntu.com/7505215/ | 11:21 |
| Cimi | module Infographics | 11:22 |
| Cimi | InfographicList 0.1 InfographicList.qml | 11:22 |
| Cimi | this is my cmakelist | 11:22 |
| Cimi | ops | 11:22 |
| Cimi | qmldir | 11:22 |
| Cimi | inside I have InfographicList.qml | 11:23 |
| Cimi | more than that I don't know... | 11:23 |
| === MacSlow is now known as MacSlow|lunch | ||
| Saviq | Cimi, push to a branch | 11:26 |
| Saviq | Cimi, btw, you're missing a "typeinfo" stanza in the qmldir | 11:27 |
| Cimi | ah maybe then | 11:27 |
| Saviq | Cimi, but that won't change | 11:27 |
| Saviq | the empty module | 11:28 |
| Cimi | Saviq, pushed | 11:29 |
| Saviq | Cimi, lp:~cimi/unity8/infographics-new-lightdm ? | 11:29 |
| Cimi | I am going quickly to the doctor, will be back in less 10 mins | 11:29 |
| Cimi | Saviq, yup | 11:29 |
| Saviq | Cimi, woah, that's a close doctor :) | 11:30 |
| Saviq | Cimi, and where's the .qml file? | 11:32 |
| xnox | Saviq: yeap, found regression in pkg-config. | 11:34 |
| xnox | Saviq: will fix, should unbreak a whole bunch of things that worked a month a go. | 11:35 |
| Saviq | xnox, yeah great to hear | 11:35 |
| Saviq | pete-woods, ↑ | 11:35 |
| tsdgeos | hmmmm | 11:39 |
| tsdgeos | guys can you think of a way to know when ListView finished moving because the currentItem changed? | 11:40 |
| tsdgeos | i tried listening to moving but seems moving is never set to true in this case | 11:40 |
| Saviq | tsdgeos, hmm sounds like a bug then? | 11:42 |
| tsdgeos | Saviq: not really | 11:42 |
| Saviq | tsdgeos, only stupid idea comes to mind | 11:42 |
| tsdgeos | "due to the user either dragging or flicking the view." | 11:42 |
| tsdgeos | the user ain't doing anything | 11:42 |
| tsdgeos | name is bad i agree | 11:42 |
| tsdgeos | should be userMoving :D | 11:43 |
| Saviq | tsdgeos, is to compare delegate's x/y to contentX/Y | 11:43 |
| tsdgeos | yeah that's what i have | 11:43 |
| tsdgeos | not cool | 11:43 |
| Saviq | Cimi, btw, InfographicsList is a singleton in the real plugin, you need to make it so in the mock, too - see wiki on QML singletons: http://qt-project.org/groups/qt-contributors-summit-2013/wiki/Evolution-of-the-QML-language | 11:44 |
| Saviq | tsdgeos, ah wait, I think velocity still changes | 11:44 |
| Saviq | tsdgeos, so moving is false, but velocity is != 0 | 11:45 |
| tsdgeos | interesting | 11:45 |
| Saviq | I think I reported this as a bug at some point | 11:45 |
| Saviq | or at least considered it one ;P | 11:46 |
| Cimi | Saviq, ok | 11:46 |
| pete-woods | Saviq: cool, thanks for the update | 11:46 |
| Saviq | Cimi, but anyway, you didn't bzr add the file | 11:46 |
| Cimi | Saviq, my doctor is 2 mins walk, cool eh? :) | 11:46 |
| Saviq | Cimi, are you sure you have that .qml file there at all? | 11:46 |
| Cimi | Saviq, it's pushed | 11:47 |
| Saviq | Cimi, | 11:47 |
| === marlinc_ is now known as marlinc | ||
| tsdgeos | Saviq: weeeeird | 11:48 |
| Saviq | tsdgeos, indeed | 11:48 |
| tsdgeos | Saviq: this is my horizontal velocity log http://paste.ubuntu.com/7505281/ | 11:48 |
| tsdgeos | why all those 0! | 11:48 |
| Saviq | tsdgeos, oh | 11:49 |
| Saviq | yikes | 11:49 |
| tsdgeos | let me see if i get that in a simple test case | 11:49 |
| tsdgeos | that's kind of bad | 11:49 |
| Saviq | Cimi, ah, Module { } can be empty indeed | 11:50 |
| Saviq | Cimi, but that's fine, it means the import works | 11:50 |
| Saviq | Cimi, that works http://paste.ubuntu.com/7505295/ | 11:51 |
| Cimi | Saviq, so I need the singletone thing | 11:53 |
| xnox | Saviq: pkg-config caused regression 0.28-1 (released in 2014-04-25) in utopic. Fix uploaded 0.28-1ubuntu1. Once that builds, everything should be tip-top. | 11:54 |
| tsdgeos | Saviq: if you are still in 5.2 can you quick try http://paste.ubuntu.com/7505308/ and see if you get all the 0s? | 11:54 |
| xnox | Saviq: e.g. unity-scopes-api crossbuilds. | 11:54 |
| Saviq | xnox, awesomes! | 11:54 |
| Saviq | tsdgeos, yes, 0s here | 11:55 |
| tsdgeos | ok, so it's not a regression | 11:55 |
| Saviq | /food | 12:14 |
| Saviq | tsdgeos, fyi bug #1322615 | 12:54 |
| ubot5 | bug 1322615 in Unity 8 "QML test failures under Qt 5.3" [Undecided,New] https://launchpad.net/bugs/1322615 | 12:54 |
| Saviq | mterry, hey, you back already? sure don't want to get more sleep? | 12:57 |
| mterry | Saviq, I had trouble sleeping last night :-/ | 12:57 |
| Saviq | mterry, sorry to hear that | 12:58 |
| Saviq | mterry, hope it's not split greeter causing it... | 12:58 |
| Saviq | mterry, where did you find the 10% CPU issue in the end? | 12:59 |
| mterry | Saviq, that is a little odd... So there was that place in qtubuntu where we were checking for "unity8" process name and disabling sensors. Seemed by visual inspection to be 100% the problem. But changing it locally didn't do anything. And adding debug statements to see if it was being run didn't show up! So I figured it was dead code at this point. But I added a branch to look for unity8-greeter too, since wh | 13:00 |
| mterry | y not in case that code did get run in a different situation | 13:00 |
| mterry | Saviq, I put it in the PPA and after having that there, the CPU bug was fixe | 13:00 |
| mterry | Saviq, so somehow I wasn't installing the deb I built right or something when testing it? :-/ | 13:01 |
| Saviq | mterry, right, now that you're saying this... I think I know what's happening | 13:01 |
| Saviq | mterry, you can see app+sensors taking 10% cpu whenever you have any app focused | 13:02 |
| Saviq | mterry, that's most probably the accelerometer sensor for orientation eating away | 13:02 |
| Saviq | mterry, I now remember we disabled sensors for unity8 since we knew it's not going to rotate | 13:02 |
| tsdgeos | Saviq: that many? | 13:02 |
| Saviq | tsdgeos, yeah it's bad - we're using the wrong sensor and generally it's bad | 13:02 |
| Cimi | Saviq, in order to get InfographicList without brackets, it means is a singleton, right? | 13:03 |
| Saviq | Cimi, yes | 13:03 |
| Cimi | Saviq, but once I added pragma singleton to the file | 13:03 |
| Saviq | Cimi, did you also add "singleton" to the qmldir file? | 13:03 |
| Cimi | nope | 13:03 |
| mterry | Saviq, yeah tht's what I figured from looking at code, but was confused why changing it and installing the deb didn't work... But I must have done it wrong | 13:05 |
| mterry | Saviq, but that was a pleasant surprise after updating from PPA :) | 13:05 |
| Saviq | mterry, good | 13:05 |
| Saviq | mterry, so yeah, as you can see from my comment we're real close | 13:05 |
| mterry | Saviq, so how bad is the "not syncing volume" issue for your? Like percent wise it happening? | 13:05 |
| Saviq | mterry, not any more at all | 13:05 |
| Cimi | Saviq, keeps saying Pragma requires a valid qualifier when I run make | 13:05 |
| Cimi | Saviq, is this a qt 5.3 thing? | 13:06 |
| Saviq | Cimi, no, 5.2 | 13:06 |
| Cimi | weird then | 13:06 |
| Saviq | mterry, once I had ofono sounds gone, but music was playing, so separate issue | 13:06 |
| Saviq | and couldn't reproduce either | 13:06 |
| Saviq | at that point sms wouldn't show up in greeter, too, but can't reproduce either | 13:06 |
| mterry | Saviq, isn't that the same bug as we have on normal Touch? | 13:07 |
| mterry | Saviq, "missed call / sms number not recognized in greeter messaging menu" is a known issue, it's on Ted's TDO | 13:08 |
| mterry | *TODO | 13:08 |
| Saviq | mterry, k | 13:08 |
| Saviq | mterry, which issue known? | 13:08 |
| Saviq | on normal Touch? | 13:08 |
| mterry | Saviq, well both I thought. The missed notification for calls was happening on normal Touch. Did we fix that? | 13:08 |
| Saviq | mterry, yeah, it was there under the greeter | 13:09 |
| Saviq | mterry, ah now you mention it... | 13:09 |
| Saviq | mterry, dual notifications... one in greeter, one in session, looked weird, too, and we probably need to address that at some point | 13:09 |
| mterry | Saviq, well... yeah. that's tricky design wise. Not sure how they want that to look (especially since notifications in session could be a different set than on the greeter theoretically) | 13:10 |
| Saviq | mterry, indeed | 13:10 |
| mterry | Saviq, infographic bubbles have lost their color? Odd | 13:11 |
| Saviq | mterry, I saw that before | 13:11 |
| Saviq | mterry, they were doing that under run_on_device for example | 13:11 |
| Cimi | Saviq, ahah found out | 13:11 |
| Cimi | Saviq, that wiki is wrong, pragma Singleton not pragma singleton, uppercase | 13:12 |
| Cimi | had to read qt code | 13:12 |
| Saviq | Cimi, indeed, it's not documented well :| | 13:13 |
| tsdgeos | Saviq: reported https://bugreports.qt-project.org/browse/QTBUG-39226 | 13:14 |
| Saviq | tsdgeos, so you can't use it anyway can you... | 13:17 |
| tsdgeos | Saviq: correct | 13:17 |
| mterry | Saviq, so are all those issues blockers? I'm wondering if it's possible to squeeze in landing today, but probably not. At least the logind using 3% CPU is probs a blocker. Maybe some of the visual flashing is too | 13:17 |
| tsdgeos | Saviq: mzanetti: tests will now run slowlier if we do the qmldebug thing | 13:17 |
| Saviq | mterry, no, they're not blockers | 13:18 |
| Saviq | mterry, that's why I said 'file bugs for indicators' | 13:18 |
| tsdgeos | not that i mind but lowers our "response time" on merging if we need to wait for a ci that runs in 2 hours | 13:18 |
| mterry | Saviq, yeah but the other stuff I was thinking | 13:18 |
| tsdgeos | but since i'm not the one doing the merges that's ok to me :D | 13:18 |
| Saviq | tsdgeos, "qmldebug thing"? | 13:18 |
| tsdgeos | Saviq: https://code.launchpad.net/~cimi/unity8/qmljsdebugger/+merge/217474 | 13:18 |
| Saviq | tsdgeos, isn't that optional? | 13:18 |
| Saviq | tsdgeos, like only for tryFoo? | 13:19 |
| tsdgeos | no? | 13:19 |
| Saviq | it should be | 13:19 |
| mzanetti | I think so yes, let me make sure it is | 13:19 |
| Saviq | tsdgeos, it's only there for qmlscene | 13:19 |
| Saviq | tsdgeos, not for qmltestrunner | 13:19 |
| tsdgeos | ah right | 13:19 |
| tsdgeos | sorr | 13:19 |
| mzanetti | yep | 13:19 |
| mzanetti | nw | 13:19 |
| Saviq | tsdgeos, we're running 2h anyway now... we should not do xvfb in qmluitests on jenkins... | 13:20 |
| Saviq | not as long as we're in a VM there | 13:20 |
| mzanetti | Saviq: huh? | 13:20 |
| mzanetti | does that make such a difference? | 13:20 |
| Saviq | mterry, I think yeah, the white flash on manta would be a thing to fix - I'd like the "SIM PIN" not fullscreen to be fixed, the blank frame on mako is very difficult to reproduce | 13:20 |
| Saviq | mzanetti, well, sure, everything's going on the CPU instead of the GPU | 13:20 |
| Saviq | mzanetti, so if we have 3D accel there, but not a lot of CPU power, it'll be much slower | 13:21 |
| mzanetti | yeah... but... 10 minutes and 2 hours doesn't really compare | 13:21 |
| Saviq | mzanetti, well, bug #1322202 | 13:21 |
| ubot5 | bug 1322202 in Ubuntu CI Services "Jenkins logs need timestamps" [Undecided,New] https://launchpad.net/bugs/1322202 | 13:21 |
| Saviq | I've no idea what takes so long | 13:21 |
| mzanetti | ok | 13:21 |
| mzanetti | mterry: found issues with the left edge animation branch you did | 13:22 |
| greyback | sorry was offline, updated to utopic, forgot to unpin the wireless driver, wireless driver failed with newer kernel, hilarity ensued | 13:22 |
| mterry | mzanetti, the one that changed the speed? oh bummer, what's up? | 13:22 |
| mzanetti | mterry: left comments | 13:22 |
| Saviq | but yeah the UNSTABLE runs (not the FAILED ones of course) take up to 2h now :| | 13:22 |
| Saviq | http://s-jenkins.ubuntu-ci:8080/job/unity-phablet-qmluitests-utopic/67/ | 13:22 |
| mzanetti | mterry: https://code.launchpad.net/~mterry/unity8/launcher-drag-animation/+merge/219216 | 13:22 |
| Saviq | 30 minutes of that is probably dependencies + build time | 13:23 |
| Saviq | as that's how long FAILED runs take | 13:23 |
| Saviq | so it's rather bad there already | 13:23 |
| dpm_ | hi Saviq, I've noticed that the Search box at the top of the screen is no longer translated in languages that IIRC were translated before. Do you know of any recent changes in unity that could have affected this? Actually, is this part of Unity?, or scopes? | 13:23 |
| mterry | mzanetti, odd. Those actions you describe (swipe/cancel, swipe/complete) are the only two things I would have tested. I'll look at it again later | 13:24 |
| Saviq | dpm_, that's unity, and that string is going away real soon | 13:24 |
| mzanetti | mterry: did you really pull from the edge? | 13:24 |
| mzanetti | mterry: it works fine if just dragging the greeter | 13:24 |
| Saviq | dpm_, in favour of a button on the page header itself | 13:24 |
| mterry | mzanetti, that's the only pull affected by the change | 13:24 |
| Saviq | dpm_, so I'd rather not deal with that, wdyt? | 13:24 |
| mterry | mzanetti, dragging on greeter isn't changed | 13:24 |
| === _salem is now known as salem_ | ||
| mterry | mzanetti, I must have screwed something up, I'll re-examine | 13:25 |
| mzanetti | ok | 13:25 |
| dpm_ | Saviq, we're preparing this Simplified Chinese image for MAE to be ready by the end of next week. Do you think the change with the transition to a button might land before that? I'm just trying to get a feel for what the best thing to do | 13:25 |
| mterry | Saviq, I can't reproduce the logind 3% cpu issue | 13:26 |
| dpm_ | is | 13:26 |
| Saviq | mterry, good | 13:27 |
| Saviq | dpm_, let me have a quick look (we didn't change it, not sure why it wouldn't work)... | 13:27 |
| dpm_ | Saviq, ok, thanks! | 13:27 |
| Saviq | dpm_, bad news: https://bugs.launchpad.net/ubuntu/+source/ubuntu-ui-toolkit/+bug/1322630 | 13:48 |
| ubot5 | Ubuntu bug 1322630 in ubuntu-ui-toolkit (Ubuntu) "Setting i18n.domain breaks translation" [Undecided,New] | 13:48 |
| * Saviq goes to #sdk | 13:48 | |
| Saviq | brb | 13:52 |
| dpm_ | ok, thanks Saviq, let me follow on #sdk | 13:55 |
| === salem_ is now known as _salem | ||
| === MacSlow|lunch is now known as MacSlow | ||
| mhr3 | dpm_, ping? | 14:02 |
| dpm_ | hey mh | 14:02 |
| dpm_ | hey mhr3 | 14:02 |
| mhr3 | dpm_, i think you need to restart the phone to get the scopes scope translated | 14:02 |
| dpm_ | mhr3, I did several times. It's actually mostly translated already, though | 14:03 |
| mhr3 | so, all good? | 14:03 |
| paulliu | tsdgeos, Saviq: https://code.launchpad.net/~paulliu/unity8/fix-zoomableimage-test1/+merge/218649 | 14:04 |
| dpm_ | mhr3, it seems so, facundo is looking at the rest of untranslated strings, and probably Monday they should appear translated | 14:05 |
| Saviq | paulliu, bad indentation in line 66 of the diff | 14:05 |
| paulliu | Saviq: ok. fixed. | 14:07 |
| paulliu | Saviq: forget to run ctest | 14:07 |
| Saviq | paulliu, testZoomableImage seems to just hang for me indefinitely | 14:08 |
| paulliu | Saviq: you mean xvfbtestZoomableImage? | 14:09 |
| paulliu | Saviq: It has some delays in testZoomableImage because we need some delay for consequent two separate pinch events. | 14:10 |
| Saviq | ok, I'm leaving it be | 14:11 |
| Saviq | let's see | 14:11 |
| mterry | Saviq, notification sized fixed... now looking at white flash on manta | 14:28 |
| Saviq | mterry, coolz | 14:29 |
| paulliu | BTW, what is /usr/lib/i686-linux-gnu/mesa/libGL.so.1 I only have i386 one. | 14:30 |
| Saviq | paulliu, there shouldn't be i686 on ubuntu... | 14:37 |
| paulliu | Saviq: ok. so there's some bugs. I'll try to figure that out. | 14:38 |
| Saviq | paulliu, where did you see the i686 path? | 14:40 |
| paulliu | Saviq: If I try to use xvfb test. It tries to PRELOAD it. | 14:41 |
| Saviq | paulliu, oh weird | 14:42 |
| Saviq | mzanetti, ↑ | 14:42 |
| Saviq | paulliu, what does `gcc -dumpmachine` say? | 14:42 |
| paulliu | Saviq: i686-linux-gnu | 14:43 |
| Saviq | _weird_ | 14:43 |
| Saviq | paulliu, don't use xvfb then, use testFoo | 14:43 |
| paulliu | Saviq: ok | 14:43 |
| Saviq | paulliu, and we need to fix the arch detection then | 14:43 |
| Saviq | paulliu, but that's not your issue | 14:43 |
| paulliu | Saviq: ok. | 14:44 |
| Saviq | paulliu, how about `dpkg-architecture -qDEB_BUILD_MULTIARCH` ? | 14:44 |
| paulliu | Saviq: i386-linux-gnu | 14:45 |
| Cimi | Saviq, I have this error http://paste.ubuntu.com/7505878/ with make testPhone | 14:47 |
| Cimi | new to you? | 14:47 |
| Saviq | Cimi, yes, I even wrote on your MP about those | 14:47 |
| Saviq | Cimi, https://code.launchpad.net/~cimi/unity8/infographics-new-lightdm/+merge/220662/comments/527505 | 14:48 |
| Saviq | mterry, could you even repro the white flash? | 14:49 |
| Cimi | Saviq, yes but I thought was something else | 14:49 |
| mterry | Saviq, this is after locking? no, I haven't been able to yet | 14:50 |
| mzanetti | Saviq: interesting... ok. will fix it | 14:51 |
| Saviq | mterry, yeah, I just pressed power to wake, pressed power to lock, waited a sec, BLAM | 14:51 |
| Saviq | mzanetti, am doing already | 14:51 |
| mzanetti | ah ok | 14:51 |
| mterry | Saviq, :-/ haven't seen it yet. Was it 100% reproducable for you? | 14:52 |
| Saviq | mterry, yes, not sure if reboot didn't fix it though | 14:53 |
| Saviq | mterry, just flashed and installing silo again | 14:53 |
| Cimi | Saviq, maybe because it needs updated lightdm? | 14:54 |
| Cimi | Saviq, I am confused with this permission denied | 14:55 |
| Cimi | doesn't tell me much about what's wrong | 14:55 |
| Saviq | Cimi, do those files even exist? | 14:55 |
| Cimi | Saviq, those are folders | 14:55 |
| Cimi | cimi:cimi users and group | 14:56 |
| Cimi | with permissions | 14:56 |
| Cimi | inside, a liblightdm-qt5-2.so | 14:56 |
| Saviq | Cimi, "env" is trying to execute those directories | 14:57 |
| Cimi | let me see the diff | 14:58 |
| Saviq | Cimi, so it's basically missing LD_PRELOAD_PATH | 14:58 |
| Saviq | Cimi, -v should show you the actual command | 14:58 |
| Cimi | Saviq, I used your branch, and something might have been removed when you removed usermetrics | 14:58 |
| Saviq | Cimi, sure, I never said my branch was correct, it was rather quick'n'dirty instead | 14:58 |
| Cimi | Saviq, not blaming you, just trying to understand what changed in this branch | 14:59 |
| Cimi | correct | 15:00 |
| Cimi | a LD_PRELOAD_PATH was removed in tests/qmltests/CMakeLists.txt | 15:01 |
| Cimi | library_path | 15:02 |
| Cimi | Saviq, works now! | 15:03 |
| === jono is now known as Guest70896 | ||
| === _salem is now known as salem_ | ||
| Saviq | paulliu, mzanetti, https://code.launchpad.net/~saviq/unity8/fix-qmltest-preload/+merge/220819 | 15:06 |
| mzanetti | paulliu: the code looks good to me, can you confirm it fixes your issue too? | 15:14 |
| === salem_ is now known as _salem | ||
| * greyback heading to the airport, see you folks in Malta | 15:21 | |
| mzanetti | o/ greyback | 15:22 |
| mterry | Saviq, any luck with manta flashes? I just got a volume syncing issue. Something is occasionally messing that up... | 15:28 |
| Cimi | entire session dying running unity8... risky... | 15:50 |
| Saviq | mterry, nope, can't repro | 15:54 |
| mterry | saviq, I don't like these sometimes issues | 15:55 |
| mterry | Saviq, but good that it's not reproducable | 15:55 |
| mterry | Saviq, so by my count it looks like volume not syncing sometimes is only big bug left | 15:56 |
| * mterry goes for lunch, will see if that's easily-solvable later today | 15:57 | |
| Saviq | mterry, yeah, let's sit down early in Malta and put an ACK stamp on ot | 16:04 |
| Saviq | it | 16:04 |
| mterry | Saviq, missed my goal of pre-Malta, but I'll live | 16:04 |
| Saviq | mterry, almost ;) | 16:05 |
| Cimi | catch you in malta! | 16:50 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!