/srv/irclogs.ubuntu.com/2013/07/24/#ubuntu-unity.txt

sgx1hi. now there're two mir PPAs, which one to use? installing_prebuilt_mir_on_pc.md suggests using system-compositor-testing ppa. but i find packages in Mir staging ppa are newer.04:17
=== wgrant_ is now known as wgrant
larsuSaviq: I have unity8 running on my desktop, but can't unlock it. Is there a trick?07:43
larsuSaviq: also, good morning :)07:43
larsulol, maximizing the window allows "tap to unlock", that works. Responsive ftw!07:44
larsustill can't bring down the indicators though07:47
sil2100jibel: morning!08:09
sil2100jibel: so, it seems that the same problem as yesterday struck intel this time!08:10
sil2100jibel: https://jenkins.qa.ubuntu.com/job/autopilot-saucy-daily_release/580/label=autopilot-intel/console08:10
Saviqlarsu, use ./run08:11
Saviqlarsu, it will convert mouse to touch08:11
Saviqlarsu, ./run -h has some options08:11
Saviqveebers, the "Declined" was for next week when I'm in IoM08:16
veebersSaviq: heh, I figured that out later on when I actually bothered to look at the date :-P08:16
larsuSaviq: ah cool, but that doesn't seem to work from a build-area created by bzr bd08:17
larsu./run: 71: ./run: ./builddir/unity8: not found08:17
Saviqlarsu, -mousetouch, then08:17
larsuSaviq: awesome. thanks :)08:17
Saviqlarsu, but we got a nic ./build script that you should probably use :)08:18
larsuSaviq: last time I tried that, a very polite french guy told me I shouldn't ;)08:19
Saviqlarsu, if you want to hack on it - yes you should - otherwise it's available in ppa:ubuntu-unity/next08:20
larsuheh, thanks08:21
nic-doffaySaviq, did you see those questions I shot off to you yesterday?08:23
Saviqnic-doffay, yeah, but I'm not sure I understood them :)08:23
nic-doffaySaviq, here's an example: http://img824.imageshack.us/img824/4346/yk3k.png08:23
Saviqoh goo08:24
Saviqd08:24
Saviqnic-doffay, just clip: true it08:24
Saviqnic-doffay, also, the highlight goes out of the shape08:24
Saviqnic-doffay, you need to use the trick we have in UbuntuShapeForItem.qml08:25
Saviqnic-doffay, and that's actually all you need to do - no clipping required08:25
Saviqnic-doffay, as UbuntuShape will take care of that for you08:25
=== jasoncwarner__ is now known as jasoncwarner_
=== seb128_ is now known as seb128
jibelsil2100, I applied the same changes to intel than ATI. That should fix the problem, but it is not normal that tests consume so much memory.08:43
sil2100jibel: indeed...08:43
sil2100didrocks: ^08:43
sil2100jibel: thanks!08:43
didrocksjibel: ah, you think the memory is fragmented on it?08:44
jibeldidrocks, I increased memory limit to 5G on ATI yesterday08:44
didrocksand so tests don't start08:44
didrocksok08:44
nic-doffaySaviq, I dced there did you get those last few messages?08:44
jibeland just did the same on intel08:44
Saviq<Saviq> nic-doffay, you need to use the trick we have in UbuntuShapeForItem.qml08:44
Saviq nic-doffay, and that's actually all you need to do - no clipping required08:44
Saviq nic-doffay, as UbuntuShape will take care of that for you08:44
didrockslxc-attach: failed to open log file "/var/log/lxc/saucy-i386-20130723-0916.log" : Read-only file system08:44
didrocksjibel: hum? ^08:45
didrocks-bash: /usr/bin/less: Input/output error08:45
nic-doffaySaviq, could you tell me more about the trick?08:46
jibeldidrocks, the machine is badly damages08:46
jibeldamaged08:46
didrocksjibel: yeah, seems so…08:47
jibelI'll reboot it08:47
didrocksthanks!08:47
Saviqnic-doffay, read UbuntuTouchForItem.qml, is all you need08:47
Saviqnic-doffay, it's in unity808:47
Saviqnic-doffay, it takes any Item from QML and shapes it08:47
Saviqsil2100, didrocks, hey, how are we looking on unity8 daily release?08:53
didrocksSaviq: we are trying to get stuff releasing first, I'm working on Mir, not sure sil2100 had the time since yesterday evening to tests with trying to release unity708:53
sil2100Saviq: ah, right! Let me finish tweaking the branch and getting it in08:53
Saviqdidrocks, sil2100 thanks!08:54
didrocksjibel: are you collecting anything? I can try rebooting, we never know by chance…08:55
dandraderdednick, hey, you owe me a code review! -> http://paste.ubuntu.com/5906906/08:56
* dandrader never forgets08:57
jibeldidrocks, the machine must be rebooted electrically but for this to happen, I need a working java plugin08:57
didrocksargh, ok08:57
dednickdandrader: woops.08:57
dednickdandrader: ok. i'll get on it this morning08:58
jibeldidrocks, it is rebooting08:58
didrocksthanks jibel08:59
jibeldidrocks, sil2100 ATI seems to be back up08:59
didrocksjibel: thanks, let's see how next test behave09:00
sil2100jibel: thanks!09:01
sil2100didrocks: should I re-run QA later?09:01
didrockssil2100: yeah09:01
Saviqtsdgeos, any idea about http://pastebin.ubuntu.com/5906920/ ?09:06
Saviqtsdgeos, it's what happens in the VM I sent out last week every 3rd unity8 autopilot test or so09:06
tsdgeosnope09:07
tsdgeosbut i think i've seen it sometimes09:07
tsdgeoscrash at startup¿09:07
Saviqtsdgeos, not sure if at startup09:07
Saviqtsdgeos, as it says handling keyboard09:07
Saviqmardy, actually you said you know the xcb backend ↑↑09:07
tsdgeoswindow=0x009:08
Saviqtsdgeos, ah! but it only happens for the lockscreen tests09:08
tsdgeosi guess that's bad :D09:08
Saviqtsdgeos, so very much keyboard related09:08
Saviqtsdgeos, autopilot might be injecting something badly09:08
tsdgeosor the xcb plugin may not be "that good"09:09
Saviqtsdgeos, re: crash on startup, I'm trying to get a core on manta now09:09
Saviqtsdgeos, as 1 in some 50 ap tests we're indeed crashing on startup09:09
Saviqobviously now that I enabled cores09:10
Saviq50 tests later - no crash09:10
tsdgeos:D09:10
tsdgeosobviously09:10
Saviqtsdgeos, yeah, the crash above - definitely related to keyboard - only crashes on passphrase tests and not on startup - as I can see the thing09:11
tsdgeosoka, no clue then ;./09:11
Saviqok, /me gets some time back, biab09:12
sil2100didrocks: https://code.launchpad.net/~sil2100/cupstream2distro-config/add_check_to_unity8/+merge/176407 <- I think this should be more like it09:15
sil2100didrocks: probably the packages list will need tweaking, but that's all I could do without having the test environment09:16
didrockssil2100: as told yesterday, unity-launcher-impl-2 isn't a virtual package?09:16
didrocksah, diff not updated maybe09:16
sil2100It is, it's removed from the package list ;)09:16
didrockssil2100: yeah, ensure the package list is good :)09:16
didrocksthen, will just ack it09:16
sil2100didrocks: we'll know for sure if the list is OK after the check job is ran at least once ;p09:17
didrockssil2100: yeah, just do a manual run once it's free09:17
sil2100didrocks: AH! So we have unity8 packages in the daily-build PPA?09:18
sil2100didrocks: I have no idea why I though that they didn't build even once ;/09:18
gatoxhi, do you guys know if ./run_on_device is working??09:18
didrockssil2100: daily-build-next09:18
sil2100didrocks: eeeh, so my manual mambo-jumbo was unneecessary09:18
didrockssee ppa and dest:09:18
sil2100Right09:18
gatoxdidrocks, hi, do you  know if ./run_on_device is working??09:22
didrocksgatox: sorry, I never use that command09:22
gatoxack, thx09:22
mardySaviq: I only touched QXcbWindow :-p09:23
gatoxdidrocks, do you know who can i ask that?09:23
didrocksgatox: maybe try on #ubuntu-touch as well?09:23
nic-doffaySaviq, I just looked at UbuntuShapeForItem.qml09:32
nic-doffayI don't see what could be of help there?09:32
sil2100Saviq_: hi!10:19
sil2100Saviq_: so, I fixed up the test packages list to enable the check job for the unity8 stack, I also did a test run in jenkins10:20
sil2100Saviq_: there are some test failures though - could you and your team take a look?10:20
sil2100Saviq_: https://jenkins.qa.ubuntu.com/job/autopilot-saucy-daily_release/591/10:20
=== dandrader is now known as dandrader|afk
Saviqsil2100, uhm, that's not "some" failures - every single test failed10:53
Saviqsil2100, "09:58:41.723 INFO __init__:333 - DBusException while attempting to get PID for org.freedesktop.DBus: DBusException("Could not get PID of name 'org.freedesktop.DBus': no such name",)"10:53
Saviqsil2100, that sounds bad?10:54
sil2100Every single test? I saw there were 20 tests and 11 failed, so hm10:54
sil2100But maybe I looked wrong?10:54
Saviqsil2100, https://jenkins.qa.ubuntu.com/job/autopilot-saucy-daily_release/591/ says "22 errors"?10:54
sil2100Saviq: in overall, 11 on one machine and 11 on the other10:55
Saviqsil2100, yeah, 11 tests with multiple scenarios10:55
sil2100Saviq: so it's 11/2010:55
sil2100So, it looks as if 9 still passed?10:55
Saviqsil2100, ah, what's your guys' resolution?10:56
Saviqsil2100, and could we have recordmydesktop?10:56
sil2100Saviq: there is recordmydesktop :) Let me paste you the link to the videos10:57
sil2100Saviq: https://jenkins.qa.ubuntu.com/job/autopilot-saucy-daily_release/591/label=autopilot-ati/artifact/results/autopilot/videos/ <- from ATI10:57
Saviqsil2100, so, for the two indicator_client ones we need indicator-battery10:57
sil2100Ok, noted10:58
sil2100Will add that10:58
Saviqsil2100, except it's not available in distro...10:58
sil2100Saviq: so it's not the same as indicator-power?10:58
Saviqsil2100, no, not yet10:58
sil2100hmmm10:58
Saviqsil2100, we're moving *onto* indicator-power soon, though10:58
Saviqsil2100, so it might not make sense to do anything there10:58
Saviqsil2100, and https://jenkins.qa.ubuntu.com/job/autopilot-saucy-daily_release/591/label=autopilot-ati/artifact/results/autopilot/videos/unity8.shell.tests.test_hud.TestHud.test_hide_hud_click%20%28Desktop%20Nexus%2010%29.ogv10:59
Saviqsil2100, our window didn't fit, 'cause of the launcher...10:59
Saviqlet me see if we can make that better10:59
sil2100Saviq: so as didrocks mentioned, some errors might be related to interaction with unity7... ok11:00
Saviqsil2100, we do adapt to smaller resolutions11:00
Saviqsil2100, by means of halving the size of the window when we don't fit...11:00
Saviqsil2100, but obviously didn't take struts into account11:00
didrocksyeah, it seems struts are not taken into account11:01
Saviqdednick, what's the status of us switching onto indicator-power, btw?11:01
didrocksSaviq: on the indicator-power thing, maybe just skip the test for now with a big blinking TODO/FIXME? :)11:01
sil2100;)11:01
Saviqdidrocks, yeah, will probably skip them11:02
dednickSaviq: we need new qmenumodel from larsu before we do any new indicators. new ones are somewhat less supposed than we thought.11:03
Saviq:/11:03
Saviqdidrocks, sil2100 for the other one, can we do something? like stop unity7 for the time of the test?11:03
dednicks/supposed/supported11:03
Saviqdidrocks, sil2100, or maybe hide the launcher at least11:03
Saviqdidrocks, I get that you'd probably rather not :)11:03
Saviqbut maybe we can solve it somehow without us reducing the res even more11:04
didrocksSaviq: I would be on the "hide the launcher"11:04
didrocksthan stopping unity711:04
didrocks(as it will respawn without the upstart job)11:04
Saviqdidrocks, yeah, /me too11:04
didrocksSaviq: do you have a way to do that in a setUp() in your tests?11:05
Saviqdidrocks, probably - via a gsetting11:05
didrocksright, let me find the key, one sec11:05
sil2100+111:05
dednickSaviq: i was looking into getting them working without it, but it's quite a bit of work that will most likely be removed once unitymenumodel is around.11:05
Saviqdednick, ok11:06
dednickand all the icons are different between battery and power :( so will need to update assets11:06
didrocksSaviq: it's a relocatable key FYI11:07
didrocksSaviq: org.compiz.unityshell11:07
=== MacSlow is now known as MacSlow|lunch
didrocksunder /org/compiz/profiles/unity/plugins/11:07
didrockslauncher-hide-mode11:07
Saviqdidrocks, relocatable?11:07
didrocksSaviq: yeah, gsettings relocatable schema11:07
Saviqdidrocks, no idea what that means :D11:08
didrockshttps://developer.gnome.org/gio/2.32/GSettings.html11:08
didrocksNormally, a schema has as fixed path that determines where the settings are stored in the conceptual global tree of settings. However, schemas can also be 'relocatable', i.e. not equipped with a fixed path. This is useful e.g. when the schema describes an 'account', and you want to be able to store a arbitrary number of accounts.11:08
didrocksSaviq: just a warning the way to access it is under a specific path11:08
didrocksSaviq: this is what I'm doing in gnome-control-center:11:09
didrockspriv->unity_settings = g_settings_new_with_path (UNITY_GSETTINGS_SCHEMA, UNITY_GSETTINGS_PATH);11:09
didrockswith #define UNITY_GSETTINGS_SCHEMA "org.compiz.unityshell"11:10
didrocks#define UNITY_PROFILE_PATH "/org/compiz/profiles/unity/plugins/"11:10
didrocks#define UNITY_GSETTINGS_PATH UNITY_PROFILE_PATH"unityshell/"11:10
didrocksthen:11:10
didrocksg_settings_set_int (priv->unity_settings, UNITY_LAUNCHERHIDE_KEY, value);11:10
Saviqtsdgeos, any idea about http://pastebin.ubuntu.com/5907226/ ?11:10
didrocks(value being an int, 0/1)11:11
didrocksUNITY_LAUNCHERHIDE_KEY: "launcher-hide-mode"11:11
didrocksSaviq: does it make sense?11:11
Saviqdidrocks, kinda :)11:11
Saviqdidrocks, do I have to reset it afterwards?11:11
Saviqdidrocks, or are the machines reverted?11:12
didrocksSaviq: the whole machine is snapshotted first, and then reverted11:12
Saviqdidrocks, ok11:12
didrocksSaviq: so, don't worry about the revert ;)11:13
sil2100Saviq: you want to change that in the test's setUp?11:13
Saviqsil2100, yeah, probably11:13
sil2100Saviq: then the clean way is to add the revert back in a cleanUp call, since setUp is called on every test anyway11:13
=== dandrader|afk is now known as dandrader
Saviqsil2100, yeah, I need to revert in case people run it on their desktops11:14
tsdgeosSaviq: woot11:14
tsdgeosSaviq: where does that come from?11:15
Saviqtsdgeos, crash on manta11:15
Saviqtsdgeos, on startup11:15
Saviqtsdgeos, 1 of every 50 runs or so11:15
tsdgeoswow11:15
tsdgeosnope :-/11:15
Saviqtsdgeos, although I'm not sure the bt is correct, gdb is not working with me11:15
Saviqtsdgeos, would you care to spend a minute with the core dump?11:15
tsdgeos5.0 or 5.1?11:16
Saviqtsdgeos, 5.011:16
Saviqtsdgeos, good point11:16
* Saviq will have to try with 5./111:16
Saviq-/11:16
tsdgeosyeah i think it's better you try 5.1 first11:16
tsdgeosand if it still crashes then we put some time on it11:16
Saviqtsdgeos, yeah11:17
Saviqok, /me tries to get qt5.1 going on the device...11:17
Saviqdidrocks, sil2100 hmm http://pastebin.ubuntu.com/5907259/11:30
Saviqdidrocks, I seem to be changing something else11:30
Saviqdidrocks, as the setting changes, but not in dconf11:31
seb128Saviq, what's the issue?11:35
Saviqseb128, I'm trying to hide the unity7 launcher for unity8 tests on the machines11:38
Saviqseb128, http://pastebin.ubuntu.com/5907259/11:38
Saviqseb128, but I seem to be changing something else...11:38
seb128Saviq, how do you check it's not written in dconf?11:40
Saviqseb128, dconf-editor11:40
Saviqseb128, and the fact unity7 doesn't react :)11:40
seb128Saviq, what's the env? writting to dconf requires a connection to the service over dbus11:41
Saviqah11:41
Saviqmissing /11:41
Saviqinteresting11:41
seb128it works with it?11:41
Saviqseb128, yeah, trailing slash is needed11:41
Saviqfor new_with_path11:41
seb128that seems error prone, maybe mention it to desrt11:42
Saviqseb128, yeah, will do11:43
Saviqjeez and it segfaults with wrong schema...11:44
seb128yeah, gsettings abort() on missing schemas11:44
seb128they are considered as part of the installation11:45
Saviqdidrocks, sil2100 bug #1204480 btw11:54
ubot5bug 1204480 in Autopilot "Display should include strut information" [Wishlist,New] https://launchpad.net/bugs/120448011:54
mzanettiSaviq: pong11:57
mzanettierr... ping11:57
Saviqmzanetti, did I ping?11:57
Saviqlol11:57
mzanettiSaviq: do you have a couple of minutes to give some opinion?11:57
dandraderdednick, Saviq: do you mind if I rename the context property ApplicationArguments to applicationArguments? Otherwise it's impossible to mock it from within a QML file.11:59
Saviqdednick, +1, it should never have been uppercase12:00
sil2100Saviq: true, +112:00
dednickdandrader: fine12:00
Saviqmzanetti, gimme 5?12:00
mzanettiSaviq: sure12:00
sil2100Saviq: let's poke the autopilot guys with that12:00
Saviqseb128, ugh, how can I check if a schema is installed? list_schemas() doesn't include org.compiz.unityshell...12:08
Saviqdidrocks, ↑?12:09
=== MacSlow|lunch is now known as MacSlow
Saviqah list_relocatable_schemas :)12:11
seb128Saviq, right12:15
didrocksSaviq: back, yeah, I was doing that in g-c-c12:34
Saviqmzanetti, I remember about you, will ping when off the phone12:34
mzanettiSaviq: np12:34
mzanettitsdgeos: I'm sure you're just waiting for the next ListView bug to fix it, right?12:39
mzanetti:P12:39
Saviqlol12:40
dednickSaviq: ping12:57
Saviqsil2100, https://code.launchpad.net/~saviq/unity8/tweak-autopilot-for-daily/+merge/176679 ← can you test with a branch?12:57
tsdgeosmzanetti: lol no :D12:57
tsdgeosmzanetti: you found one?12:57
Saviqdednick, FIFO :D12:57
Saviqmzanetti, here12:58
mzanettitsdgeos: yeah... if an item shrinks and the others move with a displace animation and at the same time there is a move() operation in the model which would also trigger the displace animation it leaves a gap between items12:58
Saviqsil2100, should I maybe Recommend gi there? so that it gets installed if it isn't?12:59
mzanettiSaviq: lp:~mzanetti/unity8/dnd-and-quicklists12:59
mzanettiSaviq: I kinda fixed what I wanted to ask before, but still it doesn't look good and as vesar is on vacation I'd like someone else to brainstorm how to do it12:59
dednickSaviq: unping :)12:59
mzanettiSaviq: with that branch, try the drag and drop a bit13:00
Saviqdednick, oof13:00
tsdgeosmzanetti: mind -> explosion13:00
mzanettitsdgeos: hehe13:00
mzanettitsdgeos: ok. again, a bit slower:13:00
mzanettitsdgeos: you have a ListView with items and a displace animation13:01
mzanettitsdgeos: if one of the items shrinks, the others will move according to that animation13:01
mzanettitsdgeos: so far that works13:01
mzanettitsdgeos: now, while this moving happens, there can be a beginMoveRows()/endMoveRows() operation (especially with drag'n'drop)13:02
mzanettitsdgeos: because of the moving item, the displace animation would be triggered again to animate the reordering13:02
mzanettitsdgeos: if that collides with the already running animation it breaks somehow and just stops the animation, causing a gap between items13:03
sil2100Saviq: hmm, currently the jenkins job that allows us to test certain 'branches' is not working, so it would be hard to get this tested ;/13:03
Saviqsil2100, ok, can you review, then?13:03
mzanettitsdgeos: you can reproduce with my launcher branch: ~mzanetti/unity8/dnd-and-quicklists13:03
tsdgeosmzanetti: ok, i see, i was thinking of the LVWPH first13:03
tsdgeosand was confused as we don't have displacement animations there13:04
tsdgeos:D13:04
Saviqmzanetti, few things: a) the quicklist should disappear as soon as you move the item out if its initial position (by a minmal threshold like 2dp or something)13:04
sil2100Saviq: reviewing in a sec13:04
Saviqb) the space where the item is supposed to get back in should be the size of the item, not just a small gap, if the item is over the launcher13:05
Saviqsmall if its outside of the launcher (see unity7's behaviour here)13:05
mzanettiSaviq: not really according to design13:05
mzanettiah ok13:05
Saviqmzanetti, when I drop it back in, it's folded at first and unfolds13:06
mzanettiSaviq: yeah. thats requested from design13:06
Saviqwha?13:06
Saviqbut you're dragging it unfolded?13:06
* Saviq no get it13:06
mzanettiSaviq: let me forward the design video13:06
Saviqmzanetti, something weird is happening with the folded items at start/end - they get unfolded and refolded when you drop the item13:07
Saviqmzanetti, which, I believe, is a problem again with the small gap and unfolding item13:08
mzanettiSaviq: yeah... those are the things where I wanted to brainstorm13:08
mzanettiSaviq: but wait for the design video so we have a common starting point13:08
Saviqmzanetti, yeah13:08
dandraderdednick, updated https://code.launchpad.net/~dandrader/unity8/lp1116207/+merge/17516313:11
mzanettiSaviq: http://ubuntuone.com/5LVsZIjJUxwSB3dT1pitJa13:15
mzanettiSaviq: I was told that they agreed on a combination of the first one and the second one13:16
mzanettiSaviq: which means the unfolding when dropping and with the small space13:16
Saviqjeez why is u1 so slow :/13:16
Saviq160kBps13:17
mzanettiSaviq: but yeah... I think I need more precise specs13:17
Saviqon a 60Mbps link...13:17
mzanettiyeah... took me quite a while to upload it13:17
dednickdandrader: qmenumodel-qml is part of the install deps for unity8. :(13:17
Saviqmzanetti, people.c.c seems much better recently13:17
Saviqmzanetti, (than u1, that is)13:17
mzanettiSaviq: ok. next time13:18
* mzanetti feels slightly sick :/13:18
Saviqmzanetti, from all the beer and water you drank on Monday?13:18
mzanettiSaviq: I was fine yesterday... only started like an hour ago13:18
mzanettibut yeah... can't exclude that possibility13:19
sil2100Saviq: looking at the branch - it's not urgent, but maybe indeed a gir Recommends might be welcome, but we should be temporarily fine without that13:21
Saviqsil2100, I'll add13:21
Saviqsil2100, will gir1.2-glib-2.0 be enough?13:22
Saviqsil2100, or something for python to support gir, too?13:22
Saviqsil2100, python-gi?13:23
dednickdandrader: gr.. ok missing from control source deps, it needs to be in the build-deps then. :(13:24
sil2100Saviq: I guess we could add that too, since python-dbus recommends python-gi, which is being used by pthon-autopilot but it's best to have both13:25
sil2100Saviq: so python-gi and gir1.2-glib-2.013:26
Saviqsil2100, pushed13:27
dandraderdednick, ermm.. missing what?13:27
dednickqmenumodel-qml13:27
dednickSaviq: if an external library is needed for unity8 tests, then it needs to be in build-deps right?13:28
dednickSaviq: CI qmluitests tests.13:28
Saviqdednick, yes, and runtime13:28
dednickdandrader: ^ can you add qmenumodel-qml to the Source dependencies of debian/control and remove the mock code? should fix the problem.13:30
Saviqdednick, actually13:30
Saviqdednick, I wonder if we should add it to the job directly13:30
Saviqmzanetti, what's your say?13:31
Saviqmzanetti, runtime deps for qmluitests?13:31
dednickSaviq: bzr bd wont work though.13:31
dednick?13:31
dednickor does it only go though unit tests?13:31
Saviqdednick, is it a qmltest or a qmluitest?13:31
mzanettiSaviq: huh? why would that be required?13:31
greybackSaviq: standup13:31
* greyback suspects Saviq secretly enjoys taking notes13:32
mzanettilol13:32
dednickSaviq: i'm guessing it's a qmluitest.13:33
kgunntsdgeos: can you describe a little more "flaky"13:34
kgunnmterry: you forgot to mention you'll support 3 diff optional configs for 3 months ;)13:36
sil2100Saviq: one small thing before approving globally!13:37
mterrykgunn, :)13:37
kgunnmterry: which btw...i'm only a fan of the one (greeter as regular client)13:38
sil2100Saviq: could you just add some XXX: comment in the fn_auto_brightness check?13:38
tsdgeoskgunn: apps randomly closing sometimes, and stuff like that13:38
sil2100Saviq: so that it would be directly visible that it's a workaround13:38
kgunntsdgeos: oh...wow...yeah, that'd fall in to the category of "flaky"13:38
mzanettidammit... unity8 still keeps on freaking out on the Nexus4... 80% CPU all the time :(13:39
dednickSaviq: i'm presuming you meant qmlunittest vs qmluitest?13:42
tsdgeoskgunn: after a reboot all is better, i'm guessing that it was the phone was "dirty" after a day of using if for all kind of weird development stuff13:43
dandraderdednick, ok, will try that13:43
mterrygreyback, did lp:~unity-team/unity/unity8-mir/ get moved somewhere?13:45
greybackmterry: yep, is now lp:unity-mir13:45
mterrygreyback, OK13:45
dandraderdednick, done13:46
Saviqdednick, can you get back on mumble for a sec?13:47
mterrygreyback, hrm, hard to merge from it, after having merged from old unity8-mir now that unity8-mir is deleted (bzr can't recreate history)13:49
greybackmterry: oh sorry, I didn't read your message properly13:50
=== dandrader is now known as dandrader|lunch
greybackmterry: lp:unity-mir is a library+qml plugins which unity8 will use. lp:~unity-team/unity8/unity8-integrate-mir/ i spin of unity8 which use unity-mir13:51
greybackmterry: sorry for the change, but this should stabilize now13:51
mterrygreyback, ah...  ok.  the unity8-integrate-mir branch used to be out of date in favor of unity8-mir, IIRC.  Now it's back in vogue13:51
* mterry 's head spins13:51
Saviqdednick, yeah, unit ones are run in debhelper's autotest, ui ones are not13:51
dednickdandrader|lunch: sorry to dick about. We want to have it mocked.13:52
greybackmterry: confusion understood, sorry about that13:52
mterrygreyback, no worries, I get that we're in flux13:52
dandrader|lunchdednick, ok, will just rollback my last commit then13:55
Saviqsil2100, yeah, doing13:55
nic-doffaySaviq, can you get in touch when you have a moment.13:55
Saviqsil2100, pushed13:56
Saviqnic-doffay, 'fraid it's gonna be in at least an hour, going into a meeting now13:57
sil2100Saviq: thanks!13:57
Saviqnic-doffay, but write away, if it's a simple one I might push it through in the mean time13:57
nic-doffaySaviq, no problem. It's not too simple sadly.13:57
nic-doffayCan crack on with the selector MP13:58
dednicklarsu: how goes qmenumodel ? :)14:00
larsudednick: reviewing my code from back then right now. MR will come today.14:04
larsudoesn't look like there's much missing14:04
larsuuntil you look at it I assume :D14:04
olligreyback, Saviq nice video14:09
ollithanks to everybody involved14:10
olliracarr, I guess too14:10
greybackolli: thanks. Working on making it even better14:10
olligreyback, we should get tthat into the test build14:11
ollisoon14:11
olli:)14:11
Saviqgreyback, you get a slap for not CC'ing olli on the email yesterday ;)14:11
* olli will bug saviq and kgunn in a bit about that14:11
dednicklarsu: \o/14:11
greybackolli: Saviq: I was respecting the chain of command :)14:12
Saviqlol14:12
olliyeah, I get respect14:12
ollid'oh ;)14:12
mterrykgunn, aw crap, I forgot about launching the camera and phone apps in the greeter14:14
mterrykgunn, that may be a blocker for client-only greeter14:15
mterryduh, I'm not used to actually launching those, forgot we do that in greeter session rather than user's14:16
seb128mterry, hey, does the greeter have an option to make sound on lock/unlock atm? if not is that on the roadmap?14:21
mterryseb128, does not make a sound right now.  I don't believe that's on the design mocks.  katie ?14:22
seb128mterry, the system settings mockups have the option: https://wiki.ubuntu.com/Sound?action=AttachFile&do=get&target=phone-settings-sound.png14:23
seb128mterry, bottom item there14:23
katiemterry, seb128 its not in the design mocks... there is, will be a sound in the future14:23
mterrykatie, so not for 13.10?14:23
seb128hum14:23
katiemterry, seb128 let me find out14:24
seb128katie, thanks14:24
seb128we either need to implement it or to drop the item from the system settings for 13.1014:24
Cimimm weird14:25
CimiSaviq, I removed crossfadeimage, added import Ubuntu.Components 0.1 to the tst_CrossFadeImage.qml we have in the shell, just to cross-test14:26
CimiSaviq, it complains about line 27, "Invalid alias location"14:26
Cimiproperty alias crossFade: crossFadeImage.crossFade14:26
Cimibut I read SDK code, and crossFade is a public property of CrossFadeImage14:27
* Cimi epic fail14:27
Cimiprice to pay when you have too many files opened and you were looking at the wrong one14:28
ollidandrader|lunch, ping14:29
katieseb128, mterry, there are no sounds at the moment.. there will be at some point in the future - that's all I can tell you!14:30
SaviqCimi, ;)14:31
Saviqmzanetti, tsdgeos can you guys do https://code.launchpad.net/~diegosarmentero/unity8/unity-scope-data/+merge/17627414:31
mzanettiSaviq: ack14:31
mzanettiwas just searching for one I could do14:31
Saviqmzanetti, tsdgeos I'll still look through it, just don't want to block gatox14:31
seb128mpt, ^ what katie said ... it would be useful to figure out of the "lock sound" is on the roadmap for v1 or not14:33
CimiSaviq, I realised the SDK has less properties14:33
CimiSaviq, shall we add them to the SDK?14:34
=== boiko_ is now known as boiko
SaviqCimi, dunno, you tell me14:35
SaviqCimi, since the CFI was taken from us, it was removed for a reason, probably?14:35
CimiSaviq, it's simply removes two properties that were controlling some corner cases14:36
Cimimzanetti, CFI in SDK lack of crossFade and fadeInFirst properties, you think they are important?14:36
SaviqCimi, not me you should talk to about those, remember? ;)14:36
Cimimzanetti, it behaves like with crossFade: false14:36
Cimimzanetti, and fadeInFirst: false14:36
SaviqCimi, I think mterry might know, too ↑14:37
mzanettiCimi: yeah. they are needed otherwise the Greeter will look weird at startup14:37
Cimiok14:37
sil2100greyback: oh noes!14:37
Cimiwill ask SDK to add them14:37
greybacksil2100: what??!!14:37
mzanettiCimi: I'm a bit confused about that testImage you added in the Shell.qml here: https://code.launchpad.net/~unity-team/unity8/unity8.background_gsettings/+merge/17495814:37
mzanettiCimi: can't see any reason why this is all needed14:38
sil2100greyback: hi! I wanted to test some new packaging changes to unity-mir but again there seems to be a FTBFS, I suspect an API change14:38
greybacksil2100: correct, I'm working on it. Gimme an hour14:38
sil2100<o>14:38
sil2100greyback: excellent!14:38
greybacksil2100: in then, have a lie down. Nothing to worry about :)14:38
Cimimzanetti, to fallback when the image is broken?14:38
greybacks/in/until/14:39
mzanettiCimi: still don't see why the CrossFade image wouldn't be enough for that14:39
Cimimzanetti, what?14:39
Cimimzanetti, we're talking on two different things14:39
mzanettiyes14:39
Cimimzanetti, one is the branch for gsettings14:39
Cimimzanetti, one is I am looking into removing crossfadeimage from shell14:40
mzanettiCimi: yeah. I'm talking about the gsetting branch14:40
mzanettianyways, I'll check it out and test myseldf14:40
mzanetti-d14:40
Cimimzanetti, we fallback to the default background14:40
Cimimzanetti, when the image is broke14:40
CimiI thought it was a good idea might be wrong14:41
mzanettiCimi: seems like a rather dirty hack to me... but there could well be things I don't see yet... gimme a few minutes to check details14:41
mterryCimi, (forgive me if I missed this in scrollback, but) crossfade is used in greeter14:42
Cimimterry, indeed14:42
kgunnmterry: hmmm, wondering....would camera app be any different than rendering infographic/launcher/pin screen....in a way, its just another window14:42
Cimimterry, we were wondering if the crossfadeimage in the sdk is enough14:42
kgunnmterry: i don't think this kills the greeter-as-normal-mir-client concept14:42
mterryCimi, probably.  I'm not super familiar, but we don't use anything special from ours14:43
mterrykgunn, camera app is different because it's a separate process14:43
mterrykgunn, we *could* go down the road of making bits of the camera app re-usable in the greeter as native widgets14:44
kgunnmterry: its what i was thinking....but yeah...kinda gives you heartburn thinking about it14:44
kgunnmterry: what you really want it seems is a special way to load camera into greeter process...at least the rendering bits14:45
mterrykgunn, like xembed!  ;)14:46
mzanettiCimi: is there a way to set the background image through command line?14:46
kgunnmterry: surely there's a qt version14:47
Cimimzanetti, probably through dcong14:47
Cimidconf14:47
kgunne.g. just sounds like a problem that would;ve come up enough to do something about14:47
mterrykgunn, I wouldn't be surprised if not.  xembed was mostly reviled as a hack14:47
kgunnmterry: one man's hack is another man's much needed soln ;)14:48
mterrykgunn, I mean, we have a solution for "showing a window inside a session" and we call it mirserver, eh?  :)14:48
kgunnmterry: true14:49
mterrykgunn, if we want to make camera and phone apps giant qml plugins with the barest of main() wrappers around them, we could re-use them as special cases in the greeter14:50
mterrykgunn, that wouldn't work later when we want to support more apps, but could work for 13.1014:50
kgunnmterry: yeah14:50
kgunnmterry: i was trying to think...what reasons14:50
mzanetti*shrug* The infographics looks quite bad with a non-purple background14:50
kgunnwe had for not wanting to be server side...and only one i can come up with14:50
Saviqjeez what is it, security breach week or something?14:50
mzanettiSaviq: what happened?14:51
Saviqmzanetti, ubuntu forums, OVH, now Simple Machines forums14:51
kgunnis adding a bunch of naff api's to the shell/internal client api of mir14:51
kgunnmterry: or was there something else?14:51
kgunnracarr: ^14:51
mzanettiSaviq: not to forget developer.apple.com14:51
Saviqmzanetti, didn't know that, not there ;)14:51
* Saviq likes him a Password Hasher14:52
mterrykgunn, hold on one moment14:52
kgunnracarr: we forgot about greeter needing to launch camera app...so we're discussing whether or not we can still go "regaular mir client" with the greeter14:52
mterrykgunn, you're asking why we didn't put the greeter in u-s-c?  or why we didn't want to be a mirserver ourselves like the shell?14:53
kgunnmterry: i might need re-education....i thot out of 3 choices, there was 1) greeter in shell (security killer), 2) greeter "like shell" server side of u-s-c's mir, 3) greeter as regular client to u-s-c's mir14:55
kgunnmterry: if this is correct, i assume we're talking about choosing between #2 & 314:56
mterrykgunn, there was a fourth option (greeter in u-s-c itself) that was also a security killer14:56
mterrykgunn, so "server side" is a confusing term (in u-s-c or a mirserver talking to u-s-c)14:57
mterrykgunn, but we didn't want to be #2 because there is risk of not getting multiple simultaneous mirservers performant14:57
mterrykgunn, we may *have* to go #2 though14:58
=== dandrader|lunch is now known as dandrader
kgunnmterry: sorry...u-s-c doesn't really do anything but launch a mir server....so, when you say "in usc" i assume you mean the mir it just launched14:59
dandraderolli, pong14:59
sil2100Saviq: hmm, CI seems a bit irritated with your branch?15:00
sil2100Saviq: https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-saucy/768/consoleFull15:00
Saviqsil2100, it hanged15:00
mterrykgunn, I'm going to make a quick drawing15:00
Saviqsil2100, it should merge fine15:00
sil2100Saviq: it's still autolanding, but hm15:00
sil2100Saviq: ok15:00
kgunnmterry: thanks! needed i think15:00
ollidandrader, is there any chance my magic trackpad can do 2 finger horizontal swipes15:01
ollidandrader, like I can scroll vertically, but not horizontally15:02
olli13.1015:02
bregmaI don't believe the synaptics driver provides horizontal scrolling15:03
Saviqolli, hmm mine only does with shift15:03
dandraderolli, +1 on what bregma said15:04
Saviqkgunn, you joining?15:04
Saviqgreyback, do you remember the google doc where we discussed the orientation?15:04
Saviqgreyback, it was some "window management apis" or something15:05
greybackSaviq: not really. Am looking15:06
Saviqgreyback, it was your review of the app manager apis15:07
greybackSaviq: https://docs.google.com/a/canonical.com/document/d/1A0ZmFpVdIhkz7zwmdee7749RjdoKiz7erwytbPz4UoE/edit15:07
Saviquh15:07
greybackhad very little to do with orientation15:08
mterrykgunn, shared doc with you15:08
kgunnmterry: will take a look....(busy for a next hour or so...we'll talk more later)15:08
mterrykgunn, k15:09
tsdgeosmzanetti: can you do https://code.launchpad.net/~diegosarmentero/unity8/unity-scope-data/+merge/176274 ? I have the phone setup for unity-mir and going back to "regular unity" it's like 2 hours forth and back at least15:11
mzanettitsdgeos: ack15:11
mzanettiCimi: I'd have a IMHO cleaner proposal for this. You ok with me pushing to your branch?15:24
mzanettiCimi: in case you don't agree you can revert it ofc15:24
Cimimzanetti, obviusly15:25
Cimimzanetti, that's why I push with unity-team15:26
Cimimzanetti, obviusly15:26
Cimiops15:26
mzanettiCimi: still I think its not nice to push to someones branch without asking first15:28
mzanettiCimi: pushed15:28
mzanettiCimi: I hope I didn't miss some corner cases15:28
mzanettiCimi: the sourceSize thing is gone, but I think your's didn't work either because CrossFadeImage's sourceSize property is readonly for some reason15:29
Cimimmm was working for me15:29
Ciminevermind15:29
mzanettiCimi: ah right... the backgroundImage was a regular image before... I just changed it to be a CrossFadeImage15:31
mzanettiCimi: with your latest revision only the Greeter would crossfade, but not the background image15:31
Cimiyes15:32
Cimimzanetti, and in theory is fine15:32
Cimimzanetti, because you change the bg from the settings app15:32
Cimimzanetti, so you won't see the crossfade15:32
mzanettiCimi: what if the settings app runs in the side stage?15:32
mzanettiCimi: or windowed on the desktop (in the future)15:32
Cimimzanetti,  I don't have a nexus 10 :P15:32
mzanettiright...15:32
Cimiyou're right15:33
mzanettithe easiest way to deal with a problem is to ignore it15:33
mzanetti:P15:33
Cimiwhich problem?15:33
Cimihah15:33
mzanetti:D15:33
Cimibb in a bit15:34
Cimitaking the bus to the office15:34
ollikgunn, bregma, Saviq ... quick follow up thought from the u8/main discussion15:50
olliwe have been discussion our opinion whether asac will proceed of getting a FFe or not, but not really what the impact is on us if U8 lands in main15:51
olliSaviq indicated that there ain't none, other than the time it would take to do the watermark15:51
ollias for us it doesn't matter if we are in (Main & FFe) or PPA15:52
ollifor touch images15:52
Saviq+115:52
olliis that right?15:52
kgunnolli: right....cause we can basically ignore all bugs15:52
Saviqolli, we will just stop releasing into main on FF15:52
olliSaviq, unless asac pulls out the FFe (which we are doubtful of)15:52
bregmaso, like doing an upstream release and freezing that version in distro?15:53
CimiSaviq, filed https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/120456515:53
ubot5Launchpad bug 1204565 in Ubuntu UI Toolkit "CrossFadeImage needs properties to control crossfade and fade" [Undecided,New]15:53
ollibregma, I guess and then continue in PPA if we don't have a FFe15:53
Saviqs/main/distro/15:54
Saviqolli, yup15:54
olliSaviq, and the point about having a better experience in few weeks... if we get a FFe, we can still improve it also for the desktop (basically for free) but if not, we will just take what we have for the desktop15:56
ollibut this isn't impacted by or impacting landing u8 in main either, is it?15:56
Saviqolli, no, as long as we're not blocked by FF to work on unity8 or any of its dependencies15:57
Saviqpaulliu, hey, with http://pastebin.ubuntu.com/5908021/ I can enable home, but the generic preview fails sometimes with http://pastebin.ubuntu.com/5908027/15:58
Saviqpaulliu, could you have a look tomorrow?15:58
paulliuSaviq: sure15:58
olliSaviq, bregma, kgunn, so to conclude this topic: U8 is being pulled from a PPA today for the touch build, pushing u8 into main will increase the build quality, in the worst case (i.e. no FF for u8 and all its dependencies) we switch back to use a PPA at FF which for us is not an issue... is that a fair statement?16:00
Saviqolli, yes16:00
bregmaolli, agreed16:00
ollilooking for a +/- 1 from all of you, don't want to override what we just discussed, but I think we discussed the wrong question16:00
ollikgunn, up to you now ;)16:01
kgunnolli: agree16:01
Saviqbiab o/16:01
ollikgunn, Saviq, bregma the only impact/question is... in what position are we with regards to daily landing/daily image tests16:02
olliare we prepared to be green on the dashboards when we land u8 in main16:03
Saviqolli, yes, the three platform-api bugs I posted yesterday have fixes (merged or MR'd at least)16:03
Saviqolli, the occasional crash (about 2% of runs) I'm investigating is in Qt, need to check whether 5.1 helps there16:04
Saviqsil2100, didrocks can you guys push the button on another unity8 test run?16:04
sil2100Saviq: it's rolling now16:05
Saviqsil2100, ah, awesome16:05
sil2100Saviq: I already re-deployed and re-ran the stack, waiting for builds to finish16:05
Saviqsil2100, what's the "source" jenkins for it? can I access it through the VPN?16:06
sil2100Saviq: yes, let me give you the link to the stack16:08
sil2100greyback: I see your merge to fix the FTBFS in unity-mir!16:21
greybacksil2100: no you don't16:21
sil2100:(16:22
greybacksil2100: /almost/ there16:22
sil2100;)16:22
didrockssil2100: Saviq: passed \o/16:35
didrocksSaviq: do you still want blocking publication?16:35
didrocks(in next)16:35
greybacksil2100: quick fix for FTBFS landed, no point blocking you16:37
=== om26er is now known as om26er_afk
=== om26er_afk is now known as om26er
dandradermzanetti,  is autolanding disabled, broken or something of the like?17:25
dandradermy MP is consistently and utterly failing https://code.launchpad.net/~dandrader/unity8/lp1116207/+merge/17516317:27
mzanettidandrader: /tmp/buildd/unity8-7.81.3+13.10.20130718ubuntu.unity.next/tests/qmltests/tst_Shell.qml: bad whitespace in line 19417:29
dandradermzanetti, excellent! (I'm surprised you're still on duty) where did you find this line?17:30
dandradermzanetti, nevermind. got it. I should check all those different urls...17:30
dandraderbad dandrader17:31
mzanettidandrader: cd .bazaar && make isntall17:31
mzanettidandrader: cd .bazaar && make install17:32
dandradermzanetti, right, your magic bazaar plugin. thanks17:35
dandraderwell, such false alarms should no longer happen :)17:35
mzanettidandrader: the latest version of the bazaar plugin is really usable imo17:36
mzanettidandrader: I understand that the first version was way too annoying for anyone to actually use it17:37
mzanettidandrader: but now it only checks for such nasty mistakes as whitespaces which makes it really fast17:37
dandradermzanetti, yes, just tried it. works fine17:37
dandradermzanetti, it simply issues "make test", right?17:38
mzanettidandrader: yeah. and if it fails, it backups your commit message so you can easily uncommit, fix and commit -F message.txt17:38
mzanettidandrader: so you don't have to type the message again17:38
dandraderhmm, interesting17:38
dandraderEOD17:39
sil2100Saviq: \o/ did you see the results?17:47
mzanetticyphermox: ping17:47
sil2100Saviq: the check job is green, all tests passed it seems17:48
sil2100Saviq: sadly, Didier is away so I can't ask him if we have permission to publish the stack17:48
mzanettisil2100: if it compiles, ship it!17:48
mzanetti:P17:48
sil2100pff ;p17:49
* Saviq reads all the pings...18:06
Saviqsil2100, yeah, we don't want that yet18:16
Saviqgreyback, hey, I asked Jamie to file the bugs against unity-mir as well, so that we make sure we have them fixed/not applicable18:18
Saviqgreyback, feel free to reassign if needed https://bugs.launchpad.net/unity-mir18:18
greybackSaviq: ack18:18
kgunnmterry_: ok...i added mir's to your picture & think we are totally thinking the same18:23
kgunnalso, in the instance of option4 - i think there are other issues besides security18:24
kgunnthat is...what does window management18:25
kgunnacually scratch that18:25
kgunnthat's an issue in both 3 & 418:25
mterry_kgunn, yup, looks good.  And yeah, window management is a problem.  I had previously thought #3 would work because I forgot that the greeter will have clients for 13.1018:26
kgunnmterry_: so phone and camera...18:30
kgunnmterry_: how does the launcher work? is the shell technically a client ?18:31
kgunnof greeter that is18:32
mterry_kgunn, no...18:36
mterry_kgunn, the greeter acts like its own shell.  And the launcher widget itself is just a widget, built into the shell (so shell and greeter can share the code, but each have their own launcher)18:36
kgunnmterry_:18:37
kgunnah18:37
mterry_kgunn, that's why the launcher and indicators are not a problem, they are just internal widgets18:37
mterry_kgunn, but obviously phone and camera are different18:37
kgunnmterry_: right...i guess, in order to avoid18:37
kgunntaking in camera/phone components18:38
kgunnwe would need camera/phone to18:38
kgunncreate a thin main as you were suggesting earlier18:38
kgunnallowing us to "widgetize" them18:38
mterry_kgunn, yeah18:38
mterry_have the bulk of their code be in a plugin form, that the greeter could load18:39
kgunnmterry_: i don't really see a way around it18:39
mterry_kgunn, well, we could do the original option #218:39
mterry_which has its own risks18:39
kgunnmterry_: i actually think tvoss is right on that one tho....18:39
kgunnmainly we will have performance issues for a while18:39
kgunnas we have deprioritized android bypass18:40
kgunn(+ even if we reprioritize android bypass....its not obvioulsy easy because we use18:40
kgunnbinaries and there's lots of assumptions under the hood on many of these hw platforms)18:40
kgunnalso...memory18:40
kgunnthat's just too damn many framebuffer sized memory chunks floating around18:41
mterry_kgunn, :-/18:43
mterry_kgunn, what about 14.04?  Do we think we can go back to option #2 then?18:45
kgunnmterry_: maybe Saviq  or others have maybe another idea besides widgetizing18:45
kgunnmterry_: wrt going option#2 for 14.04...possibly18:45
kgunnits still kinda not the most effecient i think...but if bypass is there18:46
* Saviq reads backlog18:46
kgunnthen yeah18:46
* kgunn shares option diagram with saviq18:46
Saviqmterry_, kgunn I think I assumed that "greeter" apps would be launched in a guest session18:47
Saviqbut then assumed we'd have bypass...18:48
cyphermoxmzanetti: pong?18:48
mterry_Saviq, (a) that's a tad slower and (b) would still introduce another Mir server floating around18:48
Saviqmterry_, kgunn "widgetizing" camera and phone would be good for those, maybe a few other core/trusted apps18:50
Saviqmterry_, kgunn but it's supposed to be possible for all apps to have an unlocked mode18:50
mterry_Saviq, not for 13.10, but yeah18:50
Saviqmterry_, kgunn like, for example, you can create notes in... notes, events in calendar18:51
mterry_and even then, theoretically only those that have added explicit support for it.  So that could mean this widgetization process18:51
Saviqmterry_, but that would mean running arbitrary code in greeter18:51
Saviqmterry_, I can tell you already security will nix it ;)18:51
mterry_Saviq, greeter isn't root18:52
Saviqmterry_, but it could still crash it18:53
Saviqmterry_, and do stuff which apps shouldn't be able to do, if it's in the same process18:53
mterry_Saviq, true, which sucks.  But that's a design issue not a security one18:53
mterry_Saviq, but I agree there are problems with things being in-process18:54
mterry_Saviq, kgunn: long term, if we want the "arbitrary app can run unlocked in greeter" mode, we'll want the greeter to be a real mirserver18:55
Saviqmterry_, or for it to launch a guest session, right?18:56
mterry_(or run them in greeter, which is another mirserver anyway)18:56
mterry_*guest18:56
Saviqmterry_, we could think of a stripped down, no-shell session that would launch almost instantly18:57
kgunnSaviq: you mean "no-shell" session for the apps in question?18:58
Saviqkgunn, yeah, if app is launched in unlocked mode18:58
Saviqkgunn, the app could get launched in a minimal shell18:58
Saviqkgunn, where greeter would still handle launcher and panel18:58
Saviqkgunn, overlaid over the user session (although I understand this may be costly)18:59
Saviqso if people tell me that's not feasible - that's ok18:59
mterry_Saviq, tough for 13.10 for sure  :)18:59
Saviqmterry_, weren't we talking long-term? ;)19:00
Saviqmterry_, for 13.10 I'm good with widgetizing just the core apps19:00
Saviq+few19:00
kgunnSaviq: is that something we should take on? (to minimize external deps :)19:00
kgunnor ask first...19:00
Saviqkgunn, TBH it's QML...19:01
mterry_kgunn, who is in charge of camera and phone apps?19:01
Saviqmterry_, bfiller19:01
mterry_Saviq, thanks19:01
Saviqkgunn, so it's already "widgetized"19:01
Saviqkgunn, as long as we set the environment up the way QML expects it, you can just load the app's main QML and be done with it19:02
* kgunn loves it when we catch a break19:02
SaviqQML of the app expects it, I mean19:02
Saviqso it might mean a) fixing the apps to not do stuff in their main() if possible19:02
mterry_Saviq, depends how easy their qml makes it I imagine19:02
Saviqmterry_, if their main()s are heavy it's wrong anyway19:03
=== mterry_ is now known as mterry
Saviqmterry, and if everything is in plugins - we're done19:03
kgunnSaviq: mterry_ shouldn't the environ be set up proper i mean we are using it for infographic, launcher, pin screen19:03
Saviqkgunn, what I mean by that is that apps may be doing stuff in their main()19:03
Saviqkgunn, like setting some context properties19:03
Saviqkgunn, initializing some stuff19:04
Saviqkgunn, so either we need to do the same in Greeter's main() - but that'd not be nice since we'd be doing it regardless of whether you actually load the app in the greeter or not19:04
Saviqkgunn, or move those out of main() into plugins19:04
kgunnSaviq: gotcha19:05
Saviqkgunn, at which point you just load the QML of the app and you're done19:05
Saviqkgunn, only thing is, of course, that's insecure19:05
mterrySaviq, insecure in that we're loading "arbitrary" qml?19:06
Saviqkgunn, but would be fine for some core apps (even long-term - for them to be faster)19:06
Saviqmterry, yes19:06
kgunnright...but we trust these guys19:06
Saviqmterry, and whatever that QML brings with it19:06
mterrySaviq, not arbitrary, and we're already relatively untrusted19:06
Saviqmterry, with core apps that's not arbitrary - of course19:06
kgunnyea! we're back to greeter as a regular mir client19:06
Saviqmterry, that's why I said it can be fine for core apps19:06
kgunnracarr: ^19:06
mterrySaviq, yar19:06
Saviqmterry, but long-term that's not going to cut it19:07
SaviqIMO19:07
* mterry starts working on that pure-client branch again19:07
Saviqdid you see guys? we've passed the $5M mark a while ago19:07
mterrySaviq, yeah... if we ever want 3rd party apps to do that, I agree19:07
Saviq$40k ago, to be exact19:07
mterry$32 million is so much19:07
Saviqmterry, that's not an if, AFAIK :D19:07
Saviqmterry, it is19:07
Saviqmterry, but the 2.5 days look promising19:08
Saviqbtw, just got the Firefox OS phone19:08
Saviqwill put out a bigger report later about my feelings19:08
mterryinteresting19:08
Saviqbut if anyone wants to know anything in particular, ping me19:08
Saviqit's *cheap* for sure... but it's ~€100 with no contract...19:08
racarrkgunn: Err. 5 minutes sorry19:09
racarralmost finished this input stuff19:09
kgunnracarr: no worries...was just telling you we're back to greeter not needing to sully up those internal client i/f's19:09
racarrkgunn: Just to see I understand19:11
racarrthe greeter, is running apps, etc (i.e. CAmera in unlocked mode), and they aren't running in a guest session19:12
racarrand are integrated with the greeter UI19:12
racarrso the thought was gthe greeter may have to be a mir compositor19:12
racarrbut instead19:12
racarrsend around QML for now?19:12
kgunnracarr: bingo19:14
kgunnracarr: basically....those apps being widgets to the greeter19:16
racarrOk sounds good19:18
racarrI actually have a secret fondness that dates back to college for using javascript as an IPC protocol :p19:18
mzanetticyphermox: is the pong still valid?19:25
cyphermoxmzanetti: yes. the alternative is really boring debugging of bluez19:25
mzanetticyphermox: ah... thats exactly the topic :)19:25
cyphermox:)19:26
cyphermoxmzanetti: what's up?19:26
mzanetticyphermox: what do I need to do to get the bluetooth chip enabled on the Nexus 4?19:26
cyphermoxah! this is really cool :)19:26
cyphermoxmzanetti: take this branch (https://code.launchpad.net/~mathieu-tl/phablet-extras/brcm-rename) and build it on the device, then install the bluetooth-touch package it will create19:27
cyphermoxyou'll need build-essential debhelper bzr bzr-builddeb19:27
cyphermoxand when installing the package you also need rfkill19:27
mzanettiright19:28
cyphermoxit's just a few upstart jobs to poke the right things19:28
mzanettiok... kinda in the middle of something right now (EOD already and this is spare time activity) but I'll probably bug you again in a few if I run into troubles.19:28
mzanetticyphermox: what I want to do is to check out if QtBluetooth works as expected on our device19:29
cyphermoxcool19:29
cyphermoxI suspect it probably does, but yes it's a very valuable test19:29
cyphermoxwhat device are you going to try?19:29
cyphermoxI'd favor a mouse or headset19:29
cyphermoxI'm seeing some weird crashes with a keyboard, which I'm quickly debugging while I do other stuff19:30
mzanetticyphermox: my use case is mostly a2dp for my car and the stereo headset for running19:30
cyphermoxperfect19:30
cyphermoxa2dp does work with bluez, so if QtBluetooth uses bluez you should be good19:30
cyphermoxmzanetti: feel free to ask if you want pointers on how to link devices from the command line19:31
mzanettiQtBluetooth is not involved in a2dp actually... I'm interested in QtBluetooth because I have been the one porting it from Qt4 to Qt5 back then when I was at Nokia19:31
cyphermoxok19:31
mzanettiand as we never shipped a device with Qt5 I'm still not perfectly sure how good I did :D19:31
mzanettiok... bbiab19:32
Saviqmterry, kgunn three more things about greeter - "widgetizing" - I'd refrain from trying to support more than QML in the long run, unless we go for multiple processes - surfaces - and simply a mirserver19:41
Saviqmterry, kgunn QML has the added benefit that we can set QML2_IMPORT_PATH to supply the same interface, but a different implementation (proxy, write-only or whatnot) for "locked" apps19:42
Saviqmterry, kgunn, and IIUC greeter/lightdm would still remain a mirserver for the system compositor, right? controlling session surfaces?19:43
mterrySaviq, yeah, plus our greeter client19:43
Saviqmterry, right, so lightdm would be mirserver for system compositor, and greeter and sessions would be its clients?19:44
Saviqmterry, assuming here that greeter is a special kind of client? or is sessions geometry management built into lightdm?19:45
mterrySaviq, lightdm would talk to u-s-c, which would be the system mirserver19:46
mterrySaviq, greeter and sessions would be u-s-c's clients19:47
mterrySaviq, I think greeter would be normal sort of client (the same way a session is a normal client that happens to also be its own server for subclients)19:47
Saviqmterry, yeah, but something needs to control sessions geometry19:47
Saviqmterry, so something needs to tell u-s-c what to do19:48
Saviqmterry, or is that built into u-s-c19:48
Saviqmterry, and in that case, why can't greeter be u-s-c, same as unity8 is session-compositor?19:48
mterrySaviq, that bit I don't know19:48
mterrySaviq, security in that case, since u-s-c is root19:48
Saviqmterry, right19:48
Saviqmterry, so we need a mini-shell inside u-s-c to deal with those, or a protocol between greeter and u-s-c I'd say19:50
Saviqmterry, who can we clarify that with?19:50
mterrySaviq, that sounds like a u-s-c job19:51
mterrySaviq, but robert_ancell would know for sure19:51
Saviqmterry, ok, I'll grab him when he comes on19:53
tvossmterry, let's assume usc would not require root19:54
mterrySaviq, u-s-c is only guy in picture I can imagine doing that.  Out of scope for greeter and lightdm, I'd think (though lightdm does talk to u-s-c to help it know who is on top)19:54
tvossmterry, then greeter could be the system-level shell on top of usc19:54
Saviqmterry, yeah, but then it's the greeter that know that it's supposed to slide away and animate the underlying session19:54
Saviqtvoss, yeah, that's what I thought would happen, otherwise we introduce the same process split we prevented for user session19:55
mterrySaviq, yes and no.  lightdm and u-s-c will probably between them mark the greeter as special or "on top" and when transitions happen, u-s-c can do the animations19:55
Saviqmterry, that's a difficult split to make19:56
Saviqmterry, see that panel is not supposed to be swiped away on greeter unlock19:56
Saviqmterry, do you know why u-s-c needs to be root?19:56
mterrytvoss, Saviq: as for why u-s-c is root...  I'm guessing because it controls hardware?19:56
tvossmterry, we can control that, so no reason it _has_ to be root19:57
Saviqmterry, it probably just needs to write access to some devices19:57
mterrySaviq, right, on greeter unlock we make the greeter semi-transparent and the session beneath bleeds through19:57
Saviqmterry, so facls should give us enough granularity19:57
tvossmterry, is that still the active blur?19:57
mterrytvoss, it's the "something" blur19:58
mterrydoesn't need to be active, but needs to be periodic at least19:58
mterryi.e. active with a really slow pulse  :)19:58
tvossmterry, ah okay. at any rate, that would require a special flag on the surface (I don't like that) or the greeter to be able to control the compositing iiuc19:58
tvossmterry,  can we have something that at least is calcutable in one pass?19:59
mterrytvoss, the way we had been talking with mir folks was that the lightdm process has to name all the sessions for u-s-c anyway.  So it can name the greeter as "Greeter" or some such, so that u-s-c knows to treat it different19:59
tvossmterry, sure, we can do that, not talking about technical feasibility here20:00
mterrytvoss, as for one-pass, I'm not familiar with the various blur algorithms.  Haven't researched that much.  mzanetti was last person to investigate blur performance I think20:01
mterrytvoss, but you don't like the idea of u-s-c treating greeter specially?20:01
tvossmterry, I think it separates logic where it shouldn't. But you already know that I'm a huge fan of the greeter being a system-level shell20:02
mterrytvoss, well, let's ask robert_ancell why/if u-s-c needs root20:03
mterrytvoss, that would enable greeter to be in u-s-c if we wanted, from a security standpoint anyway...20:03
Saviqmterry, I agree with tvoss here that I always envisioned greeter to be to the system compositor what unity8 is to the session compositor20:06
Saviqmterry, so to *be* the system compositor, as unity8 is the session compositor20:06
Saviqok, I think I didn't eat anything today, is it a good time for breakfast?20:07
mterrySaviq, :)20:10
tvossSaviq, enjoy :)20:12
Saviqtvoss, oh, good, was waiting for someone to actually *reply* mterry :P20:12
Saviqdidn't know whether it was a good time20:12
Saviqit was a valid question, you know!20:12
tvosswhich one20:12
Saviqtvoss, which one what?20:13
tvosswhich question was valid?20:13
Saviqtvoss, the one whether it was a good time for breakfast20:13
tvossSaviq, ah :)20:13
* Saviq is gonna go now, probably has some sugar-level-related brain issues20:14
Saviqjeez $5M...20:14
Saviqtwo days...20:14
Saviqif only we can keep it up :)20:14
Saviqtvoss, btw, got a Firefox OS phone today, if you have any particular questions20:15
Saviqwill do a bigger report of my feelings (objective, of course)20:15
Saviqtvoss, and will bring to IoM20:15
tvossSaviq, oh that's great20:15
KaleoSaviq: is it any good?20:15
SaviqKaleo, not from the first impressions :/20:16
SaviqKaleo, http://pastebin.ubuntu.com/5908811/ my first-5-minutes notes20:17
Kaleook20:17
mterryIs there a PPA with unity-mir in it?20:54
mterryaha!  ~phablet-team/+archive/mir20:55
mterryrobert_ancell, good morning20:55
robert_ancellmterry, hello20:56
=== racarr is now known as racarrlunch
mterryrobert_ancell, oh, I had some questions for ya21:03
* mterry digs around21:03
mterryrobert_ancell, ah yes, why does u-s-c need to be root?21:05
mterrySaviq, tvoss ^21:05
tvossrobert_ancell, hey there :)21:05
robert_ancellmterry, to get the DRM device I believe.21:05
Saviqrobert_ancell, o/21:05
tvossrobert_ancell, we could theoretically work around that, couldn't we?21:05
Saviqwe're not banding against you at all ;)21:05
robert_ancelltvoss, According to RAOF we can't yet. It's the same reason X has to run as root.21:06
mterryrobert_ancell, (this is in service of maybe putting the greeter in-process with u-s-c to avoid having the greeter run another mir server)21:06
robert_ancellI'd love it not to be root21:06
tvossrobert_ancell, okay, can you check with RAOF?21:06
robert_ancelltvoss, I'll ask again, but last time I asked it wasn't something we could work around21:06
robert_ancellPutting the greeter in the system compositor will break some assumptions in lightdm21:07
tvossrobert_ancell, what are those?21:07
robert_ancellmterry, what's wrong with running it as a Mir client? Or what's wrong with running it as a Mir server? Note it will be a nested Mir server, so it wont be as heavy as u-s-c (won't have to access hardware directly)21:07
robert_ancelltvoss, that lightdm launches greeters like normal sessions whenever it wants21:08
mterryrobert_ancell, running as a mirserver is just the standard worries about performance21:08
tvossrobert_ancell, it's still overhead on the phone that we could save from my pov21:08
robert_ancellAre we sure this is a problem yet?21:08
mterryrobert_ancell, running as a mir client we realized today will hit some problems when we want to launch some apps like camera or phone in greeter mode21:08
robert_ancellyes21:09
tvossrobert_ancell, I can tell you we need to optimize Mir even without a system-compositor on the phone21:09
mterryrobert_ancell, not sure, but likely.  I think the work to fix it got de-prioritized (according to kgunn)21:09
robert_ancelltvoss, obviously we need to optimise. But both u-s-c and the shell are going to get this performance hit. Is special casing the greeter going to give us any benefit?21:09
tvossrobert_ancell, it would close the loop on a system-level shell and a session-level shell21:10
mterryrobert_ancell, if we can't run u-s-c as non-root, our current thinking is to have the greeter be a normal client, but run camera and phone in-process as local qml plugins...21:10
tvossrobert_ancell, iiuc, the greeter apps need to run in some sort of session21:10
Saviqrobert_ancell, who controls sessions' geometry?21:10
robert_ancelltvoss, yes, or be hacked like mterry said21:10
tvossrobert_ancell, out of curiosity: could lightdm talk to just one greeter?21:11
Saviqtvoss, long run, yes, we need a session for them, short run we'll build them into the greeter21:11
robert_ancellSaviq, Mir will detect geometry (in u-s-c) and pass that to the shell/greeter. Then those pass policy back up the chain and u-s-c applies the policy of the active session21:12
tvossSaviq, sure, but if they are qt-apps, they could as well be clients of the usc (system-level shell, if it's a full-blown shell)21:12
robert_ancelltvoss, yes, in theory. But it would be a non-trivial amount of work21:12
Saviqrobert_ancell, how do we animate the session coming in? or its opacity / scaling?21:12
robert_ancellSaviq, the compositor in u-s-c does the animation21:12
Saviqrobert_ancell, yeah, so that's the problem from my PoV21:12
tvossrobert_ancell, how do we implement that? with a custom toolkit?21:12
robert_ancellWhen not animating, u-s-c is just running in bypass21:12
Saviqrobert_ancell, as the greeter needs to sync its own animation (swiping away) with the session coming in animation21:13
kgunnwe won't have android bypass for 13.1021:13
mterrySaviq, as I mentioned before, we are thinking of handling that via telling u-s-c which session is next and it stacks them21:14
robert_ancellwhat mterry said21:14
Saviqmterry, robert_ancell, but that's a one-time thing21:14
mterryand the top session will be partly transparent21:14
robert_ancellkgunn, really? I thought that would be a blocker on the phone?21:14
mterrySaviq, one-time?21:14
Saviqmterry you put your finger on the right edge, start swiping left21:14
kgunnrobert_ancell: well...the more we "need" it depending on the outcome of this discussion that might be true21:15
Saviqmterry, the animation of the greeter going away and the session coming in behind it21:15
robert_ancelltvoss, the UI implementation of the greeter in u-s-c wouldn't be hard, but it's a matter of making the link with the daemon and handling the concept of the greeter not being launched and owned by the daemon21:15
Saviqmterry, they need to be synchronized21:15
mterrySaviq, yeah, but u-s-c has both surfaces and can sync them21:15
kgunnbut if we are 10% worse than we could be under load....maybe we could live with it21:15
Saviqmterry, it's not about surfaces21:15
kgunnin the end we want it...but its an optimization post 13.10 at this point21:15
Saviqmterry, because the greeter's surface can't move21:15
Saviqmterry, because the panel needs to be drawn all the time21:15
Saviqmterry, it's not supposed to be swiped away21:16
Saviqmterry, although I agree there's a conflict there21:16
Saviqmterry, because how do we transition between the greeter panel and the session panel21:16
tvossmterry, robert_ancell I'm afraid that the compositing logic in usc if we do it without qt will take significant effort21:16
robert_ancelltvoss, why can't we use qt in u-s-c?21:17
mterrySaviq, interesting point about the panel21:17
tvossrobert_ancell, hmmm, I thought that was one of the reasons to keep the greeter out of the usc21:17
Saviqmterry, robert_ancell, tvoss I'm afraid of more of the same that we're now facing with syncing two scenegraphs...21:17
mterrySaviq, I had assumed panel would be animated too, but I'm guessing you're right that mockups don't show panel moving21:17
Saviqmterry, yeah, and that's how it's implemented now21:18
robert_ancelltvoss, no, the greeter is out of u-s-c because a) that's the way display management traditionally works (to avoid the cost of supporting that) and b) security benefits (greeter is unprivileged user, we can destroy / recreate it and ensure it has safe state)21:18
mterryrobert_ancell, tvoss: qt in u-s-c is partly bad because of security issues, which are only true if u-s-c is root21:18
robert_ancellmterry, do we have security concerns with Qt?21:19
mterrySaviq, well, how it's implemented now was just convenience, not because that was the final word21:19
Saviqmterry, sure, that's why we're discussing it at all ;)21:19
mterryrobert_ancell, we just don't typically run it as root21:19
robert_ancellObviously, the less we put in u-s-c the safer from security and stability, but I don't see other issues21:19
Saviqrobert_ancell, both a) and b) don't convince me when we run u-s-c out of root21:19
Saviqrobert_ancell, we had the same discussion for user session21:19
mterryrobert_ancell, but it's not just qt as root that's the problem, it's running the whole greeter as root21:19
robert_ancellSaviq, a) doesn't come for free21:20
tvossrobert_ancell, what do we gain from the traditional approach? trying to understand that21:20
Saviqrobert_ancell, that unstability is an issue21:20
Saviqrobert_ancell, which we shot down21:20
robert_ancelltvoss, not having to rewrite a whole bunch of code21:20
robert_ancellFinishing in time for 13.10!21:20
Saviqrobert_ancell, I think we're trying to look beyond 13.1021:20
Saviqrobert_ancell, at least I am21:20
mterrySaviq, I'm mostly looking at 13.10  :)21:21
robert_ancellSure, but do we have to solve this immediately? Do we even know there's a performance problem yet?21:21
Saviqrobert_ancell, mterry, what's possible for 13.10 is one thing - and cutting corners is fine21:21
Saviqrobert_ancell, not a performance one21:21
Saviqrobert_ancell, but a complexity one - from my PoV - yes21:21
robert_ancellSaviq, implementing design?21:21
Saviqrobert_ancell, yes, implementing the design in a split u-s-c/greeter is going to be difficult21:22
mterrySaviq, handling the panel transition?21:22
robert_ancellSaviq, right, which is why I thought the greeter would be in the shell for 1.021:22
Saviqmterry, switching between greeter apps, too21:22
robert_ancellThat's the only way you're going to have true flexibility for complex transitions, at the security cost21:22
robert_ancellUltimately wherever you put the greeter process, if it's going to be run as a different user then there's going to be complexity in doing a transition21:23
Saviqrobert_ancell, sure, but we're adding another level of complication for no security benefit, IIUC21:23
mterrySaviq, well, that is something either the normal shell code handles or we do manually if we've got core-app qml plugins21:23
kgunnrobert_ancell: i think its pretty firm we're not gonna have greeter in shell21:23
Saviqmterry, assuming all greeter apps run in a single guest session - which might be ok21:24
robert_ancellkgunn, right, so that's why I don't think there's an easy solution for this. Putting the greeter in u-s-c is not going to make things significantly easier than having it in a separate process.21:24
Saviqmterry, and probably is better for performance reasons21:24
mterrySaviq, or more likely for 13.10, the greeter's own session21:24
tvossmterry, tbh, both approaches seem to be hacky21:24
Saviqrobert_ancell, it is, IMO21:24
Saviqtvoss, you'd see a session per-greeter-app?21:25
robert_ancellSaviq, I'm not seeing an argument here showing what will be easier21:25
tvossSaviq, nope, I would like them to be clients of the usc/system-level-shell21:26
Saviqrobert_ancell, session management is not going to be split between greeter and u-s-c21:26
tvossSaviq, in my ideal world, admittedly21:26
Saviqtvoss, that means running under the greeter user21:26
Saviqtvoss, which might not be desirable21:26
Saviqtvoss, even if it doesn't run as root21:26
tvossSaviq, fair point, so you are thinking about a guest session?21:26
Saviqtvoss, yes, at least one - or even one per greeter app21:26
robert_ancellSaviq, what part exactly of session management21:26
kgunnSaviq: i'm thinking like tvoss...but i've missed why its be undesirable21:27
Saviqrobert_ancell, geometry, applying effects, transitions, animations21:27
kgunni know something about panel handling....but i don't understand21:27
mterrySaviq, seems to be complicating things even more to have each app run as a different user21:27
mterrythan the greeter21:27
Saviqmterry, I agree it might be overkill - a single session would probably suffice21:28
Saviqkgunn, panel - look at your device, swipe the greeter away21:28
mterrySaviq, even there, we've now got apps in greeter running as a different user than the greeter.  We've already got a complicated setup before that...21:28
Saviqkgunn, panel stays in place21:28
tvossSaviq, let's assume the greeter user is not root. apps would be severely restricted/confined21:28
Saviqkgunn, if u-s-c treats greeter and any other session the same way21:28
tvossSaviq, don't see the benefit of a separate user session21:28
Saviqkgunn, swiping the greeter away would mean swiping the panel away21:29
robert_ancellSaviq, when you say "panel stays in place" you mean the panel shown in the greeter and the panel in the session21:29
Saviqrobert_ancell, yes, which is actually conflicting slightly21:29
mterrySaviq, u-s-c knows which is greeter.  It could animate the rest of the screen and ignore the panel pixels...21:29
Saviqrobert_ancell, as user session can have more indicators than system "session"21:29
mterrySaviq, and then swap into normal session indicators after animation is done, "instantly" changing the set of indicators21:30
Saviqmterry, see? exactly my point ;)21:30
mterrySaviq, it is?21:30
Saviqmterry, for ever such exception (sure - with current design I can only think of one now) we'll be dealing weird things between greeter and u-s-c21:30
Saviqmterry, but design isn't set in stone, I'm afraid ;)21:31
Saviqas we've learned21:31
robert_ancellSaviq, both panels are owned by different processes and different users in any case - so the transition is always going to be hard21:31
robert_ancellUnless you put the greeter in shell21:31
kgunnmterry: so is panel like launcher while greeter is shown?21:31
mterrySaviq, OK...  but we can adjust as we go.  As long as u-s-c knows about the greeter, we can treat it special21:31
Saviqmterry, which is exactly something I want to avoid21:31
mterrykgunn, what?21:31
SaviqI dunno, I feel like we're having the same discussion again21:32
kgunnmterry: widgestized21:32
mterrySaviq, but...  special-greeter solves several problems.  You want to avoid it because you think it is overcomplicated architecture?21:32
Saviqunity8 at least has some control over surfaces21:32
mterrykgunn, yeah, panel and launcher are separate widgets, shared between unity and greeter code21:32
Saviqgeometry, opacity and whatever we say is needed21:32
Saviqwith greeter, you seem to be moving even that away from the greeter21:33
=== racarrlunch is now known as racarr
Saviqand treat is as a dumb application, shoe-horning some exceptions like for the panel etc.21:33
tvossmterry, how do you envision greeter and usc to communicate?21:33
SaviqIMO it's going to be a huge pain21:33
kgunnmterry: so when you swipe....panel stays in place...but shell takes over ? (e.g. new render is from shell process)21:33
mterrytvoss, actually lightdm and u-s-c communicate, and they already have a channel for it21:33
Saviqkgunn, there needs to be some transition21:34
Saviqkgunn, it could be just a cross-fade synchronized with how far you've dragged the greeter21:34
tvossmterry, so greeter talks to lightdm, lightdm talks to usc and vice versa on what is essentially a side-channel?21:34
Saviqkgunn, there actually isn't design for it, 'cause all design assumes they're going to be the same (which actually might still be the case)21:34
mterrytvoss, for this sort of stuff, we don't need to add anything more to the greeter<->lightdm channel21:35
Saviqkgunn, i.e. the indicators are going to look the same, just their contents may be different for greeter (like no messages, just their count, for example)21:35
mterrytvoss, but lightdm and u-s-c do talk on a side channel right now21:35
Saviqkgunn, so there's no visible transition between the two - so as soon as the user session is unlocked - we'd just flip the greeter one "off"21:35
tvossmterry, how do we synchronize to the right edge swipes?21:35
tvossmterry, who is the leader in this scenario? usc, greeter or lightdm?21:36
mterrytvoss, the plan so far was to have the greeter be semi-transparent and the session below would bleed through21:36
mterrytvoss, u-s-c is the "leader" I'd guess?  Not sure what you mean by that21:36
kgunnbrb21:36
Saviqmterry, which is inconsistent with visual design21:36
tvossmterry, who controls the operatoin? who controls what happens when?21:36
mterrySaviq, how is that inconsisten?21:37
tvossand I guess the control flow21:37
Saviqmterry, or by "bleed through" you mean that the whole greeter is swiped away21:37
Saviqmterry, and the session is just there?21:37
mterrytvoss, lightdm tells u-s-c when the user changes and thus which session to place underneath the greeter; u-s-c can manage the transitions21:37
tvossmterry, who tells lightdm that the user has changed?21:38
mterrySaviq, right.  so the greeter marks the section on the right as transparent.  u-s-c places the next session beneath the greeter.  So you'll see the session underneat21:38
Saviqmterry, so u-s-c is going to have handle input as well to follow the finger when swiping the greeter away, right?21:38
mterrytvoss, greeter does (like it does now on desktop)21:38
Saviq-have21:38
mterrySaviq, no...  it just puts the session beneath21:38
mterrySaviq, greeter is animating itself moving out of the way, thus making more and more of itself transparent21:39
Saviqmterry, ok, who scales the session in and un-darkens it?21:39
mterrySaviq, maybe you and I have seen different mockups for this21:39
Saviqmterry, no I think that's actually fine21:39
tvossSaviq, it's working like that on the phone right now, isn't it?21:39
Saviqtvoss, yes, until that point that's correct21:40
mterrySaviq, but regardless, u-s-c can scale and modify the session surface, since it owns it21:40
Saviqmterry, yes, who tells u-s-c what scale and darkening / opacity to apply at any given time?21:40
Saviqmterry, and by what means?21:40
mterrySaviq, we could build it in, we could have a config file, we could make it part of the protocol between lightdm and it...  I hadn't thought that was a big problem21:41
mterrySaviq, oh you mean during the animation?21:41
Saviqmterry, yes21:41
mterrySaviq, again, I feel like you are operating on a different mockup than I have.  I imagine the current greeter right now, which just shows the session below21:42
Saviqmterry, what this means to me is that we need a protocol that will, for each frame, communicate from the greeter to u-s-c what geometry, scale, opacity, and any other effects, to apply on the session surface21:42
Saviqmterry, no it doesn't21:42
Saviqmterry, don't unlock with password21:42
Saviqmterry, or pin21:42
Saviqmterry, drag away21:42
mterrySaviq, ah yes, you are right21:42
Saviqmterry, and even with a button21:42
Saviqmterry, there's a progressive transition of the "session" coming in21:42
mterrySaviq, I so rarely get past the greeter, my memory was fuzzy.  Did we change that, or have I always been so unobservant  :)21:43
Saviqmterry, it was always like that21:43
Saviqmterry, it isn't when app's in focus21:43
Saviqmterry, which is a bug, I think21:43
Saviqotherwise we have a problem of syncing the greeter/u-s-c *and* user session21:43
mterrySaviq, that does complicate things a bit more yeah21:43
Saviqbut that could actually be ok, if we just pass a 0.0 ÷ 1.0 progress value from the greeter to the session21:44
SaviqI don't expect that we'd need anything more than that21:44
tvossSaviq, but still: it is a side-channel we are opening up21:45
robert_ancellSaviq, mterry, we did discuss that with the input from the greeter being converted into a transition percentage and being given to u-s-c21:45
mterrySaviq, sure21:45
mterryrobert_ancell, yar21:45
Saviqtvoss, yes, but I don't think we can avoid it completely, but we should try and minimize it21:45
mterrytvoss, that side channel already exists, but yeah, it would be using it more21:46
Saviqrobert_ancell, yeah, but IMO it's different if we're talking about that between greeter and session, which obviously have a huge boundary between them - and that's correct21:46
Saviqrobert_ancell, and different when we've split u-s-c and greeter21:46
mterrySo what's the outcome here?  We continue with greeter-as-usc-client for 13.10...  We can handle transitions in usc by marking greeter and talking to it?21:48
mterryFor 14.04 maybe we can make greeter a real mirserver, assuming performance is ok...21:49
mterryOr a hail-mary of being able to run usc as non-root21:49
robert_ancellI agree with mterry. We don't know of performance problems yet, the complexity is manageable, the risk is lowest continuing with what we have. And we have options up our sleeve if we need them21:51
mterryrobert_ancell, well, I was mentioning trying to make greeter a pure-client for 13.10, and go back to a mirserver for 14.0421:52
Saviqmterry, robert_ancell, I'm game with that for 13.10, I'm just asking for an open mind (and paths) for the future :)21:52
mterryrobert_ancell, but either way, performance needs to be investigated21:52
robert_ancellmterry, It has no effect on me if it's a client or a server - it just makes it more complex for you if you need to launch apps from the greeter21:53
mterryrobert_ancell, yup.  I'd rather it is mirserver, that is far easier to implement21:53
mterryrobert_ancell, but tvoss and kgunn are worried about the performance there, since we won't have bypass it sounds like21:54
Saviqrobert_ancell, racarr, on another topic:21:55
Saviqmirserver/mir/default_server_configuration.h: No such file or directory21:55
Saviqare we missing a build dep? or was something not yet released?21:55
mterrysounds like missing libmirserver-dev21:56
Saviqmterry, yeah, that's what I thought21:57
robert_ancellkernel panic, please repeat anything directed at me since "robert_ancell, yup.  I'd rather it is mirserver, that is far easier to implement"21:58
Saviq robert_ancell, but tvoss and kgunn are worried about the performance there, since we won't have bypass it sounds like21:58
Saviqrobert_ancell, ↑ that was mteryy21:58
Saviqmterry, even21:59
robert_ancellSaviq, ta21:59
mterryAlso:21:59
mterry<Saviq> robert_ancell, racarr, on another topic:21:59
mterry<Saviq> mirserver/mir/default_server_configuration.h: No such file or directory21:59
mterry<Saviq> are we missing a build dep? or was something not yet released?21:59
mterry<mterry> sounds like missing libmirserver-dev21:59
Saviqmterry, yeah, and that, didn't think it was needed ;)21:59
mterryrobert_ancell might have some deep insights!22:00
robert_ancellSaviq, are you using pkg-config?22:00
robert_ancellalso, full log?22:00
Saviqrobert_ancell, this is the unity-mir integration effort, not cleaned up enough, yet22:00
Saviqrobert_ancell, https://launchpadlibrarian.net/145781977/buildlog_ubuntu-saucy-armhf.unity8_1%3A7.81.3%2B13.10.20130718ubuntu.unity.next-0%2B201307241850~129_FAILEDTOBUILD.txt.gz22:01
* Saviq tries to sbuild22:01
robert_ancellSaviq, oh, that's the wrong include22:04
robert_ancellit should be <mir/default_server_configuration.h> and the include path set by pkg-config22:05
Saviqrobert_ancell, wrong as in it won't work? or wrong as it shouldn't be used?22:05
Saviqrobert_ancell, ok, so that's different - we're just missing a build dep22:05
Saviqrobert_ancell, we'll be cleaning this up later22:05
robert_ancellbut it appears to be <mirserver/mir/default_server_configuration.h>22:05
robert_ancellIt works because the file is /usr/include/mirserver/mir/default_server_configuration.h22:06
robert_ancellsure22:06
Saviqrobert_ancell, yeah, noted22:06
Saviqrobert_ancell, ok, needs to be fixed now, as the includes inside there fail anyway22:07
robert_ancellSaviq, lp:unity-mir?22:08
Saviqrobert_ancell, no, that's lp:~unity-team/unity8/unity8-integrate-mir/22:08
Saviqor is it22:09
Saviqrobert_ancell, yes, lp:unity-mir, actually22:09
Saviqrobert_ancell, and it seems to be missing libmirserver-dev dependency22:09
robert_ancellSaviq, that would do it :)22:09
robert_ancellSaviq, in the .pro files? It does have it in debian/control22:10
Saviqrobert_ancell, only for build22:10
Saviqrobert_ancell, not for runtime22:10
Saviqrobert_ancell, which it needs for anyone building against it22:11
Saviqrobert_ancell, there's no unity-mir-dev package split yet22:11
Saviqrobert_ancell, and *that's* what should have the dep22:11
robert_ancellOh the binary unity-mir needs a dep on libmirserver-dev?22:11
robert_ancellbinary package unity-mir22:11
Saviqrobert_ancell, yes, because it doesn't have -dev split out of it22:11
robert_ancelland does shlibs not work? It has a bunch of explicit dependencies on libraries22:12
Saviqrobert_ancell, libraries, shlibs will never add a -dev dep, will it?22:12
Saviqrobert_ancell, -dev packages usually depend on -dev ones explicitly, IIRC22:12
robert_ancellSaviq, no, but it should add the dependency on libmirsever022:12
robert_ancellSaviq, correct22:12
Saviqrobert_ancell, ok, but the build in question (the failed one)22:13
Saviqrobert_ancell, is unity822:13
Saviqrobert_ancell, *against* unity-mir22:13
robert_ancellSaviq, if unity-mir is a dev package it should be called libunity-mir-dev or similar22:13
robert_ancellah22:13
Saviqrobert_ancell, of course22:13
Saviqrobert_ancell, and it needs the -I for mirserver in the .pc22:13
Saviqhmm it doesn't seem to have a .pc at all22:14
robert_ancellSaviq, it needs the dependency on the libmirserver .pc file, not an explicit -I22:14
robert_ancellyeah, just noticed that too :)22:14
Saviqrobert_ancell, or that, indeed22:14
* Saviq adds22:14
Saviqrobert_ancell, ah, it actually generates one22:16
robert_ancellkgunn, oh, nice diagram of the greeter options btw. Just saw it now22:19
kgunnanything settled in my short hiatus ?22:20
Saviqrobert_ancell, hum http://pastebin.ubuntu.com/5909168/22:22
Saviqrobert_ancell, that's the .pc it generates22:22
Saviqrobert_ancell, so it's looking good, not sure why it doesn't pick up the Requires :/22:23
robert_ancellkgunn, did you see from <mterry> So what's the outcome here...?22:24
kgunnrobert_ancell: nope22:24
robert_ancellSaviq, it shouldn't require the mircommon, but otherwise looks correct22:24
robert_ancellkgunn, See http://irclogs.ubuntu.com/2013/07/24/%23ubuntu-unity.html22:25
robert_ancellI tried to copy and paste it, but it only wants to copy one line for some reason22:25
robert_ancellSaviq, what does a manual pkg-config --cflags show?22:26
Saviqrobert_ancell, yeah, it's right, I'm trying to build now22:26
mterryrobert_ancell, do you know where "Failed to load platform plugin "ubuntumir"." message is coming from?  (it lists ubuntumir as one of the available platforms, trying to find why it didn't load)22:27
Saviqrobert_ancell, might be something was b0rked in my sbuild22:27
Saviqrobert_ancell, yeah, seems to build fine now, verifying22:27
robert_ancellmterry, no, racarr knows perhaps? Sounds like something in the Qt layer22:27
Saviqrobert_ancell, http://bazaar.launchpad.net/~mir-team/unity-mir/trunk/revision/20 looks sane?22:38
robert_ancellSaviq, yes22:40
Saviqrobert_ancell, how about http://pastebin.ubuntu.com/5909228/22:45
robert_ancellSaviq, the compile line doesn't have the -I includes in it for Mir22:46
robert_ancellSo looks like a build system issue22:46
Saviqrobert_ancell, right, I wonder why it at all wants to include it, though...22:47
Saviqrobert_ancell, actually, that's -j922:47
Saviqrobert_ancell, so the message probably doesn't really count..22:47
* Saviq tries -j122:47
robert_ancellSaviq, if the code is as it was, it's explictly including default_server_configuration.h using the whole path (bad) and that needs the header from mircommon. But since the -I/usr/include/mircommon is not on the command line (should be provided by pkg-config) it fails22:48
Saviqrobert_ancell, ok, let me fix those includes, then22:49
mterryrobert_ancell, so where are we with the create-surface-with-session-pointer branch?22:58
robert_ancellmterry, otp, will get back to you22:58
mterryk22:58
Saviqrobert_ancell, d'oh... stupid cmake doesn't fail if pkgconfig's requires are not met... we were missing libplatform-api1-dev and that caused pkgconfig to crap out, but cmake let it go through - never setting cflags up23:07
mterrymzanetti, oh btw, did I point you at the ofono backend branch I started?23:10
Saviqasac, see the pass rate for unity8 in http://91.189.93.67/staging/daily/ ? ;)23:15
robert_ancellmterry, so the short answer as I understand it is "will be better solved once the scene graph work is completed". There was a suggestion which was the initial method I was planning to use (grudgingly) of watching the surfaces be created and creating the mapping yourself. Do you think that is feasible as a stop gap?23:22
robert_ancellmterry, I've also talked with racarr, and hopefully he might be able to help out with the mir side of u-s-c next week23:23
mterryrobert_ancell, what is the scene graph work?23:25
robert_ancellmterry, https://lists.ubuntu.com/archives/mir-devel/2013-July/000307.html23:26
mterryrobert_ancell, watching the surfaces be created is what I was originally trying to do, but we don't have a connection to the session at surface creation time...23:26
mterryrobert_ancell, will read that link.  I'm about to go out door23:26
robert_ancellmterry, ok, talk to you more tomorrow23:27
robert_ancellmterry, the solution might be to have the link in the creation method, not in the object (as it can go out of scope)23:27
mterryrobert_ancell, that's exactly what my branch does, I thought...23:27
robert_ancellmterry, oh, I'll re-check it23:28
robert_ancellmterry, yes, you pass it to creation, but you also store it in mir::shell::Surface23:33
mterryrobert_ancell, ah....  to pass it on?  I don't remember.  I didn't make an accessor for it or anything I don't thin23:33
mterryk23:33
mterrybut yah, we shouldn't store it.  I can clean that up23:34
mterryunless we have to store it to pass it on...23:34
=== qengho_ is now known as qengho

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