/srv/irclogs.ubuntu.com/2013/10/08/#ubuntu-unity.txt

ricmmracarr: https://code.launchpad.net/~ricmm/unity-mir/shell-surface-monitor/+merge/18974300:01
kgunnkdub: what is interesting, is that i can get 80-100% cpu pretty consistently on top when i scroll the application lens on the dash with suggested apps expanded00:01
ricmmthat kinda fixes all (plus your shell keyboard focus for keys)00:01
racarrricmm: *deep bow*00:02
Saviqricmm, racarr and why did we not have that by default?00:02
Saviqs/by default/to start with/00:02
ricmmgood question00:02
ricmmwe do in SF00:03
racarrbecause through the whole twisted chain of input00:03
racarrit was at one point decided we wouldn't use it in mir00:03
racarrthen I forgot it was an option because we've never used it in mir00:03
* Saviq was under the impression we couldn't do it00:03
Saviqugh00:03
racarrits still not...00:03
racarrwell00:03
racarrI mean with the monitor stuff00:03
racarryou end up with doubled events00:03
racarri.e. your gestures end up going throuhg to the app as well00:03
ricmmyup00:03
ricmmwell not to the app, they go to the shell00:04
racarrsure00:04
ricmmbut the shell can still reorganize its input regions00:04
kgunnracarr: ricmm ...so net-net...forget the branchs "default-input-focus" and "mir-map-audio-keys"....and work on landing "shell-surface-monitor"00:04
ricmmand it wont be delivered events outside of those regions, no?00:04
racarrkgunn: Ok well either way we get mir-map-audio-keys00:04
ricmmno no, this is just an idea00:04
ricmmand we still need the keyboard focus00:04
racarrricmm: No, it will get all events if its a monitor (irregardless of input region)00:04
ricmmas its a different path00:04
Saviqricmm, racarr, I think we'll get dupe events for InputAreas, though?00:05
ricmmyes00:05
ricmmwell not really dupe, as those events will be eaten and wont make it to the application surface00:05
Saviqricmm, dupe for shell00:05
ricmmin the non-monitor case00:05
racarrin the monitor case, the shell should only get one event (the monitor event)00:06
Saviqricmm, once we get them due to monitoring, second time due to InputAreas set up00:06
racarrbut the shell will never eat events...00:06
racarrI think the input areas wont get input if monitoring is on00:06
racarrbecause the input areas are just a filter on the surface input00:06
Saviqright, so won't be able to block input - no go00:06
racarrbut would have to do some investigation00:06
racarrmm00:07
racarrthats the problem00:07
racarrI really need to stretch my legs...00:07
racarrbe back soon00:08
ricmmSaviq: right, thats the thing... in the android app manager InputArea's also injected a null input area in the regions specified00:08
ricmmso that the event wouldnt hit the application00:08
ricmmalthough I think that is happening right now00:08
Saviqricmm, except the shell *only* gets input in InputAreas00:09
ricmmwhats a good way to test it?00:09
Saviqricmm, it's a flipped model00:09
ricmmin the SF model the shell gets input everywhere00:09
Saviqricmm, under SF, we got all input, and were able to tell SF to not deliver events to apps in certain areas00:09
Saviqricmm, under Mir, we add areas we want input for, and those don't get to the app00:09
Saviqricmm, so currently it's either-or, not both, as in SF00:10
Saviqricmm, either the app gets the events or the shell takes them over00:10
ricmmI think we are still blcokign where theres an InputArea00:10
Saviqricmm, yes, of course00:10
Saviqricmm, that's what I mean00:11
ricmmso whats the issue?00:11
Saviqricmm, we need places where we listen, but *don't* block00:11
Saviqricmm, i.e. monitor - but it can't be a blanket like your branch00:11
ricmmwe listen in all places whrees theres no InputArea00:11
Saviqricmm, no can do, according to what racarr said00:11
Saviqricmm, once you're a monitor, you're a monitor and that's it, you don't have control over the events00:12
Saviqricmm, i.e. InputAreas don't matter anymore00:12
Saviqricmm, and don't receive any input - so can't block it, either00:13
Saviqanyway...00:13
Saviqo/00:13
Saviqmore tomorra00:13
ricmmone sec00:13
ricmmwell it doesnt seem to be true00:14
ricmmSaviq: I just tested and I'm not receiving input events in the InputArea's00:14
ricmmin client apps00:14
ricmmwhere an input area is set, the shell sees the monitor event but the app is not delivered anything00:14
ricmmso it looks like it does block the region00:14
ricmmSaviq_: two saviqs !00:15
ricmmSaviq_: yea, input is blocked fine by the InputAreas00:17
ricmmracarr: ^00:17
kgunnricmm: hey...do you know if we're on a motley, trinity or M-kernel ?...or none of the above ? (or do any of those even mean anything to you?)00:22
ricmmthey mean nothing to me00:23
ricmmwhat are they?00:23
kgunnricmm: so...i was reading about gpu clocking on xda...http://forum.xda-developers.com/showthread.php?p=3637381100:28
kgunnricmm: i know we use CM kernels...wondered if they were related in any way...seems not....00:29
ricmmcertainly not a random kernel00:29
ricmmonly CM-standard ones00:29
ricmmracarr: looks like being a monitor helps to get key events... but only when the application receives them00:30
ricmmfor some reason the webbrowser is not getting such events00:30
ricmmwhile calculator is00:30
ricmm:00:30
ricmmooohhhhh00:30
ricmmI get it now, the webbrowser spawns a secondary client which is the one in focus, but it is a transparen one00:32
ricmmthats why events arent being delivered to the main client00:32
ricmmbut all other apps work fine and you can even see the volume keys00:32
ricmmalthough it needs the qtubuntu branch to fix the keycode mappings00:32
ricmmracarr: kgunn so I think we can agree that a one line fix is less risky than the injector interface MR + associated injection code in unity-mir00:33
ricmmfor now, post 13.10 is another story00:33
kgunnricmm: yes...one line sounds better than 2 related code changes in 2 diff components :) racarr ?00:39
ricmmwell its not just that00:45
ricmmit needs the focus-setter branch as well00:45
ricmmotherwise there will never be key evens unless a window is focused00:45
ricmmbut at least it saves us the injector stuff, that one sounds scary00:45
ricmmfor pre-freeze00:45
ricmmkgunn: sending this out in an email00:52
ricmmas I need to bounce00:52
ricmmracarr: ill be back later for further discussion00:53
racarrBack for potential further discussion02:02
=== _salem is now known as salem_
=== salem_ is now known as _salem
Mirvtsdgeos: pong03:50
=== _morphis is now known as morphis
tsdgeosSaviq: https://code.launchpad.net/~aacid/unity8/start_ssh07:48
Saviqtsdgeos, yup, saw that07:48
tsdgeosok07:48
Saviqpete-woods, ping07:52
pete-woodsSaviq: hi07:52
Saviqpete-woods, hey, think we should get an ABI bump on upstart-app-launch?07:52
Saviqpete-woods, was able to build unity-mir against u-a-l trunk yesterday and it didn't pull the new library07:53
Saviqand was missing symbols for the observers07:53
pete-woodsSaviq: er, I think you have me confused with someone else. I have nothing to do with that lib07:53
Saviqpete-woods, orly?07:53
Saviqpete-woods, sorry, then :)07:53
pete-woodsSaviq: I can certainly do an MR for you07:54
Saviqpete-woods, right, it's all Ted's07:54
pete-woodsokay07:55
Saviqpete-woods, we should probably wait for him, not sure what his policy is on the -dev package version07:55
Saviqas it's slightly different than the default07:55
tsdgeosSaviq: on the "locking" thing, i think i know how to fix it (in Qt), not sure it's the proper fix though (and not sure it'll work, it's still all on my head), but at least it's making sense in my head :D08:04
Saviqtsdgeos, ROCK ON08:07
SaviqMirv_, ping08:16
Mirv_Saviq: pong08:17
SaviqMirv_, hey, do you know if unity-mir is going to be published?08:17
SaviqMirv_, or are we waiting for more?08:18
Mirv_Saviq: I don't know, I'd guess landing of it would be talked in 10 mins in our call08:18
SaviqMirv_, ok thanks08:18
Mirv_"trying to land first thing in the morning", the unity8 part was to me and my morning is earlier, thus it's in08:19
Mirv_lool may land the unity-mir part then08:20
SaviqMirv_, yeah, we might actually wait for the two more fixes down below08:28
Mirv_Saviq: just let lool know08:31
SaviqMirv_, k, will do08:33
tsdgeosaaaaaaaaaaaaaandddddddd it's fixed08:35
* tsdgeos does the "i'm fixing too much of Qt" dance08:35
nic-doffaySaviq, mind taking a look at this? https://code.launchpad.net/~nicolas-doffay/unity8/dismiss-keyboard/+merge/18859908:41
Saviqtsdgeos, \o/08:42
Saviqnic-doffay, you'll need to find someone else, I'm swamped08:42
tsdgeosSaviq: http://pastebin.kde.org/pgn5xeua908:42
tsdgeoscreating the codereview entry now08:42
tsdgeoslet's see what the guys think08:42
Saviqtsdgeos, awesome08:43
nic-doffaySaviq, cool.08:45
SaviqCimi, ↑↑08:45
nic-doffayCimi, can you do a small review for me dude?08:53
Ciminic-doffay, y08:53
tsdgeosmzanetti: greyback: photo of the booth?08:59
nic-doffayCimi, https://code.launchpad.net/~nicolas-doffay/unity8/dismiss-keyboard/+merge/18859909:03
nic-doffayta09:03
nic-doffaymzanetti, ping09:03
tsdgeosSaviq: done at https://codereview.qt-project.org/#change,6752309:24
Saviqtsdgeos, distro-patch straight away?09:25
tsdgeoshmmm09:25
tsdgeosSaviq: tbh i think it should be fine, but then it can always be an awful patch09:26
tsdgeosotoh with devdays around i'm not hoping for people to review the patch "soon"09:27
tsdgeosif this is something we want to have09:27
tsdgeosi'd go for the distro patch i guess09:27
Cimidistro patch! :)09:28
Saviqtsdgeos, do09:32
* tsdgeos does09:32
tsdgeosMirv_: https://bugs.launchpad.net/ubuntu/+source/qtdeclarative-opensource-src/+bug/123676509:42
ubot5Ubuntu bug 1236765 in qtdeclarative-opensource-src (Ubuntu) "[PATCH] Fix infinite loop in QQmlIncubator::forceCompletion" [Undecided,New]09:42
tsdgeosSaviq: ↑↑09:42
=== iahmad is now known as iahmad|afk
nic-doffaySaviq, quick question, any idea what file the category expansion is handled for the scopes?09:50
Saviqtsdgeos, tx!09:54
Saviqnic-doffay, E_SYNTAXERROR09:54
Saviqdandrader, want another ~simple task re: input?09:55
Saviqdandrader, with packages from http://people.canonical.com/~msawicz/mir-input/09:55
nic-doffaySaviq, looking into this bug: https://bugs.launchpad.net/unity8/+bug/122455209:55
ubot5Ubuntu bug 1224552 in Unity 8 "[Dash] Category expansion transition has varaible speeds" [High,Triaged]09:55
dandraderSaviq, yeah, I was actually looking at what to do next, considering the scramble to get unity in mir as default...09:55
Saviqdandrader, input goes through OSK when in shell09:56
Saviqdandrader, we have a MouseArea to catch that in Shell.qml09:56
Saviqdandrader, but unity-mir does not have ApplicationManager.keyboardVisible .keyboardHeight implemented09:56
Saviqdandrader, it does, though, have OSKController09:57
Saviqdandrader, we need to get the visible and height out of the OSKController09:58
dandraderSaviq, didn't get the  "input goes through OSK when in shell" part09:59
Saviqdandrader, search in dash, type - you'll launch stuff from the dash09:59
Saviqdandrader, both OSK and shell gets the input, so we need to eat it behind the OSK in Shell.qml09:59
dandraderah, ok.10:00
dandraderwe are starting to have a scary amount of workarounds :)10:00
Saviqdandrader, yeah...10:00
mhr3eh, flashing failed for me, can get into recovery, but phablet-flash doesn't work from there, ideas?10:00
mhr3ehm, not here10:01
=== davmor2_ is now known as davmor2
Saviqdandrader, bug #123677310:02
ubot5bug 1236773 in Unity 8 "[mir] Need an input trap in shell behind the OSK" [Critical,In progress] https://launchpad.net/bugs/123677310:02
Saviqmhr3, pass -d codename10:02
Saviqmhr3, i.e. -d mako or something10:02
mhr3thx, got help in -touch already10:03
Saviqmhr3, how do I test scope-isactive easiest?10:13
Saviqmhr3, start with no network, connect?10:14
mhr3Saviq, no, it doesn't have the networking part yet10:14
mhr3Saviq, installing a click should trigger it10:14
mhr3Saviq, or adding media over mtp... theoretically10:14
Saviqmhr3, k, checking10:14
Saviqmhr3, the qml test got fixed somehow, btw? ;)10:15
mhr3Saviq, oh???10:15
mhr3Saviq, i thought you said you *really* really need to fix that10:15
Saviqmhr3, == make it pass ;)10:15
Saviqmhr3, the test was correct - passing locally10:15
Saviqmhr3, just not on jenkins10:15
Mirv_tsdgeos: ok. is it currently under control without the patch or are there infinite loops happening elsewhere? I'm just wondering about the (proposing) scheduling of the fix, with the mir switch etc upon us10:15
mhr3hmm, magic fixes.. why not10:15
=== dandrader is now known as dandrader|afk
mhr3if we have magic breaks, this just balances it10:15
Saviqmhr3, we'll see in a few - have restarted the qmluitests10:16
Mirv_I can anyhow fire up a PPA with that it can be tested from10:16
SaviqMirv_, we had to revert something in unity8 that caused a lock up with this10:16
SaviqMirv_, that's not to say we won't get them with what we already have in unity8...10:16
SaviqMirv_, it's sheer luck we didn't hit it yet10:16
SaviqMirv_, but yeah, a PPA to test would be nice10:17
tsdgeos+1 for the ppa10:17
Mirv_adding DEP-5 headers etc and uploading then10:19
om26erMacSlow, re: Snap decisions. we don't have a button for 'show password' while connecting to a wireless network ?10:25
om26eratleast its not in the implementation. I was wondering what the design says10:25
Saviqdandrader|afk, got everything you need?10:27
Saviqom26er, it's not there in the spec10:27
om26erSaviq, uh! regression for some (not me)10:28
om26ermy wif is open :)10:28
Saviqom26er, https://docs.google.com/a/canonical.com/document/d/1puQ9Z0yKqzsQ1VQ1OOBkxgp78iWGnAhAkFXWJFTWIrE/edit#heading=h.30kqxmjkvr4b10:28
om26er*wifi10:28
Saviqom26er, scary!10:28
Mirv_yeah I missed it while testing, otherwise it was neat10:28
om26erSaviq, right, will report a bug for that. I think for people with complex passwords that can be a pain10:29
Saviqom26er, I completely agree10:29
Saviqom26er, I remember in Windoze when you had to type a 26-character DSL router password...10:29
Saviqblind...10:29
SaviqTWICE...10:30
om26erSaviq, btw re: Mir slowness. Its seem kdub is getting closer. The performance is satisfactory with the command he gave :)10:30
Saviqom26er, yup, saw that10:30
om26erouch!10:30
MacSlowom26er, that's what the design is like atm... no option for that10:31
nic-doffaySaviq, do you know where I can find the code to fix this bug report? https://bugs.launchpad.net/unity8/+bug/122455210:31
ubot5Ubuntu bug 1224552 in Unity 8 "[Dash] Category expansion transition has varaible speeds" [High,Triaged]10:31
=== dandrader|afk is now known as dandrader
mhr3Saviq, would you know what dimensions of a preview image do we need for apps?10:32
Saviqtsdgeos, can you point nic-doffay somewhere ↑?10:32
Saviqmhr3, 22x13 GU by default10:33
dandraderSaviq, I think so10:33
mhr3Saviq, and 1gu on mako is...?10:33
Saviqmhr3, 18 I think10:33
mhr3thx10:33
tsdgeosSaviq: nic-doffay: give me a sec10:33
Saviqmhr3, yes10:33
tsdgeosnic-doffay: filterAnimation in FilterGrid.qml10:35
Saviqnic-doffay, you need SmoothedAnimation with velocity set instead of Duration10:36
nic-doffaytsdgeos, ta10:36
Saviqnic-doffay, http://qt-project.org/doc/qt-5.0/qtquick/qml-qtquick2-smoothedanimation.html#velocity-prop10:36
nic-doffaySaviq, cool reading up on that SmoothedAnimation.10:37
* om26er thinks there is room for improvement re: scrolling in the Dash. 10:39
Saviqom26er, sure there is10:51
Saviqom26er, UShape breaks asynchronicity10:51
tsdgeoslol someone created in Qt a bytearray in my honour :D10:51
tsdgeos./tests/auto/corelib/io/qurl/tst_qurl.cpp:2251:        QByteArray tsdgeos("http://google.com/c?c=Translation+%C2%BB+trunk|");10:51
Saviqtsdgeos, !! congratz!10:52
om26erSaviq, so the UIToolkit needs to fix UbuntuShape ?10:52
Saviqom26er, it's coming10:52
Saviqom26er, there's a rewrite of UShape in the works10:52
Saviqom26er, and we'll revisit when it's there10:52
om26erSaviq, good to know. Thanks!10:53
Saviqmhr3, icon theme ubuntu-mono-dark? in -isactive?10:53
mhr3really? did i do it again?10:53
SaviqOOPS10:53
Saviqcan we have staging in bzr plz? pretty plz? kthxbai10:54
=== MacSlow is now known as MacSlow|lunch
mhr3Saviq, pushed10:54
Saviqmhr3, tx10:55
mhr3maybe i should make a real fix that will set it to mono-dark on desktop :)10:55
Saviqmhr3, damn, we broke autopilot tests there ;)10:56
Saviqmhr3, they depended on isCurrent ;)10:56
mhr3nooooooooooooooooooooooooooo10:56
Saviqwhich doesn't happen now until you unlock10:56
om26erSaviq, one last question :) If you open an app from inside another app (gallery -> pull toolbar -> tap 'camera') there is no indication that the app is being opened unless it fully appears. I was told its something to be fixed in unity810:56
om26erits more of an issue when you open a video/song from dash10:57
Saviqom26er, yes, and unity-mir10:57
Saviqmhr3, ugh, click scope refreshes More suggestions onIsActiveChanged...10:58
dandraderSaviq, I'm getting that when runnning unity8 manually  -> "what():  display factory cannot create fb display"10:58
dandraderSaviq, what am I missing?10:58
Saviqdandrader, stale app running?10:58
dandraderSaviq, no10:59
Saviqdandrader, if you didn't close the apps before killing unity810:59
mhr3Saviq, well... not a shell problem10:59
Saviqdandrader, try power button then10:59
Saviqmhr3, of course it's not, but...10:59
Saviqbut but... ;10:59
Saviq;(10:59
dandraderdoesn't help...11:00
Saviqdandrader, hrmpf - you running as phablet?11:02
Saviqdandrader, ssh or sudo -u phablet -i ?11:02
* dandrader tries "adb reboot"11:02
mhr3Saviq, click scope is being super stupid, emits invalidation inside search... looks easy to fix11:02
dandraderSaviq, I logged with "ssh phablet@..."11:02
Saviqdandrader, that's a mako?11:03
dandraderSaviq, maguro11:03
Saviqdandrader, try in #ubuntu-mir, I'm empty11:03
Saviqmhr3, ok, /me looks into fixing ap tests, then - the feature is working11:04
mhr3Saviq, thanks11:04
Saviqdandrader, I assume it is working from upstart?11:05
Saviqdandrader, you sure unity8 is stopped?11:05
dandraderSaviq, yes11:05
Saviqdandrader, yeah, I'm empty...11:05
dandraderSaviq, but at least rebooting solve it (as in I can see a unity8 again)11:05
dandraderSaviq, btw, I do have to "phablet@ubuntu-phablet:~$ export QT_QPA_PLATFORM=ubuntumirserver" right?11:06
Saviqdandrader, no11:07
Saviqubuntumirclient is fine - unity8 "converts" that internally11:07
Saviqdandrader, but should work nevertheless11:07
dandraderSaviq, hmm.... gonna try with QT_QPA_PLATFORM=ubuntumirclient then11:07
dandradernah, same problem11:07
Saviqdandrader, that ↑ is exported for you already11:11
Saviqmhr3, at least it's easy to see that isCurrent is working ;)11:12
om26erWho is the QE with unity8 team ?11:12
Saviqom26er, veebers11:12
mhr3Saviq, that reminds me, http://paste.ubuntu.com/6208875/11:12
Saviqmhr3, pushed fix for autopilot11:13
mhr3Saviq, wanna add that to the branch ^?11:13
Saviqmhr3, doing11:13
om26erSaviq, ack, we need a test: bug 1236796 -- I might work on it in the weekends if its not written by someone else.11:13
ubot5bug 1236796 in unity8 (Ubuntu) "[Autopilot test needed] Make sure closing an app from dash keeps dash in focus" [Medium,Triaged] https://launchpad.net/bugs/123679611:13
Saviqmhr3, lowercase B11:14
mhr3hmm, i could swear i tested it on the device11:15
mhr3then again, oneliner, i guess i just rewrote it11:15
Saviqmhr3, pushed11:15
Saviqmhr3, let's see what jenkins says11:15
Saviqmhr3, interrupting the current run to incorporate the latest two commit11:16
Saviqs11:16
SaviqI love jenkins... "Stop this build forever"11:16
SaviqFOREVER11:16
Saviq(alone)11:16
mhr3makes you feel like you have superpowers, doesn't it? :P11:17
Saviq;)11:17
mhr3Saviq, with do qt fix do we get weather and scopes renderer back?11:17
mhr3s/do/the/11:18
Saviqmhr3, yes!11:19
mhr3and we really, really need video renderers fixed, they look horrible11:19
mhr3i'm going to bother Cimi with that :)11:19
Saviqmhr3, yup, do that ;)11:20
dandraderanyone here uses a maguro for development?11:21
mhr3i do11:21
Saviqdandrader, I just flashed, will try11:21
dandraderSaviq, seems I have a fix for you bug. gonna push the proposals11:22
dandraderit was simple indeed11:22
Saviqdandrader, yup, cool11:25
Saviqdandrader, unity8 by hand worked fine here11:26
Saviqdandrader, just stop unity8; unity811:27
Saviqdandrader, making sure that display is lit11:27
dandraderbugger11:27
dandraderbtw, is there a powerd command or something to ensure the display doesn't turn off?11:27
=== alan_g is now known as alan_g|lunch
Saviqdandrader, yeah, sudo stop powerd ;)11:32
Saviqdandrader, there is powerd-cli that should work, in theory11:32
Saviqguys use it in automation11:32
=== hikiko is now known as hikiko|lunch
dandraderSaviq, https://code.launchpad.net/~dandrader/unity8/fix_lp1236773/+merge/189821 and https://code.launchpad.net/~dandrader/unity-mir/fix_lp1236773/+merge/18982211:38
Saviqdandrader, yup, looking good11:39
Saviqdandrader, we *could* do OSKController for SurfaceFlinger, but it probably doesn't make sense now ;)11:40
om26erDo we plan to merge this anytime soon? https://code.launchpad.net/~saviq/unity8/fix-frequent-app-ids/+merge/18853211:40
om26erif not how about removing Facebook and gmail icons from the Home scope ?11:41
Saviqom26er, after https://code.launchpad.net/~ted/url-dispatcher/app-id-url/+merge/187913 most probably11:41
dandraderSaviq, not so fast. spotted an issue11:44
Saviqdandrader, ;P11:45
Saviqmhr3, isactive lookin' good, passed ap11:47
Saviqmhr3, passed qmluitests11:47
mhr3Saviq, yey!11:47
Saviq-touch is to follow11:47
Saviqbut I'm happroving already11:47
dandraderSaviq, yeah, running unity8 from console works after stopping powerd. \o/11:50
Saviqdandrader, cool11:55
=== MacSlow|lunch is now known as MacSlow
dandraderSaviq, ok, now the fixes seem to be really ready to go12:01
Saviqdandrader, coolio12:01
=== dandrader is now known as dandrader|lunch
Saviqdandrader|lunch, hum? you disable the InputArea? isn't that how the OSK *ever* got input/12:02
=== Mirv_ is now known as Mirv
=== alan_g|lunch is now known as alan_g
=== _salem is now known as salem_
=== hikiko|lunch is now known as hikiko
=== iahmad|afk is now known as iahmad
=== jhodapp|afk is now known as jhodapp
Ciminic-doffay, do we have a bgreport or a design guideline for that?12:48
Ciminic-doffay, I tested it, works but I don't like the interaction12:48
CimiI think we can do better12:48
Saviqmhr3, it's our lucky day - merged on first try ;D12:52
mhr3Saviq, feels like sweeping an issue under the rug and waiting for another branch to fail on it :/12:54
Saviqmhr3, no, really - the test was correct12:54
Saviqmhr3, I didn't see what could be happening there12:54
mhr3Saviq, then we're sweeping under the rug how much the ci infra is broken :)12:55
Saviqmhr3, btw, YOU SHOULD SEE MY RUG!12:55
mhr3lol12:55
mhr3i can imagine those dead bodies there... you probably have a big rug, right?12:55
=== dandrader|lunch is now known as dandrader
dandraderSaviq, that's just the initialization of that value13:04
CimiSaviq, who is responsible of the dash design?13:05
CimiSaviq, was reviewing mic branch but I don't like the UX13:05
dandraderSaviq, it's overwritten later by a manual connection at the end of OSKController.qml13:05
Ciminic13:05
Saviqdandrader, right!13:05
SaviqCimi, visual? Rosie13:06
CimiSaviq, UX13:06
SaviqCimi, Patricia13:06
CimiSaviq, ok I'll come up with my proposal of UX and wait her feedback later13:06
om26ermhr3, the carousel in the music scope does not make sense. That should only be used for Albums not individual songs.13:07
mhr3om26er, design wanted that, we raised the same question, they had their reasons13:08
om26erright now I am starting songs blindly :/13:08
om26ermight be better to just open their preview first13:08
mhr3om26er, you'd want https://code.launchpad.net/~unity-team/unity8/switching-previews/+merge/189556 then :)13:09
mhr3Saviq, is that one going in btw ^?13:09
Saviqmhr3, if mzanetti fixes13:09
om26erSaviq, that branch opens the preview instead of directly opening the song ?13:09
Saviqom26er, yes13:09
Saviqmhr3, I'd like it to, but there's an issue13:09
mhr3mzanetti, it's not just pretty, but useful as well, pls fix ;)13:09
om26ermhr3, heh, I meant to ask you :) but Saviq already answered. Thanks13:10
dandraderping kgunn13:19
kgunndandrader: pong13:22
tsdgeosCimi: can you test the packages at https://bugs.launchpad.net/bugs/1236765 with https://code.launchpad.net/~aacid/unity8/unrevertr388/+merge/18983913:22
dandraderkgunn, is that a blocker for Oct 10th? https://bugs.launchpad.net/unity-mir/+bug/123460013:22
ubot5Ubuntu bug 1234600 in unity-mir "Contacts Cancel/Save buttons don't work with OSK showing when running Mir" [High,In progress]13:22
ubot5Ubuntu bug 1236765 in qtdeclarative-opensource-src (Ubuntu) "[PATCH] Fix infinite loop in QQmlIncubator::forceCompletion" [Undecided,In progress]13:22
tsdgeosCimi: i'll do the same and thus we have two people testing13:22
Cimitsdgeos, yup13:23
Cimitsdgeos, packages are already built or I build on phone?13:23
tsdgeosCimi: read the bug13:24
kgunndandrader: its not a blocker - but will be on the high visbility list when mir gets turned on...13:24
tsdgeoswhere it says "packages are at foobar"13:24
Cimitsdgeos, ok13:24
Cimitsdgeos, https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-daily/+build/508492813:25
Cimitsdgeos, it's easier to add the ppa13:25
Ciminevermind13:25
tsdgeosyes, it's easier to add the ppa :-)13:25
dednickmhr3: what was wrong with the datetime indicators?13:32
mhr3dednick, not showing up for me in --channel=saucy-proposed13:33
dednickmhr3: um, is that the same as --pending?13:33
mhr3dednick, i was screamed at to not use cdimage13:33
dednickmhr3: er. ok13:34
mhr3but theoretically, yes :)13:34
dednickmhr3: can you see if indicator-datetime is installed?13:35
mhr3dednick, running upgrade right, so apt-cache doesn't talk to me, a file i can check to know?13:36
dednickmhr3: /usr/share/unity/indicators/com.canonical.indicator.datetime13:37
mhr3yep, it's there13:37
nic-doffayCimi, I just tried out my branch, it def dismisses here when you press on anywhere aside from the keyboard/text area.13:39
mterrykgunn, "feeling froggy"13:40
mterrynever heard that13:40
kgunnmterry: you're not american :)13:42
Ciminic-doffay, :-\13:48
seb128is there a known bug that once unity8 is in "close running software" mode (e.g long press in the app lens leading the red cross on the images to close what is running) it's stucked in there until you are done closing running softwares?13:52
tsdgeosseb128: you jut long press again and the [x] should go away, doesn't that work?13:55
Saviqdednick, ping13:56
dednickSaviq: pong13:56
Saviqdednick, re: indicator redrag - using a single EdgeDragArea didn't work well?13:57
Saviqdednick, it's probably too intertwined with the Revealer?13:57
seb128tsdgeos, oh, you need to long press again on one of the program ... I close one, I would expect the mode to end or that starting a new software would stop it13:57
seb128tsdgeos, I didn't expect I should try to click on one of the icons of the softwares I don't want to close and keep pressing13:58
seb128tsdgeos, e.g I was unsure that wouldn't close the thing I was clicking on13:58
tsdgeosseb128: the UX wasn't really well defined when we implemented the feature afair13:58
tsdgeosso yeah, it can for sure be better13:58
dednickSaviq: the Areas fo the initial drags are anchored to the bottom of the page. when you drag down, they go to the bottom of the page.13:58
tsdgeosseb128: needs people that know about that to think about it13:59
Saviqseb128, if you go away from the apps scope it will reset, too13:59
Saviqseb128, at least in trunk - we just merged a fix13:59
Saviqdednick, orly? ok then13:59
dednickSaviq: hang on, i might take another look13:59
dednickSaviq: possibly might be able to reuse the drag down.13:59
seb128Saviq, oh ok, is it worth opening a bug "starting a new software should exit that mode as well"?13:59
Saviqseb128, no, that will happen14:00
Saviqseb128, with trunk14:00
seb128ok, great14:00
seb128you guys are one step ahead as usual ;-)14:00
seb128Saviq, tsdgeos: thanks14:00
seb128Saviq, (if I get what you said, "exit the app scope", includes going to an app, not only changing lens in the dash)14:01
Saviqseb128, yup14:01
Saviqseb128, going to indicators, too14:01
Saviqseb128, or locking the phone14:01
seb128excellent14:01
* seb128 is happy14:01
seb128no bug from me for this one then ;-)14:01
dednickSaviq: hm. doesnt work. EdgeDragArea doesnt have the autocomplete/reject. That's why we need the DragHandles.14:10
dednickalso for the hinting animation.14:10
Saviqdednick, right, another question then - did you talk about the "commit" phase with design guys?14:11
Saviqdednick, it feels too easy to change maybe?14:11
SaviqCimi, you'll do bug #1231937 ?14:12
ubot5bug 1231937 in Unity 8 "Video & music renderers are not to spec" [High,Triaged] https://launchpad.net/bugs/123193714:12
dednickSaviq: not specifically. They said just drag your finger left to right and it should change... i had originally done it so that you had to drag down first.14:12
dednickas in left to right across the indicator  bar14:13
Saviqdednick, ah14:13
Saviqdednick, ok, let's land this14:13
SaviqCimi, bug #1226288 is related14:13
ubot5bug 1226288 in Unity 8 "Carousel should only be used when there's enough items" [Medium,Triaged] https://launchpad.net/bugs/122628814:13
Saviqtsdgeos, can you review https://code.launchpad.net/~stolowski/unity8/category-reordering/+merge/189263 - you've been involved with that (can be *after* you do delegates ;))14:15
tsdgeosSaviq: sure, will do14:16
dednickSaviq: let me just double check with oren14:16
Saviqdednick, already happroved ;P14:16
dednickSaviq: not anymore ;)14:20
Saviqdednick, you!14:20
dednickSaviq: apparently the design has not gone through... JohnLea told me to do it, but now oren says no...14:21
Saviqdednick, eh...14:21
dednickyup14:21
Saviqdednick, k14:21
dednickmhr3: datetime is present for me. Can you check if the indicator-datetime process is running?14:25
mhr3dednick, i updated to daily, it's fine now14:26
dednickmhr3: ok :/14:26
CimiSaviq, yep both14:26
SaviqCimi, cool14:26
kgunngreyback: are you actually out? https://plus.google.com/hangouts/_/6f7b22053ff6d243c9ba67afbf759cbf9d0c5e6a14:31
kgunnkdub: ^14:32
Saviqdandrader, were you ever able to look at / reproduce https://bugs.launchpad.net/unity8/+bug/1228336 ?14:33
ubot5Ubuntu bug 1228336 in Unity 8 "[DDA] Crash when revealing indicators" [Critical,Triaged]14:33
mhr3sil2100, ping?14:33
dandraderSaviq, no, is it showing up its ugly face?14:33
Saviqdandrader, no14:34
Saviqdandrader, does the trace make any sense, though?14:34
Saviqdandrader, or shall we just mark incomplete?14:34
dednickFree Karma: https://code.launchpad.net/~nick-dedekind/unity8/indicators-client.item-title/+merge/18986814:34
dandraderSaviq, it does14:35
dandradermake sense14:35
=== charles_ is now known as charles
dandraderSaviq, although it's a mystery how is it possible for it to get into such state14:37
Saviqdandrader, mhm, mark Incomplete then?14:37
sil2100mhr3: pong14:37
sil2100mhr3: what's up?14:37
Saviqdednick, MINE14:38
dednickSaviq: :)14:38
mhr3sil2100, hey, we're seeing a crasher in mediascanner tests, identified it to be grilo bug, mind distropatching it?14:38
dednickone of my easier ones14:38
dandraderSaviq, yeah, incomplete as there are no steps to reproduce it14:38
dandraderin a reasonably reliable way14:38
mhr3sil2100, it's in  lp:~mhr3/+junk/grilo-fix-123663714:38
sil2100mhr3: sure thing! Where's the bug/patch/branch?14:39
mhr3^^14:39
sil2100Doing14:39
Ciminic-doffay, not everywhere14:39
nic-doffaydednick, got a moment to test out a branch for me?14:39
Ciminic-doffay, searching from the home screen14:39
CimiI still can click elements on the application section14:39
dednicknic-doffay: funny man.14:39
dednicknic-doffay: but yeah, sure14:39
nic-doffayCimi, I just tried it again on the home screen.14:40
nic-doffayIt dismissed when I select anywhere?14:40
Ciminic-doffay, I think your improves the situation14:41
Ciminic-doffay, with the keyboard14:41
Ciminic-doffay, but it's still unclear to the user when it will dismiss or not14:41
nic-doffaydednick, i'm a comedian, what can I say? https://code.launchpad.net/~nicolas-doffay/unity8/expansion-transition-fix/+merge/18987214:41
nic-doffayCimi, that's not part of the bug though?14:41
Ciminic-doffay, which bug?14:42
nic-doffayWhen it lands design can go over it.14:42
nic-doffayCimi, I thought I'd linked to the branch, apparently not.14:43
nic-doffayCimi, here's the bug: https://bugs.launchpad.net/unity8/+bug/121303414:43
ubot5Ubuntu bug 1213034 in Unity 8 "Can't dismiss keyboard by tapping outside of search entry" [High,In progress]14:43
nic-doffayI've lined it to the MP now too.14:43
nic-doffay*linked14:43
Ciminic-doffay, so, it's not fully working for me14:46
Ciminic-doffay, I invoke the search entry in the home screen14:46
Ciminic-doffay, keyboard and recent searches menu appear14:46
Ciminic-doffay, I touch on "applications" category header14:47
sil2100mhr3: do you have a bug for the crash btw.?14:47
Ciminic-doffay, it expands the applications category but does not dismiss the search box, even if it's empty14:47
mhr3sil2100, don't you wonder what is that number in the branch name? :P14:47
sil2100mhr3: pfff, some random numbers14:48
sil2100;)14:48
mhr3sil2100, or do you want upstream bug?14:48
mhr3i'll open that14:48
kdubkgunn, ping14:49
dednicknic-doffay: reviewed.14:50
nic-doffaydednick, cool ta14:50
nic-doffayCimi, that's weird man it def works for me...14:51
tsdgeosnic-doffay: both of you on surfaceflinger or mir?14:52
nic-doffaytsdgeos, no mir here14:52
Cimidednick, can you try https://code.launchpad.net/~nicolas-doffay/unity8/dismiss-keyboard/+merge/188599 ?14:54
kgunnkdub: pong14:54
nic-doffaydednick, switched to equivalent grid units dude.14:55
nic-doffayTotally slipped my mind.14:55
dednicknic-doffay: i'm not sure about this. we should probably have a max time. it can take quite a while to open if there are lots of entries... have you checked with design?14:58
=== dandrader is now known as dandrader|afk
nic-doffaydednick, not yet Oren wasn't online.15:02
nic-doffayLet me check if he's around.15:02
dednicknic-doffay: he's not at his desk15:02
nic-doffayI assumed there shouldn't be since it wasn't mentioned in his bug report.15:02
nic-doffayBut lets hold off until I double check.15:02
dednicknic-doffay: my file scope has a hundred entries or so, and it takes quite some time to close...15:03
dednickCimi: testing now15:04
dednicknic-doffay: lp:~nicolas-doffay/unity8/dismiss-keyboard has conflicts.15:05
mhr3dednick, https://wiki.ubuntu.com/Touch/Testing15:09
dednickmhr3: thanks15:09
mhr3sil2100, once you have the patch, can you link it to lp:1236637 pls?15:13
kgunnSaviq: need this one https://code.launchpad.net/~dandrader/unity8/fix_lp1236773/+merge/18982115:13
kgunnas well as your mp that's waiting to merge right?15:13
Saviqkgunn, yeah15:13
sil2100mhr3: suar15:13
kgunnanymore other than those 2 ?15:13
Saviqkgunn, no15:14
kgunncool15:14
mhr3sil2100, txh15:14
Saviqkgunn, refocus-shell should land within minutes (and if it fails on mako I'll just push it through)15:14
Saviqah, SUCCESS15:14
Saviqscared, huh!15:14
=== dandrader|afk is now known as dandrader
Saviqkgunn, the other one just started15:16
Saviqaaand I'm off for now... did not get up from this chair yet today15:16
kgunnSaviq: please don't injure yourself15:16
mhr3kgunn, on the other hand that would ensure he doesn't move from there :)15:17
sil2100dednick: hi! Is ubuntu-settings-components ready for release? And daily-release?15:18
kgunn:)15:18
dednicksil2100: yup15:18
sil2100dednick: awesome!15:18
Saviqmhr3, ;D15:19
mhr3Saviq, j/k, go enjoy the sun, while there's still some left15:20
Saviqmhr3, I *did* merge scope-isactive today did I not!15:24
Saviqmhr3, you can try and fix switching-previews in your spare time ;)15:24
mhr3Saviq, yes, you restored my faith in humanity, thank you15:24
sil2100mhr3: package uploaded! Sponsored by kenvandine, it's building now ;)15:31
mhr3sil2100, awesome, thanks15:32
mhr3Cimi, here's the spec https://docs.google.com/a/canonical.com/presentation/d/1X3_G6reCpcFJxzxn1Xtv9wK5n3-wOW-6z2l0qSZ8-sg/edit#slide=id.g103714763_4715:35
kgunndandrader: since greyback is out...do you know if unity-mir launching mir is running mir with user permissions or as root ?16:07
kgunnracarr: ^ ?16:07
dandraderkgunn, afaik it's running as phablet user16:08
dandraderkgunn, why?16:09
kgunndandrader: :)16:09
kgunndandrader: we're talking about moving the mir_socket to a diff dir associated with a running user session16:09
Saviqkgunn, /run/user/32011/mir_socket or something16:11
kgunnright talking about how to do that w/o breaking xmir as well16:12
Saviqright16:12
Saviqfginther, http://10.97.0.26:8080/job/generic-mediumtests-runner-maguro/2161/console stuck?16:14
fgintherSaviq, look like16:14
fgintherSaviq, I took the node offline16:15
kgunnSaviq: yeah...i just reapproved dandrader's mp to get some jenkins love....16:17
dandraderEOD16:25
=== jhodapp is now known as jhodapp|vet
CimiSaviq, albert is OED16:41
CimiSaviq, I tested the qt fix and is working16:41
dednickcharles: ping16:52
dednickkgunn: it would appear that it's the location service backend that is misbehaving.16:55
om26ermterry, hey :)17:04
=== alan_g is now known as alan_g|EOD
dednickcharles: location indicator doesnt seem to be working correctly. action activations arent doing anything on backend. Seem not to be getting updates from platform api.17:08
SaviqCimi, comment on bug #123676517:08
ubot5bug 1236765 in qtdeclarative-opensource-src (Ubuntu) "[PATCH] Fix infinite loop in QQmlIncubator::forceCompletion" [Undecided,In progress] https://launchpad.net/bugs/123676517:08
SaviqCimi, ah you did17:08
SaviqCimi, ok then, Timo will put it in our qt tomorrow and we'll unrevert then17:08
mterryom26er, what's up?17:10
om26ermterry, bug 123696017:10
ubot5bug 1236960 in Unity 8 "[Mir] greeter matrices are not updating, requires unity8 restart" [Medium,New] https://launchpad.net/bugs/123696017:10
mterryom26er, marked as dup17:11
mterryom26er, of bug 123490417:11
ubot5bug 1234904 in unity-mir (Ubuntu) "[mir-only] infometric values are not updated" [Critical,Confirmed] https://launchpad.net/bugs/123490417:11
om26erhaah17:11
mterryom26er, a fix on the way, just needs to be reviewed17:11
om26ermterry, cool. thanks17:11
MirvSaviq: I'll need to run all AP:s to convince the rest of the integration team, but yes I'll look at that in the morning and find a sponsorer then17:22
Mirvg'night.17:22
=== jhodapp|vet is now known as jhodapp
mterrySaviq, if the greeter is visible, should the focused app still be focused?17:53
* mterry thinks not for the same reason that we unfocus on display-off17:56
Saviqmterry, no18:21
mterrySaviq, cool, fixing that optimization makes it easier to detect when a snap decision launches an app18:21
kgunnSaviq: fginther ...i notice we're still waiting on https://code.launchpad.net/~dandrader/unity8/fix_lp1236773/+merge/18982118:30
kgunnhowever, we're hoping to get that into qa test...18:30
kgunnare we confident ci will run ok ?18:30
kgunnor should we consider manually landing onto trunk (ci run be damned)18:31
Saviqkgunn, let me push that through18:31
fgintherkgunn, it's building right now, let me see how far it is18:31
Saviqkgunn, right, it's building atm18:32
kgunnwhat's the normal turn time....like 30 minutes?18:32
Saviqkgunn, more, currently, I'm afraid18:32
Saviqkgunn, 48 mins on a good run18:33
Saviqkgunn, 1:48 on a bad one (when queued up)18:33
fgintherkgunn, Saviq, the mako/maguro tests are queued waiting for the current build to finish.18:34
fgintherkgunn, Saviq, it should start within 10 and another 20 to run18:34
Saviqfginther, kgunn yup18:34
Saviqkgunn, it's green up 'til now - if it fails on touch, I'll push through18:35
kgunnack, thanks Saviq18:35
fgintherkgunn, Saviq, the desktop tests passed18:35
Saviqfginther, yup18:37
Saviqkgunn, for future reference, you got access to rebuild jobs on s-jenkins? you want them?18:38
kgunnooo...i like that18:39
kgunnhow do i get that18:39
Saviqkgunn, register at http://10.97.0.26:8080/signup18:41
Saviqkgunn, and fginther will set you up for access to triggering rebuilds18:42
Saviqkgunn, looks like you won't get the chance this time - things look green for that MP18:43
kgunnSaviq: never been so happy to be disappointed :)18:47
fgintherkgunn, Saviq is this stuck? http://10.97.0.26:8080/job/unity-phablet-qmluitests-saucy/2178/console19:04
fgintherkgunn, Saviq, it's a ci run for https://code.launchpad.net/~unity-team/unity8/albumart-provider/+merge/18992319:05
fgintherbut it looks to be blocking the autolanding you're interested in19:05
kgunnwell that explains why the merge wasn't showing up19:05
fgintherkgunn, should I kill and retrigger the -ci jo?19:06
kgunnfginther: yes, please makes sense19:06
Saviqfginther, yeah, looks like it19:07
fgintherSaviq, kgunn, it failed a qmluitest, do you want it pushed as is19:10
fginther?19:10
Saviqfginther, already pushed19:10
fgintherSaviq, :-)19:10
Saviqfginther, we should probably skip that test - it's unstable under Qt 5.019:11
Saviqkgunn, it's merged19:11
kgunnSaviq: ack, thanks19:12
tedgSaviq, ping, you want to bump ABI?19:22
Saviqtedg, or even API - I managed to build unity-mir packages against new u-a-l and install them, just to get missing symbols19:25
tedgOh, this is because we don't have a symbols file...19:26
tedgNo one tell didrocks19:26
Saviqtedg, ;)19:26
tedgSaviq, So I think if I just add a symbols file we should be good there, because then dpkg will make the version you depend on based on the symbols you use.19:26
thomi_Saviq: still awake?19:28
=== thomi_ is now known as thomi
Saviqthomi, yup, here19:32
thomiSaviq: do you know what would be required to fix this? https://bugs.launchpad.net/unity8/+bug/123702219:32
ubot5Ubuntu bug 1237022 in Unity 8 "Silence unknown option messages" [Undecided,New]19:32
thomiSaviq: is it as easy as not passing those merssages to Qt itself?19:32
thomiif so, I'll propose a branch myself that fixes it19:33
Saviqthomi, sure, but how would you know?19:33
thomihow would you know what?19:33
Saviqthomi, which options are "known" and which are not?19:33
thomiwell, I assume that unity8 knows "I handle options X, Y, and Z", everything else pass to Qt itself19:33
Saviqthomi, and then there's Mir19:34
Saviqthomi, and the QPA19:34
Saviqthomi, but anyway, it's not unity8 that spits out those unknown options warnings19:34
Saviqthomi, it's Qt, AFAICT19:34
thomiSaviq: I thought it was Qt19:35
thomiyeah19:35
Saviqthomi, yeah, so you'd have to know all options Qt and Mir and QPA handle to filter out those unknown19:35
thomiso if I make sure -testability and -fullscreen have been removed before passing argv to Qt, that should be OK?19:35
Saviqthomi, sure, we can pop19:35
thomiSaviq: ideally yes, but in reality it's only ever these two options that get printed19:35
Saviqthomi, but that won't mean we'll actually silence it ;)19:35
Saviqthomi, granted, false negatives will be silenced19:36
thomiSaviq: it'll still print if someone uses some mir / QPA option, but that seems rare19:36
Saviqthomi, not a huge gain if you ask me ;)19:36
Saviqthomi, we generally need to silence Qt's and QML warnings19:36
thomihmmm19:37
Saviqthomi, only problem is... that can only be done build-time with current Qt's logging facilities19:37
thomiugh, really? bum19:37
thomiSaviq: do you know whwre I should start looking inside Qt for that stuff?19:37
thomiroughly which class/file it's in?19:37
Saviqthomi, yeah, 5.3 is supposed to have a more sophisticated logger19:37
Saviqthomi, http://qt-project.org/doc/qt-5.0/qtcore/qdebug.html19:37
thomiSaviq: ahh, it uses QDebug - where is that called though?19:38
thomitell you what, I'll stop being lazy and look for it myself :)19:38
Saviqthomi, good boy!19:38
thomi:)19:38
thomineed more coffee first though19:38
Saviqthomi, or well - http://qt-project.org/doc/qt-5.0/qtcore/qtglobal.html#qInstallMessageHandler19:40
Saviqthomi, we could make and install one that would be quiet by default19:41
Saviqand only enable debug on some env var and/or argument19:41
thomiSaviq: I was hoping to find where in QApplication (or wherever) those log messages are generated, and see if I couldn't override that code to be silent19:41
Saviqthomi, that does *not* sound like the correct solution ;)19:41
thomiwhy's that? We want the other Qt debug output, just not those messages19:42
Saviqthomi, what's so bad about those two>19:42
Saviqthomi, and anyway - just pop them from argv when you handle them, is all19:42
thomiSaviq: OK, that makes more sense - I'll do that19:42
thomiSaviq: they really clutter up the AP test logs19:43
thomimaking the test logs much less useful19:43
Saviqthomi, I usually use QStringList::removeAll(const QString&)19:44
Saviqthomi, it spits out the number of items removed19:44
Saviqthomi, so if(argv.removeAll('-fullscreen') > 0) { foo; }19:45
thomiSaviq: gotchya19:46
tedgSaviq, https://code.launchpad.net/~ted/upstart-app-launch/symbols-file/+merge/18995119:46
Saviqtedg, should you do REPLACEME there?19:48
tedgSaviq, Since I bumped the version I don't need the merge to do it.19:48
Saviqtedg, right, it will always be >= 0.219:48
kgunnSaviq: lool> kgunn: so I wanted to run the webbrowser-app-autopilot, these stop unity8 and start it again immediately, but it doesn't seem to come up19:50
kgunn<lool> kgunn: like can't unblank display thing19:50
kgunndid we test the webbrowser-app19:51
kgunnap suite?19:51
thomikgunn: presumably the reason it does that is because we don't run in testability mode by default.19:51
thomi...and it needs the shell to be introspectable for some reason19:52
Saviqthomi, no19:52
Saviqthomi, it was lool running with -n19:52
Saviqphablet-test-run I mean19:52
thomiahhh19:52
thomiwell, you can't do that with the mir stack any more I guess19:52
Saviqthomi, well, you *should* be able to19:53
thomiwell, sure, if you start the shell yourself19:53
Saviqthomi, but the powerd issue + the leftover socket makes it difficult enough19:53
thomiSaviq: exactly19:53
thomiand it shouldn't really be the concern of the test case19:53
thomiunless it's testing the shell itself19:54
Saviqthomi, of course19:54
kgunnSaviq: i wanna top approve this one...as i think it will solve _some_ crashes... https://code.launchpad.net/~afrantzis/unity-mir/fix-1236898/+merge/18989420:06
kgunnSaviq: shold i wait for gerry...seems sensible and safe20:06
kgunntho20:06
Saviqkgunn, no need to wait, just top-approve if you think is good - if in doubt - ask someone else to look (like racarr)20:07
kgunnSaviq: he already approved :)20:08
Saviqkgunn, right, so yeah - go for it20:08
thomiSaviq: is the recommended way to build unity8 still './build -s && ./build ' ?20:19
thomiI'll assume that 'CODING' is accurate...20:20
Saviqthomi, yeah20:20
Saviqthomi, bzr bd does fine, too20:20
Saviqthomi, no scary PPAs added this time ;)20:20
thomiSaviq: sweet - it still asks me for my password though, which is kind of scary :)20:21
veebersSaviq: ping, hey is it possible to launch unity8 without the greeter?20:28
thomiSaviq: hey, so when I run unity8 locally with -testability I don't see that output... is there some kind of race condition, do you think?20:30
Saviqveebers, nope20:32
Saviqveebers, there is something mterry added - it listens to "ShowHome" on com.canonical.Unity.Greeter.Broadcast on  the system bus20:33
veebersSaviq: ah ok, so that's a potential solution if people are having 'greeter swipe' issues?20:34
veeberselopio: ^^ I don't know if that helps you at this point?20:34
Saviqveebers, elopio, we *could* add a env var we'd look at, but hopefully we won't need to?20:35
Saviqmterry, ideas about unlocking unity8 for testing ↑↑ ?20:35
elopioveebers, Saviq: at this point, I think we have to live with the fact that our app tests interact a lot with unity.20:35
elopiobut for the future, I would like to split the suites. If we are not testing any integration with unity on a test suite, I would like unity to be as invisible as possible.20:36
Saviqelopio, actually, if you start them with upstart after https://code.launchpad.net/~mterry/unity8/hide-greeter-on-app-launch/+merge/189948 is merged20:36
Saviqelopio, it should just unlock and focus the app20:36
Saviqmterry, correct ↑ ?20:37
elopiooh, that's handy.20:37
thomiSaviq: so that "unknown option" string is not being printed by Qt it seems20:39
thomiSaviq: I wonder if it's unity-mir maybe?20:39
Saviqthomi, no, not there for sure20:40
thomihmmm20:40
thomiI grepped for "unknown option" through the unity source code, and couldnt' see anything that was related to what we're doing20:41
thomierr, I mean "Qt source code"20:41
Saviqthomi, the interesting thing... if I go ./builddir/unity8 -blahblah20:42
Saviqthomi, I don't get it printed out?20:43
thomiSaviq: yeah, me neither20:43
thomiSaviq: which made me think maybe it only happened when you ran on top of mir?20:43
Saviqthomi, https://bugs.launchpad.net/mir/+bug/1226227 then?20:43
ubot5Ubuntu bug 1226227 in unity8 (Ubuntu) "libmirserver parses arguments and fails if it's not something it understands" [Medium,Fix released]20:43
thomiSaviq: no, that's something separate20:44
thomiSaviq: what's more, I'm *sure* this happened with unity8 on SF as well20:44
Saviqthomi, yeah, so QPA-specific maybe?20:44
thomiSaviq: which project is that?20:45
mterrySaviq, yeah, right.  That should work20:45
Saviqthomi, that'd be qtubuntu20:46
thomiI looked there already :-/20:46
Saviqthomi, it's "invalid option" - grepped for that?20:47
thomiyeah20:47
Saviqthomi, so I'm getting it on the device, yes20:47
Saviqthomi, sure it's not QApplication? it shows up just after "WARNING: QApplication was not created in the main() thread."20:47
thomiSaviq: It might be constructing that string in a way that makes it un-greppable20:48
Saviqthomi, yeah, looks like it20:49
Saviqthomi, so - both on SF *and* Mir indeed20:50
thomiSaviq: hmmmm20:51
Saviqthomi, but not on desktop20:51
Saviqthomi, so arguably not QApplication20:51
Saviqanyway, I'm afk20:51
thomiSaviq: thanks for your help20:52
=== salem_ is now known as _salem
mterryIs the ListViewWithPageHeaderTest.testMaximizeVisibleAreaMoveUpAndShowHeader qmltest causing problems for others?22:14
Saviqmterry, we have one that's flaky in Qt 5.0 and it's failing in jenkins sometimes - not sure that's the one, though22:56

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