kklimondayeah, too bad I get some other random issues with unity after restarting it... fun times00:00
bschaefermhr3_, thanks you! Yes it does, but getting it to display the pretty pixels is step one :)00:00
kklimondaI guess unity is just telling me not to play games, and do work00:00
kklimonda(when I restart unity, from time to time, gnome-terminal gets into a weird state where I can't alt-tab to it - i can still click on it to get focus though)00:01
bschaeferkklimonda, :(, usually restarting the terminal fixes that00:02
bschaeferthough i thought we fixed that issue...00:02
kklimondahas you fixed it in the last few days?00:02
bschaeferkklimonda, nope that should have been like a year or so ago...so thats a different bug...00:03
kklimondaah, I'll report it then when I get it to happen again00:03
bschaeferkklimonda, thank you!00:03
bschaeferkklimonda, also the fact that your settings don't change dynamically through ccsm is a bug (as it should be dynamic...)00:06
kklimondabschaefer: I know, I'm really confused on why it's happening00:06
bschaeferyeah.. if you go to preferences and see what backend you're using?00:07
bschaeferas it should be set to gsettings00:07
bschaeferand the intergeneration box should be checked00:07
bschaefer(in ccsm)00:07
kklimondawhere in ccsm?00:08
bschaeferkklimonda, bottom left00:08
bschaeferabove close, and advanced serach00:08
bschaeferyeah its in a strange spot00:08
kklimondayeah, it's GSettings, and the integration box is checked00:08
bschaeferthen im not sure what its not dynamic as i've never ran into that issue...00:09
bschaeferkklimonda, what version are you running of unity?00:09
* bschaefer should have asked this first really...00:09
kklimondabschaefer: the one that comes with 14.0400:10
bschaefero so you've got the most update to date version hmm00:10
bschaefer(thats around for download)00:10
kklimonda7.1.2+14.04.20131106.1-0ubuntu2, I just restarted it00:10
bschaeferthats the version i have installed...let me try running that version as im on trunk untiy00:11
bschaeferkklimonda, did you upgrade to 14.04 or clean install?00:12
kklimondait's a clean install00:12
bschaeferas it still seems to work here hmm00:12
bschaeferkklimonda, ccsm is not a nice application at all00:14
bschaeferTrevinho, if you're still around, have you ever seen the problem where CCSM doesn't actually save the changes?00:14
kklimondait isn't, but then i have used it a few times in the past, and never had this particular issue :)00:15
bschaeferyeah, its having problems saving the changes it seems...00:15
bschaeferkklimonda, could you try install 'dconf-editor' and see if it'll change it this way00:16
bschaeferonce you've dconf-editor, press Ctrl+f (for find), then search for 'show-hud'00:18
bschaeferfrom there you should see the '<Alt>' key, if you just change it to '', it should stop the hud from opening right away00:18
kklimondayeah, I just run apt-get dist-upgrade and unity exploded..00:18
kklimondasigh, that will teach me ;)00:18
kklimondacan't load plugin 'ccp'00:19
bschaeferthats not a good error...umm, this is after a dist-upgrade and reboot?00:19
bschaefercompiz is having a problem starting (not unity this time!)00:19
kklimondano, during dist-upgrade unity just segfaultet00:19
kklimondawell, compiz00:20
kklimondaand then when I try to run in again, it hangs at this error00:20
kklimondadconf-editor shows that the show-hud command is changed, and it works after I restart unity00:20
bschaeferbut not right away?00:20
bschaeferie. not dynamic.00:20
bschaeferthat is very very strange...00:21
kklimondaI can make a clean user and see if it still happens00:21
bschaeferthat isn't a bad idea00:21
kklimondaonce I get my system into a usable state again :)00:21
bschaeferyeah :), umm getting back from that depends on the actual seg fault reason00:21
bschaeferbut hopefully continuing with the upgrade00:22
bschaeferthen a reboot...should fix it00:22
kklimondaccp is not loading because the new libprotobuf changed ABI..00:30
kklimonda(I had to downgrade some stuff)00:30
kklimondait seems to be working under a new account, so I'll just assume something weird happened on my main00:34
kklimondawhich is weird, this installation is like a week old..00:34
bschaefersounds like its looking in the wrong place...00:38
bschaeferwell at lease now its working (though thats never fun to have to make a new account just because of that...)00:38
bschaeferkklimonda, if you get some time to file a bug against compiz?00:38
kklimondayeah, I wonder if it's related to that other bug I reported, where ccsm is not saving changes I make (I tried to disable "Initiate Window Move" but it's always being reverted to the default value if I disable it, and when I set a different combination it's just ignored)00:40
kklimondabug 127369000:40
ubot5bug 1273690 in unity (Ubuntu) "can't disable Alt+Button1 shortcut for moving windows around" [Undecided,New] https://launchpad.net/bugs/127369000:40
bschaeferkklimonda, sounds like it, it seems to be running of different files. ie, its reading one file and saving to a different file00:41
bschaeferthen when you relog it looks at the saved one00:42
kklimondamhm, sounds plausible00:43
bschaeferas all the settings are stored in a single compiled file hmm if you would be able to umm00:45
bschaeferkklimonda, get a strace off of the ccsm that fails to load dynamically00:45
bschaeferthat way I can see if its reading/writing to the wrong files00:46
bschaeferkklimonda, or just upload that log to the bug00:46
kklimondaok, I'll do it tomorrow when I get up :)00:46
bschaefer(it'll be a large file)00:46
kklimondathanks for the help00:46
bschaeferkklimonda, thanks a lot!00:46
bschaeferand np!00:46
Saviqtsdgeos, 'fraid I overridden your review a bit on TextSummary08:23
tsdgeosSaviq: seen it, no worries, good stuff you found :)08:28
Saviqtsdgeos, did you ever notice weirdness with childrenRect when it's supposed to "shrink"? I had to do lines 11-12 of https://code.launchpad.net/~unity-team/unity8/unity8-card-overlay/+merge/204790 in Card.qml for it to behave properly08:30
Saviqtsdgeos, otherwise it'd grow and stay grown, even if the contents shrunk08:30
tsdgeosSaviq: i'm going to approve ted's the table is anywhere MR https://code.launchpad.net/~ted/unity8/on-the-screen/+merge/20385108:30
Saviqtsdgeos, ok ;)08:31
tsdgeosactually no, needs a pot rebuild08:31
Saviqtsdgeos, the other possibility is there's a binding loop and it just doesn't propagate up (which, btw, Qt complains about), but I found no way to break that loop08:32
tsdgeosno, never noticed it08:33
tsdgeosif we have a binding loop it may be that08:33
tsdgeosand maybe the way to break it is how you did08:33
Saviqtsdgeos, only I hate the solution ;)08:39
Saviqbut well :|08:40
tsdgeosSaviq: ok, pushed a branch of the image_preview thing that now is able to merge corrrectly08:51
tsdgeosmhr3_: Saviq: the actions definition is different in "preview definition example" and in "Preview JSONs" on the document09:05
tsdgeoswhich is right? the second one'09:05
mhr3_tsdgeos, yea, second one09:16
=== mzanetti- is now known as mzanetti_
mhr3_tsdgeos, i mean... both are correct :)09:17
tsdgeosmhr3_: ?09:20
tsdgeosah you changed it09:21
tsdgeoscome on!09:30
tsdgeosnow other tests fail!09:30
* tsdgeos jumps out of the window09:30
=== lool- is now known as lool
mhr3_uh oh, developer down!09:38
CimiSaviq, hey man09:44
CimiSaviq, you asked me to change implicitHeight with height09:44
Cimibut it's implicitHeight also the minimum allowed height?09:44
mhr3_sil2100, hey, can we land libunity without the train?09:46
mhr3_sil2100, as in the old lib09:46
tsdgeosmhr3_: Saviq: where do i get the colour of the button for the action?09:54
tsdgeosi've seen some stuff as green, other as orange09:54
mhr3_tsdgeos, design?09:54
Cimiwyluzuj poslady Saviq!09:54
tsdgeosmhr3_: design wants some green, some orange, so where does that come in the json?09:55
CimiSaviq, (I don't know what it means, just told my gf you're picky with my reviews this morning :D)09:56
mhr3_tsdgeos, i'm not convinced that should be configurable... way to make the previews eeek09:56
mhr3_and inconsistent09:56
tsdgeosmhr3_: talk to design :D09:56
mhr3_tsdgeos, pinged rosie09:59
Mirvbregma: heya. I started looking at unity stack a bit yesterday. compiz needed some changelog syncs.  it's working nicely (I updated nux + compiz + unity) on my machine, but I filed a bug #1276516 about the autopilot situation on cu2d.10:42
ubot5bug 1276516 in Unity "unity trunk failing autopilot tests (02/2014)" [High,New] https://launchpad.net/bugs/127651610:42
Mirvbregma: to approach releasibility, we'd need some solution to those or how we'd validate the release via cu2d results.10:43
Saviqtsdgeos, the green part will be defined by the scope author10:44
Saviqtsdgeos, but don't worry about that yet10:44
tsdgeosSaviq: ?10:44
tsdgeosyou mean a property10:44
SaviqCimi, lol10:45
Saviqtsdgeos, yeah, but later10:45
Saviqtsdgeos, for now we're only going orange10:45
tsdgeosSaviq: there's also the width thing10:46
tsdgeosbuttons are "smart" enough to give themselves a proper width10:46
tsdgeosbut the OptionSelector doesn't seem to be able to do that10:46
tsdgeosso do we force say 30gu of width?10:46
tsdgeosor we have a property for that?10:46
SaviqCimi, re: implicitHeight vs. height, it's not "minimal", it's only "default", but in this case the widget won't adapt if you change its height, so it's not really default, it's the height it's meant to be10:46
tsdgeosor try to fix optionselector to be smart?10:47
Saviqtsdgeos, optionselector? where do we use optionselector? you mean combo button?10:47
tsdgeosis there a combo button?10:47
Saviqtsdgeos, there isn't indeed10:47
tsdgeoswell the docu says there is10:47
Saviqtsdgeos, there isn't in the SDK10:47
Saviqtsdgeos, hence we just need to go for multiple buttons now10:47
tsdgeosso i thought it meant option selector10:47
Saviqtsdgeos, nah, it's a different component that's "meant to be" in the SDK10:48
tsdgeoswhich confusingly we then have something called valueselector also10:48
tsdgeosSaviq: so just a row of buttons? or a column?10:48
Saviqtsdgeos, column, yeah10:48
Saviqtsdgeos, actually10:49
Saviqtsdgeos, take what's in Notification.qml10:49
Saviqtsdgeos, it's a poor man's combo button10:49
tsdgeosit's a column10:51
tsdgeosisn't it?10:51
Saviqtsdgeos, there's two buttons, and the left one opens the column10:51
* tsdgeos reads the code more carefully10:53
CimiSaviq, length can be null?10:53
Cimijust wondering about the function10:54
Cimiin case it's null10:54
SaviqCimi, no, but foo["foo"] can be null10:54
tsdgeosSaviq: besides that, you saw that tests failed in mako now?¿10:54
CimiSaviq, modelData["length"]10:54
CimiSaviq, I'm passing this to lengthToString(s)10:54
SaviqCimi, let me see what I commented about10:54
SaviqCimi, I commented about modelData["subtitle"] that it could be null10:55
CimiSaviq, yes10:56
CimiSaviq, but what happens if length is too?10:56
CimiSaviq, it goes to that JS10:56
SaviqCimi, ah well, yeah, you should return "" if it's not a number10:57
CimiSaviq, I just have !== undefined10:57
SaviqCimi, if (typeof s !== "number")10:57
CimiI can drop undefined then10:57
Saviqtsdgeos, grr10:58
tsdgeosSaviq: i wonder how that happened, is because the tests were stopping before because the network ones failed? or?10:58
Saviqtsdgeos, no, a rewrite of the datetime indicator was released, probably changed the name or something, checking10:59
Saviqtsdgeos, or it's a valid failure and the indicator was broken10:59
SaviqCimi, and well, me being picky, is that a new thing?11:04
SaviqCimi, wanna know what "wyluzuj poślady" means?11:05
CimiSaviq, that could be good :)11:05
SaviqCimi, "loosen your cheeks", and it's not about your face ;)11:05
CimiSaviq, she said at first that all polish people are picky :)11:05
CimiSaviq, so what does it means? :)11:06
SaviqCimi, look up ↑↑11:07
CimiSaviq, thought it was something related to smiling11:07
SaviqCimi, in a sense...11:08
CimiSaviq, she said it's something I can tell you in case we are friends11:09
CimiSaviq, so I wrote you :)11:09
SaviqCimi, yup11:10
SaviqCimi, not offensive at all (or well, as usual - depends on how you say it)11:10
CimiSaviq, I fixed your things and added test for NAN11:12
tsdgeosSaviq: can't copy directly Notifications since it expands with left button instead of right, so you meant adapt right? I was thinking to do a base class that both Notifications.qml and this would use, but it'd be a work since behaviour is not exactly the same, so you think it's worth or just copy/adapt the code and wait for the combobox to land in SDK?11:12
Cimiguys, I'll be away for few mins, need to reconnect router with new cat 5e cables11:13
Saviqtsdgeos, we need to expand left button I believe11:17
Saviqtsdgeos, https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgUUY0MjFTQThBZFE/edit11:17
tsdgeosSaviq: that is not "the left button"11:18
tsdgeosSaviq: each button is it's own widget11:18
tsdgeosif the expectation is that in that screenshot11:18
tsdgeosis that left button will do something like notification does11:18
tsdgeoswe can't do it11:18
tsdgeosthey don't know eachother11:18
tsdgeosor we need to rethink the actions thing11:19
Saviqtsdgeos, well, that's the question - don't they...11:19
tsdgeosSaviq: i don't see why, it'd just see left one expanding down11:19
tsdgeosand that's it11:19
Saviqtsdgeos, I think we can make a "smart combo button" i.e. one button - on the right, two buttons - next to each other11:19
Saviqmore buttons11:19
Saviqexpanding from the elft11:19
tsdgeosSaviq: i didn't understand at all what you meant11:20
Saviqtsdgeos, think of a "row of buttons" as one widget of type: "actions"11:21
tsdgeoswhich is not what how we were thinking of it, right?11:21
Saviqtsdgeos, first one of those actions would be the button on the right, second - the button on the left, any more - expanded from the left one11:21
Saviqtsdgeos, not before it was redesigned, no ;)11:21
Saviqtsdgeos, but I think it might've been bad thinking11:22
tsdgeosbut you don't want the left one to do expansion11:22
tsdgeosyou want it to be a button and also expand11:22
tsdgeosotherwise how do you actually click "Add to queue"11:23
Saviqtsdgeos, oh yeah, well, that's the missing combo button11:23
Saviqtsdgeos, but we should be able to do something of the sort the Notification has11:23
tsdgeoswell that is what i was saying, put two buttons together and let it be similar to a combobox11:24
tsdgeosi'll try to code it and we can talk over it later11:24
tsdgeosSaviq: what is actually interesting if you see https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgUUY0MjFTQThBZFE/edit11:24
tsdgeosis why are the buttons one next to another11:25
tsdgeosand not on top of the other11:25
tsdgeoslike all the other preview components11:25
tsdgeoswho decides that?11:25
Saviqtsdgeos, that's the thing - I think that's a single widget11:25
Saviq+meant to be11:25
Saviq/should be11:25
Saviqtsdgeos, so yeah, let's implement it with that in mind11:27
tsdgeosneed a new json for it11:27
Saviqtsdgeos, no we don't11:27
Saviqtsdgeos, type: "actions" takes one or more actions11:27
tsdgeoshow come11:27
tsdgeosand more actions11:27
tsdgeosis a combobox11:27
Saviqtsdgeos, well, that's the thing11:28
Saviqtsdgeos, not any moer11:28
tsdgeosbut how do i put a button and a combo box in one actions11:28
tsdgeoswith that json?11:28
Saviqtsdgeos, put more than 2 actions11:28
tsdgeosand i have 3 buttons11:28
Saviqtsdgeos, no11:28
Saviqtsdgeos, 2 buttons, left a combo11:28
tsdgeosdoesn't make much sense11:28
tsdgeoswhat if i am in a tablet11:29
Saviqtsdgeos, you get two columnt11:29
tsdgeoswhy shouldn't i get to use all my width11:29
Saviqtsdgeos, the widget shouldn't change whether on tablet or phone at all11:29
Saviqtsdgeos, because that's wasting the width11:29
tsdgeosinstead of a button and a combo?11:29
Saviqtsdgeos, that's output from Cape Town11:29
Saviqtsdgeos, no button to span the whole width of a phone11:29
Saviqtsdgeos, as that wastes space11:29
tsdgeossure, i'm not saying that11:30
tsdgeosi'm saying 2 buttons (one a combo) when i could have 3 is a bit weird11:30
tsdgeosbut ok11:30
tsdgeosif you want that11:30
tsdgeosi'll code that11:30
Saviqtsdgeos, if you want three, that's when you put two widgets - one with two actions, one with one action11:31
mzanetti_man... I got a headache today11:32
Saviqtsdgeos, I'll add a comment to the dash spec, 'cause it doesn't deal with that, but I think it's the best route11:33
Saviqtsdgeos, found the issue with the tests - datetime takes longer to load now it seems, and the test didn't wait for it11:35
tsdgeosSaviq: so just to confirm, with this new definition on how the json works, you can never have just a combobox, if you want a combobox, you'll always hace a button on it's right too, correct?11:36
Saviqtsdgeos, yes11:36
Saviqtsdgeos, you get one button, two buttons, or a button and a combobox11:37
Saviqtsdgeos, if you disagree with that, comment please on https://docs.google.com/a/canonical.com/document/d/1n880Fih5KyGPcoP5chidnHDG_8TxXUgSuij7f4rHpuk/edit?pli=1&disco=AAAAAH2p9p411:41
tsdgeosSaviq: i think it limits what people can do11:41
tsdgeosbut i'm not the designer so i'm ok11:41
Saviqtsdgeos, we're doing that all over the place don't we :)11:41
cimitsdgeos, what you disagree?11:41
cimitsdgeos, "I'm not a designer so I'm ok" is not the answer I want to hear from anyone working on my team :)11:42
cimitsdgeos, in some areas you have more experience than designers, so share your issue with me :)11:42
tsdgeoscimi: i'm not disagreeing with designers, i'm disagreeing with Saviq :D11:43
cimitsdgeos, amazing!11:43
tsdgeosi do think that in some case it may make sense to just have a combobox11:43
tsdgeosbut since i am not designing the interface i don't know if that makes sense in the previews or not11:44
tsdgeosand if it does, we can add it later, is not that as if the code i'm writing now won't even change11:44
Saviqtsdgeos, we could add a prop to actions11:44
Saviqtsdgeos, that would say "I want all to go into combo"11:44
Saviqtsdgeos, but yeah, we can add that later if needed11:45
Saviqtsdgeos, https://code.launchpad.net/~saviq/unity8/fix-indicator-tests/+merge/20488911:47
* tsdgeos runs the autopilot tests11:48
* tsdgeos dies of old age11:48
Saviqtsdgeos, try to run them on emulator ;D11:49
mzanetti_he'll be dead before the emulator has booted :P11:49
tsdgeosSaviq: you miss the guidelines thing11:50
Saviqtsdgeos, refresh11:50
tsdgeosah, not anymore11:50
Saviqcimi, 56  if (s <= 0 || typeof(s) !== "number") return "";11:54
Saviqcimi, I think those should be reversed11:54
Saviqcimi, we don't want to even look at whether s <= if it's NaN11:54
Saviqcimi, text: (modelData["subtitle"] && modelData["subtitle"] !== undefined) ? modelData["subtitle"] : ""11:55
Saviqcimi, text: modelData["subtitle"] || ""11:55
Saviqtsdgeos, just a small tweak to the docs: https://code.launchpad.net/~aacid/unity8/image_gallery_preview_widget/+merge/204712/comments/47892712:01
=== MacSlow is now known as MacSlow|lunch
tsdgeosSaviq: do you like the new styling i just pushed more?12:08
Saviqtsdgeos, +112:09
Saviqtsdgeos, I'll put * everywhere after the next release12:09
tsdgeosSaviq: so on the "combo - button" scenario, i'm still unsure of how we want the thing to work12:30
tsdgeosobviously pressing the right button is pressing it, that's easy12:30
tsdgeosbut do we want the combo to have two sensitive areas, one for activate and one for "open"?12:30
tsdgeosand when the combo opens, do we want the first button to hide like in notificationss?12:30
tsdgeosand when the combo is opened, does clicking on one of the opened options activate it? or just select it and close the combo and then you have to click on the combo?12:31
tsdgeosin the non "open" area to activate it12:31
Saviqtsdgeos, *we* do the simplest thing that makes sense12:33
Saviqtsdgeos, I think the combo button is desgined... somewhere...12:33
Saviqtsdgeos, Jouni should have them, he should be online in an hour or two (as should SDK folks)12:37
tsdgeosSaviq: no worky https://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/5078/#showFailuresLink12:46
tsdgeosor maybe it did fix 112:47
Saviqtsdgeos, that's another issue...12:56
Saviqtsdgeos, they broke app launching in the latest image...12:56
Saviqtsdgeos, we'll have to wait for the next one12:56
=== alan_g is now known as alan_g|lunch
dandraderis launchpad slow today or is it my connection?13:13
dandraderhmm, getting several launchpad "Oops!" pages now...13:14
=== MacSlow|lunch is now known as MacSlow
Saviqdandrader, oops from launchpad suggest LP issues, your connection shouldn't have any bearing, no?13:33
Saviqkarni, hey13:33
dandraderSaviq, yes, I got those Oops after I notices the overall launchpad slowness13:33
=== mardy_ is now known as mardy
karnihi Saviq :)13:33
Saviqdandrader, so yeah, it seems pretty slow here13:34
Saviqmhr3_, so, scope::preview() only gives out a single PreviewModel now, is there a multi-column version anywhere yet?13:40
=== alan_g|lunch is now known as alan_g
mhr3_saviq, not finished yet13:47
Saviqaaagh stupid QtCreator... every single time I create a folder with the name of the file I want to create...13:51
tsdgeosSaviq: so arrpove yours that indeed seems to decrease the number of failing tests?13:56
Saviqtsdgeos, well, it fixes that one failure13:56
Saviqtsdgeos, so yeah13:56
Saviqtsdgeos, and we wait for improvement on the other side13:57
mzanetti_Saviq: duude14:01
Saviqmzanetti_, what up?14:01
mzanetti_I constantly get: mzanetti Nickname already in use, try a different one.14:01
mzanetti_Saviq: do a whois mzanetti14:01
Saviqmzanetti_, I am!14:02
* mzanetti_ wonders why you can hold that one even though I am identified @ NickServ14:02
* tedg wonders if IRC explodes if you try to whois yourself14:02
Saviqwhois mzanetti14:03
=== dandrader is now known as dandrader|afk
mzanetti_well, with the /14:03
Saviqyeah yeah :P14:03
Saviqneed to log in to disconnect...14:03
Saviqmzanetti_, here14:04
=== mzanetti_ is now known as mzanetti
mzanettiSaviq: cheers14:05
mzanettistill wondering how this is possible14:05
mzanettibut I never really got irc authentication anyways14:05
Saviqmzanetti, you have to explicitly ask NickServ to get your nick back14:09
tsdgeosghost him!14:09
tsdgeostedg: seen the comment i made in your anywhere MR?14:10
tedgtsdgeos, Yup, just did.14:10
tsdgeostedg: that's going to conflict14:10
tedgK, I'll deal.14:11
tsdgeoswith https://code.launchpad.net/~unity-team/unity8/unity8.previews_textSummary/+merge/20417714:11
tsdgeosbut that's not approved for the moment14:11
tsdgeosso do yours and see if we can sneak it in14:11
tsdgeosthis delayed landing is a pain for merges14:12
tsdgeosbecause you have to think about billions of branches that may or may not conflict with yours14:12
tedgHah, you're telling me!  :-)14:12
tedgI'm starting to loose track of which branches I've told other people to run so they get the functionality they need.14:14
=== sdeziel is now known as sdeziel_
Saviqtsdgeos, http://pastebin.ubuntu.com/6879267/ ?14:17
tsdgeosthat is what i have yes14:17
tsdgeoskind of14:18
tsdgeosyour docu is better14:18
=== dandrader|afk is now known as dandrader
tedgtsdgeos, So I ran it, then I deleted the changes that weren't related to my change.14:19
tedgtsdgeos, That should reduce the conflict14:19
tsdgeosor not14:19
tsdgeoswho knows :D14:19
tedgWell, yes, it should reduce the likelihood of conflict ;-)14:20
karniSaviq: re: your comment, I did that for title only because that was critical fix for us. I'd be happy to continue work in anothe branch, if that's okay with you. https://code.launchpad.net/~unity-team/unity8/new-scopes-title-alignment/+merge/20467714:24
Saviqkarni, ok14:24
Saviqtsdgeos, PreviewActions.qml?14:27
tsdgeosignore that14:27
tsdgeosSaviq: you're complaining on the "s" ?14:27
Saviqtsdgeos, lol14:27
Saviqtsdgeos, no, asking whether that's the name of the widget ;D14:27
* Saviq should be less picky in reviews14:28
tsdgeosi had PreviewActions14:28
tsdgeosmoved to PreviewAction14:28
tsdgeosbut i guess i have to move it to PreviewActions again14:28
tsdgeosgiven the new meaning we gave the json14:28
Saviqtsdgeos, yeah, plural :)14:28
Saviqtsdgeos, well, the json was always "actions" for one-or-more actions :)14:28
tsdgeosi know14:28
tsdgeosbut didn't like it ^_^14:29
mhr3_tsdgeos, saviq, what's the conclusion on the colors? do we have new prop?14:29
Saviqmhr3_, not yet14:29
tsdgeos mumble14:29
tsdgeosmumble: symbol lookup error: mumble: undefined symbol: _ZN6google8protobuf18GoogleOnceInitImplEPlPNS0_7ClosureE14:29
Saviqtsdgeos, yup, no mumble today14:29
tsdgeosi can trying using the android client on the Z1014:30
tsdgeoslet's see how it flies :D14:30
Saviqtsdgeos, I can't :P14:30
Saviq@unity: hangout today, mumble doesn't load with ↑↑14:30
dandraderSaviq, will take me a while to jump on ghangout (have to turn on my laptop, etc)14:31
Saviqdandrader, sure14:31
karniSU notes: attending sprint sessions, letting Saviq kick ass and save the world before mwc, ha ;) reviewing code, etc14:31
Saviqkarni, you'll have to put it in yourself ;P14:31
tsdgeosoh there's a new protobuf deb comin in dist-upgrade14:32
tsdgeosmaybe fixes the thing14:32
karniSaviq: come again :)?14:32
elopioI'm trying to join too, but need to install the plugin.14:32
sdeziel_Hi all14:32
sdeziel_I'm running Trusty and installed some updates (one of them was lightdm) and rebooted. Since then, Unity doesn't autostart so I get a bare desktop. Any idea where to look for debugging?14:32
Saviqkarni, in the doc I mean ;)14:32
karniSaviq: haha, I will14:32
karniguys, mind throwing the SU doc at me?14:33
karnifound it14:34
mzanettikgunn: -mumble +hangout14:36
mzanettikgunn: https://plus.google.com/hangouts/_/72cpiqua3mnkr7dti836vn0ui014:36
MacSlowSaviq, hm... I'm not allowed to connect to that hangout...14:36
SaviqMacSlow, https://plus.google.com/hangouts/_/72cpiqua3mnkr7dti836vn0ui0?authuser=114:36
Saviqelopio, ↑↑ we're doing hangout today as mumble's dead14:38
elopioSaviq, I'm in, thanks.14:39
Saviqelopio, d'oh14:39
elopiomterry, ping. Do you have time now?14:45
mterryelopio, sure!  Let me dig into how system-settings is called over dbus14:46
kgunntsdgeos: so when would that qt release actually hit ? (...thinking of risk & mwc being right around the corner)14:48
sdeziel_any idea on what's wrong with Unity not starting after login?14:48
kgunnaltho...maybe not so big deal since Saviq says we're _virtually_ on  that rel anyway14:48
Saviqkgunn, wait wait wait14:49
Saviqkgunn, we're not switching for mwc14:49
Saviqkgunn, we have 5.2 (and some of 5.2.1) in qt5-beta2 ppa14:49
Saviqkgunn, so there's little risk between that and full 5.2.114:49
kgunnok, np....14:50
Saviqkgunn, but we still have bugs to fix for it (although apparently not unity8 ones!)14:50
kgunnright, is that phonedations that makes the call for when to pull in a new Qt rel ?14:51
Saviqkgunn, QA rather14:51
Saviqkgunn, basically when we get green on 5.2 and no bugs can be identified - that's when14:52
kgunnof course....14:53
kgunnwas wondering if there was even a "conformance test suite" in addition to that...14:53
kgunnguessing not14:53
Saviqkarni, https://code.launchpad.net/~unity-team/unity8/new-scopes-title-alignment/+merge/204677/comments/47909214:59
Saviqmhr3_, action activation will happen on the preview object?15:16
Saviqmhr3_, any name for the method yet?15:17
mhr3_saviq, didn't pick yet, feel free to15:18
Saviqmhr3_, we're using triggered(string, string, QVariantMap), how's that?15:18
mhr3_fine with me15:19
Saviqmhr3_, I'm starting to feel like we need a "separator" widget...15:19
Saviqmhr3_, as I can't find a rule in the visuals for where to put it and where not15:20
mhr3_i can see scopes specifying margins in px soon15:21
Saviqmhr3_, let me see if katie has an idea15:21
Saviq<-- katie has quit (Ping timeout) FAIL15:21
Saviqok actually found a rule...15:22
Saviqmhr3_, btw, we discussed roleNames a while ago, and decided there's no point in keeping them in m_roles, as they will only ever get called once, so we're usually creating them in roleNames directly and letting it drop when going out of scope15:31
mhr3_saviq, hm, not like it used up insane amounts of memory15:32
Saviqmhr3_, of course, just saying15:32
mhr3_yea, will keep in mind15:32
=== alan_g is now known as alan_g|tea
CimiSaviq, is this fine now? https://code.launchpad.net/~unity-team/unity8/unity8.previews_audioPlayer/+merge/20338715:39
mhr3_    "card-size": "medium", // "small", "medium", "large", 12 ÷ 38 (in grid-units) when "category-layout": "vertical"15:39
mhr3_should have been card-layout, right?15:40
Saviqmhr3_, of course15:40
mhr3_i was confused if it wasn't supposed to be vertical-journal15:40
mhr3_wasn't it supposed to be "category-layout": "vertical-journal"?15:42
=== alan_g|tea is now known as alan_g
mhr3_saviq, is there max for xollapsed-rows?15:48
Saviqmhr3_, man, I should probably read what you write15:49
mhr3_you should :P15:49
Saviqmhr3_, it should be journal15:49
Saviqmhr3_, in vertical size is s, m, l - only in journal you can say what height every row is15:49
Saviqmhr3_, spec defines collapsed-rows per category layout15:50
mhr3_saviq, i wonder if we have a maximum value there15:51
mhr3_1024? :)15:51
Saviqmhr3_, no, like... 215:51
mhr3_so, 1 or 2... lots of options15:51
Saviqmhr3_, indeed15:52
mhr3_collapsed-rows Number of result rows displayed while the category is collapsed; possible values: any integer between 1 and 215:52
mhr3_doesn't sound right :)15:52
Saviqmhr3_, it only affects grid and journal, too ;)15:52
Saviqmhr3_, sounds like we should make it bool15:52
mhr3_saviq, i'm starting to think you should have been writing this ;)15:52
Saviqmhr3_, collapsed-2-rows: true ;)15:53
tsdgeosSaviq: can get lp:~aacid/unity8/action_preview_widget and run "make tryPreviewActions"15:59
tsdgeosand tell me what you think?15:59
tsdgeosI think it's not great, but unless I should be coding the combobox (which will take some time) I don't see anything better we can do15:59
Saviqtsdgeos, hmm yeah that looks fine, we'll only have two buttons at the most in our previews I think16:05
Saviqmhr3_, will I still have "id" and "type" in preview widget properties, or are you stripping them?16:20
mhr3_saviq, they're separate roles, not in the big variantmap16:21
Saviqmhr3_, bleh :P16:21
mhr3_you should be glad!16:25
mhr3_it's guaranteed to always be there16:25
mhr3_and always a string16:25
CimiSaviq, ping? maybe I missed your answer16:40
CimiSaviq, how is the preview widget now?16:40
SaviqCimi, you didn't, just ACK'ed it16:41
CimiSaviq, maybe tsdgeos could review it16:41
SaviqCimi, he did already, didn't he16:42
SaviqCimi, if you get him to do it - be my guest ;)16:43
CimiSaviq, now again16:43
tsdgeosdo what?16:47
Saviqtsdgeos, do a last review of audioPlayer (probably just the last commits would be enough)16:49
Saviqmhr3_, properties^Wattributes... widgetId, type... yay for consistency ;P16:50
mhr3_consistency is relative :)16:51
tsdgeosSaviq: looks ok to me16:51
=== dandrader is now known as dandrader|lunch
pstolowski_sil2100, hey, any idea why https://code.launchpad.net/~marcustomlinson/unity-scopes-api/smartscopesproxy_fixes/+merge/204827 hasn't been merged yet?17:03
tsdgeosshould give me the first paramter of the first signal that was spied, right?17:04
greybackSaviq: I'm still waiting for 2 branches of unity-mir to land. Did they fall off the CI train?17:23
Saviqgreyback, http://i.dailymail.co.uk/i/pix/2009/05/25/article-1187119-051592F9000005DC-459_634x421.jpg17:24
greybackSaviq: in that case, can I squeeze this in: https://code.launchpad.net/~gerboland/unity-mir/fix-readme/+merge/20457217:24
Saviqgreyback, sure17:24
greybackSaviq: it needs a quick review plz17:24
* greyback appalled Saviq reads the daily mail17:25
Saviqgreyback, you should stop supporting in-source builds17:25
greybackpatches welcome :)17:26
Saviqgreyback, why the -DCMAKE_INSTALL_PREFIX=/usr ?17:26
greybackSaviq: otherwise installs in /usr/local17:27
Saviqgreyback, well, yeah, you shouldn't "make install" it anyway, should you..17:27
Saviqgreyback, but yeah, ok17:29
greybackSaviq: it's fast & dirty, just wanted to update the wrong info, tis all17:29
Saviqgreyback, got it17:29
Saviqgreyback, ACK17:30
Saviqmhall119, hey, so we've started generating docs from unity8 (doxygen style), think it'd make sense to publish them somewhere?17:31
Saviqmhr3_, hrmpf "Error: Unknown method return type: scopes_ng::PreviewModel*"18:02
Saviqbut Q_DECLARE_METATYPE(scopes_ng::PreviewModel*)!!18:02
Saviqmzanetti, ↑ you solved that error for yourself recently with exactly that, right?18:02
Saviqah hmm18:03
mzanettinot exactly this18:03
mzanettiit was a parameter, not the return type18:03
mzanettibut yeah. I guess its the same18:03
karniSaviq: I'll address the rest of the comments on card-overlay branch18:03
Saviqmzanetti, I think in this case it's because it's not wrapped in QVariant maybe18:04
Saviqkarni, cool, I got previews *almost* working18:04
karniSaviq: fantastic :)18:04
mzanettihuh... no. its a pointer. don't wrap that in QVariants18:04
=== dandrader|lunch is now known as dandrader
Saviqmzanetti, yeah, don't listen to me18:08
Saviqmzanetti, was thinking of different place in the code18:08
mzanetti:) no worries18:08
Saviqah, it's not qmlRegistered18:08
mhall119Saviq: why doxygen?18:10
Saviqmhall119, as opposed to?18:10
=== alan_g is now known as alan_g|EOD
karniSaviq: I think that unity8-card-overlay willl have to be against new-scopes. CardCarousel.qml is not present in unity8 (only contains DashCarousel, while it is CardCarousel.qml that should override whether overlay is shown on current item)18:17
karniwhatcha think?18:17
karnis/against/rebased on new-scopes18:18
Saviqkarni, https://code.launchpad.net/~unity-team/unity8/newscopes-card-overlay/+merge/20479718:18
* karni marks WIP18:18
karniSaviq: shall I just continue work in that branch?18:18
Saviqkarni, yup, is why I pushed it under unity-team18:21
karnithanks bug18:21
karni*bud :D18:22
mhall119Saviq: I ask about qdoc because that's what all of the SDK uses already, and I have an importer for qdoc into the API website for publishing18:26
mhall119for doxygen somebody would have to write an parser/importer to get it published to the API website18:26
Saviqmhall119, oh well, I thought doxygen was meant to be used...18:27
Saviqmhall119, fortunately they're compatible18:27
mhall119in that case I can set you up on developer.ubuntu.com/api/unity/<version>/ for qdoc generated stuff18:28
mhall119developer.u.c/api is just a reverse proxy too, so we can set up unity.u.c/api/ to point to it too18:28
mhall119well, ok, on second thought that would take some work18:29
mhall119mostly because of the website theme18:29
Saviqmhall119, if you can try and see how this will work out, that'd be nice18:31
mhall119Saviq: worst case scenario, we deploy a separate instance of the API website just for Unity18:33
mhall119it's all Juju charmed now, so that shouldn't be hard18:33
mhall119(famous last words)18:33
Saviqmhr3_, http://paste.ubuntu.com/6880650/18:34
=== boiko_ is now known as boiko
karniSaviq: I assume the shader should fade in together with CardHeader (as if they were one thing)18:44
karniI'll do it for text first. Once that works, I'll include the shade effect.18:45
Saviqkarni, yeah, fade in text, too18:49
=== jono is now known as Guest56083
mzanettiSaviq: another one of my cmake hacks :D https://code.launchpad.net/~mzanetti/unity8/make-single-testfunction/+merge/20503519:11
Saviqmzanetti, huh, is $(FUNCTION) something cmake knows?19:12
mzanettiSaviq: well, sort of yes19:12
mzanettiits make19:12
mzanetti$(FOO) can be used to pass arguments to make19:12
mzanettibut not real arguments19:12
mzanettionly vial FOO=blabla19:12
Saviqmzanetti, will it work with ninja?19:13
* mzanetti tries19:13
mzanettiworks with nmake according to stackoverflow19:13
ubot5Launchpad bug 1276747 in autopilot (Ubuntu) "after starting a new app cannot get proxy object for unity8" [Undecided,New]19:14
mhr3_saviq, eh, yea, sorry19:14
Saviqom26er, yeah, saw that - affected autopilot, too, we're not doing anything special around this19:14
mhr3_saviq, will add to my previewstack branch19:14
Saviqmhr3_, nw, it excepts anyway19:14
mhr3_sucks :P19:15
om26erSaviq, thanks for that19:16
mzanettiSaviq: too bad... ninja doesn't like the $19:17
mzanettiwants $$19:17
mzanettibut then it breaks19:18
elopiomterry, https://code.launchpad.net/~elopio/unity8/url-dispatcher_test/+merge/20503719:23
Saviqok, finishing early today ;P19:33
karniSaviq: early you say :D 8:30 haha. If you happen to drop around, the isCurrentItem attached property doesn't seem to work for me :| http://paste.ubuntu.com/6881024/19:46
karnibtw your overlay is lovely19:47
karnifont color: http://paste.ubuntu.com/6881030/19:47
Saviqkarni, "=== true" can probably be dropped20:04
karnijust pushed the version that doesn't work with attached property isCurrentItem20:05
karnileft a comment, don't know why it doesn't work.20:05
Saviqkarni, http://developer.ubuntu.com/api/qml/sdk-1.0/Ubuntu.Components.Themes.PaletteValues/ use these for the default color20:05
karniif you put true/false, it toggles overlay on all items, so the attached property should work20:05
karniSaviq: I asked katie for those, she just said "white" in overlay xD thanks!20:05
Saviqkarni, attached properties are only available on the direct delegate item20:06
karnithe Loader in that case..20:06
Saviqkarni, yeah, for overlay white, but for the default - use one foregroundColor20:06
* karni nods20:06
Saviqkarni, not Loader, Card?20:06
Saviqkarni, ah I know what you mean20:07
karnithat in that case, that'd be  showHeader: isCurrentItem20:07
karniand that doesn't work either20:08
karniactually, it says  ReferenceError: isCurrentItem is not defined20:08
karniwe use Loader as the delegate in Carousel20:08
karnimaybe that's the reason20:08
Saviqkarni, no, you're right - ListView.isCurrentItem would only be available in the Loader20:08
karniparent.ListView.isCurrentItem works20:08
karniwithin Card { } in CardCarousel20:09
karnijust found out by trial and error ;P20:09
Saviqkarni, tricky ;)20:09
karnimakes sense, as Loader would be parent of Card, but the Card becomes the delegate so.. not very intuitive20:09
karnihahahah, when you click - isCurrentItem changes. when you scroll the carousel, it doesn't :D20:10
karnilet my try that on my own, tho! xD20:10
karniactually, got a session in 5 minutes, but I'll try to set current item from within Carousel component20:11
Saviqkarni, showHeader: explicitlyScaled20:11
karnismart xD20:11
Saviqkarni, that's what it's there for (but the name is tricky) Cimi, you and your prop names :P20:12
karniyeah, naming is one of two worse problems in programming. the other is cache invalidation ;P20:12
karni*proper naming20:12
Saviqkarni, to only use a single animation, you can go: opacity: header.opacity * 0.6 for the overlay20:18
karniyeah, I thought it'd be better to use one (or pull Shader under CardHeader with z: -1 )20:19
Saviqkarni, not sure what that has to do ;)20:23
Saviqkarni, opacity is inherited, but z-order does not change parentship20:23
karniSaviq: I thought I could use one "Behavior" declaration, but I guess I can't (like, one animator, if that even matters)20:23
karniok, now I get it heh, sorry :)20:24
Saviqkarni, you could, though, set parent: of the header to the overlay, when in overlay mode20:24
Saviqkarni, and sure, you could use a single Animation object20:24
Saviqkarni, but that's effectively what opacity: header.opacity * 0.6 does20:24
* karni nods, fixed20:25
Saviqkarni, with less code, 'cause you don't have to explicitly configure the Animation20:25
karniCorrect, sir. Pushed20:25
karniShall I leave it WIP to center align title within overlay (whish works in vertical mode, so Left aligned) ?20:26
karnibut seriously, you should go enjoy your evening pal :)20:26
Saviqkarni, not sure it should center in overlay20:26
karniSaviq: that's what's in design doc, katie added overlay section20:26
Saviqkarni, I'll enjoy my week starting Saturday, when I'm off snowboarding in Austria, but first I need to earn it ;)20:27
karni:O wohooo \o/20:27
karniSaviq: leaving for a week? you've certainly earned it20:27
Saviqkarni, yeah, I just want to leave you guys in a state where you can mostly apply tweaks to the whole system, rather than have to implement huge portions of it20:28
karniVery much20:29
kgunnSaviq: booo...leaving20:29
kgunnhope there's snow20:30
Saviqkarni, you should run stuff before pushing ;)20:30
Saviq        opacity:  opacity: header.opacity * 0.620:30
karnithat's what happens when trying to work in middle of session, my bad :/20:30
karniwas sure I :s <enter>'ed ;P20:30
Saviqkgunn, hold your horses, I'm still here :D20:30
karnifixed, sorry about that ://20:31
kgunnSaviq: oh right...8 not 620:31
Saviqkarni, re: margins, we should probably push them down into CardHeader, no need for them to be in two places20:34
Saviqkarni, /me does20:34
karni:D was about to say "consider it done"20:34
karniok. /me listens to the session20:34
Saviqkarni, height: header.height is not good, we don't want to make overlays of different height in case there's only one line of title or missing subtitle20:41
Saviqkarni, CardTool needs to give up headerHeight20:41
karniSaviq: ack20:43
karniI'll figure it out20:43
karniSaviq: I assume it'd be either 2 lines (for title) or 3 lines (for title+subtitle, if both are mapped)20:44
karniThat sucks, cose we'll have 2 lines even if no title wraps to two lines20:45
karnibut I guess that's better than have different height, yes20:46
Saviqkarni, it sucks less than overlay going up and down20:46
* karni agrees20:46
Saviqkarni, or well, maybe it depends20:46
Saviqkarni, as in Carousel you only ever show a single overlay20:46
karnimaybe I should ask katie?20:46
Saviqkarni, but in Grid you show them all20:46
karnigood point20:46
karniactually, very good observation20:46
Saviqkarni, so yeah, asking for design input would be good20:46
karniI'll do that20:46
Saviqkarni, I moved most of your changes into unity8-card-overlay20:48
Saviqkarni, and merged+pushed into newscopes-card-overlay20:49
* karni looks20:50
karniLooks great, Saviq, thank you20:51
* Saviq really likes it how we managed to keep the diff between old and new scopes really small20:53
karniYea, you did it fantastic20:56
=== shiznix_ is now known as shiznix
bregmatedg, I'm having trouble launching apps through upstart-app-launch, are there any cunning debug tricks?21:45
tedgbregma, Which branch are you using?21:46
bregmatrunk ... in a Unity8 session on Mir on the desktop21:46
tedgbregma, K21:47
tedgbregma, So I'd set the glib debug flag21:47
tedgbregma, initctl set-env --global G_MESSSAGES_DEBUG=all21:47
tedgbregma, That'll put more interesting stuff in ~/.cache/upstart/application*21:47
bregmasounds like a good start21:48
tedgbregma, Wait, are you using trunk or main?21:48
tedgLike what's released in archive21:48
bregmawhatever's in the release archive21:48
tedgK, that should be your next step then.21:48
bregmatedg, does upstart-app-launch/desktop-exec reset the environment or something?22:01
tedgbregma, It configures it, upstart creates a clean environment each time.22:02
tedgbregma, You can get it with initctl get-env22:03
bregmawell, I'm setting the QT_QPA_PLATFORM explitcitly with initctl set-env (and env directives) but the app crashlog does not show anything but a few basic variables22:04
tedgset-env --global ?22:05
tedgWhat is generally best is to put that in an upstart job that is "start on starting dbus" so then it gets into the dbus env as well.22:05
tedgbregma, This is my ~/.config/upstart/glib-debug.conf as an example: http://paste.ubuntu.com/6881793/22:06
elopiomterry: are you still here?22:51
mterryelopio, yup22:51
elopioI don't understand why url-dispatcher stops working when we do the initctl stop unity822:51
mterryelopio, I'm looking at things now too22:51
elopiois the unity process we launch on the tests missing something?22:51
mterryelopio, does autopilot know the right DBUS_SESSION_BUS_ADDRESS?22:51
elopiomterry: I bet not, but you said that comes from an environment variable, right?22:53
mterryelopio, you might be thinking of me saying about MIR_SOCKET22:53
mterryelopio, I'm talking about url-dispatcher being able to see dbus22:53
mterryelopio, I'm testing this idea now22:53
mterryelopio, no, that's fine22:54
elopiomy previous tests restart unity8 and still can talk to the session dbus. But I barely know what I'm talking about :)22:55
elopiomy guess is that initctl start unity8 does something we are not doing on the test's launch unity.22:56
mterrywas just a though22:56
mterryelopio, if I enter a bogus url or whatever to dispatcher, I get an error22:59
mterryI don't see that here22:59
elopiomterry: could the camera app be behind unity?23:01
mterryelopio, camera-app isn't running23:01
elopiomterry: it is here.23:02
mterryoh , let me try again23:02
mterryelopio, right you are!23:03
mterryHow does AP launch unity8?  I thought it was via upstart too23:04
elopiogood :)23:05
elopiosadly, that leaves me as clueless as before, jaja.23:05
elopiomterry: it does initctl start unity8, but it also does a lot of other things I'm looking for the first time23:05
elopio                os.getenv('MIR_SOCKET',23:05
elopio                          os.path.join(os.getenv('XDG_RUNTIME_DIR', "/tmp"),23:05
elopio                                       "mir_socket")))23:05
elopioI don't know what's that about.23:05
mterryelopio, that's just cleaning up from previous runs23:14
mterryelopio, shouldn't be anything to worry about23:14
kgunnmterry: you got your wish...0.1.4 mir is landing now...its actually in devel-proposed image now23:19
mterrykgunn, sorta.  I want 0.1.5 though!23:19
mterryelopio, odd results23:19
kgunnand it has the orientation argument in the configure_output23:19
kgunnmterry: it took so long....23:19
kgunni just remerged devel into the MP :)23:20
mterrykgunn, oh good!23:20
kgunnduflu set it up perfect so the so bumps were in devel...but the deb bump is only in trunk23:20
kgunnso i didn't have to wait23:20
mterrykgunn, now we just have to get the new AP tests sorted.  Neither one is working even against current image right now23:20
mterrykgunn, tests are hard23:20
kgunnmterry: no kidding i keep telling people this23:21
kgunnthey're like hey...just get a guy to do a test for 2 weeks...23:21
mterryelopio, so when I manually run camera-app (sudo -u phablet -i camera-app) I get different results between the two runs (normal and AP)23:21
kgunni'm like...uh, no...it takes maintenance23:21
mterryelopio, normal gives me "QUbuntu: Could not create application instance"23:21
kgunnok, gonna drop for abit...later23:21
mterryelopio, AP gives me http://paste.ubuntu.com/6882127/23:22
mterrykgunn, bye23:22
mterryelopio, why would we get those results?23:22
mterryelopio, neither brings up camera-app23:23
elopiomterry: I have no idea.23:24
elopioveebers should know more about this code, but I think new zeland is on holidays.23:24
elopiomterry: at what point of the autopilot test are you trying to launch the camera?23:26
mterryelopio, I'm doing my current testing in a separate adb terminal and running the camera-app manually either during AP test or during normal operation23:27
mterryelopio, not running it from the test, just during the test23:27
elopioright. It's weird, the set up of the unitytestcase must be playing tricks with the environment.23:28
mterryelopio, but it's weird that during AP test, it seems to get further23:29
elopiomterry: this is how we are starting unity on the tests:23:38
elopio/sbin/initctl start unity8 BINARY=/usr/bin/unity8 ARGS= XDG_DATA_DIRS=/usr/share/unity8/mocks/data:/usr/share/ubuntu-touch-surfaceflinger:/usr/local/share/:/usr/share/ LD_LIBRARY_PATH=/usr/lib/arm-linux-gnueabihf/unity8/qml/mocks/libusermetrics:/usr/lib/arm-linux-gnueabihf/unity8/qml/mocks/LightDM/single QML2_IMPORT_PATH=/usr/lib/arm-linux-gnueabihf/unity8/qml/mocks QT_LOAD_TESTABILITY=123:38
elopioif I do that manually, and manually launch the camera, it doesn't appear in front.23:38
mterryok, good23:38
mterryso we can start taking away things  :)23:38
mterryoh we have quite a few mocks there23:39
mterryelopio, /usr/share/unity8/mocks/data in XDG_DATA_DIRS23:43
mterrymaybe something else too23:44
mterryelopio, that and dropping QML2_IMPORT_PATH=/usr/lib/arm-linux-gnueabihf/unity8/qml/mocks23:45
elopiomterry: yes! without those two, it works.23:46
elopiothere is a fake application manager, maybe not updated to work with url dispatcher?23:49
elopioit seems to be faking too much.23:50
mterryelopio, why are we faking that much in AP mode?23:51
* mterry looks at the delta for the mock and real thing23:51
elopiomterry: I don't know. I'm a new comer. I added one fake, that was to avoid talking to the click servers23:52
elopioI suppose there are some good reasons for these others.23:52
* elopio makes a mental note to always add a comment on the reason for using a fake.23:52

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