=== elopio_ is now known as elopio
=== duflu_ is now known as duflu
tsdgeosmzanetti: ping08:16
tsdgeosmzanetti: goint to need a qmluitests VM detached when you have time to try to repro that test that is now suddenly blocking us from out of the blue08:22
mzanettitsdgeos: hey08:42
mzanettitsdgeos: I'll get you the vm now08:42
mzanettitsdgeos: ps-trusty-server-amd64-1 it is08:46
mzanettikgunn: o/08:46
kgunnmzanetti: \o yo!08:47
mzanettikgunn: fixed the last glitches08:47
kgunncool...i'm all set up...i'll resync this afternoon08:47
* tsdgeos does the evil eyes at Saviq09:29
* tsdgeos does the evil eyes to himself09:30
tsdgeosmzanetti: https://code.launchpad.net/~aacid/unity8/fix_test_dash_shown/+merge/20189909:35
tsdgeosmzanetti: you can bring back the vm to the pool09:36
* mzanetti wonders why tsdgeos doesn't do the evil eyes to him09:36
mzanettivm is back in pool09:36
tsdgeosmzanetti: you didn't do nor review the change, no?09:36
mzanettiah right... no, I didn't do that change09:37
tsdgeosyou're saved from evil eye-ing09:37
Saviqtsdgeos, uh oh, sorry09:37
tsdgeosSaviq: there's a few more checks against undefined that should be probably null09:37
tsdgeosbut they are == or != and that still works09:37
tsdgeosit's === or !== that doesn't09:38
mzanettitsdgeos: hmm... but how could this have gotten merged? I mean, shouldn't those tests have failed already with the bad commit?09:38
tsdgeosmzanetti: because it's a timing thing09:38
tsdgeosthat function there09:38
tsdgeoschecks that everything we're going to need is loaded09:38
mzanettioh, I got it09:38
tsdgeosby making sure they are there09:38
tsdgeosi guess we were unlucky and the merge ran on a fast machine :D09:38
mzanettiyeah, the vm's performance is quite variable depending on how many jobs are running09:39
seb128MacSlow, hey, could you review https://code.launchpad.net/~sam92/notify-osd/focusfollowdefault/+merge/198541 as well?10:29
mzanettigreyback: hi10:31
mzanettigreyback: just merged trunk and as I feared,everything broke badly10:31
mzanettiTaskController::startApplication appId='dialer-app' FAILED10:31
mzanettiAsking Upstart to start application 'dialer-app' failed10:31
mzanettiany idea? ^10:31
mzanettigreyback: if I try for a second time it crashes: (null):dbus_error.c:69: Unhandled error from nih_dbus_error_raise: Name "com.ubuntu.Upstart" does not exist10:32
MacSlowseb128, ok10:33
seb128MacSlow, thanks10:33
MacSlowseb128, but mpt also approved it already... but it's good to go from my side too10:34
MacSlowseb128, I can top-approve it10:34
MacSlowseb128, done10:34
seb128MacSlow, thanks10:34
mzanettigreyback: forget about the above... no idea what went wrong, now it works - sorry about the noise10:36
=== _salem is now known as salem_
=== pete-woods is now known as pete-woods|posti
Saviqmzanetti, hey, you happy to show what you have for right edge?11:25
mzanettiSaviq: yip yip11:26
Saviqmzanetti, what do I do?11:26
mzanettiSaviq: when did you last flash your phone?11:26
mzanettiSaviq: do you already have the sidestage code from ricmm on your device?11:26
Saviqmzanetti, I don't think so, checking11:26
SaviqI'm at image 12711:27
mzanettiSaviq: hmm.. no idea which one that is11:27
mzanettiSaviq: better flash it with a fresh one to be on the save side11:27
Saviqmzanetti, 0.2+14.04.20140108.1-0ubuntu111:28
Saviqmzanetti, will do, thought it was "you should have an old one"-type of a problem11:28
mzanettiSaviq: I just merged everything with trunk11:28
mzanettiSaviq: if your image is an older one you could use my branches before the "merge trunk" commit11:29
Saviqmzanetti, upgrading now11:29
mzanettiSaviq: so, you need this branch: lp:~mzanetti/unity-api/new-screenshot-and-focusing-api unity-api11:29
mzanettiit's enough if you copy the Application*.h files to /us/include/...11:30
mzanettithen you need this one: bzr branch lp:~mzanetti/unity-mir/screenshotting-focusing-api unity-mir11:30
mzanettineed to compile + install11:31
greybackmzanetti: hey, sorry was away, that error sounds like upstart broke a bit, as that message says the dbus message to upstart failed to send11:31
mzanettiand then this one: bzr branch lp:~mzanetti/unity8/appmanager-rework11:31
mzanettiyou can run_on_device this one ^11:31
mzanettigreyback: no worries. I still don't know what happened, at some point it was working again... the first 2 reboots I always got this error11:32
mzanettigreyback: but can't reproduce it any more. so everything fine I guess11:32
greybackmzanetti: I had it once before too11:32
Saviqricmm_, greyback https://bugs.launchpad.net/ubuntu/+source/unity-mir/+bug/1269414 you saw that?11:39
ubot5Ubuntu bug 1269414 in unity-mir (Ubuntu) "Sound is cut when another application is launched" [High,Triaged]11:39
Saviqtsdgeos, mzanetti, can you please do https://code.launchpad.net/~mterry/unity8/hide-greeter-on-focus-request/+merge/201817 ?11:55
Saviqwe should add an ap test for that...11:55
mzanettiSaviq: ack11:57
kgunngreyback: so i was trying to build unity-mir on my device...which i did yesterday, i just updated11:59
kgunnthe branch....but now, when i hit qmake...it spews the help12:00
kgunnever seen this12:00
ricmm_kgunn: migrated to cmake12:00
ricmm_build with dpkg12:00
greybackyeah, we switched to cmake12:00
greybackneed to see if I can X-compile it now12:01
mzanettigreyback: if you manage to, let me know12:01
mzanettilast time I tried click chroot didn't like my system at all12:01
greybackmzanetti: will do12:01
Saviqmzanetti, FOOK 6k diff? think we could split it up in smaller chunks?12:04
mzanettiSaviq: no12:04
mzanettiSaviq: and it'll get about double the size12:04
mzanettiSaviq: because we're not supposed to break anything12:04
mzanettiSaviq: so the new sidestage has to be in this too12:04
mzanettiSaviq: but most of it is dropping old FIXME's :)12:05
Saviqmzanetti, we're hammering out the side stage case this week, too, so hopefully there will be direction (or there might be an interruption anyway)12:05
mzanettiSaviq: yeah, I heard. vesar is busy prototyping it. looking forward to it12:06
=== MacSlow is now known as MacSlow|lunch
tsdgeosmzanetti: you doing mterry's review, then, right?12:06
mzanettitsdgeos: yes, can do12:06
=== pete-woods|posti is now known as pete-woods
mzanettiSaviq: where did you see a 6k diff btw?12:20
Saviqmzanetti, unity812:20
mzanettiSaviq: https://code.launchpad.net/~mzanetti/unity8/appmanager-rework/+merge/19981512:20
mzanetti1.5, no?12:20
Saviqmzanetti, against your branch12:20
Saviqmzanetti, hmm wait12:21
Saviqmzanetti, merged on wrong branch12:21
Saviqmzanetti, 1.5k is fine12:22
mzanettiSaviq: yeah. it's ok. so this is only the Phone stuff... the tablet will come in with another 500 - 1000 lines approx12:22
mzanettiSaviq: does it work?12:26
Saviqmzanetti, not yet, couldn't cross-build :/12:26
=== alan_g is now known as alan_g|afk
tsdgeosMirv: if you have not started a qtdeclarative rebuild12:31
tsdgeosyou want to include this one12:31
tsdgeosMirv: https://codereview.qt-project.org/#change,7568912:31
Mirvtsdgeos: ok. well I "started" but KDE is eating all the builders so it has not started yet. reuploading with that added.12:32
tsdgeosMirv: awesome, it should fix that crash in i38612:32
Mirvtsdgeos: do you have qtdeclarative release branch git checked out? could you check if I should update from the cb78684ae or if the 10 commits on top of that are not crucial? like by looking at the commit descriptions.12:34
Saviqxnox, hey, something changed and Qt cmake modules are not found any more when cross-building :/ any idea?12:34
tsdgeosMirv: sure, let me check12:34
Mirvto me maybe thiago's d580411ca looks important, but not much else12:35
Mirvon the other hand if it doesn't look too important, it shouldn't hurt either...12:35
tsdgeosMirv: 2f9099443d9acd6583e92785afbb38b2e4dcbfd5 looks like something we want to have (no clue what it does but sounds complicated, i.e. could break/fix stuff :D)12:36
tsdgeosMirv: if updating to current tip of release is not hard i'd do it12:36
Mirvok then :)12:37
Saviqmzanetti, ApplicationImage.qml: File not found?12:40
Saviqdamn ^W12:40
mzanettisalem_: where do you get this?12:40
mzanettiSaviq: ^ (sorry salem_)12:40
Saviqmzanetti, hmm it didn't build, let me12:41
Saviqmzanetti, ah that's "real" u812:41
mzanettiSaviq: ?12:41
Saviqmzanetti, distro u8 with new libunity-mir112:41
Saviqmzanetti, so makes sense12:41
mzanettiah ok, yeah12:42
mzanettiI dropped AplicationImage.qml12:42
Saviqmzanetti, yup12:42
mzanettiSaviq: btw... u8 trunk won't start with my libunity-mir, which causes libhud to go to 100% cpu12:43
Saviqmzanetti, mhm12:43
mzanettiSaviq: so make sure to reinstall unity-mir from the repo before leaving the device unattended for a while12:43
mzanettior it'll get hooot12:43
Saviqmzanetti, :)12:43
mzanettiactually I should file a bug about it12:44
cwayneSaviq: ping: are we eventually planning on having the unity automatically detect the screen resolution settings? or will we always need to drop a file in /etc/ubuntu-touch-session.d/?12:59
Saviqcwayne, you mean DPR / GRID_UNIT_PX?13:00
cwayneyeah, specifically GRID_UNIT_PX13:00
Saviqcwayne, those are arbitrarily set, so AFAICT there will always be a per-device file there13:00
Saviqcwayne, 'cause it's not really DPI13:00
cwayneright, i was worried about that :)13:01
cwayneSaviq: where are those files actually read?13:01
cwaynei.e. is it just unity8 that actually reads the contents of those files?13:01
Saviqcwayne, no, it's being exported in the session I think13:01
Saviqmzanetti, can't launch apps, and run_on_device complains about pkilling processes that're not running?13:02
Saviqmzanetti, ah wait, got dialer13:02
Saviqmzanetti, what's the chopping at the top?13:03
Saviqmzanetti, is the "I can swipe enough to go back to the app when reached the end of the stack?13:06
Saviq+designed behaviour?13:06
=== MacSlow|lunch is now known as MacSlow
mzanettiSaviq: no. that's todo still13:14
mzanettiSaviq: design is not exactly sure yet where the point of no return shall be13:14
Saviqmzanetti, you mean todo to fix?13:14
Saviqmzanetti, ok, how do we fix the cropping?13:15
mzanettiSaviq: what cropping?13:15
Saviqmzanetti, - yOffset is not needed any more13:16
Saviqmzanetti, application.cpp in libunity-mir13:16
Saviqmzanetti, the screenshots coming from mir are now the actual size of the surface13:16
mzanettiSaviq: re launching of apps: currently you need to launch apps with the launcher as we didn't merge the fix to unity-scopes yet13:17
Saviqmzanetti, 'stood13:17
mzanettiSaviq: I don't see any cropping13:17
Saviqmzanetti, merge trunk libunity-mir13:17
Saviqmzanetti, you'll see it13:18
ricmm_Saviq: hi13:18
=== ricmm_ is now known as ricmm
Saviqricmm_, FUCK OFF13:18
mzanettiagain? I did that today already13:18
mzanettidamn. conflicting again13:18
ricmmso yea yOffset is not needed as app buffers are now the size of the render area requested13:19
ricmmin accordance to the available geometry in the selected stgage13:19
Saviqricmm, in the selected what?\13:20
ricmmSaviq: sthfgage13:20
Saviqmzanetti, you're being missed here13:20
mzanettiam I? why is that?13:21
Saviqmzanetti, just because ;D13:21
Saviqmzanetti, it's late, I didn't sleep much, hungover, don't listen to me ;D13:21
mzanettiis the beer any good down there?13:21
mzanettilol. do you have the drunken voice again?13:22
Saviqmzanetti, it's *CHEAP* to start with13:22
Saviqnot any more ;) that was Monday :D13:22
Saviqmzanetti, a pound a beer, 3-course dinner with a bottle a wine a head, 16 pounds13:22
ricmmSaviq: hi13:23
Saviqand they give you half a cow13:23
Saviqricmm, FUCK OFF13:23
Saviq↑↑ he's sitting →→13:23
* mzanetti just realizes that the app suspend whitelist will mess with the screenshot stuff we're doing13:25
mzanettii.e. the screenshots are most likely always outdated13:25
ricmmoh not at all13:26
ricmmit still takes screenshots of suspended app's buffers13:26
ricmmand technically the app wont have changed if its suspended13:26
mzanettiyeah, but the app is not suspended so it will change after the screenshot has been taken13:26
mzanettithat's what I mean... the whitelist13:26
ricmmoh the whitelist13:26
Saviqmzanetti, the two last (bottom ones) screenshots get swapped sometimes13:27
Saviqmzanetti, you know about that?13:27
ricmmso the whitelisting is a hack13:27
mzanettiSaviq: yep... that will implicitly be fixed with the point of no return13:27
ricmmlets not write special code to support that outdate screenshot of music app13:27
ricmmultimately it wont use whitelisting13:27
Saviqmzanetti, ok13:27
ricmmbut the media hub13:27
mzanettiricmm: ack13:27
Saviqmzanetti, lookin' good!13:39
=== alan_g|afk is now known as alan_g
mzanettiSaviq: nice :)13:40
Saviqmzanetti, noticed that fullscreen apps might need to be handled differently13:40
Saviqmzanetti, or are we not meant to put the toolbar on them?13:40
mzanettiSaviq: yeah, that's one of the questions I had for design13:40
mzanettithey don't know a solution yet13:40
mzanettithe prototype and mocks only dealt with non-fullscreenones13:40
Saviqmzanetti, no I mean that weren't we supposed to put the panel on the non-fullscreen ones?13:41
mzanettiSaviq: hmm... dunno. I'd think that looks weird, but who knows. maybe that'll be designs answer13:41
Saviqmzanetti, we will probably have to "composite" the OSK on the apps anyway13:42
mzanettiwell, it solve some more issues tho13:42
Saviqmzanetti, yeah, and I thought videos were doing that (panel)13:42
mzanettiyeah, martins videos did13:42
mzanettiSaviq: vesar's prototype doesn't13:43
Saviqmzanetti, I think the video is more of what we  should be looking at, but yeah a confirmation is needed13:43
mzanettihmm... no that was the mistake I did at first... I implemented the video while the prototype is the one where the changes from the reviews are in already13:44
mzanettibut we'll see... I have another design feedback call either today or tomorrow - depending on how busy things are on the southern hemisphere13:45
Saviqmzanetti, right, visual vs. behavioral13:45
Saviqmzanetti, great13:46
mzanettibut overall it's quite cool isn't it?13:46
mzanettimakes the right edge so much more useful13:46
mzanettiSaviq: I merged with trunk again, still nothing clipped off13:50
Saviqmzanetti, there's been changes to platform api and qtubuntu that changed that13:51
Saviqmzanetti, so apt-get update/upgrade should show you that13:51
* mzanetti wishes he waited for another hour with the whole updating/merging thing :D13:52
Saviqmzanetti, conflicts?13:52
mzanettiyeah, lots of them13:52
mzanettibut merged already... just seems I'm not up to date again13:53
mzanettistill good after an upgrade... /me tries dist-upgrade13:56
mzanettinope, nothing in there13:56
mzanettiSaviq: did you manually install something on top? ^14:01
karniHey guys. Is the Y axis in Qt layouts pointed downwards from upper left corner, or upwards from lower left? I seem to be calculating correct values for vertical journal row spacing, but with the opposite sign. second.y - (first.y + first.height)14:02
mzanettikarni: 0, 0 -> upper left14:02
karnimzanetti: Thank you14:02
Saviqmzanetti, no, I flashed latest trusty-proposed and installed your unity-mir, built unity814:03
mzanettistrange... looks fine here14:03
karniSaviq / tsdgeos: https://code.launchpad.net/~unity-team/unity8/new-scopes-vj-integration/+merge/20193214:11
karnitsdgeos: Thank you for your e-mail :)14:11
tsdgeoskarni: no worries14:11
* karni already notices copyright header wrong year heh14:11
Saviqkarni, measuredTwoLinesHeight → out14:13
karniSaviq: ow WOW sorry, that wasn't supposed to land there. yes, we talked about it14:15
* karni removes14:15
Saviqkarni, no worries14:15
Saviqkarni, I'd also s/columns/maxColumns/ probably14:16
karniSaviq: The thing is, that's useful for ResponsiveGridView. In this case, we never make the number of columns less than "columns" set, because of large column spacing or other factors.14:17
karniSaviq: That's why I thought naming it just columns would be more appropriate, as it always matches maxColumns. Thoughts?14:17
Saviqkarni, not sure I followed14:18
karniSaviq: ResponsiveGridView - when you set maxColumns = 1000, it may happen number of columns will end up 23, because of delegate item with and column spacing14:18
Saviqkarni, yup, why is here different?14:19
karniSaviq: in ResponsiveVerticalJournal - whatever you set the columns (i.e. maxColumns) value, that WILL always be the value of columns14:19
Saviqkarni, ah14:19
Saviqkarni, I don't think that should be the case14:19
karniThat's where my question came from if we shouldn't limit the number of columns from arbitrary values14:19
Saviqkarni, the number of columns should be dynamic14:19
karniin which case I'd agree we should name it maxColumns instead14:19
Saviqkarni, we *know* the size of the cards (so column width)14:20
Saviqkarni, what we don't know is spacing or column number14:20
karniI ended with conclusion we don't know delegate size, but what you're saying makes sense.14:21
Saviqkarni, we do14:21
karniThat'll need fixing, and with that, columns -> maxColumns makes sense14:21
Saviqkarni, it's small, medium or large14:21
karnimy bad!14:21
Saviqkarni, no worries14:21
karniSaviq: -> Needs fixing :)14:21
tsdgeoskarni: https://code.launchpad.net/~aacid/unity8/journal_misc_fixes ?14:22
Saviqkarni, done14:22
* karni looks14:22
tsdgeossince daniel is back pained14:22
tsdgeosi'll let you review it14:22
tsdgeoswho reviews https://code.launchpad.net/~aacid/qtubuntu-camera/no_priv_headers/+merge/201933 now that gusch is gone?14:23
tsdgeosMirv: ↑↑↑14:23
tsdgeossil2100: ↑↑↑ ?14:23
tsdgeosrsalveti: ↑↑↑↑ ?14:23
Saviqtsdgeos, start with bfiller14:23
karnitsdgeos: Jenkins failure is expected on that branch, right?14:24
Saviqkarni, one interesting thing to know would actually be whether cards should be resized or spaced out14:24
tsdgeoskarni: i retriggered the build14:24
tsdgeoswe have a pretty bumpy CI14:24
karnitsdgeos: approved14:25
karniSaviq: Correct, I came through that, gave it all much thought. Should I consult Katie?14:25
dednickanyone know if it's possible to detect a click on a qml area, but not consume the events for it? ie. it gets passed still gets passed on.14:25
tsdgeosdednick: MouseArea has a property for that afair14:27
karnidednick: perhaps http://qt-project.org/doc/qt-5.0/qtquick/qml-qtquick2-mousearea.html#propagateComposedEvents-prop14:27
sil2100tsdgeos: right, bfiller is the right person, but I guess I can take a look at this as well14:27
dednicki've tried, but seems to get messed when there are mouse areas at a different/deeper parent level...14:28
mzanettiSaviq: added the "point of no return" as I think it makes sense. fixes the weird flipping issue you had14:29
tsdgeosdednick: standup?14:31
dandradertsdgeos, still no feedback from design regarding half-filled organic grids/14:52
tsdgeosdandrader: nope :_/14:52
tsdgeosSaviq: can you get katie to answer the email we sent?14:52
tsdgeosmzanetti: you're preview man? you doing https://code.launchpad.net/~diegosarmentero/unity8/purchase-service/+merge/201807 ?14:53
dandradertsdgeos, or I would exercise common sense and do what gallery does14:53
mzanettitsdgeos: ack14:53
tsdgeosdandrader: ok, let me finish what i'm doing and i'll have a look at what the galllery does14:54
elopiogood morning everybody.14:56
elopioI now need help with the cmake task to install the fake scopes and previews. Can somebody help me?14:57
mzanettimterry: hey, would you agree that an AP test makes sense here? https://code.launchpad.net/~mterry/unity8/hide-greeter-on-focus-request/+merge/20181715:03
mterrymzanetti, uh, sure.  The bug I fixed I don't think needs it (because the bug is in how we respond to events from upstart), but that code in general could use it (for testing integration boundary with upstart)15:04
mterryI'll look at it15:04
mzanettimterry: the functionality and the unit test look good to me. I'll approve once we have an AP test. Just drop it into the other app_lifecycle ap tests15:05
=== alan_g is now known as alan_g|afk
=== alan_g|afk is now known as alan_g
mterrymzanetti, do the autopilot tests not have an example of locking the screen? ...  I'm not sure how to induce a fake power button press15:49
mzanettimterry: hmm... good question15:49
mzanettimterry: how does it actually work?15:51
mzanettimterry: I remember we were using Qt.KeyPower at some point, but that seems gone15:51
mterrymzanetti, how does locking actually work?  we get a signal from powerd that the display setting is changing15:52
mzanettimterry: well, I guess in theory we'd need to change the display setting then15:52
mterrymzanetti, seems like something we should mock15:53
mzanettimterry: not really in autopilot tests15:53
mzanettiideally autopilot would not use any mocks at all15:53
mterrymzanetti, I get that they are integration tests, but I feel like actually changing hardware settings might be too much?15:54
mterrymzanetti, cameras watching the devices and all might not like the screen going out.  Though I suppose it would only be for a moment15:54
mzanettiand that's what we're striving for. the only reason we're using some mocks now is because we don't have an applicationmanager for the desktop yet15:54
mzanettimterry: the more I think about it, the more I think we should do that... its the proper thing to do15:56
mterrymzanetti, actually change display brightness?15:56
mzanettimterry: yes.15:56
mzanettimterry: anyways, that's not really related to your tests here15:56
mzanettimterry: so why don't you just not unlock it after starting up unity and unlock it directly by starting some other app?15:57
mterrymzanetti, well.  That's one test.  That test always worked though.15:57
mterrymzanetti, the thing that broke was focusing an already open app while the greeter was up15:57
mzanettiah, I see15:57
mterrymzanetti, but...15:57
mzanettihmm... so yes. my opinion is that we shouldn't mock in autopilot tests15:58
mterrypowerd-cli should be able to futz with screen for us15:58
mzanettiyeah, that sounds like the proper thing to do15:58
elopionot that anyone was asking, but /me agrees :)15:59
elopiomzanetti: can you help me with the installation of the fake scopes ?15:59
mzanettielopio: hi15:59
mzanettielopio: what up? :)15:59
mzanettiwhat exactly do you want to install?16:00
elopiomzanetti: I need the scopes to be copied to builddir/install/lib/{arch}/unity8/qml/Unity/16:00
elopiobut I don't know anything about cmake.16:00
mzanettielopio: afaik they should already be packaged up into unity8-fake-env16:01
tsdgeosneed to reboot, something broke in here16:01
elopiomzanetti: some of them, but not the ones I need.16:01
elopiobuilddir/tests/mocks/Unity has more files than16:02
elopioso I suppose I'm missing the install function in tests/mocks/Unity/CMakeLists.txt16:04
elopiobut I don't know what to write in it16:04
mzanettielopio: oh boy... that's going to be tricky now :/16:04
mzanettielopio: so the thing is this:16:04
mterrymzanetti, hm.  powerd-cli doesn't let you turn off display, only on.  Do you happen to know a better way?16:04
mzanettielopio: ideally we wouldn't use any mocks at all but we need to for now. that's why we install some of them16:05
mterrymzanetti, sorry if you replied, I had connection issue16:05
mzanettielopio: the problem now is, if you just install your mock along with the others it will always be used for all tests16:05
mzanettielopio: so we'd need a way in autopilot to tell which mocks to use and which not16:05
mzanettimterry: no, I didn't reply yet. still thinking about something16:06
elopiomzanetti: in this case, I think that's a must. Otherwise, all the tests will access the u1 production servers.16:06
elopiomzanetti: we will cover the tests for the real scopes and servers in a different branch.16:06
mzanettiSaviq: help me :)16:06
mterrymzanetti, no rush, sorry16:07
elopiomzanetti: oh, but btw, what you said is doable. We can tell the tests to use just some of the fakes, but for that we would need them to be in a separate folder.16:10
elopioI suppose cmake can do that.16:10
mzanettielopio: yeah, cmake can do that16:12
mzanettimterry: really weird... powerd-cli help says <on|dc> and the descroption of that says "Off, On or Don't Care"16:13
mzanettimterry: does that seem like a bug to you too?16:13
mterrymzanetti, maybe.  It has always been like that.  There is a documentation bug for certain.  Maybe also a missing functionality bug16:13
mzanettimterry: hmm... interesting... if I manually suspend the screen and then call powerd-cli display on it turns on16:14
mzanettimterry: but as soon as I kill powerd-cli again, it goes off again16:14
mzanettimterry: so off wouldn't work anyways as unity still holds the "on" state afaiu16:15
dandradermzanetti, a powerd-cli command is valid just as long as it's running16:15
mterrymzanetti, yeah.  powerd-cli will install an 'on-hold' that goes away when it does16:15
dandradermzanetti, so you have to keep it running in a separate tab/ssh session16:15
mterrymzanetti, fair...16:15
mzanettiso I guess that architecture doesn't allow an off if someone else holds a on-lock16:15
mterrymzanetti, and that's why it doesn't accept off as an argument, sure16:16
mzanettistill a doc bug there I guess16:16
mterryAnd yet, the user can press a button and the android drivers deliver a signal to powerd that a screen turn off should happen16:17
mterrymzanetti, we could emulate/create that driver event?16:17
mzanettimterry: yeah... either that, or there is some admin mode to powerd where you can kill other locks16:17
mzanettimterry: I guess you need to ask someone from phonedations16:18
mzanettielopio: check out tests/mocks/Unity/Application/CMakeListsts.txt16:19
mzanettielopio: there are to install() statements at the end16:19
mzanettielopio: add the same to tests/mocks/Unity/CMakeLists.txt for the target FakeUnityQml and put some other directory in there16:20
elopiomzanetti: let me give it a try.16:20
mzanettielopio: along with that, please group the add_subdirectory() from the top and the bottom of the file16:22
elopiomzanetti: it doesn't work.16:27
elopiothe libFakeUnityQml.so gets copied to the install lib, but I suppose we are still missing some files.16:27
mzanettielopio: yeah, the qmldir file16:28
elopioah, lets see16:28
elopiomzanetti: thank you!16:32
elopionow, I think we need to discuss a little more about that never use fakes policy.16:33
elopiowe are actually aiming at closing external network access from the CI lab, at least for the tests run in merge proposals.16:33
karniSaviq: tsdgeos: I sent Katie an e-mail. Without her opinion, I don't know whether I should spread the cards or resize them depending on available space in VerticalJournal. I'll shelve that untill I get a response, and will look into why Scott couldn't surface the summary field in the Card header.16:34
elopioso many of the troubles we have now is because we didn't disallowed it when we started writing tests.16:34
tsdgeoskarni: i think that's on the spec16:34
karni"Width of cards is determined by art width."16:35
karnisorry, wrong row16:35
tsdgeosor was16:35
karni"Cards within a category have equal width. Card widths may be one of the 3 pre-defined sizes: S, L or M."16:35
tsdgeosi had a question there with an answer16:35
karninot much besides that16:35
tsdgeosand i think it was marked as resolved16:35
tsdgeosso it's not shown anymore16:35
=== dandrader is now known as dandrader|afk
tsdgeosdon't know if you can get them back16:35
karniSaviq: Do we have revision with old comments of the Future Dash spec? (if you're still around)16:36
mzanettikarni: seems he isn't any more. they closed down the location16:36
karnimzanetti: ack, np16:36
karnitsdgeos: honestly, it it says card widths maybe on of of the 3 pre-defined sizes, I guess I shouldn't resize them, space them out instead16:37
karnithat would also match the behavior of ResponsiveGridView, I believe16:37
mzanettikarni: I'd say yes16:38
karniI just re-read what I wrote. I clearly can't type heh, sorry ;)16:38
tsdgeosdandrader|afk: but the gallery does exactly my thing does16:39
=== dandrader|afk is now known as dandrader
elopiomy branch is ready for review.17:39
elopiomzanetti: can you?17:40
mzanettielopio: not today any more but I'll get it through the queue17:42
elopiomzanetti: that's just fine, thank you.17:43
* karni is having serious problems finding the delegateWidth of VerticalJournal, since the card-size is defined within the delegate Component17:48
karnimzanetti: You EODed yet? If not, maybe you could help me figure out one bit.17:48
=== alan_g is now known as alan_g|EOD
elopiokarni: which one is your team now? unity?18:06
karnielopio: Nope, I'm here temporarily, pushing a bit on pieces Phone Delivery needs18:07
karnito build scopes for MWC18:07
karniI'm enjoying it here :)18:07
elopiokarni: ah, nice.18:12
dobeywhere's the right place to file UX bugs as relates to global menu bar?18:18
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== dpm_ is now known as dpm
mhall119Saviq: are all the packaged needed to build Unity 8 on Saucy built and published in the PPA now?20:58
=== salem_ is now known as _salem
karni@all Unity8 won't start with packages installed from demo-stuff PPA: http://paste.ubuntu.com/6764850/23:44
* karni flashed ubuntu-system --channel trusty-proposed -b + demo-stuff setup23:44
karni /usr/lib/arm-linux-gnueabihf/unity8/qml/Unity/libUnity-qml.so: undefined symbol: _ZTIN5unity6scopes12ReceiverBaseE23:45

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