[08:43] <mhr3_> tsdgeos, ping?
[08:44] <tsdgeos> mhr3_: hi
[08:44] <mhr3_> tsdgeos, hey, last week saviq had an issue where the scope results were not showing up cause there was something broken in the renderer
[08:44] <mhr3_> tsdgeos, and we're seeing the same issue in new-scopes branch
[08:45] <mhr3_> tsdgeos, wonder if you know what the issue was
[08:45] <tsdgeos> no :/
[08:45] <tsdgeos> mhr3_: is it white text vs white background?
[08:45] <tsdgeos> or is it something more "broken"?
[08:45] <mhr3_> tsdgeos, no, cause there's no icons either
[08:46] <tsdgeos> no idea then
[08:46] <mhr3_> tsdgeos, only the category header shows up and nothing else
[08:46] <tsdgeos> but if you give me a way to reproduce it i can have a look
[08:46] <mhr3_> tsdgeos, just branch new-scopes and run the unity-scope-tool
[08:46] <mhr3_> perhaps run `restart scope-registry` in between
[08:47] <mhr3_> tsdgeos, oh plus the onlinemusic scope from the demo ppa
[08:49] <tsdgeos> ok
[08:51] <tsdgeos> mhr3_: "unity-scope-onlinemusic is already the newest version." ?
[08:51] <tsdgeos> 0.1+14.04.20131029.1-0ubuntu1 ¿?
[08:51] <tsdgeos> ah
[08:51] <tsdgeos> silly me
[08:51] <tsdgeos> missing an update
[08:51] <mhr3_> tsdgeos, you need this one https://launchpad.net/~unity-team/+archive/demo-stuff/+packages
[08:51] <tsdgeos> yes yes
[08:51] <tsdgeos> i always forget to update after add-apt-repository
[08:57] <mhr3_> tsdgeos, if you don't even see the category header then something else is wrong :)
[08:57] <tsdgeos> so i see "Popular online"
[08:57] <tsdgeos> and that's all
[08:57] <tsdgeos> should i see more?
[08:57] <mhr3_> cool, yea that's the problem
[08:57] <mhr3_> you should also see the results
[08:59] <tsdgeos> ok fix is easy
[08:59] <tsdgeos> don't know if it's the correct fix yet though
[08:59] <tsdgeos> http://paste.ubuntu.com/6908000/
[08:59] <tsdgeos> mhr3_: ↑↑↑ if it's blocking you
[08:59] <mhr3_> tsdgeos, yea, we mostly need to fix new-scopes
[09:00] <mhr3_> tsdgeos, can you explain why that messes stuff up?
[09:00] <tsdgeos> because you are assigning to something that is readonly
[09:00] <tsdgeos> and qml says
[09:00] <tsdgeos> no
[09:00] <tsdgeos> won't do that
[09:00] <tsdgeos> and then all breaks
[09:01] <tsdgeos> need to find out if it should not be readonly
[09:01] <tsdgeos> or the assignment isn't neede
[09:01] <tsdgeos> d
[09:01] <mhr3_> tsdgeos, so why is it there in the first place? isn't trunk's cardtool broken too then?
[09:01] <tsdgeos> sure
[09:01] <tsdgeos> i mean
[09:01] <tsdgeos> no
[09:01] <tsdgeos> because trunk isn't merged since ten years ago
[09:02] <tsdgeos> because we have unreproducible autopilot errors in a phone we are not supposed to support anymore blocking stuff
[09:02] <tsdgeos> yeah \o/
[09:02] <tsdgeos> mhr3_: but yes it is a mistake in some branch that we want to eventually merge to trunk
[09:02] <tsdgeos> i'll hunt it down
[09:03] <mhr3_> tsdgeos, thx
[09:05] <tsdgeos> karni: you there?
[09:07] <tsdgeos> mhr3_: i'm going to propose a branch for merging to new-scopes, can you review it? Do you know if we have automerging in there or we manually merge stuff in it?
[09:07] <mhr3_> tsdgeos, manual merging afaik
[09:14] <tsdgeos> mhr3_: https://code.launchpad.net/~aacid/unity8/no_readonly_headerheight/+merge/205538
[09:14] <tsdgeos> if you approve it i'll manually merge it
[09:15] <mhr3_> tsdgeos, acked
[09:18] <tsdgeos> ok, merged
[09:21] <mhr3_> tsdgeos, thx
[09:21] <mhr3_> tsdgeos, do you know which category layouts are properly supported these days?
[09:21] <tsdgeos> mhr3_: grid, carousel
[09:21] <tsdgeos> i have coded all the rest, but they are not wired
[09:21] <mhr3_> tsdgeos, i'm looking at the designs and wondering if we can even do them :)
[09:22] <tsdgeos> i.e. lots of qml glue is missing
[09:22] <tsdgeos> mhr3_: Saviq told me about the addSpecialCategory thing you're working on, is it done, what's your timeframe if not
[09:22] <mhr3_> tsdgeos, can we do all of this for example? https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgZDJvd3l4U2tKYnM/edit
[09:22] <mhr3_> tsdgeos, yea, there's mp for it
[09:23] <mhr3_> tsdgeos, feel free to review :)
[09:23] <tsdgeos> we can't do the seemore/less
[09:23] <tsdgeos> i mean we can
[09:23] <tsdgeos> but it's the old expansion pattern
[09:23] <tsdgeos> not that one
[09:23] <mhr3_> tsdgeos, https://code.launchpad.net/~mhr3/unity-scopes-shell/special-categories/+merge/205427
[09:23] <tsdgeos> besides that, it's just grid and carousel no?
[09:23] <mhr3_> tsdgeos, i think cimi is working on the more/less
[09:23] <mhr3_> tsdgeos, the carousel is overlaid though
[09:23] <tsdgeos> just for the text summary
[09:24] <tsdgeos> we do support overlay
[09:24] <mhr3_> ah, good
[09:28] <tsdgeos> mhr3_: make tryCard and select the overlaid one (the last)
[09:32] <mhr3_> tsdgeos, missing title when using {
[09:32] <mhr3_>   "template": {
[09:32] <mhr3_>     "overlay": true,
[09:32] <mhr3_>     "category-layout": "carousel"
[09:32] <mhr3_>   },
[09:32] <mhr3_>   "components": {
[09:32] <mhr3_>     "title": "title",
[09:32] <mhr3_>     "art": "art",
[09:32] <mhr3_>     "subtitle": "subtitle"
[09:32] <mhr3_>   }
[09:32] <mhr3_> }
[09:32] <mhr3_> (and subtitle)
[09:34] <tsdgeos> trueth
[09:34] <tsdgeos> and if you leave the mascot there it looks out of position
[09:35] <mhr3_> but at least it's there :)
[09:35] <tsdgeos> so i guess someone needs to look at it and that someone is me?
[09:37] <mhr3_> tsdgeos, well, i'm not your team lead :) but would be nice if you did
[09:38] <tsdgeos> there is no team lead :D
[09:38] <tsdgeos> someone hinted i was acting one this week
[09:38] <tsdgeos> don't believe it
[09:40] <tsdgeos> j/k
[09:40] <tsdgeos> i'll have a look
[09:41] <tsdgeos> sil2100: do you have any idea what happened to the unity8 silo/landing/train thing? still blocked by autopilot failures?
[09:41] <tsdgeos> and any idea where i can check the status of that?
[09:54] <tsdgeos> mhr3_: it works
[09:54] <tsdgeos> it's just the tryCard thing that is not as good as it should
[09:55] <tsdgeos> and doesn't set all the stuff needed for carousel to work
[09:55] <tsdgeos> mhr3_: but if you use tryCardTool
[09:55] <mhr3_> oh?
[09:55] <tsdgeos> you'll see it work
[09:55] <mhr3_> great then
[09:55] <tsdgeos> even a bit crappy since it's just wireframes either
[09:55] <tsdgeos> i can fix tryCard to work if you want
[09:56] <mhr3_> tsdgeos, working tests are always good, primary thing is that it will work
[10:01] <tsdgeos> mhr3_: i pushed the fix of the test to new-scopes
[10:03] <mhr3_> cheers
[10:09] <sil2100> tsdgeos: hi! Not sure if anything changed, on Friday Didier wanted the maguro failures on unity8 resolved, right?
[10:10] <mhr3_> sil2100, ping, ping, libunity landing?
[10:10] <tsdgeos> sil2100: i guess, why are we still caring about galaxy nexus if we don't support it and almost noone has one anymore to try to reproduce the problems?
[10:16] <sil2100> tsdgeos: the rationale is that maguro is slow, and we'll start using the emulator soon which is even slower - so making stuff work on maguro will help things working on the emulator
[10:17] <sil2100> mhr3_: ping! Give us a few moments
[10:17] <mhr3_> sil2100, i did, i asked last week :P
[10:18] <mhr3_> sil2100, it's a unity7 pkg, so it better land before ff
[10:18] <sil2100> Aaaah, libunity!
[10:18] <tsdgeos> sil2100: the rationale is fine, but if it's just blocking our builds and noone can reproduce/fix it, not really helping the flow
[10:18] <tsdgeos> anyway i know i'm not going to win
[10:19] <tsdgeos> so i go to my cave
[10:19] <sil2100> Mirv: ^ could you give it a shot maybe?
[10:19] <sil2100> Mirv: could you see if you could use the good-old cu2d to land the new libunity? (stand-alone)
[10:19] <sil2100> Mirv: I'll add a landing for that in the old spreadsheet
[10:20] <mhr3_> +1, there probably won't be anymore changes there, so no need to use train for it
[10:30] <Mirv> sil2100: sure in theory, but it's known that the tests don't pass at the moment for unity stack so I'm not sure how to validate it
[10:32] <sil2100> Mirv: I guess mhr3_ can help out with finding the right set of tests to run/do manually in the meantime :)
[10:34] <Mirv> ok. and on device I guess unity8 AP etc.
[10:35] <Mirv> my device is as usual in Qt 5.2 mode, but I'll see if I can do that still today or tomorrow morning (provided mhr3_ will instruct how to test on desktop)
[10:35] <Mirv> the changelog is essentially the dropping of gee
[10:38] <sil2100> Sure, thanks!
[10:40] <mhr3_> Mirv, indeed, it's just dropping gee, so what could break is scopes themselves (especially aggregation) and then sound indicator (the play/pause buttons in there)... if that works it should be good
[10:44] <Cimi> tsdgeos, when will we have branches merged?
[10:44] <tsdgeos> Cimi: when some tests we can't reproduce failing don't fail
[10:45] <Mirv> mhr3_: ok, but no problems known? should it work fine with the unity itself from october? (since nux+compiz+unity haven't been updated in trusty at all)
[10:45] <tsdgeos> Cimi: meanwhile, can you review https://code.launchpad.net/~aacid/unity8/preview_header
[10:45] <mhr3_> Mirv, sure
[10:46] <Cimi> tsdgeos, in your code seems all of them are optional
[10:47] <Cimi> tsdgeos, shall we have title required?
[10:47] <tsdgeos> it's not required afaics
[10:47] <tsdgeos> at least the tryCard doesn't make it mandatory either
[10:47] <Mirv> mhr3_: hmm, I updated libunity (the packages that come from the source package) from daily-build and now my windows don't have decorations for some reason
[10:47] <Cimi> tsdgeos, but if you set to use it, you might have the title no?
[10:48] <Cimi> tsdgeos, what's the point of using the header if you have all the json null?
[10:48] <mhr3_> Mirv, ?? compiz weirdness?
[10:49] <tsdgeos> Cimi: makes no sense
[10:49] <tsdgeos> Cimi: but that's the guy that created the json fault
[10:49] <tsdgeos> Cimi: what do you suggest us to do?
[10:49] <Mirv> mhr3_: hmm ok I guess compiz weirdness, not seeing the problem on guest account
[10:50] <Mirv> and yes I was testing compiz+nux+unity for fun before that, probably related
[10:50] <Cimi> tsdgeos, fix the json
[10:53] <tsdgeos> Cimi: right, so if the solution is "fix the json", what change do you want me to do in the qml code of that branch?
[10:53] <Mirv> mhr3_: ok thanks, I should be able to test the desktop side
[10:53] <Cimi> tsdgeos, 38	-        title: root.widgetData["title"] || ""
[10:53] <Cimi> +        title: root.widgetData["title"]
[10:54] <tsdgeos> Cimi: but that makes no sense
[10:54] <Cimi> why?
[10:54] <tsdgeos> why are you forcing me to have a title?
[10:54] <Cimi> tsdgeos, because if I use a header, I must provide a title
[10:54] <tsdgeos> where does it say that?
[10:55] <Cimi> tsdgeos, it's like having a button without a label and an icon
[10:55] <Cimi> it makes no sense
[10:55] <mhr3_> Mirv, and if desktop didn't break, unity8 won't either :)
[10:55] <tsdgeos> Cimi: but i can have a header with an emblem and no title
[10:55] <Cimi> tsdgeos, title is always important
[10:56] <Cimi> it's the title of the app, no?
[10:56] <tsdgeos> sure
[10:56] <tsdgeos> it's the title of something
[10:56] <tsdgeos> this is a preview
[10:56] <tsdgeos> not a preview of an app
[10:56] <tsdgeos> title is important
[10:56] <Cimi> tsdgeos, well you got my point
[10:56] <Cimi> tsdgeos, I'd kinda force developers to keep things right
[10:56] <tsdgeos> but since we provide stuff for randomguy to do a preview, i don't see why we should force title on him
[10:57] <Cimi> because otherwise he uses no title making it not consistent?
[10:57] <tsdgeos> Cimi: honestly your change won't force anything
[10:57] <tsdgeos> only will give us a console warning
[10:57] <tsdgeos> and the developer will still don't care
[10:57] <Cimi> tsdgeos, following your thought we should have those checks for empty strings all around unity
[10:58] <tsdgeos> and we mostly do
[10:58] <tsdgeos> and it's not a check for empty string
[10:58] <tsdgeos> is a check for undefined data in a map
[10:59] <tsdgeos> ./plugins/Unity/Indicators/qml/MenuItemFactory.qml:229:            text: menuData && menuData.label || ""
[10:59] <tsdgeos> ./qml/Dash/Card.qml:71:        title: cardData && cardData["title"] || ""
[11:02] <tsdgeos> mhr3_: about the addSpecialCategory thing, can you explain why are we doing that?
[11:02] <mhr3_> tsdgeos, because of the running apps
[11:02] <tsdgeos> mhr3_: but it works as we have it now, no?
[11:02] <tsdgeos> so, why are we doing it?
[11:03] <mhr3_> tsdgeos, i'm not sure i understand, the new plugin didn't allow shell to "override" a category
[11:03] <mhr3_> the old one did yes, so we're mostly bringing it back
[11:04] <mhr3_> in a slightly different shape
[11:04] <tsdgeos> mhr3_: ok, so overrideResults is gone
[11:04] <mhr3_> right
[11:05] <tsdgeos> hmmm
[11:05] <tsdgeos> so we need addSpecialCategory for running apps and for the dashhome then
[11:06] <mhr3_> tsdgeos, there's no dashhome with new scope
[11:06] <mhr3_> s
[11:06] <tsdgeos> ah right
[11:06] <tsdgeos> forgot about that
[11:07] <mhr3_> tsdgeos, so far we don't have apps scope, so try to test it with music or something, but fwiw apps scope id will most probably be "clickscope"
[11:12] <Cimi> tsdgeos, this is not always
[11:12] <Cimi> tsdgeos, I'm scared of developers adding empty headers
[11:12] <tsdgeos> Cimi: ok
[11:13] <tsdgeos> Cimi: do you understand that your change won't fix anything?
[11:13] <Cimi> I do
[11:13] <tsdgeos> ok
[11:13] <Cimi> because just gives a warning
[11:13] <tsdgeos> that noone will see anyway
[11:13] <Cimi> ok
[11:13] <tsdgeos> if you want to force CardHeader to have a title, discuss with design and propose a code change that actually breaks the thing if it's not there
[11:13] <Cimi> tsdgeos, but warning is better than nothing, no?
[11:14] <tsdgeos> but later please
[11:14] <tsdgeos> we need to get all this done by friday
[11:23] <mhr3_> if i do `system-image-cli -b 0` i'm getting the same thing as if i did a full reflash, right?
[11:25] <mhr3_> ie i can do it on dirty device? (one with ppa pkgs installed)
[11:27] <mhr3_> also, is it faster than phablet-flash?
[11:28] <sil2100> tsdgeos: anyway, I know it's all a very risky topic, but did anyone of you have any success with those maguro unity8 failures in overall? I see one, test_hide_hud_click, seems to happen on almost every maguro test in smoketesting
[11:29] <sil2100> tsdgeos: you think it would be easy to identify and/or fix?
[11:29] <tsdgeos> sil2100: i don't have a maguro
[11:29] <tsdgeos> i've been running the test for 1 hour in a loop
[11:29] <tsdgeos> and nothing failed
[11:29] <tsdgeos> in my nexus4
[11:30] <tsdgeos> if we are fine being blocked, fine
[11:30] <greyback> sil2100: is a test app crashing occasionally on maguro?
[11:30] <sil2100> Do you know if unity8 tests are going fine on the emulator now?
[11:30] <sil2100> greyback: yes, but this failure seems to be unrelared to the qmlscene-crash
[11:31] <tsdgeos> sil2100: i have not touched the emulator yet, and honestly don't feel like doing
[11:31] <tsdgeos> sil2100: we have one target which is nexus4, and stuff works there
[11:31] <tsdgeos> if people feels we need to get stalled fine
[11:32] <sil2100> tsdgeos: I'm thinking that if the problem here is the slowness of maguro, maybe if the timeout could be bumped for the failing assert in that test?
[11:32] <tsdgeos> but i'm not going to take any blame for stuff not working on stuff we don't support and stuff i can't reproduce
[11:32] <greyback> sil2100: ok
[11:32] <sil2100> A hacky way around it, but hmmm
[11:32] <tsdgeos> sil2100: hey, maybe, if i had a maguro, i would get a fix, but i don't
[11:36] <tsdgeos> sil2100: i doubt very much the problem is slowness
[11:36] <tsdgeos> i've run the test on nexus4
[11:36] <tsdgeos> while running "stress --cpu 8 --io 4 --vm 2 --vm-bytes 128M " also
[11:36] <tsdgeos> and it's just fine
[11:36] <mhr3_> tsdgeos, if you want i could give you ssh tunnel to a maguro :)
[11:37] <tsdgeos> mhr3_: i guess that could help
[11:37] <sil2100> \o/
[11:38] <mhr3_> tsdgeos, you sure? you won't see much :)
[11:38] <tsdgeos> mhr3_: i don't need to see anything
[11:38] <tsdgeos> mhr3_: just need to run the test
[11:38] <tsdgeos> and make it succeed
[11:38] <tsdgeos> well
[11:38] <tsdgeos> see if it fails first
[11:38] <tsdgeos> and then make it succeed
[11:38] <tsdgeos> seeing would help
[11:38] <tsdgeos> but it's not critical
[11:39] <tsdgeos> i can just add billions of prints
[11:39] <tsdgeos> mhr3_: can you phablet flash it with latest stuff?
[11:39] <mhr3_> ok, let me try to setup everything
[11:39] <mhr3_> first i need to charge it a bit though
[11:39] <tsdgeos> mhr3_: thanks
[11:40] <sil2100> mhr3_: thanks!
[11:49] <mhr3> tsdgeos, :/ someone changed our network setup, can't open ports, so.. not good
[11:49] <tsdgeos> :/
[11:49] <mhr3> tsdgeos, i can do it at the office tomorrow, if that's good enough for you
[11:50] <tsdgeos> unity: anyone has a galaxy nexus ?
[11:50] <tsdgeos> @unity: anyone has a galaxy nexus ?
[11:50] <tsdgeos> i mean
[11:50] <Cimi> tsdgeos, I have
[11:50]  * mzanetti too
[11:51] <tsdgeos> Cimi: mzanetti: you have two options, fix the autopilot failures we have or give me ssh so i can try to
[11:51] <sil2100> Cimi, mzanetti: can anyone of you flash it to the latest version and provide ssh access for tsdgeos ? :)
[11:51] <tsdgeos> Cimi: mzanetti: you choose :D
[11:51] <sil2100> (or that)
[11:51] <sil2100> ;D
[11:51] <sil2100> It seems to be something easily reproducible on maguro from what I see
[11:52] <mzanetti> ok... need to charge and flash tho
[11:52] <Cimi> tsdgeos, I can do both
[11:52] <Cimi> tsdgeos, I have to run to the doctor though now :)
[11:53] <Cimi> tsdgeos, maybe i can setup ssh before
[11:53] <tsdgeos> Cimi: that'd be great
[11:53] <Cimi> oh yeah
[11:53] <Cimi> tsdgeos, so what want me to do?
[11:53] <tsdgeos> Cimi: setup ssh so i can login from here, if you have time to phablet-flash it first the better
[11:54] <tsdgeos> if not i guess i can manage from here
[11:54] <Cimi> tsdgeos, you want a linux pc or ssh to the phone?
[11:54] <tsdgeos> Cimi: ssh to the phone
[11:54] <tsdgeos> or ssh to a pc where i can adb to the phone
[11:54] <tsdgeos> you choose
[11:54] <tsdgeos> but i guess the first is easier
[11:55] <Cimi> tsdgeos, ok if I set static ip
[11:55] <Cimi> tsdgeos, battery is dead
[11:55] <Cimi> tsdgeos, hah
[11:55] <mhr3> tsdgeos, don't forget that flashing would remove the ssh server
[11:55] <Cimi> tsdgeos, charging then flashing
[11:55] <Cimi> tsdgeos, 30 mins+
[11:55] <tsdgeos> Cimi: ok
[11:55] <sil2100> Bad luck!
[11:55] <sil2100> :\
[11:56] <mzanetti> yeah... mine won't turn on even with power attached
[11:56] <mzanetti> will take a little here too
[11:56] <mzanetti> Cimi: if you gotta run to the doctor I can set it up
[11:57] <Cimi> mzanetti, I can do it
[11:57] <mzanetti> ok
[11:57] <Cimi> mzanetti, I don't have appointment, just grab medicine
[11:57] <mzanetti> ah ok
[11:58] <tsdgeos> mhr3: about the progress preview type
[11:58] <tsdgeos> mhr3: i don't understand this
[11:58] <tsdgeos> "id": null, // mandatory here! action $id triggered on completion/error(/cancel) with argument for status
[11:59] <mhr3> tsdgeos, there should be just a regular widget_id
[11:59] <mhr3> tsdgeos, when triggering actions, we're sending widget_id + action_id
[12:00] <tsdgeos> is this something i care at all from the UI POV?
[12:00] <mhr3> tsdgeos, for progress it should be the widget_id as specified by the scope author + action_id specified by us
[12:00] <tsdgeos> i mean for progress i just show stuff, don't send anything back, no?
[12:01] <mhr3> tsdgeos, the action should be auto-triggered when stuff happens
[12:01] <mhr3> http://developer.ubuntu.com/api/devel/ubuntu-14.04/cplusplus/libunity-scopes/classunity_1_1scopes_1_1_preview_widget.html#progress
[12:02] <tsdgeos> mhr3: what do you mean with auto-triggered?
[12:02] <mhr3> tsdgeos, download finishes - shell triggers "finished" action on the progress widget
[12:04] <tsdgeos> can i say that doesn't make any sense?
[12:05] <mhr3> no
[12:05] <tsdgeos> i knew it :)
[12:06] <mhr3> tsdgeos, the entire progress widget really needs to tied to the progress provider service
[12:06] <tsdgeos> mhr3: why is that on the frontend and not the backend?
[12:06] <tsdgeos> imho the frontend all it should care is "show 50", "show 75", "done"
[12:07] <mhr3> tsdgeos, it is, but shell is the bridge
[12:07] <tsdgeos> ok, i don't think it makes sense
[12:07] <mhr3> tsdgeos, scope might not be running to receive those notifications
[12:07] <tsdgeos> but we're of course not going to change it
[12:07] <mhr3> tsdgeos, all it wants is result of the operation
[12:07] <tsdgeos> so let's pretend i think it makes sense
[12:08] <mhr3> tsdgeos, and since shell does know both the progress and the result, it makes sense to notify the scope when the result is available
[12:08] <mhr3> tsdgeos, plus the completion will change the preview
[12:14] <tsdgeos> mhr3: the contract we have for previews right now is http://paste.ubuntu.com/6908739/ does it mean that besides triggered your model is also going to expose finished and failed?
[12:14] <tsdgeos> does it do that already?
[12:15] <mhr3> tsdgeos, no, that's the progress provider's job
[12:15] <mhr3> (the thing that gives you progress == 60%)
[12:16] <tsdgeos> what is its job?
[12:16] <mhr3> to notify you of the progress and whether the operation was successful/failed
[12:17] <tsdgeos> you're killing me :D
[12:17] <tsdgeos> you told me the shell has to notify the scope
[12:17] <mhr3> wait, let me try to find something
[12:17] <tsdgeos> yes?
[12:18] <mhr3> yes, as in you're supposed to call .triggered(progress_widget_id, "finished", meta)
[12:18] <mhr3> but you do that in response to progress indicator telling you that is what happened
[12:18] <mhr3> *provider
[12:19] <tsdgeos> ok
[12:19] <tsdgeos> so that is what i was asking
[12:19] <tsdgeos> if we use triggered
[12:19] <tsdgeos> or something else
[12:19] <tsdgeos> but the docu you pointed says
[12:20] <tsdgeos> "Upon completion the scope will receive preview action activation with id "finished" or "failed" depending on the outcome of the operation."
[12:20] <tsdgeos> will you take care of converting that .triggered(progress_widget_id, ) to that?
[12:20] <mhr3> sure
[12:20] <mhr3> that is what's happenning with any "triggered" method call
[12:21] <mhr3> tsdgeos, this might help to see the entire flow https://docs.google.com/a/canonical.com/drawings/d/1AKbddm35Tw9IoxZ8T33HBO30qO8c35B_IwEzacmf3bI/edit
[12:21] <mhr3> the 3rd part especially
[12:21] <tsdgeos> mhr3: so can you tell me what triggered call exactly you want for success and failure?
[12:22] <sil2100> mhr3: the unity-scope-scopes landing - will this have some impact on the current image? i.e. is this scope enabled by default now?
[12:22] <mhr3> sil2100, no
[12:22] <mhr3> .triggered(progress_widget_id, "finished", progress_widget_properties_map)
[12:23] <mhr3> tsdgeos, ^ + same for "failed"
[12:23] <dednick> mh. my nexus 10 keeps showing device out of space when installing packages...
[12:23] <tsdgeos> mhr3: but s/finished/failed
[12:23] <mhr3> right
[12:23] <tsdgeos> mhr3: what's in progress_widget_properties_map ? the same thing that you gave us?
[12:24] <mhr3> tsdgeos, yep
[12:24] <tsdgeos> mhr3: we don't do that on buttons, saviq told me not to, do you expect it too for buttons?
[12:24] <mhr3> yea, it should be there
[12:24] <tsdgeos> why?
[12:25] <mhr3> tsdgeos, but needs to per-action
[12:25] <mhr3> to be*
[12:25] <tsdgeos> i mean you gave it to us, you sure know its contents, no?
[12:25] <mhr3> tsdgeos, cause scope can push there extra data that you don't care about, but it does
[12:26] <tsdgeos> ok
[12:26] <tsdgeos> so that needs changing in the regular actions too
[12:26] <sil2100> mhr3: did this package get pre-NEWed by an archive admin already?
[12:27] <mhr3> tsdgeos, ie if actions = [{"id": "foo", "label": "Foo", "session-id": "0x1234ef"}, {...}], i expect triggered(widget_id, "foo", {"session-id": "0x1234ef", ...})
[12:27] <mhr3> sil2100, no idea, robru was dealing with that
[12:27] <tsdgeos> mhr3: you'll get back id and label too
[12:27] <mhr3> tsdgeos, yea, fine
[12:28] <tsdgeos> ok, i added it to my todo
[12:28] <tsdgeos> thing is growing too much :D
[12:29] <mhr3> tsdgeos, btw i saw designs where there is an action combo + regular button on the right side, do we support that?
[12:30] <tsdgeos> kind of
[12:30] <tsdgeos> we don't support "combos"
[12:30] <mhr3> tsdgeos, https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgUUY0MjFTQThBZFE/edit
[12:30] <tsdgeos> we support a button that when you click on it
[12:30] <tsdgeos> it grows
[12:30] <tsdgeos> so it's an "ugly combo"
[12:30] <tsdgeos> but yes
[12:30] <tsdgeos> if you give us 3 actions
[12:30] <tsdgeos> you'll end up with the first action in the right and then on the left a More/Less buttons that shows the other 2 actions
[12:31] <mhr3> ah, ok
[12:31] <mhr3> so the right one is "primary" action?
[12:31] <tsdgeos> so yes but if looks a bit ugly because there's no proper combo in the SDK yet and we didn't feel like coding one :D
[12:31] <tsdgeos> yep
[12:31] <mhr3> ok
[12:31] <tsdgeos> also if you give us two actions
[12:32] <tsdgeos> they are laid right to left
[12:32] <tsdgeos> so first is on right and second on left
[12:32] <mhr3> makes sense
[12:32] <tsdgeos> see make tryPreviewActions
[12:35] <mhr3> tsdgeos, how hard is it to get support for "image" and "video" widget types?
[12:36] <mhr3> we kinda... use those
[12:36] <tsdgeos> mhr3: saviq doesn't want to do video
[12:36] <tsdgeos> until it's out of process
[12:36] <tsdgeos> so basically never :D
[12:36] <mhr3> tsdgeos, let's just make it same as image, but on click it will open the uri?
[12:37] <tsdgeos> non zoomable?
[12:37] <tsdgeos> the uri would be a new param in the json?
[12:37] <mhr3> tsdgeos, it's already defined
[12:37] <mhr3> see the json def
[12:37] <tsdgeos> is it?
[12:37] <tsdgeos> i'm looking at it :D
[12:38] <tsdgeos> type, source, zoomable
[12:38] <tsdgeos> whiere's uri?
[12:38] <mhr3> tsdgeos, oh you meant image itself, no, it's there only for video
[12:38] <mhr3> tsdgeos, but the source is a uri
[12:38] <mhr3> in both cases
[12:38] <tsdgeos> sure
[12:39] <tsdgeos> the source of the image :D
[12:39] <mhr3> but in video's it's video/*
[12:39] <tsdgeos> but ok, in video we have screenshot
[12:39] <mhr3> indeed
[12:39] <tsdgeos> so you want video to show the screenshot and when the guy presses launch the video
[12:39] <mhr3> tsdgeos, if not specified, just use the thumbnailer
[12:40] <mhr3> tsdgeos, yep
[12:40] <dednick> anyone know if it's possible to redistribute freespace on touch device? i keep running out of root partition space!
[12:41] <tsdgeos> dednick: i clean the apt cache manually
[12:41] <tsdgeos> but not anything else
[12:41] <tsdgeos> if you know how to do it, 'll be interested too!
[12:42] <tsdgeos> mhr3: well, it should not be "hard", but we need more work in getting stuff to have feature parity with what we had
[12:42] <dednick> only 540MB free on root partition on clean install. it's a bit useless
[12:42] <mhr3> tsdgeos, that's what this week is about :)
[12:42] <tsdgeos> mhr3: right, so that's why i'm saying "no" to video and image
[12:42] <tsdgeos> since we didn't have that
[12:42] <tsdgeos> so let's achieve parity first
[12:43] <tsdgeos> and then talk about those?
[12:43] <mhr3> tsdgeos, what do you mean we didn't have those? isn't opening an album preview using an image?
[12:44] <tsdgeos> and where did we do that in previews?
[12:44] <mhr3> tsdgeos, i don't mind if some props will get ignored, i just don't want some of the widget types to be ignored
[12:45] <mhr3> tsdgeos, so if "image" is an alias for gallery with 1 image for now... i can live with it
[12:45] <mhr3> but ignoring all "image" widget types sucks
[12:47] <mhr3> and same for video really, if it's aliased image.. i'm happy
[12:49] <tsdgeos> ok
[12:49] <tsdgeos> we'll see if we can get anyone to do it
[12:49] <mhr3> cool
[14:06] <kgunn> mzanetti: ping
[14:06] <mzanetti> hi kgunn
[14:07] <kgunn> mzanetti: hey...did Saviq leave landing responsibility with you ?
[14:07] <kgunn> or someone else ?
[14:07] <mzanetti> kgunn: yeah, he asked me if I could do if something needs to be landed
[14:07] <kgunn> mainly just needs a shepherd to watch for reasons builds might fail and/or failures on AP's
[14:07] <mzanetti> but no list of things to be landed
[14:07] <tsdgeos> mzanetti: i can give you a list, we have a list :D
[14:07] <mzanetti> I guess so :D
[14:08] <mzanetti> 53 branches
[14:08] <tsdgeos> kgunn: i'm trying to get a maguro setup for me to find out how we fix those two failing autopilot tests
[14:08] <tsdgeos> mzanetti: well no, look at the document, it's 13 branches
[14:08] <kgunn> tsdgeos: is that for this silo ?
[14:08] <kgunn> https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AuDk72Lpx8U5dFlCc1VzeVZzWmdBZS11WERjdVc3dmc&usp=drive_web#gid=22
[14:08] <tsdgeos> 3 unity-mir and 10 unity8
[14:08] <tsdgeos> yes
[14:08] <kgunn> ok
[14:08] <mzanetti> tsdgeos: yeah... but in total of branches we have was 53 on friday
[14:10] <mzanetti> tsdgeos: so those 13 should be released this week?
[14:10] <tsdgeos> mzanetti: no, those 13 need to be released today
[14:10] <tsdgeos> well
[14:10] <tsdgeos> on friday
[14:10] <tsdgeos> last friday
[14:10] <tsdgeos> this week we need to release *much* more stuff
[14:32] <tsdgeos> elopio: karni: you joining?
[14:32] <elopio> hectic morning, sorry.
[14:37] <mhr3> tsdgeos, we'll need someone to fix the issues with previews in new-scopes
[14:37] <tsdgeos> mhr3: karni, Cimi or me
[14:37] <tsdgeos> mhr3: write them somewhere, or tell me and i'll write them down and try to get them passed around
[14:37] <mhr3> tsdgeos, k, just reminding that it's another thing for this week :)
[14:37] <tsdgeos> i know i know
[14:38] <tsdgeos> mhr3: if you know what exactly fails, please tell me
[14:38] <tsdgeos> my todo item says "new preview needs work"
[14:38] <mhr3> tsdgeos, 1) preview doesn't open when clicking on non-first result
[14:38] <mhr3> 2) progress indicator never hides
[14:39] <mhr3> 3) i'm hungry
[14:39] <mhr3> 3 might seems unrelated, but it's because i need to check previews :P
[14:40] <Cimi> hah
[14:42] <tsdgeos> @unity: so the news scopes branch is https://code.launchpad.net/~unity-team/unity8/new-scopes/
[14:42] <tsdgeos> we can merge stuff in there
[14:42] <tsdgeos> but we should be as strict as we are with landing to trunk
[14:45] <paulliu> Is there any way to test the pinch event if I don't have multiple mouses? Is there any emulators for that?
[14:46] <greyback> dandrader: so here's my notepad of nexus7 jobbies to do: http://studio.sketchpad.cc/IOYP51EBZk
[14:47] <dandrader> greyback, I'm currently stuck with wifi
[14:47] <dandrader> greyback, no wifi device is listed
[14:49] <tsdgeos> dandrader: i've heard you need to flash some special android radio firmware or something
[14:49] <tsdgeos> greyback probably knwos which one, but if not ask in #ubuntu-touch
[14:50] <greyback> dandrader: that issue was mentioned in the ML thread. Which image did you flash? I flashed ogra's image
[14:50] <dandrader> tsdgeos, I'm currently poking people on #phablet
[14:50] <tsdgeos> okidoki
[14:51] <dandrader> greyback, I flashed the regular http://cdimage.ubuntu.com/ubuntu-touch/daily-preinstalled/pending/trusty-preinstalled-touch-armhf.zip
[14:51] <dandrader> greyback, ok, I will dig into this e-mail thread..
[14:52] <greyback> dandrader: I used http://people.canonical.com/~ogra/ubuntu-touch/ubuntu-touch-4.4.2-mir.rootfs-armhf.zip
[14:55] <mzanetti> greyback: kgunn: just to be sure. don't expect the current branch to perfectly work. the question is more if we should focus on fixing mine or fixing trunk during this week
[14:56] <greyback> mzanetti: yep, understood
[14:59] <kgunn> mzanetti: do you feel like we could ? get it fixed?
[15:01] <mzanetti> kgunn: not 100% I'm afraid. but afaik that wouldn't be the case for trunk too, right?
[15:01] <kgunn> mzanetti: possibly...that's why i was asking greyback what he had up his sleeve...
[15:02] <mzanetti> if we can make trunk perfectly fine for MWC, I'd vote for trunk
[15:02] <kgunn> in fact, greyback why don't you send me a list of any non MP'd branches
[15:02] <kgunn> i can do a more complete compare
[15:03] <dandrader> tsdgeos, greyback: rebooting solved my wifi issue.
[15:03]  * dandrader shrugs
[15:03] <tsdgeos> all the computer stuff is fixed that way!
[15:03] <tsdgeos> elopio: can we change wait_for timeout? do you know how one does that?
[15:04] <tsdgeos> @unity: ↑ do you remember?
[15:05] <greyback> kgunn: lp:~gerboland/unity8/sidestage-smoothness-tweaks/ and lp:~gerboland/unity8/sidestage-unlock-sidestage-appears/ and lp:/~gerboland/unity8/sidestage-handle-input-area/ will help a bit
[15:05] <kgunn> greyback: ta
[15:06] <tsdgeos> timeout=something
[15:06]  * tsdgeos tries
[15:06] <dandrader> tsdgeos, missing context.
[15:06] <tsdgeos> dandrader: autopilot wait_for
[15:06] <dandrader> tsdgeos, ah, no idea
[15:07] <elopio> tsdgeos: yes, this is the method signature:
[15:07] <elopio> def wait_for(self, expected_value, timeout=10)
[15:07] <tsdgeos> elopio: so i do
[15:08] <tsdgeos> wait_for(bla, timeout=5)?
[15:08]  * tsdgeos knows not much python :D
[15:08] <tsdgeos> seems to work
[15:08] <elopio> tsdgeos: where are you?
[15:08] <elopio> it's like self.main_window.visible.wait_for(True, timeout=60)
[15:08] <tsdgeos> ok
[15:08] <elopio> the timeout is in seconds.
[15:09] <tsdgeos> tx
[15:37] <tsdgeos> elopio:
[15:37] <tsdgeos> ping
[15:37] <tsdgeos> :D
[15:37] <elopio> tsdgeos: pong.
[15:37] <tsdgeos> so i've found i seems to be a "load" thing
[15:37] <tsdgeos> if you make the galaxy nexus be busy
[15:37] <tsdgeos> it fails more oten than when it's not busy
[15:37] <tsdgeos> i thought i had something that made the test more robust
[15:38] <tsdgeos> but just failed
[15:38] <tsdgeos> so i'm back to the ddrawing board
[15:38] <tsdgeos> sorry :D
[15:38] <elopio> tsdgeos: but why is the device busy? in jenkins, isn't running one single test at a time?
[15:38] <tsdgeos> well it also fails when it's not busy
[15:38] <tsdgeos> but less often
[15:39] <tsdgeos> for me we could just skip that failure, but i understand we need to have reliable tests
[15:40] <elopio> but it's still not clear to me.
[15:40] <elopio> tsdgeos: when the device is busy, does it take more than 10 seconds to open the scope?
[15:41] <tsdgeos> takes a lot to start
[15:41] <tsdgeos> so you swipe
[15:41] <tsdgeos> and stuff is not settled yet
[15:41] <tsdgeos> so the swipe is lost
[15:41] <elopio> hum, ok, so we need to wait for something before swiping, that makes a lot of sense.
[15:42] <tsdgeos> so my current "fix" is http://paste.ubuntu.com/6909754/
[15:42] <tsdgeos> it makes it quite a bit more stable
[15:42] <tsdgeos> but still got it to fail once
[15:44] <elopio> tsdgeos: where are you reproducing the errors?
[15:44] <elopio> I can't get it to fail.
[15:44] <tsdgeos> elopio: Cimi's galaxy nexus
[15:44] <tsdgeos> while [ true ]; do autopilot run unity8.shell.tests.test_emulators.DashAppsEmulatorTestCase.test_get_details; if [ $? -ne 0 ]; then break; fi; done
[15:45] <elopio> tsdgeos: and do you have the patch from https://code.launchpad.net/~elopio/unity8/fix1277591-open_scope_scrolling_waiting_for_wrong_property/+merge/205423 applied?
[15:45] <elopio> that will help a little too.
[15:45] <tsdgeos> actually no
[15:45] <tsdgeos> i don't ^_^
[15:45] <tsdgeos> let me apply it
[15:55] <elopio> I'm leaving the loop running while I walk the dog.
[15:55] <elopio> bbs.
[16:01] <om26er> MacSlow, ping
[16:01] <MacSlow> om26er, yup?
[16:01] <om26er> MacSlow, is there a way to disable the OSD notification on touch ?
[16:01] <MacSlow> om26er, no
[16:02] <om26er> MacSlow, they come during my autopilot test runs over the app's tabbar which results in failing test
[16:02] <om26er> since the app cannot switch tabs
[16:02] <MacSlow> om26er, unless you change unity8's sources of course
[16:02] <om26er> MacSlow, is there a plan to make them click through like on desktop ?
[16:03] <MacSlow> om26er, no... they were made explicitly non-click-through for UbuntuTouch.
[16:03] <MacSlow> om26er, do you run your AP-tests on a live device/session... just wondering
[16:04] <om26er> MacSlow, yes, we run it on mako
[16:07] <MacSlow> om26er, what kind of notifications do you get in your case? I assume you're not getting incoming calls on a test-device :)
[16:08] <om26er> MacSlow, its '<number> call ended' non-interactive
[16:10] <MacSlow> om26er, how odd... I wonder what app triggers that notification... doesn't make much sense to me.
[16:10] <om26er> MacSlow, its the dialer-app, when the call ends
[16:11] <MacSlow> Why would the user need to be notified about that?
[16:12] <MacSlow> om26er, he/she certainly knows that the moment the call was hung up :)
[16:12] <om26er> MacSlow, I was just told by the dialer-app dev that that notification is going away :)
[16:12] <MacSlow> om26er, :)
[16:37] <Cimi> tsdgeos, which is the signal I should trigger in rating when I change the rating?
[16:37] <Cimi> triggered(string widgetId, string actionId, var data)
[16:37] <tsdgeos> yes
[16:46] <elopio> tsdgeos: can you capture a video of the failure?
[16:47] <tsdgeos> elopio: the thing is running on cimi's phone
[16:47] <tsdgeos> not sure how to do that tbh
[16:47] <elopio> I am seeing it like 1/30 runs on this maguro, but that doesn't help a lot to understand where should I add a wait.
[16:47] <Cimi> tsdgeos, sorry was more a question, which data I sent?
[16:48] <tsdgeos> mhr3: which data do you want sent when someone clicks on the ratings? ↑↑↑
[16:48] <elopio> tsdgeos: you convince Cimi to hold a camera in front of the maguro until there's an error :p
[16:48] <Cimi> hah
[16:49] <tsdgeos> elopio: doesn't seem cool
[16:50] <mhr3> tsdgeos, Cimi, first of all, want a signal only when both rating and review are set
[16:50] <mhr3> well.. if "required"=="both"
[16:51] <mhr3> then keep the review widget format
[16:51] <mhr3> merge the input property dict with "rating", "author" & "review" keys
[16:52] <mhr3> see http://developer.ubuntu.com/api/devel/ubuntu-14.04/cplusplus/libunity-scopes/classunity_1_1scopes_1_1_preview_widget.html#rating-input
[16:52] <mhr3> and http://developer.ubuntu.com/api/devel/ubuntu-14.04/cplusplus/libunity-scopes/classunity_1_1scopes_1_1_preview_widget.html#reviews
[16:54] <Cimi> mhr3, rating *and* review?
[16:54] <Cimi> mhr3, why that?
[16:54] <Cimi> mhr3, a rating is fine
[16:54] <mhr3> Cimi, if required==both, both are needed
[16:54] <Cimi> I don't usually put a comment
[16:54] <Cimi> ah ok
[16:56] <mhr3> hm, not sure where you'd be getting "author" from... i guess that's server thing, so not needed from the shell
[16:58] <elopio> kgunn: how far are we from having the screen casting API to record autopilot tests executions?
[16:59] <kgunn> elopio: desktop closer, touch devices further
[17:01] <karni> tsdgeos: mhr3: I'm reading scrollback. I haven't looked into it yet, but I know the problem exactly. I believe tsdgeos suggested Saviq to make the property readonly during a code review, while it is actually used to *set* the header size. If you guys haven't fixed this yet, I'll take this on as priority once I finish reading scrollback.
[17:01] <mhr3> karni, good morning :)
[17:01] <mhr3> fixed
[17:02] <karni> Thank you. Yes, I had the fix ready, I thought I pushed it already to that MR :/
[17:02] <karni> Guess I was distracted with the sprint.
[17:02] <karni> mhr3: Morning. FYI It's 9AM here, so I won't be appearing much earlier than 8AM my TZ for next 2 weeks.
[17:03] <mhr3> someone's in california
[17:03] <karni> Yes, sir.
[17:03] <mhr3> is that what it takes to hide from us? :)
[17:04] <karni> mhr3: lol!
[17:04] <mhr3> karni, you need to go to hawaii, i still get 1 hour overlap with you
[17:04] <karni> mhr3: hahahh
[17:04] <karni> I wouldn't complain ;)
[17:05] <mhr3> actually anywhere between hawaii and central australia
[17:05] <karni> The more overlap we have, the better ;P
[17:06] <tsdgeos> mzanetti: so the blocking to fix unstable tests has been lifted, can you take care of the landing that is on the ci train document?
[17:12] <tsdgeos> elopio: do you know where the qml output go when running an autopilot test?
[17:16] <mhr3> karni, if you're looking for things to do, we'd want "image" widget type (even if it'd be just an alias for gallery with just one source)
[17:16] <elopio> tsdgeos: it's added to the test as details, and printed to stdout when it fails.
[17:21] <tsdgeos> elopio: you sure?
[17:21] <tsdgeos> i only get
[17:22] <tsdgeos> http://paste.ubuntu.com/6910208/
[17:22] <tsdgeos> and not the billions of qml output i've added
[17:22] <tsdgeos> elopio: which command line do you use?
[17:25] <mzanetti> tsdgeos: ok
[17:57] <mhr3> tsdgeos, i suppose you didn't have time to look at the special-category branch?
[17:57] <tsdgeos> mhr3: i didn't
[17:58] <tsdgeos> mhr3: are you expecting me to review it?
[17:58] <tsdgeos> or?
[17:58] <mhr3> tsdgeos, yes
[17:58] <mhr3> :)
[17:58] <tsdgeos> mhr3: i know close to nothing of unity-scopes-shell honestly
[17:59] <tsdgeos> mhr3: isn't there anyone else that can?
[17:59] <mhr3> tsdgeos, mzanetti saw it a few times
[17:59] <mhr3> but i thought that as long as you implement the shell side of it and it works you can greenlight it
[18:01] <tsdgeos> mhr3: ok, i'll have a look
[18:01] <mhr3> afterall it has good enough tests :)
[18:18] <karni> mzanetti: There's an e-mail which perhaps you could address on ubuntu-phone mailing list: [SDK] Expandable items
[18:19] <mzanetti> karni: hey... I started to reply but not sure what I should write tbh
[18:20] <karni> mzanetti: the interesting part is that I am almost positive I saw an expandable list view in action during the sprint. do we have some older component with similar functionality?
[18:20] <mzanetti> karni: not in the sdk... some apps do their own thing. and unity we do it ourselves too
[18:20] <karni> I see
[19:12] <karni> I have something that worries me. Not havin Saviq and tsdgeos around, who would I raise this with?
[19:13] <karni> A branch has been merged to new-scopes without the prerequisite being merged to unity8 trunk first. https://code.launchpad.net/~unity-team/unity8/newscopes-card-overlay/+merge/205359
[19:13] <karni> mhr3: by the way, what did you mean by saying everything will break tomorrow? we expect some minor changes?
[19:14] <mhr3> yea, we want to land scopes-api 0.3, and it changes things, see the email
[19:14] <karni> mhr3: noted the "image widget". I'll see if I can get to it today (I doubt today, though)
[19:14] <karni> mhr3: ok, I'll make sure to shut up now and read my mail :P
[19:15] <mhr3> karni, we're already using it in the previews, and shell just ignores it... so not nice
[19:15] <karni> ack
[19:22] <mhr3> karni, second major thing is that opening previews for stuff is mostly not working
[19:22] <mhr3> just sometimes it does
[19:23] <mhr3> and only when clicking on the first result
[19:23] <karni> sometimes? haha ;)
[19:23] <karni> mhr3: ack. is someone looking into this, or perhaps something I could take a look at? I guess I should get in sync with folks here how's the preview work coming along.
[19:24] <mhr3> karni, i don't think anyone is handling this yet
[19:24] <karni> noted
[19:40] <karni> mhr3: has the -readonly branch landed? can't find it
[19:42] <cwayne> 380939
[19:42] <cwayne> 431178
[19:42] <cwayne> agh crap, sorry
[19:47] <karni> mhr3: ok, found it. and its merged, great. thank you.
[21:59] <kgunn> its so quiet w.o saviq :(
[22:24] <mhr3> karni, could you send off a mail by your eod to sync up? (if you manage to do something dash/preview related)
[22:25] <karni> mhr3: Yes, although I'm test driving custom scopes atm, so I haven't even gotten to that yet. I'll try to get to the preview bug on items != 1st
[22:25] <karni> mhr3: sure
[22:26] <karni> I'll actually consider starting early tomorrow to make sure I wrap my stuff before Wed
[22:41] <cwayne> mhr3, hey, does unity8 no longer look in XDG_DATA_DIRS for scopes?
[22:41] <mhr3> cwayne, nope
[22:42] <mhr3> cwayne, you might want to comment on https://bugs.launchpad.net/unity-scopes-api/+bug/1274421
[22:43] <cwayne> mhr3, hm, that's bad for us
[22:43] <cwayne> mhr3, commented
[22:46] <cwayne> mhr3, just FYI, having unity look in XDG_DATA_DIRS is an MWC blocker at the moment
[23:03] <mhr3> cwayne, noted, will bring up tomorrow
[23:04] <mhr3> cwayne, we might want to change it to look in a specific dir instead of XDG_DATA_DIRS, but i guess you'd be fine with that
[23:04] <mhr3> of course it'd be a dir somewhere in /custom
[23:04] <cwayne> mhr3, i think we'd prefer XDG_DATA_DIRS, but i think ssweeny feels more strongly about that than I do :)
[23:08]  * ssweeny does :)
[23:09] <ssweeny> though really we should focus on where click scopes will go, since that will be how we do it eventually right?
[23:09] <ssweeny> the /custom XDG_DATA_DIRS thing is just a workaround waiting for that IIRC
[23:10] <ssweeny> cwayne, mhr3 ^^
[23:31] <cwayne> ssweeny, yeah, that was my understanding as well
[23:34] <cwayne> so maybe just adding /custom/xdg/data is fine as a stopgap then
[23:37] <ssweeny> maybe, but what about the shared libraries? can they go there as well?
[23:37] <ssweeny> the bug chatter seems to say they should be split out
[23:38] <ssweeny> but i don't think they would be in a click pkg
[23:38] <cwayne> hm, good point