/srv/irclogs.ubuntu.com/2015/05/08/#ubuntu-unity.txt

=== tmpRAOF is now known as RAOF
tsdgeosMirv: ping07:46
Mirvtsdgeos: pong08:06
tsdgeosMirv: spoke to thiago yesterday, he's shelving the dbus changes for Qt 5.608:06
tsdgeosin case you need to free a silo or something08:07
Mirvtsdgeos: ok, it'd good to know. I wonder if we can manage without them. we'll notice of course if we get QDBus related crash traces.08:08
tsdgeosMirv: i guess we will08:13
tsdgeosdednick: just tested and commented in https://code.launchpad.net/~nick-dedekind/ubuntu-settings-components/fix-maximumWaitBufferInterval/+merge/25734708:19
tsdgeosplease check i'm not testing the wrong thing :D08:19
dednicktsdgeos: there's a branch to go with it now. should probably add that to the MP08:19
tsdgeosplease08:20
dednicktsdgeos: there's a bug in the indicator mock which throws off the AP test08:20
dednicktsdgeos: done08:20
dednickhttps://code.launchpad.net/~nick-dedekind/unity8/fix-laggy-indicators-autopilot/+merge/25841108:20
dednicktsdgeos: ^08:20
tsdgeostx08:21
pstolowskitsdgeos, hey, i've sent you the proposal for edit widget08:26
tsdgeosokidoki08:26
tsdgeosi'll have a look soon-ish08:28
tsdgeosdednick: unity8.application_lifecycle.tests.test_application_lifecycle.ApplicationLifecycleTests.test_greeter_hides_on_app_open is still failing as autopilot test, did you have a look at it or should we find someone else to?08:31
dednicktsdgeos: I didn't have a look at that one. only the indicator ones08:31
dednicktsdgeos: i can take a look at that one as well though08:32
tsdgeosdednick: that'd be great08:33
tsdgeosmaybe we can get green back \o/08:33
dednicktsdgeos: lol. for a day or 2 at least ;)08:34
tsdgeosuntil the next landing :D08:34
dednicktsdgeos: um. is the greeter supposed to hide if an app opens?08:47
dednicktsdgeos: what if you have a password?08:47
dednicktsdgeos: oh, i think it might just be that the greeter is swiped away, but not "unlocked"08:48
tsdgeosdednick: right08:50
tsdgeoswhich actually the test seems to be doing08:51
tsdgeosso have to check where it's failing08:51
dednicktsdgeos: yeah. looks like it can't find the proxy or something. but checking on ubuntu-app-list shows the fake app. will need to look into it in more detail.08:52
dednicktsdgeos: works if you have the greeter open though :/08:53
dednickunlocked i mean08:53
tsdgeosdednick: you mean wihtout password?08:54
tsdgeospstolowski: why author?09:04
pstolowskitsdgeos, i suspect we may want to display your name as we do with existing reviews (i don't have visuals to confirm)?09:05
pstolowskitsdgeos, but maybe not09:05
tsdgeospstolowski: but can't we do that with a text widget or something? that's how we do it now, no?09:05
tsdgeosmaybe not09:06
pstolowskitsdgeos, now we do it with 'reviews' widget (see the doc i linked to)09:06
tsdgeosright09:06
pstolowskitsdgeos, 'reviews' widget takes multiple tuples and has ;'author' for each09:06
tsdgeospstolowski: let's have it in there, it won't hurt and then we can decide whether to show it in the ui or not09:07
pstolowskitsdgeos, +109:07
tsdgeosjust answered the email for completion09:08
pstolowskitsdgeos, cool, thanks09:09
dednicktsdgeos: no, if you enter the password manually while it's "waiting for whatever it is waiting for"09:12
dednicktsdgeos: nevermind. i'll figure it out ;)09:13
tsdgeosdednick: that's weird i don't remember that test went past the locker09:13
tsdgeosdednick: :)09:13
dednicktsdgeos: it's possible we're suspending the app before AP is able to get the proxy for introspection.09:14
tsdgeosmay be09:14
tsdgeosafaik this test used to work not very long ago09:14
Saviqtsdgeos, dednick, https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1446846/comments/209:17
ubot5Ubuntu bug 1446846 in unity8 (Ubuntu) "Three consistent autopilot test failures" [High,Triaged]09:17
Saviqtsdgeos, dednick, I think the problem there is that we're left with the lockscreen after having spawned the app09:17
Saviqso greeter hides, sure, but the lockscreen's still on09:18
dednickSaviq, tsdgeos: ubuntu-app-watch says the apps are being paused. both dash and fake app.09:18
Saviqyup09:19
tsdgeosSaviq: sure but that's always been the way it worked and i think that test passed "before"09:19
tsdgeosfor some definition of "before"09:19
tsdgeosobviously way before you filed the bug09:19
dednicktsdgeos: perhaps we need to modify the test. it's making the assumption that the app is in a "running" state at some arbitrary time after starting09:20
dednickwhere that should only definately the case after we unlock the greeter.09:20
dednickSaviq: ^?09:21
Saviqdednick, yeah, definitely a test issue09:21
dednickSaviq, tsdgeos: hm. unfortunately it's autopilot/launcher code that's waiting for the proxy :/09:25
Saviqtsdgeos, fwiw I think we "fixed" the fact that the apps were getting focused under lockscreen at some point09:26
Saviqdednick, we might need to trick the lockscreen to go away in this test09:26
Saviqtsdgeos, *or* the mock lightdm refactor09:26
* Saviq wonders if that test wasn't relying on the "single" lightdm mock09:27
tsdgeosmay be09:27
tsdgeosdednick: do we have a review checlist for https://code.launchpad.net/~nick-dedekind/ubuntu-settings-components/fix-maximumWaitBufferInterval/+merge/257347 ? or do i use the unity8 one and that's it?09:27
dednickSaviq: well, we can force the lockscreen open, but that's cheeting a bit ;)09:27
dednick*cheating09:28
dednicktsdgeos: yeah, let me add it quick09:28
Saviqdednick, I think we have the mock lightdm with no password for this use case09:28
dednicktsdgeos: oh. review. don't think we have one09:28
dednickjust an MP one09:28
tsdgeosdednick: well the test is about "opening app hides greeter", it's not cheating to put there a greeter that helps us testing09:29
tsdgeosdednick: oh, i just realized i can't top approve https://code.launchpad.net/~nick-dedekind/ubuntu-settings-components/fix-maximumWaitBufferInterval/+merge/257347 you'll need to find someone that's on the team09:30
dednicktsdgeos: i meant that it's cheating to force the lock screen open explicitly. since it skips what we're testing. not having a password enabled it's cheating.09:30
tsdgeosdednick: no no, because having the lock screen is fine09:30
tsdgeosit's what's it is supposed to do09:31
tsdgeosswipe the greeter away09:31
tsdgeosif you have no lock screen then you get the app09:31
tsdgeosif you have lock screen then you get the lockscreen09:31
Saviqdednick, tsdgeos, LIBLIGHTDM_MOCK_MODE=single will fix the test I reckon09:31
tsdgeosso we can either adapt the test to just check that the greeter went away and not test the app is running09:31
tsdgeosor we can adapt the test to check the greeter went away and the app is running09:32
dednickcimi: can you add Unity team to the ubuntu-settings-components team?09:32
cimidednick, yup09:32
tsdgeosi have no particular opinion on what's better09:32
dednickSaviq: how do i set the env var for a single test, or do i need to move it to another test suite?09:33
dednicksince u8 is started by setup09:33
Saviqdednick, grep for LIGHTDM in tests/autopilot, there's others that do it09:33
cimidednick, done09:33
Saviqeven if they're doing it in a hacky way09:33
dednickanswer is to make a new test case :)09:34
tsdgeosdednick: or that09:44
dednicktsdgeos: that's the way everything does it. can't launch unity8 before setting the env09:45
tsdgeosdednick: you lost me there :D but that's fine if you know what you're doing :D09:47
dednickSaviq: damnit. can't seem to use fake greeter. if we do, we don't get real apps...10:15
dednickSaviq: i could thread it to unlock while waiting for the proxy :/10:18
Saviqdednick, you mean that the import path is the same for fake greeter and fake apps?10:36
Saviqwe need to be able to pick'n'choose...10:37
dednickSaviq: you need to set _qml_mock_enabled apparently10:38
dednickSaviq: which i think gets all the mocks rather than just lightdm10:39
Saviqdednick, yeah10:39
dednickSaviq, tsdgeos: http://pastebin.ubuntu.com/11023987/ :S10:40
dednickSaviq: thread waits until the greeter is swiped by app open, and unlocks it. :)10:41
Saviqdednick, that kinda works...10:41
dednickpretty dodgey10:41
dednickbut will work until we can get selective mock shizzle.10:41
Saviqdednick, the other way would be to install a copy of the mock lightdm into a separate path (like we do with Unity.Application)...10:41
tsdgeosdednick: i think it's ok10:42
dednickI'll put in a FIXME for when we have mock selection.10:43
dednicktsdgeos: https://code.launchpad.net/~nick-dedekind/unity8/fix-autopilot-failures-1446846/+merge/25841110:51
dednicki've just updated the original MP.10:51
tsdgeosk10:51
=== MacSlow is now known as MacSlow|lunch
dednicktsdgeos: should I get on with the video play back in previews? or were you planning on doing it?11:40
* dednick goes for lunch11:45
=== MacSlow|lunch is now known as MacSlow
tsdgeosdednick: you can take it12:38
tsdgeospstolowski: https://code.launchpad.net/~aacid/unity8/edit_reviews/+merge/25862313:22
pstolowskitsdgeos, awesome! i'll give it a try with my code13:23
dandradermzanetti, is any sort of qml cache present on devel-proposed?13:23
tsdgeosdandrader: yes13:23
dandradermzanetti, I'm getting a very weird behavior that is driving me mad13:23
dandradermzanetti, it seems that when I change from DesktopStage to TabletStage or vice-versa13:24
dandradermzanetti, the previous stage somehow lives on13:24
tsdgeosah that kind of cache, don't know, we have the file cache to speed up startup13:24
mzanettidandrader, I think that's still the Loader bug13:24
tsdgeosbut shouldn't affect that13:24
dandradermzanetti, although I can't see it. some of its bindings still seem to act13:24
mzanettiyeah...13:24
mzanettithere is a bug in the loader, in combination with gsettings-qml13:25
mzanettiI digged into that for a day... found a workaround, but still haven't understood what's happening13:25
dandraderok, so I'm not crazy afterall13:26
mzanettiif gsettings changes and causes the loader to load something different in the same event loop pass, the loader won't free its memory13:26
mzanettiworkaround is to decouple it with a timer of interval 1 or similar13:26
dandradermzanetti, so the workaround is to have a timer to trigger the Loader.source change?13:26
mzanettiwell, actually, lemme find a branch13:26
dandraderditto13:26
dandradermzanetti, so you don't have a merge proposal with that workaround?13:27
mzanettidandrader, this one would fix it too: https://code.launchpad.net/~mzanetti/gsettings-qt/queued-processing/+merge/24822413:27
mzanettiIMO this one would be nicer as a timer in unity13:27
mzanettionly problem is: I haven't understood why this is happening13:27
dandradermzanetti, this explains why I don't get this issue when playing with "make tryShell"13:30
dandraderor when running the qml tests13:30
mzanettiyeah... this was driving me nuts too a while back13:30
dandradermzanetti, this is blocking the manual testing of my "app lifecycle logic in  unity8" work. I will have to add the TImer workaround to its branch...13:32
mzanettimhm...13:33
mzanettiI still wonder if we shouldn't land that gsettings patch...13:33
mzanettilarsu, hey, we're still suffering from this ^^13:33
mzanettiwdyt, should/can we land this? https://code.launchpad.net/~mzanetti/gsettings-qt/queued-processing/+merge/24822413:33
mzanettiif so, I'd fix the test that's failing because of the change13:35
pstolowskitsdgeos, is the editIcon already available on the phone?13:41
tsdgeospstolowski: it's part of the theme, so yeah13:42
tsdgeosshould be13:42
pstolowskiok13:42
dandraderMacSlow, wow, so we might finally getting a stable shellRotation AP test!!! what was the fix?14:20
MacSlowdandrader, the "granularity" for the faked sensor-events for "right -up" orientation seems to have been too rough14:22
MacSlowdandrader, currently at 50 successful runs in a row for "right-up"14:22
* MacSlow really hopes this is the final issue to solve for that branch!!!14:23
MacSlowdandrader, and that in turn caused qtubuntu-senros to sometimes not recognize the orientation change, because there was not enough change in between14:26
dandradermzanetti, the workaround bazaar.launchpad.net/~dandrader/unity8/app-state-handling/revision/177114:29
mzanettiyeah... looks similar to what I had14:30
mzanettidandrader, IMO we should rather land the gsettings-qml patch than this.... but good you have a way to be unblocked14:30
dandradermzanetti, I think too, but having the unity8 workaround unblocks me and removed a dependency on a different project14:31
mzanettiack14:31
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
mzanettipstolowski, tsdgeos: seems we have conflict between silo 7 and 3915:15
tsdgeosfight to death, i'm using popcorn!15:15
tsdgeosmzanetti: you mean code confligt?15:16
mzanettiI don't think any of the both has higher priority than the other15:16
mzanettilet's QA pick one and rebuild the other15:16
tsdgeosmzanetti: or "conflict because we can't land the same thing twice"?15:17
mzanettiyeah... if we'd land both as is, the merge between the two wouldn't be tested15:17
tsdgeoswhich kind of makes sense15:17
pstolowskimzanetti, tsdgeos silo 7 is older ;)15:17
mzanettiack15:17
mzanettiwfm15:17
tsdgeosyeah15:17
mhall119mzanetti: Saviq: what do I need to do to compile and run Unity8 on utopic?15:27
mzanettigood question... probably not possible15:28
mzanettimhall119, what's the scope? just want to try trunk?15:28
mhall119mzanetti: yes15:29
Saviqmhall119, you need to upgrade to vivid ;)15:29
mhall119Saviq: anything short of that?15:29
mhall119the unity-team PPA doesn't seem to have anything in it anymore15:29
mzanettiyeah... you don't want to build all the deps that updated15:29
Saviqmhall119, backport all the packages we rely on to utopic15:29
mzanettimhall119, a vmware vivid install would do too15:30
mhall119so then, I guess I should update the docs to say devs must be on vivid15:31
tsdgeosyeah15:32
tsdgeosjust vivid-ize yourself and the docs15:32
mzanettimhall119, I would go as far as saying that devs must be on the latest devel15:32
mzanettionce wily starts rolling it probably won't compile for long on plain vivid any more15:32
mhall119mzanetti: that's going to make casual contribution nearly impossible15:33
mzanettimhall119, so the thing is, there are many libraries outside of unity which just get updates because unity requires them...15:34
mzanettiyou can either update/backport those libraries or use the latest archive15:35
mhall119mzanetti: I know that backporting those into a PPA is extra work for you guys, but it will make it easier for others to get involved and contribute15:35
mzanettiI'll think about how much it would be... Can't promise anything though.15:36
mhall119thanks15:36
mhall119it's a tradeoff, I know, and if it's not worth it then that's fair15:36
dupingpingLook http://vimeo.com/12650531615:38
mzanettimhall119, it would require backporting mir and it's dependencies too... at which point you'd have like half of the archive in that ppa...15:40
mzanettiits15:40
mzanettigrr15:40
mhall119mzanetti: ok, if we can't backport can we get some documentation about how to set it up and run it in a VM? Would the dev have to write the code in the VM or on their host machine? Can we provide regularly updated VM images?15:41
mhall119anything to lower than barrier to entry15:41
mzanettimhall119, you just grab the latest image from the official downloads and do-release-upgrade --devel on it15:42
mzanettias of that, just apt-get upgrade to stay up to date15:42
mzanettieasiest is probably if you develop in there. but nothing prevents you to open the source from a shared folder on the host system and edit away there15:44
mhall119mzanetti: we're still talking 30-60 minutes of work before somebody can even start poking around in the code :(15:45
mzanettimhall119, well, you can always build the revision that is released on your system15:45
mhall119mzanetti: but that wouldn't help someone wanting to contribute new features or fixes15:46
mzanettibut how would someone contribute new things if he doesn't want to run the new things?15:46
mhall119I want to encourage community contributes to get involved in Unity8 development15:46
mhall119mzanetti: they want to run the new Unity 8, probably in a window under Unity 7, they don't care about the rest of the OS15:47
mzanettithis is quite a core piece of the software stack we're talking about... it's not an app, it's the thing where apps run inside of15:47
mhall119mzanetti: again, I'm not saying you have to backport, I'm just looking for ways to make it easier to get started contributing15:48
mzanettiI think we do publish nightly images, don't we?15:49
mhall119any additional steps, no matter how logical and necessary they are, are still going to deter some folks15:49
mhall119mzanetti: of the base distro yes, not something that already has the developer tools and all15:49
dandradergreyback, https://code.launchpad.net/~dandrader/qtmir/detach-state-from-focus/+merge/25864815:49
mzanettimhall119, from the core distro, an apt-get build-dep unity8 should work15:50
greybackdandrader: 894 lines, wow :)15:54
dandradergreyback, mostly because of the test I think15:54
greybackdandrader: yeah it's not as bad as I expected15:55
dandradermzanetti, the number of approved branches is getting unwieldy15:57
mzanettiI know...15:58
mzanettidandrader, we have 2 silos waiting for QA testing15:58
dandradermzanetti, any news no wily repos?15:58
dandraders/no/on15:59
mzanettidandrader, no... but I expect to release next week to wily15:59
=== dandrader is now known as dandrader|lunch
=== alan_g is now known as alan_g|EOW
=== dandrader|lunch is now known as dandrader
=== dandrader is now known as dandrader|afk
=== mhall119 is now known as mhall119|afk
=== dandrader|afk is now known as dandrader
=== mhall119|afk is now known as mhall119

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