karniSaviq: http://paste.ubuntu.com/6835250/ - no ideas :( We need that fix badly, you've seen the screenshots how bad it is (or make FilterGrid clip views instead of limiting the model)01:17
* karni -> eod04:26
karni(FTR I did sofly eod before, I just don't write this on every channel ;))04:27
veebersMirv: ping, hey I just saw your bug reports re: qt 5.2 && autopilot. I haven't had a proper look yet but had some thoughts/questions06:11
veebersI'm about to pop out for the night, thought I should mention it "Is it perhaps an AppArmor issue? Looks like there is a dbus exception happening" and "Is it possible that in this new version Qt have removed the ability to load the testability driver? I know we had to patch something in at some stage, but I'm not sure of the specifics off the top of my head".06:12
Mirvveebers: a good question. regarding Qt, the "load_testability_from_env_var.patch" is still applied ie support for QT_LOAD_TESTABILITY env var06:15
Mirvof course, there might be something changed with the testability driver, or then something on the apparmor side indeed06:15
MirvI updated the bug report with those thoughts06:24
tsdgeosdednick: ping08:49
dednicktsdgeos: pong08:49
tsdgeosSaviq: so what's the plan for the CI? isn't it a bit weird we are run both against released and unreelased stuff?08:50
tsdgeosdednick: can you put the new merge stuff in your prposals?08:50
Saviqtsdgeos, yeah, it is, I talked to fginther already, and we'll probably remove the local repo altogether08:50
tsdgeoshttps://code.launchpad.net/~nick-dedekind/unity8/indicator.ubuntu-settings-components/+merge/199311 https://code.launchpad.net/~nick-dedekind/unity8/indicators.better-visible-items/+merge/201789 and https://code.launchpad.net/~nick-dedekind/unity8/indicators.hint-interval/+merge/20210408:50
tsdgeosSaviq: makes sense, will help reproducing the problems too08:51
Saviqtsdgeos, what I would like is that it will be run against the CI Train silo PPA when active08:51
dednicktsdgeos: it's in https://code.launchpad.net/~nick-dedekind/unity8/indicators.better-visible-items/+merge/201789 already, will do rest.08:51
Saviqtsdgeos, otherwise we'll get build failures in CI, for example, when we'd be releasing some project we depend on at the same time08:51
tsdgeosdednick: tx08:52
tsdgeosSaviq: makes sense08:52
tsdgeosSaviq: i also was wondering if we're going to change who does the top approval09:03
Saviqtsdgeos, I don't think so09:03
Saviqtsdgeos, especially when we want to batch merges for release09:03
tsdgeosSaviq: we could use it more as a marker of "what the grand master merger wants to merge in the next run"09:03
Saviqtsdgeos, well, but that would mean he has to brood in all the MPs09:04
Saviqtsdgeos, as opposed to just Approved ones09:04
tsdgeosnot cool09:04
Saviqtsdgeos, and there's no problem with leaving one in Approved state until later09:04
Saviqtsdgeos, so yeah, I think top-Approved are basically all candidates for next release09:05
Saviqdidrocks, morning o/09:05
Saviqdidrocks, there's no 24h coverage of the landing team is there? any place where we could usually reach someone? again, chasing any one of you guys in person does not scale09:08
Saviqdidrocks, what do you think about automagically enabling silos for CI jobs when a project is active in CI Train?09:13
didrocksSaviq: in meetings, there will be 24h coverage soon, but it's a long topic about autoenabling silo assignement09:13
didrocksSaviq: we'll first try manually, if you wait too much, I'm happy to revisit09:14
Saviqdidrocks, no no, not what I was about09:14
Saviqdidrocks, please ping when you're out of meetings09:14
Saviqdidrocks, just wanted to leave you messages, not trying to drag you away from things09:14
seb128Saviq, they don't want to autoallow silos because they can want to be organize/ack what can land and not09:15
Saviqseb128, not what I was about09:15
seb128they want to organize*09:15
seb128Saviq, ok, I would like to be able to get a ppa to test without blocking on a CI person to press the ack button for me :p09:16
seb128but maybe it's only me09:16
Saviqseb128, different topic, but completely agree09:16
seb128I got stucked yesterday for half an hour waiting to get a silo so I could start a build before going away09:16
Saviqseb128, yeah, the test builds should probably be allowed, even if it won't be published straight away09:17
seb128but I guess then the issue is the number of ppas in the pool09:17
Saviqand the fact that the projects are locked when in silo09:18
Saviqseb128, anyway, what I meant was enabling the silo PPAs for -ci jobs for projects that are in an active silo currently09:18
seb128well, that's your issue as a project lander09:18
seb128Saviq, I'm not sure to understand what that means?09:18
Saviqseb128, say project x depends on a change in project y09:19
Saviqseb128, they both need to land at the same time09:19
Saviqseb128, -ci jobs for project x would fail due to (hopefully) dependencies or at least the missing change in project y09:19
Saviquntil now we had the local per-stack repo, built straight out of -autolanding jobs09:20
seb128ah, now I see what you mean09:20
seb128yeah, that makes sense09:20
seb128well to me, let's see what Dider&co think ;-)09:20
tsdgeosSaviq: i guess we should start adding 2014 to at least the (C) notices of the new files we add09:21
Saviqtsdgeos, indeed...09:23
Saviqtsdgeos, or we could just bump them all in one go...09:23
tsdgeosthat too09:23
tsdgeosnot sure what are the lawyer-guidelines for increasing the year in a file we haven't touched that year09:24
tsdgeosbut i guess changing the year includes touching the file ^_^09:25
Saviq.... 5.5GB ram free... "This computer does not have enough free memory to analyse and send this error report"...09:28
Saviq.crash file? 157K...09:28
tsdgeosyou wrapped an integer of free memory09:29
tsdgeospython was unhappy09:29
tsdgeosrandom guess09:29
SaviqMacSlow, is unity-notifications safe to release with all the changes after last release?09:55
tsdgeosnice. this year we have a much better positioned stand at MWC :)09:55
Saviqtsdgeos, where did you get that?09:56
tsdgeosgoogle does wonders :D09:56
Saviqtsdgeos, got a thing for you10:14
Saviqtsdgeos, we need to be able to tell LVWPH to start at category further down than 010:15
Saviqtsdgeos, WDYT?10:15
tsdgeoswhat for?10:15
Saviqtsdgeos, to "hide" the top-most category by default10:15
Saviqtsdgeos, so that unless you pull down you see the second category10:16
mhr3Saviq, that isn't needed atm, we said the favourites are revealed by bottom swipe10:16
Saviqmhr3, I must not have been there... and where do they show up?10:17
tsdgeosSaviq: so it's one of those "pull down extra to change the beginning of the list" type of things?10:17
mhr3Saviq, in that bottom swiped screen10:17
tsdgeossomeone used a Jolla phone? :D10:17
Saviqmhr3, not following10:18
Saviqtsdgeos, not springy, though10:18
Saviqtsdgeos, anyway, apparently I don't know something, as you were10:18
mhr3Saviq, the bottom swipe is supposed to give you a view similar to the current running view where you can remove favourites, reorder etc10:18
mhr3running apps*10:19
Saviqmhr3, yes, which is not gonna happen yet10:19
tsdgeosSaviq: i guess it's doable, not sure how much it would take, and would for sure need an exact design of how it's supposed to behave, there's like hundreds of ways it can work10:19
mhr3Saviq, the reordering yes, but the removing should10:19
Saviqmhr3, the bottom swipe was connected to the "zoom-out" gesture10:19
Saviqmhr3, which is not gonna happen10:19
Saviqmhr3, and the fall back was to have a top-most category with favourites10:20
mhr3oh good so then i missed something :P10:20
Saviqmhr3, well, not gonna happen for MWC I mean10:20
mhr3Saviq, oh, but 14.04?10:20
Saviqmhr3, no10:20
Saviqmhr3, first iteration we agreed to be just a top-most category in scopes scope, hidden by default10:21
mhr3i remember design being strongly against that10:21
mhr3although that != mark10:21
SaviqMark explicitly said that we should go for that top-most category to start with10:22
mhr3yes, and rest of design didn't agree10:22
Saviqok, I need to clarify this then10:23
mhr3ping katie once she's here10:24
mhr3Saviq, can i get someone to review https://code.launchpad.net/~mhr3/unity-scopes-shell/preview-model/+merge/203706 ?10:56
mhr3or maybe you want to? ;)10:56
=== _salem is now known as salem_
didrocksSaviq: back from all talking10:57
didrocksSaviq: so, publishing unity8 is ready it seems? awesome!10:57
MacSlowSaviq, I ran lp:unity-notifications with all notification ap-tests and the supported examples... all worked.10:58
Saviqdidrocks, yup, ready11:03
didrocksSaviq: and doneeeeeee11:03
SaviqMacSlow, that wasn't my question ;)11:03
didrocksSaviq: happy first landing!11:03
Saviqdidrocks, I need to wait now for them to be published, right?11:03
didrocksSaviq: in the release pocket yeah11:04
Saviqdidrocks, yup, thanks11:04
didrocksSaviq: if you run merge and clean before, it will tell you where things are11:04
Saviqdidrocks, I'll look at https://launchpad.net/ubuntu/+source/unity811:04
didrocksthat's how you can "hum, after 6 hours, still in proposed, there is an issue"11:04
didrocksSaviq: yeah ;)11:04
didrocksbut you don't have to11:04
didrocksyou can wait for 3 hours11:04
didrocksand rerun it11:04
Saviqdidrocks, yup, I know11:04
didrocksit will tell you :)11:04
didrocksSaviq: ok, I just meant, don't feel obliged to track right away :p11:05
MacSlowSaviq, I say yes11:05
SaviqMacSlow, ok :)11:05
MacSlowSaviq, I just also wanted to provide you with my reasons :)11:06
Saviqdidrocks, so, my question for you was: what do you think about enabling the relevant silo PPA in -ci jobs, when applicable?11:07
didrocksSaviq: so they are 2 sides of that story11:07
Saviqdidrocks, so say project x depends on a change in project y, and they're in the same silo, but still -ci jobs for them will fail11:07
didrocksfirst, a technical one, we'll need to write a jenkins plugin11:07
didrocks(but that shouldn't be a blocker)11:07
didrocksthen, there are social sides11:07
didrocks2 things as well here :)11:08
didrocksfirst, I think it's great to have the conversation on the testing plan11:08
didrockslike "you try to land the sdk, you listed only the sdk testing plan, you should as well run the unity8 one and the apps ones"11:08
didrocks-> that kind of feedback/discussion is good before assigning a silo IMHO11:08
didrocksonce this is addressed and people can make sensible decisions11:09
Saviqdidrocks, again, not my question I think ;)11:09
Saviqdidrocks, I'm not about *assigning* a silo11:09
didrockswe can have automated silos attribution for most of the case11:09
didrocksI shouldn't anticipate questions :p11:09
didrockslet me backlog again, sorry ;)11:09
Saviqdidrocks, say a silo is already assigned11:09
Saviqdidrocks, both project x and y are in there11:10
Saviqdidrocks, and x depends on the change in y11:10
Saviqdidrocks, -ci jobs for x would fail on (hopefully) dependencies, or just due to the missing change in y11:10
didrocksah, -ci jobs like the one showing on the MP?11:10
Saviqdidrocks, yes11:10
didrocksyeah, so that's in my mind11:10
didrocksand we should do that11:10
didrocks(and it's a known flaw)11:10
didrocksyou're quick to pick them up :p11:10
didrocksI just didn't want to disturb fginther before they finish their current project11:11
didrocksbut it's on my list to "poke the CI team about"11:11
Saviqdidrocks, yeah, I'm just gathering opinions11:11
Saviqso sounds like we're on the same page11:11
didrocksI completely agree that at least for the MP you set/attribute a silo for11:11
didrocksthey should add the ppa11:11
didrocksand use that11:11
didrocksI think the impact isn't low on the CI machinery to do that11:11
Saviqdidrocks, yup11:11
didrocksbut that would align us more with the Airline vision11:12
Saviqdidrocks, true11:12
didrocksso yeah, we do agree :)11:12
didrocksI just wonder if I should depress fginther right away11:12
didrocks(what I think I did pinging him twice)11:12
* didrocks has no glasses to see Francis' face on the other side of bluefinn11:13
Saviqdidrocks, ;)11:13
Saviqtsdgeos, mzanetti, could one of you do https://code.launchpad.net/~mhr3/unity-scopes-shell/preview-model/+merge/203706 ?11:15
Saviq/food for real11:16
tsdgeosi will11:16
fgintherdidrocks, ^ is what I wanted to talk about. today is not good for me, maybe tomorrow11:31
=== rachelliu_ is now known as rachelliu
tsdgeosMacSlow: added the 4 questions with their answers11:38
tsdgeosto https://code.launchpad.net/~aacid/unity-notifications/qcompare/+merge/20336211:38
MacSlowtsdgeos, thx... what do you think about "extending" unity8's new MR-process to unity-notifications?11:39
MacSlowSaviq, ^11:40
MacSlowtsdgeos, overkill or not?11:40
tsdgeosit's ok, i guess the ui question will never apply11:40
MacSlowtsdgeos, true11:40
tsdgeosbut i just answer no and that's it :D11:40
=== alan_g is now known as alan_g|afk
=== alan_g|afk is now known as alan_g
didrocksfginther: ah great!11:49
tsdgeosmhr3: is it me or testresultstest-ng is pretty unstable?12:05
tsdgeosevery second run it hands12:05
mhr3tsdgeos, yea, there's something wrong with the registry i already brought it up, fwiw with latest scopes-api it times out, and then the second try always works12:06
tsdgeosok :/12:06
tsdgeosmhr3: i always like doing while [ true ]; do make test; if [ $? -ne 0 ]; then break; fi; done to stress the tests12:10
tsdgeosbut i can't in unity-scopes-shell because of that12:10
mhr3tsdgeos, add pkill -f scoperegistry and scoperunner12:11
mhr3then it will work12:11
mhr3mostly the runner, registry should exit fine12:11
tsdgeosis that planned to work?12:12
tsdgeosi.e. it's a bug or per design?12:12
mhr3the registry "forgets" to kill the scope12:13
mhr3for some reason12:13
mhr3tsdgeos, while [ true ]; do make test; if [ $? -ne 0 ]; then break; fi; pkill -f scoperunner; done12:14
=== MacSlow is now known as MacSlow|lunch
karniSaviq: good day :) looks like we didn't land stuff yet :(? Did I miss something in the process? https://code.launchpad.net/~unity-team/unity8/unity8-fix-carousel5/+merge/20344412:17
mhr3oh.. forgetting you still need the timeout branch for that12:17
Saviqkarni, no, it just didn't happen yet - but it's almost there - waiting for things to get published in release pocket12:18
SaviqMacSlow|lunch, it will, but it's Thomas's turf12:18
karniSaviq: thanks :)12:18
Saviqkarni, did review of your branches, too12:19
* karni goes through them12:19
Saviqkarni, did your filtergrid branch actually fix the issue? I'm afraid I didn't understand well what was happening12:22
karniyes, allow me a moment to address yor comments :)12:23
Saviqkarni, ok good12:23
Saviqkarni, your fix carousel branch just got merged12:23
karniSaviq: in unity8-center-align-title - you mean the decision whether or not something should be centered should be based on mappings, not values, right?12:23
karniSaviq: \o/12:23
karniSaviq: if that's the case, I understand now, and you're right. I'll need to fix that.12:23
Saviqkarni, leave that be for now, please12:24
Saviqkarni, I'm extracting those kind of decisions (category-wide) to a CardTool12:24
karniSaviq: oh ok12:24
Saviqkarni, so it will fit in there12:24
* karni nod12:24
mhr3sil2100, could we still have automerger for non-trunk branches?12:27
tsdgeosmhr3: i don't understand the hideDash signal, what's it about?12:28
karniSaviq: addressing your filtergrid comments - you're right on the fixes. Regarding the question - yes, it does fix it. The problem seems to be that FilterGrid freaks out (rendering-wise) when backed by FilterProxyModel - that made the test implementation clean and 'smart', but made it fail at the same time. Instead, I would manually swap out the model, which would solve the failing tests - and the fix was still untouched, in that branch. ...12:28
karni... Concluding, yes - it does fix the rendering issues shown on screenshots :)12:28
* karni addresses the comment fixes12:28
Saviqkarni, right, so a FilterProxyModel over a FilterProxyModel freaks out12:29
Saviqkarni, file a bug indeed, please12:29
karniSaviq: do you know what's the toolkit project called? or shall I file against unity8?12:30
mhr3tsdgeos, it's scope saying "oh i probably launched something or invoked a dbus service, user wants to interact with that and not the dash"12:30
Saviqkarni, that's internal in unity812:30
Saviqdidrocks, sil2100, Mirv, can one of you guys please push unity-notifications through landing pipeline (old)?12:30
tsdgeosmhr3: hmmmm ok12:31
mhr3tsdgeos, shouldn't really be necessary in mir land though12:31
MacSlow|lunchSaviq, ok makes sense12:31
tsdgeosmhr3: wouldn't doing12:40
tsdgeos+ Q_EMIT previewReady(m_preview);12:40
tsdgeos+ m_preview.clear();12:40
tsdgeosget the receiver a dangling pointer?12:40
mhr3tsdgeos, the receiver owns the preview, not the scope12:41
mhr3tsdgeos, or well.. the one that invokes preview() owns it12:42
tsdgeosmhr3: ok, got it12:42
karniSaviq: Tried my best to explain this weird behavior. https://bugs.launchpad.net/unity8/+bug/127409512:43
ubot5Ubuntu bug 1274095 in Unity 8 "FilterGrid backed by FilterProxyModel renders wrong" [Undecided,New]12:43
sil2100Saviq: we'll try!12:48
Saviqsil2100, thanks :)12:48
sil2100mhr3: I think we could!12:49
Saviqkarni, yup, got it12:50
mhr3tsdgeos, but maybe i should have set the previewmodel's parent to the scope... still i wanted to express that the invoker is supposed to delete it once it's done with it12:52
tsdgeosmhr3: i think what you have works12:53
mhr3tsdgeos, well if the invoker didn't delete it, it would leak12:53
tsdgeosmhr3: maybe document in the .h that review() passes the ownership on12:53
tsdgeosit's typical for this kind of "it's your duty to delete stuff thing" i'd say12:53
mhr3tsdgeos, i have a feeling that qml will ignore me no matter what i say in a comment12:54
Saviqdednick, conflicts in hint-interval13:01
=== alan_g is now known as alan_g|lunch
CimiSaviq, can you repeat me what you told me yesterday about the models and model data? I think I'm confusing things here13:04
SaviqCimi, the model for the audio widget will be an array of dictionaries13:05
SaviqCimi, so [{"source": "foo", "title": "bar", "subtitle": "baz"}, { ... }] etc.13:05
CimiSaviq, I did this13:05
CimiSaviq, but how do I use this inside the repeater?13:05
SaviqCimi, to access those, you need to use modelData["source"], modelData["title"] etc.13:05
SaviqCimi, model: [...] works in a Repeater13:06
SaviqCimi, only then, in delegates, you don't have model.source, but modelData["source"] instead13:06
Cimioh great13:06
Cimithought that didn't work13:06
CimiI'll continue then13:06
CimiSaviq, wrote this for the test file http://paste.ubuntu.com/6837718/13:07
CimiSaviq, need to put title, length, uri with " "13:07
SaviqCimi, no, that's fine, Repeater { model: data.musicData; delegate: Label { text: modelData["title"] } } } will work now13:08
SaviqCimi, no need to wrap in the QtObject, though13:09
CimiI'll see if that works13:09
Cimithank you13:09
karniSaviq: "I'd collapse that into one line now." - I didn't want to make the line lengthy, but maybe I should just remove the { return .. } bits?13:13
karniSaviq: so it's one (broken) line13:13
karniSaviq: the readonly prop collapsedHeight: { return <formula> } -> readonly prop collapsedHeight: /n <formula>13:14
Saviqkarni, yeah, and our line lenght limit is 120 anyway13:14
karniI should have known better.13:14
Saviqkarni, nw13:14
Saviqkarni, but yeah, you shouldn't use { return ... } unless you need to wrap, or have more complicated bindings there13:14
karniSaviq: nw? :D13:14
* karni nods13:14
Saviqkarni, no worries13:18
karnithat one I didn't know haha13:18
karniSaviq: all comments addressed :) https://code.launchpad.net/~unity-team/unity8/unity8-fix-filtergrid/+merge/20365713:27
dednickSaviq: Yup, seen it13:28
Saviqkarni, dednick, k13:28
Saviqkarni, flatten 16+ readonly property int rowsWhenCollapsed: {13:33
Saviq17+ return Math.min(collapsedRowCount, Math.ceil(model.count / columns))13:33
Saviq18+ } too, please13:33
karnimissed that13:34
karniSaviq: collapsed, compacted and pushed13:35
karniSaviq: fixed one teset case tag, sorry.13:37
karni*test case tag13:37
sil2100Saviq: unity-notifications is building, we'll try releasing it later on13:37
Saviqsil2100, thanks13:37
Saviqkarni, approved13:45
karniSaviq: Thanks!13:45
karniSaviq: If carousel5 was merged to trunk, is it time to +! the new-scopes carousel fix? https://code.launchpad.net/~unity-team/unity8/new-scopes-fix-carousel5/+merge/20359713:45
karni* +113:45
Saviqkarni, yup, lemme merge trunk into new-scopes first13:45
karniSaviq: Sorry I'm bugging you, but thanks to you, we've pushed forward *significant* amount of work the phone delivery needs to make decisions for MWC :) We appreciate that.13:46
Saviqkarni, don't be13:47
Saviqkarni, conflicting, merge please13:47
* karni does that13:47
Saviqkarni, and that's why I didn't want to push before ;)13:48
karniSaviq: right, that is because getRenderer function in GenericScopeView is different in trunk and new-scopes. You're aware if I merge trunk to unity8, one day merging new-scopes back to trunk will conflict, right?13:50
karniunless we'll want that change in trunk already. for now, getRenderer is different in trunk and new-scopes13:51
karnithe latter uses the template, and has less arguments.13:51
Saviqkarni, no it won't conflict13:52
Saviqkarni, because I just merged trunk into new-scopes13:52
Saviqkarni, so your change will be correct on top of new-scopes13:52
Saviqon top of trunk13:52
=== alan_g|lunch is now known as alan_g
karniSaviq: I merge new-scopes, and bzr shows me only 1 conflict in tst_GenericScopeView.qml, while lp shows me 1 more in GenericScopeView.qml. super confusing.13:57
* karni expects 2 conflicts, like lp shows13:57
karniheh, ignore me ;P13:58
karniSaviq: fixed. https://code.launchpad.net/~unity-team/unity8/new-scopes-fix-carousel5/+merge/20359714:00
Saviqkarni, cheers14:01
elopiogood morning.14:11
elopiocan I get a review here please? https://code.launchpad.net/~elopio/unity8/generic_preview/+merge/20279314:11
tsdgeoselopio: sure, will do14:16
elopiotsdgeos: thank you.14:18
tsdgeoskgunn: Saviq: hangout or mumble today?14:20
kgunntsdgeos: let's hangout14:21
* tsdgeos fixes his hair14:22
kgunntsdgeos: you too huh ?14:23
* dandrader looks for a tshirt to put on14:23
* dednick looks for his pants14:23
dednickoh wait, dont need those14:23
karniSaviq: Can I test a custom scope using unity-scope-tool? I found there's --scope-dir argument, but couldn't make it work. If I install a scope on my machine, can I make unity-scope-tool pick it up?14:24
Saviqkarni, just switch the scope like you would on the phone14:25
Saviqkarni, either via top bar or swipe left/right14:25
karniIf someone could PM me the hangout link, that'd be neat :)14:26
karnior, just add it to calendar event14:27
mhr3they do mumble14:27
karninot today, I hear14:27
mhr3oh wow14:27
Saviqmhr3, yeah, it's a full-blown bi-weakly hangout today ;)14:27
karnimhr3: I thought you were on this team, how confusing ;D14:28
Saviqkarni, you got invite14:28
mhr3karni, anyway, --scope-dir takes a dir which has subdir and inside it needs to have subdir.ini and libsubdir.so14:28
mhr3subdir == scope_name usually14:28
karniThanks :)14:29
karnimhr3: ah14:29
mhr3so "--scope-dir /foo" if you have /foo/bar + /foo/bar/bar.ini + /foo/bar/libbar.so14:29
greyback_dammit google14:30
karnimhr3: helpful!14:30
tsdgeosyeah i need to swtich g+ identities too14:30
karnihrm. unity-scope-tool working for folks? http://paste.ubuntu.com/6838096/14:31
Saviqkarni, restart scope-registry should help14:32
* karni tries14:32
Saviqkgunn, you joining?14:32
karniSaviq: ha :> thanks14:33
Saviqkgunn, lost you on hangout...14:34
elopioSaviq: is the standup on hangout today?14:36
Saviqelopio, yes, sorry, let me paste you the link14:36
Saviqgreyback_, not happening for you?14:42
greyback_Saviq: no I only stay connected for 15-20 seconds then loose connection14:42
Saviqgreyback_, nice...14:43
greyback_Saviq: no good, I can't even connect now14:45
Saviqgreyback_, nw, just put your stuff in the doc14:45
greyback_Saviq: ack14:45
karniSaviq: what's the best way to check what has landed in trusty-proposed - the landing spreadsheet? what about new-scopes ppa? I'm fairly new to this, sorry.14:46
karnire: my question, just heard that trunk would only be a bit behind demo-stuff builds14:49
sil2100Saviq: hi! About unity-notifications, could you use CITrain for landing this maybe? Would be much of a problem?14:52
Saviqthostr_, ↑14:53
thostr_sil2100: I'm having a big timing problem tbh. So, if you could get landed I'd be more than happy14:54
Cimimzanetti, http://bazaar.launchpad.net/~unity-team/unity8/trunk/revision/55815:01
elopioSaviq: are you landing branches into a staging branch before landing them to trunk?15:01
=== dandrader is now known as dandrader|afk
Cimimzanetti, did you forget to add the ogg test files?15:01
Saviqelopio, no, that was a plan we had, but decided against it15:01
mzanettiCimi:  need to check, one sec15:02
SaviqCimi, I don't think we wanted test files like that - the Audio { } component is overridden for the tests15:02
elopioSaviq: that's the plan they have in autopilot, so can you tell me why you didn't like it? Just to see if they shouldn't do it.15:02
Saviqelopio, well, it's something the CI train folks don't want us to do, and because we can land multiple MPs at the same time in a single CI Train landing, it didn't really give many advantages15:03
MacSlowkgunn, Saviq: so first action (for the in-call notification of 2nd call) is to get latest feedback from the sdk-folks15:03
Saviqelopio, other than having a branch into which we merge before actually merging to trunk15:03
SaviqmacSlow, there's two of you...15:04
MacSlowkgunn, Saviq: yeah... I've not logged out of the other machine15:04
SaviqMacSlow, truth is first of all we need to get design to stabilize15:04
kgunnMacSlow: Saviq ...2 things really, design stable & sdk support....15:04
SaviqMacSlow, because they're changing all the buttons and their layouts to save space15:04
karnimhr3: Hey buddy, got a question. We have a aggregating scope for MWC, but new-scopes unity8 shows both aggregating and all subscopes. Is there a chance to fix that before the scope favoriting work will get done?15:04
Saviqkgunn, yeah15:04
elopioSaviq: so you land them on trunk one by one, running all the tests and making exploratory on each merge?15:05
MacSlowSaviq, sure... I'll continue with the visual snap-dec. queue for the moment... until we have the new design and the sdk-component15:05
CimiSaviq, so why having the uri specified then? :)15:05
Saviqelopio, I'm not doing it, CI Train is15:05
Saviqelopio, CI Train is building a PPA with all of the branchs15:06
Cimianyway fine, I'll do without15:06
Saviqelopio, I add that PPA and run the test plan on that15:06
elopioSaviq: ah, ok, after all of it is merged, only once.15:06
karnielopio: there's a temporary ppa build before stuff gets merged15:06
Saviqelopio, yeah, it's all merged together and put in the PPA15:06
elopioI get that.15:06
Saviqelopio, so yeah, staging branch was just more work with no particular advantage15:07
Saviqelopio, we also reduced the per-MP checklist a lot15:08
Saviqelopio, https://wiki.ubuntu.com/Process/Merges/Checklists/Unity8#preview15:08
Saviqelopio, while the landing TestPlan is more extensive https://wiki.ubuntu.com/Process/Merges/TestPlans/Unity8 - but that's what you do with the PPA15:08
elopioSaviq: ok. Sounds reasonable. I'm still missing to have trunk and a stable branch, instead of trunk being the stable one.15:09
elopiobut I suppose we can work with this.15:09
elopioI'll tell the autopilot guys about your process for them to copy the nice things.15:09
Saviqelopio, we've been doing stable trunk since the beginning, really15:10
Saviqelopio, or at least have been trying to15:10
Saviqelopio, mhr3 pointed out an important thing... you shouldn't dive a lot into the dash right now with emulators, because a lot is changing in that whole machinery15:11
elopioright. I don't mean that we should be allowed to break trunk, it should have all the tests passing always. But what I would like is to have an extra branch that changes less often, it's manually certified to be stable, and that's the one that gets into the phone.15:12
Saviqnot a lot will change conceptually, but the underlying code is changing a lot15:12
elopioSaviq: yes, but actually that means we should provide emulators to the projects consuming unity. Then, we updated the functionality, update the emulators, and for them it's transparent.15:12
Saviqelopio, yes, sure, just mentioning that the emulators will have to change internally at least, so working a lot on them right now might not be desirable15:13
Saviqelopio, [...] well, that's what happens with CI Train, only there isn't a branch, but a set of MPs that get manually certified15:13
* karni confirmed that any change to "art": "art" in unity-scope-tool, such as: "art": { "field": "art", "aspect-ratio": 0.5 } or even "art": { "field": "art" } breaks artImage (does not show at all).15:13
elopioright, I'm doing the bare minimum for the scope tests to run, and to keep running. When you change the way we open the applications scope, and the way we search, I can help updating the emulators I made to keep working.15:14
Saviqmhr3, see karni's comment ↑↑15:15
tsdgeoscan i convince someone to do https://code.launchpad.net/~aacid/unity8/delegateRangeBugfix/+merge/203498 ?15:15
tsdgeosSaviq: you?15:15
karnimhr3: artShape's are present in proper aspect-ratio, artImage within are completely gone (not shown)15:15
Saviqtsdgeos, findChild(genericScopeView, "dashCategory0") != undefined;15:16
Saviqtsdgeos, it's null now15:16
Saviqtsdgeos, but anyway15:16
Saviqtsdgeos, better to do typeof == "object" instead15:16
tsdgeosSaviq: != still works ;)15:16
Saviqtsdgeos, yeah, because it will never == ;)15:16
Saviqtsdgeos, which means you won't actually wait for the category, will you ;)15:17
tsdgeosSaviq: not really, since it's != and not !== it does that javascript magic15:17
Saviqtsdgeos, right, but still15:17
tsdgeosi'd say it does wait, i don't think the test would pass otherwise15:17
tsdgeosSaviq: want me to change the whole file? it's full of those that you forgot to update when you changed the return value15:18
=== dandrader|afk is now known as dandrader
mhr3karni, it's only about checking a settings/dconf value, we just didn't hook it up cause the default for that is not yet known15:19
mhr3karni, i could give you an envvar where you'd specify comma-separated scope names15:20
* karni consults15:20
Saviqtsdgeos, ok, leave it be now, and we'll fix it whole later15:20
tsdgeosSaviq: i'll propose a branch on top of it with the fixes15:20
elopiohey everybody, a warning. It seems that the most recent libautopilot-qt. the one that it's installed from the ubuntu-unity/daily-build is broken.15:21
Saviqtsdgeos, ok15:21
Saviqelopio, who uses daily-build any more ;)15:21
elopioyou can downgrade to the one in trusty, and things will work.15:21
elopiome and two more people who have just said that things broke everywhere :)15:22
elopioSaviq: shouldn't I be using it?15:22
karnimhr3: yes, we would very much like to know that envvar15:22
tsdgeosSaviq: mhr3: https://code.launchpad.net/~mhr3/unity-scopes-shell/preview-model/+merge/203706 looks good to me, want to top approve or prefer someone else to look at it?15:22
Saviqelopio, I don't think so, CI Train omits daily-build15:22
karnimhr3: so that we can get rid of unwanted subscopes for demoing at MWC (until we have a better solution)15:22
elopiook, I'll remove it.15:23
mhr3tsdgeos, feel free to approve, i'm confident in the tests :)15:23
Saviqtsdgeos, checked the coverage? ;)15:23
mhr3karni, will make a branch, but lunch first :)15:23
Saviqmhr3, want me to check the coverage? :D15:23
Saviqelopio, FWIW it was never "safe" as in stuff from there might never have ended up in distro15:24
karnimhr3: thank you15:24
mhr3Saviq, of course i do!15:24
tsdgeosSaviq: only visually, didn't run the coverage tool tbh15:24
tsdgeosvisually == it has a test of the new stuff15:24
Saviqtsdgeos, ah so your delegateRangeBugFix fixes the "only 9 apps visible" issue?15:26
tsdgeosi never understood that bug15:26
karnimhr3: If you could, after that envvar thing, address my comment on "art" component going haywire, I'd appreciate. I'm looking into it myself as well.15:26
tsdgeosbut it can cause that only some apps are displayed yes15:26
karnimhr3: Enjoy your lunch now, though :)15:26
mhr3karni, yea, looking at it now15:27
tsdgeosSaviq: i.e. people was teorizing about how you needed to disconnect from wifi or something15:27
tsdgeosbut maybe they did just not know how to break it properly15:27
karnimhr3: FYI Scott said (for aggregate scope thing) dconf would be fine next week. but if we can't get it before MWC, envvar would be critical to have.15:27
Saviqtsdgeos, yeah15:27
mhr3karni, i don't want dconf yet cause the default would be wrong15:28
karnimhr3: ack, let's go envvar then :)15:28
Saviqmhr3, just three holes in collectors.cpp, I'd say I'm happy ;)15:31
=== alan_g is now known as alan_g|tea
mhr3Saviq, btw QDesktopService::openUrl crashes when you don't have QGuiApplication :P15:31
Saviqmhr3, on xcb that is?15:32
Saviqsounds like a bug15:32
Saviqtsdgeos, Cimi, dandrader, elopio, kgunn, greyback_, mterry, mzanetti, karni, MacSlow, dednick, any other branch you'd like landed? I see 7 in queue already, would like to start landing them today15:34
CimiSaviq, you see the power of @uiteam ^^15:34
dandraderSaviq, no15:34
* greyback_ listens for all:15:34
mterrySaviq, https://code.launchpad.net/~unity-team/unity8/unity8.test_nested_mir/+merge/20308815:34
MacSlowSaviq, nope15:34
Saviqmterry, it's approved, so will be part of it, yeah15:34
mterrySaviq, ah OK15:34
karniSaviq: checking15:34
kgunni like @uiteam...."all" way too many false+15:34
SaviqCimi, fill in checklist ↑↑ please15:34
* mzanetti listens for "all:" too15:35
mterrySaviq, unreviewed, but I'd like to see https://code.launchpad.net/~mterry/unity8/hide-greeter-on-focus-request/+merge/201817 land15:35
Saviqkgunn, Cimi @unity?15:35
Saviqmterry, Cimi, please fill in checklist for test_nested_mir15:35
kgunnSaviq: +1 adding now15:35
mterryoh right the checklist.15:35
Saviqmterry, Cimi, as per https://wiki.ubuntu.com/Process/Merges/Checklists/Unity8#preview15:35
karniSaviq: new-scopes-fix-carousel5 and unity8-fix-filtergrid I care most about15:35
mzanettikarni: note the ":" at "all:"15:35
mzanettikgunn: ^^15:35
mzanettisorry karni15:35
karnimzanetti: np15:35
Saviqkarni, former is merged already, latter is in queue15:35
kgunnadded both15:36
Saviqtsdgeos, Cimi, dandrader, elopio, kgunn, greyback_, mterry, mzanetti, karni, MacSlow, dednick, please add "@unity" to IRC highlight words15:36
tsdgeosSaviq: with the @ ?15:36
Saviqtsdgeos, ye15:36
Cimisomeone to test pls15:37
Saviqtsdgeos, so that you don't get highlighted for unity, but @unity15:37
dandraderSaviq, I'm agains it15:37
Cimiworks fine15:37
dandraderSaviq, it will make it easier to polk people15:37
tsdgeosdidn't work15:37
Saviqdandrader, if it gets abused, we won't hold you accountable15:37
mzanettiworks here15:37
dandraderSaviq, there's a value in having to autocomplete every single person15:37
Saviqtsdgeos, @unity15:37
MacSlowtsdgeos, worked15:37
tsdgeosSaviq: stop saying my name, that breaks the test :D15:37
tsdgeosSaviq: ok, worked now15:38
tsdgeosguess doesn't highlight myself15:38
mzanettidudes! I've got work to do :P15:38
Saviqdandrader, let's see how it works15:38
Cimitsdgeos, even @cimirocks works for you I imagine :P15:38
Saviqkgunn had enough15:38
dandrader@unity makes poking more impersonal15:38
* mzanetti adds @cimirocks15:38
Saviqdandrader, yeah, but it's only meant for that15:38
=== alan_g|tea is now known as alan_g
Saviqdandrader, to poke the whole team15:39
kgunnone more time please15:39
Saviqdandrader, if wanted to poke any one in particular, you would type their name15:39
kgunnhmmm...manager probs15:39
tsdgeosdednick: there?15:39
mzanetti@unity: shouldn't it be @unity8 ?15:39
MacSlowmzanetti, now you're being german :)15:40
* karni adds hilight15:40
Saviqmzanetti, we'll be unity soon... ha ha ha.... HA Ha ha... MUAHAHAHAHAHA15:40
mzanettiSaviq: define "soon"15:40
Saviqmzanetti, whenever you implement window management for destkop15:40
mterryhmm, no highlight for me15:40
Saviqdesktop, even15:40
dandradercould someone say "@unity". testing my setting :)15:40
mterrySaviq, OK, checklist added to both branches I mentioned15:41
dandraderdidn't work15:41
Saviqmterry, cheers15:41
mzanettidandrader: really? you missed the 300 times it happened already=15:41
karniMacSlow: hahah15:41
CimiMacSlow, got that :P15:41
* elopio leaves the channel :)15:41
kgunnno workie for me either15:41
* MacSlow ROTFLBTC15:41
MacSlowkgunn, which IRC-client?15:41
kgunnMacSlow: Xchat15:41
MacSlowkgunn, hm... the moment I added @ unity (without the space) it worked15:42
dandradermaybe restarting xchat does the trick15:42
tsdgeosi broke an autopilot test ^_^15:42
MacSlowdandrader, didn't have to15:42
tsdgeosand it got merged15:42
karnifor irssi folks: /hilight -line -channels #ubuntu-unity -regexp @all15:42
mzanetti@unity (for dandrader)15:42
tsdgeosthe joy of having broken autopilot15:42
karninot @all, heh15:42
dandradermzanetti, lol15:42
kgunnMacSlow: that's what i added...in both exact words & nicknames to always highlight15:43
* mzanetti -> stupid15:43
kgunnand restarted15:43
mzanetti@unity (for daniel)15:43
* tsdgeos works on a fix fix15:43
dandradermzanetti, didn't work.... anyone using xchat here?15:43
MacSlow@unity (for kgunn)15:43
karniworks here :)15:43
mzanettiMacSlow: stupid too :D15:44
CimiMacSlow, don't add his name15:44
MacSlowkgunn, btw I'm using xchat-gnome... not plain xchat15:44
kgunnMacSlow: again...only minus kgunn :015:44
MacSlowmzanetti, Cimi: no... I like that "joke" :)15:44
mzanettioh... so its a running gag already15:44
MacSlowmzanetti, of course15:45
MacSlowkgunn, switch to xchat-gnome then ;)15:45
mzanettior something with Q or K in the beginning :P15:45
mterrykgunn, you are using plain xchat too?  Yeah, can't make it work for me15:45
MacSlowmzanetti, pfft :)15:45
* MacSlow recommends xchat-gnome15:45
mterryMacSlow, we need xchat-unity  ;)15:46
MacSlowmterry, give me two weekends ;)15:46
elopioquassel is the way.15:46
mterrydandrader, looks like we need to be using xchat-gnome15:46
* MacSlow misses the times when it was vi vs. emacs15:46
dandradermaybe I have to escape the "@" sign somehow in the setting....15:46
Saviqdidrocks, can we get a silo please?15:47
dandraderMacSlow, yeah, vim won :)15:47
MacSlowdandrader, yeah... thank god :)15:47
karniSaviq: changes will land in demo-stuff ppa first, then trunk (and image) next?15:48
Saviqkarni, all goes directly into release15:48
Saviqkarni, so demo-stuff loses its purpose soon15:48
Saviqkarni, as trunk == release with CI Train15:48
Saviqor trunk <= release for a short moment15:48
Saviqkarni, well, demo-stuff builds from new-scopes15:48
* karni nods15:48
Saviqkarni, so it should be updated already15:49
Saviqkarni, unity8 50 minutes ago15:49
SaviqSuccessfully built15:49
Cimikgunn, did you manager to get it set?15:50
kgunnCimi: nope..was googling15:51
=== mhall119_ is now known as mhall119
Saviqmhr3, is there any reason to build lp:unity-{api,scopes-api,scopes-shell} in demo-stuff ppa?15:51
Saviq(for trusty, that is)15:52
mhr3Saviq, now that trunk isn't trunk... not much15:52
Saviqmhr3, so you're CI-Training them, too?15:52
mhr3unfortunately :P15:53
karniSaviq: I don't see this branch in description of landing - I assume we skipped the landing part for it? https://code.launchpad.net/~unity-team/unity8/unity8-fix-carousel5/+merge/20344415:53
karniit's already merged15:53
didrocksSaviq: https://code.launchpad.net/~unity-team/unity8/unity8-fix-filtergrid/+merge/20365 doesn't exist15:53
Cimikgunn, ?unity "might" work15:53
karniI mean we landed it without updating spreadsheet?15:53
didrocksprobably a typo with the number15:53
Saviqkarni, Status:15:54
SaviqMerged Edit status15:54
Saviqkarni, it landed yesterday15:54
karniok, that's all I care :) thanks15:54
Saviqkarni, or well, today, but started yesterday15:54
Cimikgunn, I read it should support wildcards, so ? should replace the char @ that does not get highlighted15:54
Saviqkarni, row 9 in https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AuDk72Lpx8U5dFlCc1VzeVZzWmdBZS11WERjdVc3dmc&usp=drive_web#gid=015:54
Saviqor sil2100, could we get a silo for unity8 please?15:55
Cimikgunn, otherwise take a screenshot of your settings page and share with us :)15:56
sil2100Saviq: assigining!16:00
Saviqsil2100, thanks!16:00
Saviqsil2100, is there a list of people / a place where we could ping about that without having to poke any of you guys in person?16:01
sil2100Saviq: I think we don't have that set-up for now, but it's a valid thing to think about16:02
Saviqsil2100, in lieu of that, apart you and Didier, who else has the power to do so?16:03
=== mterry_ is now known as mterry
mterrySomeone please try at-unity again16:05
mterryPerfect!  Thanks, Cimi16:06
* mzanetti slaps Cimi16:06
sil2100Saviq: right now it's just me and Didier, but after next week's sprint we'll all be able to do that - I guess Mirv would be more or less be able to do that as well right now too16:06
mterrykgunn, dandrader: switching to xchat-gnome was painless.  It kept all my settings and even my new highlight setting from xchat16:06
mzanettinext one who uses this has to take notes in the standup for a week16:06
karnimzanetti: :DDD16:06
mterrymzanetti, I asked!16:06
kgunnmterry: ack16:06
Saviqsil2100, can I drop in one more MP?16:06
karni(uses without reason)16:06
mzanettiright... /me unslaps Cimi :D16:07
sil2100Saviq: btw.! Make sure you type in the whole MPs in it! Had to correct one since it was missing some numbers!16:07
Saviqsil2100, oups16:07
sil2100Saviq: yes, please add what you need, didn't assign it yet ;)16:07
Saviqsil2100, done16:07
sil2100Saviq: ok, assigning16:08
Saviqsil2100, thanks!16:09
CimiI'm downloading xchat source code :)16:09
MacSlowCimi, :)16:09
Cimiindeed at-unity doesn't work here16:09
sil2100Saviq: done! yw16:10
dandradermterry, hmmm, good to know. I'm just not a big fan of its GUI. but will give it another try16:11
CimiMacSlow, http://sourceforge.net/p/xchat/svn/HEAD/tree/src/common/inbound.c16:12
dandraderCimi, found the bug?16:12
Saviqseb128, could we ask you for a packaging review on https://code.launchpad.net/~nick-dedekind/unity8/indicator.ubuntu-settings-components/+merge/199311 ?16:12
Cimidandrader, I'm reading code16:13
seb128Saviq, looking16:13
Saviqseb128, thanks16:14
seb128Saviq, done, +1 from me16:16
Saviqseb128, thanks16:16
seb128yw ;-)16:16
Cimidandrader, the matching code is different between xchat-gnome and xchar16:17
Cimidandrader, http://sourceforge.net/p/xchat/svn/HEAD/tree/src/common/inbound.c#l23716:18
Cimithis is the broken function I believe16:18
dandraderCimi, from the looks of it, seems a word has to have only alpha chars16:22
Cimidandrader, indeed16:22
dandraderso the "@" is not considered part of a word16:22
Cimidandrader, I was just looking at that16:23
Cimidandrader, so, won't work16:23
Cimiunless you patch xchat16:23
dandraderCimi, or we change the calling phrase :)16:24
karniSaviq: FTR, comment on the doc from Katie "I removed art-mask height = header height for horizontal cards."16:26
dandraderSaviq, greyback_ (or whoever is more familiar with the side stage hacks): Does the QScreen have to have the side stage geometry16:27
CimiSaviq, unless you want to fix xchat, let's change to unityui :)16:27
dandraderCould QScreen:geometry have the real deal and QScreen::availableGeometry the side stage area?16:27
Cimior cimirocks16:28
Cimicimirocks works16:28
greyback_dandrader: I don't see any reason why not. If we're ok with an app knowing if it is on side stage, then I don't see the harm in letting it know the full screen dimensions16:29
dandradergreyback_, is there any benefit hiding it? the important thing is that the app window has a certain size and that it cannot grow16:31
dandraderby "hiding" I mean giving out fake info in QScreen16:31
greyback_dandrader: should a phone app run on the desktop, I think it should get the correct physical screen size like any other Qt app would. So for convergence, I think it should be the case everywhere16:32
Saviqgreyback_, dandrader, I think it depends, what benefit does an app have to know the full screen geometry?16:34
dandraderSaviq, it can tell the virtual keyboard to be in landscape even though the app itself is in portrait :-D16:35
Saviqdandrader, well, the OSK will know it, because it will have the full screen, so no telling between app and OSK needed (or wanted, for that matter)16:36
dandraderSaviq, but, seriously, I see it this way: when app code reaches for QScreen for info it's because it want to know info about the screen. why deny that?16:38
dandraderSaviq, it doesn't mean that its window is allowed to cover the entire screen, if that's the fear16:38
Saviqdandrader, sure, looking at it from that PoV, I have no ready answer for "why not", but then, I don't have one for "why", either ;)16:39
dandraderSaviq, it feels like you want to remove QScreen from the API when you deny it from giving out info16:40
dandradergiving out the info it's supposed to give16:41
dandraderit's like crippling it16:41
dandraderso, why are we crippling the API?16:42
Saviqdandrader, I'm not saying we are16:43
Saviqdandrader, I just wonder what it's there for for 99% of apps (other than shells, really)16:43
Saviqdandrader, but no, I don't want to say don't expose it16:44
Saviqdandrader, I don't see it as a security concern or anything16:44
Saviqdandrader, only thing we should be looking at is for apps to not assume that when they request fullscreen, that QScreen::geometry will be what they get16:45
Saviqand well, fullscreen in side stage will probably be prevented altogether... another good question16:45
tsdgeosmzanetti: i guess this is for you too https://code.launchpad.net/~aacid/unity8/bug1271676/+merge/20353016:54
tsdgeosmzanetti: it's a bit hacky since i need to make sure it goes though that special cased if in GenericScopeView16:54
CimiSaviq, https://code.launchpad.net/~unity-team/unity8/unity8.previews_audioPlayer/+merge/20338717:13
CimiSaviq, added tests, opinion? is the right way?17:14
SaviqCimi, looks good, yeah, s/uri/source/ please, and length, I think, was in seconds, no?17:17
SaviqCimi, could add a button or a selector for tryAudioPlayer, though17:17
SaviqCimi, and drop the bottom ThinDivider17:17
SaviqCimi, putting in review comment17:17
CimiSaviq, it was string in musicpreview17:18
CimiI can do int17:18
SaviqCimi, no if it was string then let it be it17:18
Saviqmhr3, length in Audio widget - string or int (seconds?)17:18
sil2100Saviq, thostr_: could you guys fill in a landing request in CI Train for that unity-notifications? Just fill in an empty merge request and write it in there, since we're so busy with firefighting here that we won't be able to build/push it17:19
CimiSaviq, tryAudioPlayer?17:19
mhr3Saviq, int seconds17:19
SaviqCimi, ↑17:19
SaviqCimi, when you go make tryAudioPlayer, you just get a blank page17:19
SaviqCimi, we try to add controls so that you can actually interact with the component to verify it works17:20
CimiSaviq, weird it's an int,... in music preview I see  model.length.length > 017:20
SaviqCimi, yeah, but let's go for seconds now, easier to apply locale17:20
=== dandrader is now known as dandrader|lunch
CimiSaviq, so why we had that .length > 0 ? :)17:22
SaviqCimi, OLD17:22
Cimiok fine17:22
SaviqCimi, you're doing the new thing17:23
SaviqCimi, in which we're fixing broken things like that17:23
=== boiko_ is now known as boiko
Saviqsil2100, done17:26
Saviqsil2100, can we have a silo please17:26
Saviqsil2100, unity8 is through testing, BTW17:27
CimiSaviq, which thin dividers to drop?17:27
SaviqCimi, the last one17:27
Cimiwhy only the last one017:27
SaviqCimi, because that's between preview widgets, will be handled outside of the widgets, I think17:28
SaviqCimi, if there's a "first" one, too - drop as well17:28
CimiSaviq, it was the separator17:28
CimiSaviq, between tracks17:28
SaviqCimi, ah ok, it's only between them?17:28
CimiI'd leave them both for now17:28
Cimiwell, it's the last17:29
Cimias well17:29
Cimithere's one at the top, all in between, one at bottom17:29
CimiSaviq, I can drop the first one and last one using index and model count17:29
SaviqCimi, sec, looking at visuals17:29
SaviqCimi, you should, too, btw: https://drive.google.com/a/canonical.com/#folders/0B-a_7E3tDxOgWDlCSWxkUGZlcnM17:30
sil2100Saviq: super awesome! Assigning - and we already saw unity8 and published it, it's migrating17:30
SaviqCimi, just drop them altogether https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgMnV2ZU1IamJoaEU/edit17:30
CimiSaviq, qt alternative to this? http://stackoverflow.com/questions/6312993/javascript-seconds-to-time-with-format-hhmmss17:34
Cimiif you know17:34
SaviqCimi, Qt.formatTime17:35
SaviqCimi, can't find the new docs, http://qt-project.org/doc/qt-4.8/qml-qt.html#formatTime-method17:37
CimiSaviq, but this is time17:37
CimiSaviq, I just want length in readable format17:37
SaviqCimi, that's the only thing available17:38
SaviqCimi, you should be able to use it fine17:38
SaviqCimi, you probably need "if length > 3600" to change the format to include hours, too17:38
Saviqsil2100, did you disable upstream merger for unity-notifications, though?17:39
sil2100Saviq: yes, but CI might have kicked in by accident17:40
sil2100(I redeployed it on jenkins so it should be OK)17:40
Saviqsil2100, yeah, looks fine17:40
mhr3karni, https://code.launchpad.net/~mhr3/unity-scopes-shell/non-dconf-dash-pages/+merge/20380618:01
CimiSaviq, might be ugly though18:06
CimiSaviq, I'd have to use a static date as reference18:06
CimiSaviq, add seconds to it18:06
Cimiis this ok to you?18:07
CimiI googled a bit and cannot think of anything else js-only18:07
SaviqCimi, the static date is 1.01.197018:07
Cimiyeah I know18:07
Cimiif you're fine counting from it.. ok18:07
SaviqCimi, you just ignore it anyway18:08
Cimicause I'll just print time18:08
Cimibut still I hoped I could have done just new QTime in js18:08
Ciminot possible18:08
Saviqsil2100, btw, can you approve https://code.launchpad.net/~saviq/unity-notifications/flush-through-train-ci/+merge/203797 for me, so that I don't do it myself ;)18:10
karnimhr3: looking18:11
sil2100Saviq: ;) Will do in a moment, but it's not required for it to be approved before landing - it's just for people cooperating with eachother18:12
greyback_Cimi: the SDK has a few date/time utils (I see a dateUtils.js file there anyway) - did you see that?18:13
dednickmzanetti: ping18:13
mzanettidednick: hey18:13
karnimhr3: Thank you, +1. I'm not on your team - you top-approve or me? aaaand how do we get this into the image :D?18:13
mhr3karni, we'll try to land it tomorrow18:14
karnimhr3: shweet, sir!18:14
mhr3karni, but you need to find approver, i can't approve my own branch :P18:14
* karni opens directory, haha18:15
dednick mzanetti: if you can take a look quick. https://code.launchpad.net/~nick-dedekind/unity8/plugin-qml/+merge/20381018:15
dednickmzanetti: fixes the plugins not showing qml in qtcreator18:15
mzanettidednick: oh really. cool!18:16
* mzanetti tries18:16
Saviqdednick, mzanetti https://code.launchpad.net/~mzanetti/unity8/indicators-qml-to-cmake/+merge/203351 hmm?18:16
karnimhr3: hrm :D who hangs out from your team at this time of the day? hahahah, I can find an approver tomorrow :) pstolowski would be good?18:16
mzanettiSaviq: I'll drop it if dednick's branch above works18:16
dednickSaviq: yeah, that's the nasty way to do it :)18:16
mhr3karni, Saviq, mzanetti, tsdgeos, dednick... there's lots of approvers around ;)18:17
karnimhr3: ah, I thought it had to be someone from Unity APIs team18:17
Saviqmhr3, pfft, don't you have your own team? :D18:17
Saviqman I thought for a second tsdgeos is still here... that'd be the end of the WORLD!18:17
mhr3Saviq, i'm on the edge, i can steal your people! :)18:18
karniSaviq: :D18:18
karniall your base are belong to us18:18
mhr3karni, i managed to rep the art issue with a test case18:19
mhr3now only to figure out why doesn't it work18:19
karnimhr3: that's a good start. right, I haven't found the problem yet, was tight on time. I have another 40 minutes before someone else steals my focus.18:20
Saviqseb128, think you should own ubuntu-settings-components landings or should we?18:27
seb128Saviq, those are shared widgets/components right? I would say the sdk team should own that :p18:28
=== rsalveti_ is now known as rsalveti
Saviqseb128, well, they're shared between you and us18:28
Saviqseb128, any of them that should be shared with other apps should just move to the sdk18:29
seb128Saviq, well, you guys did most of the work there so far so I would say it makes sense for you to ask for landing18:29
seb128Saviq, we can "trade" later if needed18:29
Saviqseb128, ok18:30
seb128Saviq, didrocks also said it's fine to have a "secondary lander" for stuff, e.g if I want to land something I can just give you a ping to say "I'm going to do a landing"18:30
Saviqseb128, yeah18:30
Saviqdidrocks, if still around, unity-notifications can be published now18:48
didrocksSaviq: done18:51
Saviqdidrocks, thanks18:51
Saviqdednick, ouch, forgot to add ubuntu-settings-components to the build script :/18:52
Saviqmhr3, "OnlineMusicQuery.cpp:20:32: fatal error: unity/scopes/Reply.h: No such file or directory" that expected?18:54
mhr3Saviq, yes, old onlinemusic18:55
Saviqmhr3, ah, recipe failed for saucy18:55
mhr3sigh... of course it did18:56
Saviqmhall119, sorry for the late reply, but it should now be possible to build lp:unity8 with ppa:unity-team and ppa:ubuntu-sdk-team enabled18:57
karniSaviq: landed means landed in unity8 - this is unrelated to release of trusty-proposed image, right?18:59
karniwell, demo-stuff ppa should suffice anyway19:00
Saviqkarni, yes, it's in archive, but the image is a few hours away still, but you can make your device writable19:00
Saviqkarni, and apt-get update/upgrade19:00
* karni nod and thanks :)19:01
Saviqkarni, demo-stuff will only get updated now that I merged trunk into new-scopes and pushed19:02
karniSaviq: roger that19:03
Saviqdaaaamn LP ;/19:03
Saviqoh! only 15 tries to request a build on a recipe...19:03
davmor2Saviq: see the statue you have there of the LP God, it needs to point south at all times or LP doesn't work anymore ;)19:05
Saviqdavmor2, man, south? I barely see sun here, how the heck am I to know where's south!19:05
davmor2Saviq: throw together a compass app for the phone :)19:07
mhr3karni, https://code.launchpad.net/~mhr3/unity-scopes-shell/fix-json-merging/+merge/20382019:08
* karni looks19:09
mhr3karni, oh, but you are in unity-team, so you're free to top-approve19:13
=== dandrader|lunch is now known as dandrader
karnimhr3: top-approved non-dconf-dash-pages19:14
karnimhr3: you take it from here?19:14
karnimhr3: also, trying to wrap my head aroud that second fix, very much appreciated19:15
cwaynekarni, make sure we document that envvar pls19:15
mzanettidednick: your branch works fine. I'll wait for ci to work before doing the review stuff19:15
cwayneshould be easy enough to set19:15
karnicwayne: I'll keep you in the loop19:15
cwaynekarni, kthxbye19:16
mzanettidavmor2: I have acompass app... was just waiting for the compass to actually work on the platform19:16
mzanettito polish for the platform19:16
davmor2mzanetti: haha19:17
karniSaviq: should I alredy see the filtergrid fix? I flashed trusty-proposed, installed demo-stuff, apt-get update/upgrade, and unity8-fake-env was the only package that showed up. I verified FilterGrid fix isn't there :(19:21
karnimhr3: I only don't understand line 56 of https://code.launchpad.net/~mhr3/unity-scopes-shell/fix-json-merging/+merge/20382019:23
karniyou're comparing a string to "results for: \"metadata\"" which looked suspicious to me19:23
karnithat does not look like a key or value19:23
mhr3that's the title of the result the scope provides19:24
mhr3karni, http://bazaar.launchpad.net/~mhr3/unity-scopes-shell/fix-json-merging/view/head:/tests/data/mock-scope/mock-scope.cpp#L5619:26
karnimhr3: and it should really be "result for \"metadata\"" instead of "metadata" in the test?19:27
karnibecause the RoleArt is compared against QString("art")19:28
karnithat's why RoleTitle compared to this stange string confused me19:28
mhr3because most of the time the tests are doing empty searches (/surfacing) and comparing all the titles against "" is error prone19:29
karnimhr3: so the RoleTitle ends up literally with a string saying 'result for: "metadata"' ? :)19:30
karnithat's a strange title, but let it be, if that's the case.19:30
karnimhr3: top-approved. can I leave it with you for whatever merging/landing that requires? (I honestly have no clue where that flows)19:31
mhr3sure.. and no worries... noone has these days :)19:31
karnimhr3: much thanks for prompt action, Michal19:32
mhr3i planned to write some docs in the afternoon... so that's why there are still none :P19:34
Saviqkarni, apt-cache policy unity8?19:35
karniSaviq: http://paste.ubuntu.com/6839700/19:35
Saviqkarni, 2014012819:36
Saviqkarni, apparently 0129 didn't reach your archive yet19:36
Saviqkarni, but anyway, since you want newscopes, you need to wait for 0129 to get built in demo-stuff19:36
karniI see. I guess I rushed things.19:36
* karni nods19:36
karniSorry, I'm like a kid waiting for presents these days.19:37
dandraderSaviq, still there19:38
Saviqkarni, you can always look at https://launchpad.net/~unity-team/+archive/demo-stuff/ to see when's the last thing happened19:38
Saviqdandrader, wassup?19:38
karniSaviq: yeah, there right now ^ ^19:38
karni*looking there19:38
mterrymzanetti, could you give https://code.launchpad.net/~mterry/unity8/hide-greeter-on-focus-request/+merge/201817 another lookover when you get a chance?  I added the requested AP tests19:39
dandraderThink I found a reason for having QScreen::geometry holding the side stage dimensions: QScreen::orientation concept seems to be tied to it19:39
mzanettimterry: ah ok. thanks19:39
dandradernot QScreen::availableGeometry19:39
dandraderthus if geometry is landscape but availableGeometry is portrait, the orientation will still be landscape even though the window actually has a portrait aspect ration19:40
dandraderseems like the whole orientation thing is designed with fullscreen (or "maximized") windows in mind19:41
Saviqdandrader, TBH I think that's correct19:41
Saviqdandrader, orientation of the screen is landscape19:42
Saviqeven if orientation of the window is not19:42
karninew-scopes-fix-carousel5 was merged into new-scopes, but is not listed in the changelog.19:42
dandraderSaviq, for instance, the definition of QWindow::contentOrientation is tied to the orientation of its QScreen19:43
mhall119Saviq: thanks, mhr3 told me that last week and I already have it built :)19:43
Saviqkarni, http://bazaar.launchpad.net/~unity-team/unity8/new-scopes/revision/58419:43
Saviqmhall119, well, I broke it in the mean time ;)19:43
Saviqmhall119, and well, we broke it today again...19:43
* karni shuts up ;)19:44
dandraderSaviq, yeah, makes sense indeed. problem is the content layouts are tied to the orientation and not to the window aspect ratio....19:44
Saviqmhall119, but now it's good, only you need to manually install qtdeclarative5-ubuntu-settings-components19:44
mhr3Saviq, why was it broken btw?19:45
dandraderso a window in landscape orientation but with a portrait aspect ratio will end up having a squeezed landscape layout19:45
mhall119Saviq: so what you're saying it that it builds when you're on holiday, and breaks when you're at work?19:45
Saviqmhall119, isn't that the better way?19:45
mhall119Saviq: is that new package in the PPA?19:45
Saviqmhall119, getting built now, yes19:45
Saviqmhr3, I built from lp:ubuntu/*, and that apparently isn't a good thing to do for packages that have an upstream19:45
mhall119Saviq: and will you update the build script to pull it in too?19:46
Saviqmhall119, https://code.launchpad.net/~saviq/unity8/packaging-and-cleanup/+merge/20382919:46
mhr3Saviq, but it worked a few times19:46
mhall119so there's nothing I need to update on the docs19:46
Saviqmhr3, and then it stopped ;)19:46
Saviqmhall119, nope, should be sorted tomorrow19:46
mhr3lol.. ok19:46
mhr3well.. now trunk == distro anyway19:47
bregmahey folks, I have Unity8 running natively on Mir on the desktop, and it is in need of some work -- is there an official way of tracking bugs that are specific to the desktop?19:51
bregmaand, does anyone want to try it out?19:51
karniphablet-config writable-image used to reboot the phone. that is no longer the case?20:22
cwaynekarni, it should20:25
CimiSaviq, I think I have my date in local time, cannot find reference to get UTC20:38
Cimi.UTC() doesn't seem to work20:39
greyback_bregma: I'd be interested in trying it out. Have you instructions somewhere?20:46
=== salem_ is now known as _salem
Cimigreyback_, you have any idea how to get utc time?20:47
Cimigreyback_, Qt.formatDateTime(new Date(modelData["length"] * 1000), "H:m:ss") adds 1 hour to me20:48
greyback_Cimi: have you seen this: http://qt-project.org/doc/qt-5.0/qtqml/qtqml-cppintegration-data.html#qdatetime-to-javascript-date20:49
greyback_the QDateTime class has a toUTC() method that might be what you need?20:50
Cimigreyback_, how do I use it in qml?20:51
CimiI cannot see this toUTC()20:53
Cimithere is supposed to be a UTC() in js reference, but it's not processed by qml20:53
bregmagreyback_, https://launchpad.net/~unity8-desktop-session-team/+archive/custom20:54
bregmaworks best with a touchscreen :)20:54
greyback_bregma: cool, thanks, will give it a shot20:55
greyback_Cimi: aha, this looks more like it: http://qt-project.org/doc/qt-5.0/qtqml/qml-qtquick2-date.html20:56
greyback_The QML Date object extends the JS Date object with locale aware functions.20:57
Cimigreyback_, but the command above is already locale aware, maybe20:57
greyback_Cimi: I'm not sure. the JS Date object isn't. The QML one is. So hopefully you just need to swap20:58
Cimiread with Qt.formatDateTime(new Date(30 * 1000), "H:m:ss")20:58
Cimishould be 0:0:3020:58
greyback_new Date() looks like JS20:58
Cimiinstead is 1:0:3020:58
greyback_else use the JS: http://qt-project.org/doc/qt-5.0/qtscript/ecmascript.html#date-objects20:59
karnidemo-stuff: Failed to build: amd64 armhf i38621:11
karnidpkg-buildpackage: error: debian/rules build-arch gave error exit status 221:12
=== dandrader_ is now known as dandrader|afk
karniSaviq: failed on tests/plugins/DashViews/horizontaljournaltest.cpp:28 private/v8.h: No such file or directory (missing include o_O)21:17
Saviqkarni, that's on saucy21:21
Saviqkarni, fixed in https://code.launchpad.net/~saviq/unity8/packaging-and-cleanup/+merge/20382921:22
karniSaviq: oh. do we expect a trusty build kicks off as well?21:22
Saviqkarni, checking, something failed in the recipe21:23
Saviqkarni, kicking it again21:23
* karni dances \o\ /o/ \o\21:24
Saviqkarni, you can always look at https://code.launchpad.net/~unity-team/+recipe/unity8-daily-newscopes to see what's happened, and "Request builds" at the bottom in case stuff's wrong21:24
karniSaviq: noted!21:24
karnilp time outs on me :D how dare you, lp!21:25
karniSaved the link, anyway :)21:25
=== seb128_ is now known as seb128
Saviqmterry, humpf :/ https://code.launchpad.net/~unity-team/unity8/unity8.test_nested_mir/+merge/203088 is broken on x8621:29
SaviqCimi, ↑21:30
* mterry looks21:30
mterrycurious it would be arch-specific21:30
Saviqmterry, it's probably not arch-specific, but unity8-as-shell vs. unity8-in-window21:32
mterrySaviq, hmm...  I'm guessing that network-manager is not something we can assume is installed?21:35
Saviqmterry, it definitely is installed on those machines21:35
karniIf I want ssweeny to test my locally built unity8, is the unity8 deb sufficient, or unit8-private required as well?21:35
Saviqkarni, -private, too21:35
* ssweeny takes note21:35
Saviqkarni, ssweeny, btw unity8 cross-builds nicely again21:36
Saviqkarni, ssweeny, sbuild + ccache and I can cross-build it within a few minutes for armhf21:36
=== dandrader|afk is now known as dandrader
Saviqcheck out https://wiki.ubuntu.com/CrossBuilding and https://wiki.ubuntu.com/SimpleSbuild if interested21:37
karniI will!21:37
karniI tried pbuilder some time ago, but didn't spent enough time on it.21:37
mterryWell the error is a basic "Not authorized", not that the policykit file isn't installed or working or whatever21:37
* karni notes21:37
mterryThe policy we ask about would only be rejected if the session that unity8 was in were "inactive" from a logind sense21:37
mterrySaviq, "WARNING: Could not get session id for session. Check that logind is properly installed and pam_systemd is getting used at login."21:39
Saviqmterry, they are saucy machines IIUC, does that matter?21:40
mterrySaviq, is it reasonable for the test to assume an active session?  If so, we need to fix CI.  Else we need to change the test to test a different way21:40
mterrySaviq, I don't think so...  It looks like the machine is just misconfigured21:40
Saviqmterry, yes, the test just run inside a standard unity7 session21:40
mterryfginther, heyo!21:41
mterryfginther, Cimi added a new AP test that checks if the unity session is active from a logind perspective21:41
mterryfginther, but tests are showing that sometimes that's not true, when I would expect it to be.  See above ^21:41
mterryfginther, any ideas?21:41
mterryor nicks of who I should actually bother?  :)21:41
Saviqmterry, I believe fginther's in London this wek21:43
mterrySaviq, do you know a good CI backup?21:44
* mterry hops in ci-eng21:44
Saviqmterry, and please pay more attention to CI failures next time ;)21:46
Saviqmterry, as we got the warning here https://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/2304/?21:47
Saviqwhich is the last otto run on that branch21:47
mterrySaviq, yup, that was my fault21:47
mterryI was in ignore-otto mode because of the persistent failures21:47
Saviqmterry, yeah, me too :/21:48
Saviqkarni, uploaded to ppa and is building21:48
karniSaviq: thanks, I was just sharing a local build with Scott :) Nevertheleess, that's great news!21:49
veebersSaviq: Would you know where the unity ppa get's its version of libautopilot-qt? Or who to ask at least?21:49
Saviqveebers, which unity ppa?21:50
veebersSaviq: one moment just finding out, I've been ping due to this failure and trying to figure it out: https://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/2384/consoleFull21:50
Saviqveebers, D09add_ppa~ubuntu-unity~daily-build21:51
veebersSaviq: hah, right just found that in the log :-)21:51
Saviqveebers, daily-build is enabled on that job (and if you ask me - it shouldn't be)21:51
veebersSaviq: ah ok. I'm now trying to figure out where that ppa gets it's libautopilot-qt code from21:53
veebersthere may be an issue21:53
Cimimterry, you still here?23:29
mterryCimi, sure23:30
Cimimterry, what does Qt.formatDateTime(new Date(modelData["length"] * 1000), "H:m:ss")  give you?23:30
Cimimterry, Qt.formatDateTime(new Date(30 * 1000), "H:m:ss")23:30
Cimijust on a label23:31
mterryin qml?  let me see23:31
mterryCimi, 19:0:3023:32
Cimimterry, yeah23:32
Cimimterry, it's not UTC23:32
Cimimterry, would like it to be 0:0:3023:32
CimiI have no ideas23:33
SaviqCimi, just write a formatter yourself23:34
CimiSaviq, it's what I am doing now23:34
CimiSaviq, but before giving up, wanted to understand why23:34
SaviqCimi, although it's weird that it would assume the time stamp to be UTC...23:35
CimiSaviq, on qt docs they say that there might be issues with that23:35
SaviqOTOH maybe a timestamp needs to be UTC, as that's the only way to be nonambiguous23:35
Cimino that as for dst23:36
SaviqCimi, please take on the simpler widgets next, like image / text etc., leave the more extensive ones for later23:36
Saviqon that note o/23:37
CimiSaviq, ok pushed a formatter23:45
Cimigood night guys23:45
karniSaviq: you still here, hah :) FYI Card carousel card-size only affects first card (lolwut :) )23:46
karninight Cimi23:46
* karni also EODed recently23:46
Ciminight karni23:46
karnis/recently/moments ago23:46

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