| karni | Saviq: 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 | https://code.launchpad.net/~unity-team/unity8/unity8-fix-filtergrid/+merge/203657 | 04:26 |
| karni | https://code.launchpad.net/~unity-team/unity8/unity8-center-align-title/+merge/203664 | 04:26 |
| * karni -> eod | 04:26 | |
| karni | (FTR I did sofly eod before, I just don't write this on every channel ;)) | 04:27 |
| veebers | Mirv: 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/questions | 06:11 |
| veebers | I'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 |
| Mirv | veebers: a good question. regarding Qt, the "load_testability_from_env_var.patch" is still applied ie support for QT_LOAD_TESTABILITY env var | 06:15 |
| Mirv | of course, there might be something changed with the testability driver, or then something on the apparmor side indeed | 06:15 |
| Mirv | I updated the bug report with those thoughts | 06:24 |
| tsdgeos | dednick: ping | 08:49 |
| dednick | tsdgeos: pong | 08:49 |
| tsdgeos | Saviq: 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 |
| tsdgeos | dednick: can you put the new merge stuff in your prposals? | 08:50 |
| Saviq | tsdgeos, yeah, it is, I talked to fginther already, and we'll probably remove the local repo altogether | 08:50 |
| tsdgeos | https://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/202104 | 08:50 |
| tsdgeos | Saviq: makes sense, will help reproducing the problems too | 08:51 |
| Saviq | tsdgeos, what I would like is that it will be run against the CI Train silo PPA when active | 08:51 |
| dednick | tsdgeos: it's in https://code.launchpad.net/~nick-dedekind/unity8/indicators.better-visible-items/+merge/201789 already, will do rest. | 08:51 |
| Saviq | tsdgeos, otherwise we'll get build failures in CI, for example, when we'd be releasing some project we depend on at the same time | 08:51 |
| tsdgeos | dednick: tx | 08:52 |
| tsdgeos | Saviq: makes sense | 08:52 |
| tsdgeos | Saviq: i also was wondering if we're going to change who does the top approval | 09:03 |
| Saviq | tsdgeos, I don't think so | 09:03 |
| Saviq | tsdgeos, especially when we want to batch merges for release | 09:03 |
| tsdgeos | Saviq: we could use it more as a marker of "what the grand master merger wants to merge in the next run" | 09:03 |
| Saviq | tsdgeos, well, but that would mean he has to brood in all the MPs | 09:04 |
| tsdgeos | right | 09:04 |
| Saviq | tsdgeos, as opposed to just Approved ones | 09:04 |
| tsdgeos | not cool | 09:04 |
| Saviq | tsdgeos, and there's no problem with leaving one in Approved state until later | 09:04 |
| Saviq | tsdgeos, so yeah, I think top-Approved are basically all candidates for next release | 09:05 |
| Saviq | didrocks, morning o/ | 09:05 |
| Saviq | didrocks, 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 scale | 09:08 |
| Saviq | didrocks, what do you think about automagically enabling silos for CI jobs when a project is active in CI Train? | 09:13 |
| didrocks | Saviq: in meetings, there will be 24h coverage soon, but it's a long topic about autoenabling silo assignement | 09:13 |
| didrocks | Saviq: we'll first try manually, if you wait too much, I'm happy to revisit | 09:14 |
| Saviq | didrocks, no no, not what I was about | 09:14 |
| Saviq | didrocks, please ping when you're out of meetings | 09:14 |
| Saviq | didrocks, just wanted to leave you messages, not trying to drag you away from things | 09:14 |
| seb128 | Saviq, they don't want to autoallow silos because they can want to be organize/ack what can land and not | 09:15 |
| Saviq | seb128, not what I was about | 09:15 |
| seb128 | they want to organize* | 09:15 |
| seb128 | Saviq, 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 :p | 09:16 |
| seb128 | but maybe it's only me | 09:16 |
| Saviq | seb128, different topic, but completely agree | 09:16 |
| seb128 | I got stucked yesterday for half an hour waiting to get a silo so I could start a build before going away | 09:16 |
| Saviq | seb128, yeah, the test builds should probably be allowed, even if it won't be published straight away | 09:17 |
| seb128 | right | 09:17 |
| seb128 | but I guess then the issue is the number of ppas in the pool | 09:17 |
| Saviq | and the fact that the projects are locked when in silo | 09:18 |
| Saviq | seb128, anyway, what I meant was enabling the silo PPAs for -ci jobs for projects that are in an active silo currently | 09:18 |
| seb128 | well, that's your issue as a project lander | 09:18 |
| seb128 | Saviq, I'm not sure to understand what that means? | 09:18 |
| Saviq | seb128, say project x depends on a change in project y | 09:19 |
| Saviq | seb128, they both need to land at the same time | 09:19 |
| Saviq | seb128, -ci jobs for project x would fail due to (hopefully) dependencies or at least the missing change in project y | 09:19 |
| Saviq | until now we had the local per-stack repo, built straight out of -autolanding jobs | 09:20 |
| seb128 | ah, now I see what you mean | 09:20 |
| seb128 | yeah, that makes sense | 09:20 |
| seb128 | well to me, let's see what Dider&co think ;-) | 09:20 |
| Saviq | ;) | 09:20 |
| tsdgeos | Saviq: i guess we should start adding 2014 to at least the (C) notices of the new files we add | 09:21 |
| Saviq | tsdgeos, indeed... | 09:23 |
| Saviq | tsdgeos, or we could just bump them all in one go... | 09:23 |
| tsdgeos | that too | 09:23 |
| tsdgeos | not sure what are the lawyer-guidelines for increasing the year in a file we haven't touched that year | 09:24 |
| tsdgeos | but i guess changing the year includes touching the file ^_^ | 09:25 |
| Saviq | ;) | 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 |
| tsdgeos | :D | 09:28 |
| tsdgeos | you wrapped an integer of free memory | 09:29 |
| tsdgeos | python was unhappy | 09:29 |
| tsdgeos | random guess | 09:29 |
| Saviq | MacSlow, is unity-notifications safe to release with all the changes after last release? | 09:55 |
| tsdgeos | nice. this year we have a much better positioned stand at MWC :) | 09:55 |
| Saviq | tsdgeos, where did you get that? | 09:56 |
| tsdgeos | google does wonders :D | 09:56 |
| Saviq | tsdgeos, got a thing for you | 10:14 |
| tsdgeos | shoot | 10:14 |
| Saviq | tsdgeos, we need to be able to tell LVWPH to start at category further down than 0 | 10:15 |
| Saviq | tsdgeos, WDYT? | 10:15 |
| tsdgeos | what for? | 10:15 |
| Saviq | tsdgeos, to "hide" the top-most category by default | 10:15 |
| Saviq | tsdgeos, so that unless you pull down you see the second category | 10:16 |
| mhr3 | Saviq, that isn't needed atm, we said the favourites are revealed by bottom swipe | 10:16 |
| Saviq | mhr3, I must not have been there... and where do they show up? | 10:17 |
| tsdgeos | Saviq: so it's one of those "pull down extra to change the beginning of the list" type of things? | 10:17 |
| mhr3 | Saviq, in that bottom swiped screen | 10:17 |
| tsdgeos | someone used a Jolla phone? :D | 10:17 |
| Saviq | mhr3, not following | 10:18 |
| Saviq | tsdgeos, not springy, though | 10:18 |
| Saviq | tsdgeos, anyway, apparently I don't know something, as you were | 10:18 |
| mhr3 | Saviq, the bottom swipe is supposed to give you a view similar to the current running view where you can remove favourites, reorder etc | 10:18 |
| mhr3 | running apps* | 10:19 |
| Saviq | mhr3, yes, which is not gonna happen yet | 10:19 |
| tsdgeos | Saviq: 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 work | 10:19 |
| mhr3 | Saviq, the reordering yes, but the removing should | 10:19 |
| Saviq | mhr3, the bottom swipe was connected to the "zoom-out" gesture | 10:19 |
| mhr3 | right | 10:19 |
| Saviq | mhr3, which is not gonna happen | 10:19 |
| Saviq | mhr3, and the fall back was to have a top-most category with favourites | 10:20 |
| mhr3 | oh good so then i missed something :P | 10:20 |
| Saviq | mhr3, well, not gonna happen for MWC I mean | 10:20 |
| mhr3 | Saviq, oh, but 14.04? | 10:20 |
| Saviq | mhr3, no | 10:20 |
| Saviq | mhr3, first iteration we agreed to be just a top-most category in scopes scope, hidden by default | 10:21 |
| mhr3 | i remember design being strongly against that | 10:21 |
| mhr3 | although that != mark | 10:21 |
| Saviq | Mark explicitly said that we should go for that top-most category to start with | 10:22 |
| mhr3 | yes, and rest of design didn't agree | 10:22 |
| Saviq | ok, I need to clarify this then | 10:23 |
| mhr3 | ping katie once she's here | 10:24 |
| Saviq | /food | 10:49 |
| mhr3 | Saviq, can i get someone to review https://code.launchpad.net/~mhr3/unity-scopes-shell/preview-model/+merge/203706 ? | 10:56 |
| mhr3 | or maybe you want to? ;) | 10:56 |
| === _salem is now known as salem_ | ||
| didrocks | Saviq: back from all talking | 10:57 |
| didrocks | Saviq: so, publishing unity8 is ready it seems? awesome! | 10:57 |
| MacSlow | Saviq, I ran lp:unity-notifications with all notification ap-tests and the supported examples... all worked. | 10:58 |
| Saviq | didrocks, yup, ready | 11:03 |
| didrocks | Saviq: and doneeeeeee | 11:03 |
| Saviq | MacSlow, that wasn't my question ;) | 11:03 |
| didrocks | Saviq: happy first landing! | 11:03 |
| Saviq | didrocks, I need to wait now for them to be published, right? | 11:03 |
| didrocks | Saviq: in the release pocket yeah | 11:04 |
| Saviq | didrocks, yup, thanks | 11:04 |
| didrocks | Saviq: if you run merge and clean before, it will tell you where things are | 11:04 |
| Saviq | didrocks, I'll look at https://launchpad.net/ubuntu/+source/unity8 | 11:04 |
| didrocks | that's how you can "hum, after 6 hours, still in proposed, there is an issue" | 11:04 |
| didrocks | Saviq: yeah ;) | 11:04 |
| didrocks | but you don't have to | 11:04 |
| didrocks | you can wait for 3 hours | 11:04 |
| didrocks | and rerun it | 11:04 |
| Saviq | didrocks, yup, I know | 11:04 |
| didrocks | it will tell you :) | 11:04 |
| didrocks | Saviq: ok, I just meant, don't feel obliged to track right away :p | 11:05 |
| MacSlow | Saviq, I say yes | 11:05 |
| Saviq | MacSlow, ok :) | 11:05 |
| MacSlow | Saviq, I just also wanted to provide you with my reasons :) | 11:06 |
| Saviq | didrocks, so, my question for you was: what do you think about enabling the relevant silo PPA in -ci jobs, when applicable? | 11:07 |
| didrocks | Saviq: so they are 2 sides of that story | 11:07 |
| Saviq | didrocks, 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 fail | 11:07 |
| didrocks | first, a technical one, we'll need to write a jenkins plugin | 11:07 |
| didrocks | (but that shouldn't be a blocker) | 11:07 |
| didrocks | then, there are social sides | 11:07 |
| didrocks | 2 things as well here :) | 11:08 |
| didrocks | first, I think it's great to have the conversation on the testing plan | 11:08 |
| didrocks | like "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 IMHO | 11:08 |
| didrocks | once this is addressed and people can make sensible decisions | 11:09 |
| Saviq | didrocks, again, not my question I think ;) | 11:09 |
| Saviq | didrocks, I'm not about *assigning* a silo | 11:09 |
| didrocks | we can have automated silos attribution for most of the case | 11:09 |
| didrocks | oh | 11:09 |
| didrocks | I shouldn't anticipate questions :p | 11:09 |
| didrocks | let me backlog again, sorry ;) | 11:09 |
| Saviq | didrocks, say a silo is already assigned | 11:09 |
| didrocks | yep | 11:09 |
| Saviq | didrocks, both project x and y are in there | 11:10 |
| didrocks | yeah | 11:10 |
| Saviq | didrocks, and x depends on the change in y | 11:10 |
| Saviq | didrocks, -ci jobs for x would fail on (hopefully) dependencies, or just due to the missing change in y | 11:10 |
| didrocks | ah, -ci jobs like the one showing on the MP? | 11:10 |
| Saviq | didrocks, yes | 11:10 |
| didrocks | yeah, so that's in my mind | 11:10 |
| didrocks | and we should do that | 11:10 |
| didrocks | (and it's a known flaw) | 11:10 |
| didrocks | you're quick to pick them up :p | 11:10 |
| Saviq | :D | 11:11 |
| didrocks | I just didn't want to disturb fginther before they finish their current project | 11:11 |
| didrocks | but it's on my list to "poke the CI team about" | 11:11 |
| Saviq | didrocks, yeah, I'm just gathering opinions | 11:11 |
| Saviq | so sounds like we're on the same page | 11:11 |
| didrocks | I completely agree that at least for the MP you set/attribute a silo for | 11:11 |
| didrocks | they should add the ppa | 11:11 |
| didrocks | and use that | 11:11 |
| didrocks | I think the impact isn't low on the CI machinery to do that | 11:11 |
| Saviq | didrocks, yup | 11:11 |
| didrocks | but that would align us more with the Airline vision | 11:12 |
| Saviq | didrocks, true | 11:12 |
| Saviq | indeed | 11:12 |
| didrocks | so yeah, we do agree :) | 11:12 |
| didrocks | I just wonder if I should depress fginther right away | 11: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 bluefinn | 11:13 | |
| Saviq | didrocks, ;) | 11:13 |
| Saviq | tsdgeos, mzanetti, could one of you do https://code.launchpad.net/~mhr3/unity-scopes-shell/preview-model/+merge/203706 ? | 11:15 |
| Saviq | /food for real | 11:16 |
| tsdgeos | i will | 11:16 |
| fginther | didrocks, ^ is what I wanted to talk about. today is not good for me, maybe tomorrow | 11:31 |
| === rachelliu_ is now known as rachelliu | ||
| tsdgeos | MacSlow: added the 4 questions with their answers | 11:38 |
| tsdgeos | to https://code.launchpad.net/~aacid/unity-notifications/qcompare/+merge/203362 | 11:38 |
| MacSlow | tsdgeos, thx... what do you think about "extending" unity8's new MR-process to unity-notifications? | 11:39 |
| MacSlow | Saviq, ^ | 11:40 |
| MacSlow | tsdgeos, overkill or not? | 11:40 |
| tsdgeos | it's ok, i guess the ui question will never apply | 11:40 |
| MacSlow | tsdgeos, true | 11:40 |
| tsdgeos | but i just answer no and that's it :D | 11:40 |
| === alan_g is now known as alan_g|afk | ||
| === alan_g|afk is now known as alan_g | ||
| didrocks | fginther: ah great! | 11:49 |
| tsdgeos | mhr3: is it me or testresultstest-ng is pretty unstable? | 12:05 |
| tsdgeos | every second run it hands | 12:05 |
| tsdgeos | hangs | 12:05 |
| mhr3 | tsdgeos, 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 works | 12:06 |
| tsdgeos | ok :/ | 12:06 |
| tsdgeos | mhr3: i always like doing while [ true ]; do make test; if [ $? -ne 0 ]; then break; fi; done to stress the tests | 12:10 |
| tsdgeos | but i can't in unity-scopes-shell because of that | 12:10 |
| mhr3 | tsdgeos, add pkill -f scoperegistry and scoperunner | 12:11 |
| mhr3 | then it will work | 12:11 |
| mhr3 | mostly the runner, registry should exit fine | 12:11 |
| tsdgeos | is that planned to work? | 12:12 |
| tsdgeos | i.e. it's a bug or per design? | 12:12 |
| mhr3 | bug | 12:12 |
| mhr3 | the registry "forgets" to kill the scope | 12:13 |
| mhr3 | for some reason | 12:13 |
| tsdgeos | oki | 12:13 |
| mhr3 | tsdgeos, while [ true ]; do make test; if [ $? -ne 0 ]; then break; fi; pkill -f scoperunner; done | 12:14 |
| === MacSlow is now known as MacSlow|lunch | ||
| karni | Saviq: 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/203444 | 12:17 |
| mhr3 | oh.. forgetting you still need the timeout branch for that | 12:17 |
| Saviq | karni, no, it just didn't happen yet - but it's almost there - waiting for things to get published in release pocket | 12:18 |
| Saviq | MacSlow|lunch, it will, but it's Thomas's turf | 12:18 |
| karni | Saviq: thanks :) | 12:18 |
| Saviq | karni, did review of your branches, too | 12:19 |
| karni | wohoo | 12:19 |
| * karni goes through them | 12:19 | |
| Saviq | karni, did your filtergrid branch actually fix the issue? I'm afraid I didn't understand well what was happening | 12:22 |
| karni | yes, allow me a moment to address yor comments :) | 12:23 |
| Saviq | karni, ok good | 12:23 |
| Saviq | karni, your fix carousel branch just got merged | 12:23 |
| karni | Saviq: 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 |
| karni | Saviq: \o/ | 12:23 |
| karni | Saviq: if that's the case, I understand now, and you're right. I'll need to fix that. | 12:23 |
| Saviq | karni, leave that be for now, please | 12:24 |
| Saviq | karni, I'm extracting those kind of decisions (category-wide) to a CardTool | 12:24 |
| karni | Saviq: oh ok | 12:24 |
| Saviq | karni, so it will fit in there | 12:24 |
| * karni nod | 12:24 | |
| mhr3 | sil2100, could we still have automerger for non-trunk branches? | 12:27 |
| tsdgeos | mhr3: i don't understand the hideDash signal, what's it about? | 12:28 |
| karni | Saviq: 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 fixes | 12:28 | |
| Saviq | karni, right, so a FilterProxyModel over a FilterProxyModel freaks out | 12:29 |
| Saviq | karni, file a bug indeed, please | 12:29 |
| karni | Saviq: do you know what's the toolkit project called? or shall I file against unity8? | 12:30 |
| mhr3 | tsdgeos, 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 |
| Saviq | karni, that's internal in unity8 | 12:30 |
| karni | ok | 12:30 |
| Saviq | didrocks, sil2100, Mirv, can one of you guys please push unity-notifications through landing pipeline (old)? | 12:30 |
| tsdgeos | mhr3: hmmmm ok | 12:31 |
| mhr3 | tsdgeos, shouldn't really be necessary in mir land though | 12:31 |
| MacSlow|lunch | Saviq, ok makes sense | 12:31 |
| tsdgeos | mhr3: wouldn't doing | 12:40 |
| tsdgeos | + Q_EMIT previewReady(m_preview); | 12:40 |
| tsdgeos | + m_preview.clear(); | 12:40 |
| tsdgeos | get the receiver a dangling pointer? | 12:40 |
| mhr3 | tsdgeos, the receiver owns the preview, not the scope | 12:41 |
| mhr3 | tsdgeos, or well.. the one that invokes preview() owns it | 12:42 |
| tsdgeos | mhr3: ok, got it | 12:42 |
| karni | Saviq: Tried my best to explain this weird behavior. https://bugs.launchpad.net/unity8/+bug/1274095 | 12:43 |
| ubot5 | Ubuntu bug 1274095 in Unity 8 "FilterGrid backed by FilterProxyModel renders wrong" [Undecided,New] | 12:43 |
| sil2100 | Saviq: we'll try! | 12:48 |
| Saviq | sil2100, thanks :) | 12:48 |
| sil2100 | mhr3: I think we could! | 12:49 |
| Saviq | karni, yup, got it | 12:50 |
| karni | :) | 12:50 |
| mhr3 | tsdgeos, 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 it | 12:52 |
| tsdgeos | mhr3: i think what you have works | 12:53 |
| tsdgeos | lunch! | 12:53 |
| mhr3 | tsdgeos, well if the invoker didn't delete it, it would leak | 12:53 |
| tsdgeos | mhr3: maybe document in the .h that review() passes the ownership on | 12:53 |
| tsdgeos | it's typical for this kind of "it's your duty to delete stuff thing" i'd say | 12:53 |
| mhr3 | tsdgeos, i have a feeling that qml will ignore me no matter what i say in a comment | 12:54 |
| Saviq | dednick, conflicts in hint-interval | 13:01 |
| === alan_g is now known as alan_g|lunch | ||
| Cimi | Saviq, can you repeat me what you told me yesterday about the models and model data? I think I'm confusing things here | 13:04 |
| Saviq | Cimi, the model for the audio widget will be an array of dictionaries | 13:05 |
| Saviq | Cimi, so [{"source": "foo", "title": "bar", "subtitle": "baz"}, { ... }] etc. | 13:05 |
| Cimi | Saviq, I did this | 13:05 |
| Cimi | Saviq, but how do I use this inside the repeater? | 13:05 |
| Saviq | Cimi, to access those, you need to use modelData["source"], modelData["title"] etc. | 13:05 |
| Saviq | Cimi, model: [...] works in a Repeater | 13:06 |
| Saviq | Cimi, only then, in delegates, you don't have model.source, but modelData["source"] instead | 13:06 |
| Cimi | oh great | 13:06 |
| Cimi | thought that didn't work | 13:06 |
| Cimi | I'll continue then | 13:06 |
| Cimi | Saviq, wrote this for the test file http://paste.ubuntu.com/6837718/ | 13:07 |
| Cimi | Saviq, need to put title, length, uri with " " | 13:07 |
| Cimi | ? | 13:07 |
| Saviq | Cimi, no, that's fine, Repeater { model: data.musicData; delegate: Label { text: modelData["title"] } } } will work now | 13:08 |
| Saviq | Cimi, no need to wrap in the QtObject, though | 13:09 |
| Cimi | ok | 13:09 |
| Cimi | I'll see if that works | 13:09 |
| Cimi | thank you | 13:09 |
| karni | Saviq: "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 |
| karni | Saviq: so it's one (broken) line | 13:13 |
| karni | Saviq: the readonly prop collapsedHeight: { return <formula> } -> readonly prop collapsedHeight: /n <formula> | 13:14 |
| Saviq | karni, yeah, and our line lenght limit is 120 anyway | 13:14 |
| karni | oh | 13:14 |
| karni | I should have known better. | 13:14 |
| karni | :) | 13:14 |
| Saviq | karni, nw | 13:14 |
| Saviq | karni, but yeah, you shouldn't use { return ... } unless you need to wrap, or have more complicated bindings there | 13:14 |
| karni | Saviq: nw? :D | 13:14 |
| * karni nods | 13:14 | |
| Saviq | karni, no worries | 13:18 |
| karni | that one I didn't know haha | 13:18 |
| karni | Saviq: all comments addressed :) https://code.launchpad.net/~unity-team/unity8/unity8-fix-filtergrid/+merge/203657 | 13:27 |
| dednick | Saviq: Yup, seen it | 13:28 |
| Saviq | karni, dednick, k | 13:28 |
| Saviq | karni, flatten 16+ readonly property int rowsWhenCollapsed: { | 13:33 |
| Saviq | 17+ return Math.min(collapsedRowCount, Math.ceil(model.count / columns)) | 13:33 |
| Saviq | 18+ } too, please | 13:33 |
| karni | missed that | 13:34 |
| karni | Saviq: collapsed, compacted and pushed | 13:35 |
| karni | Saviq: fixed one teset case tag, sorry. | 13:37 |
| karni | *test case tag | 13:37 |
| sil2100 | Saviq: unity-notifications is building, we'll try releasing it later on | 13:37 |
| Saviq | sil2100, thanks | 13:37 |
| Saviq | karni, approved | 13:45 |
| karni | Saviq: Thanks! | 13:45 |
| karni | Saviq: 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/203597 | 13:45 |
| karni | * +1 | 13:45 |
| Saviq | karni, yup, lemme merge trunk into new-scopes first | 13:45 |
| karni | Saviq: 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 |
| Saviq | karni, don't be | 13:47 |
| Saviq | karni, conflicting, merge please | 13:47 |
| * karni does that | 13:47 | |
| Saviq | karni, and that's why I didn't want to push before ;) | 13:48 |
| karni | ^-^ | 13:48 |
| karni | Saviq: 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 |
| karni | unless we'll want that change in trunk already. for now, getRenderer is different in trunk and new-scopes | 13:51 |
| karni | the latter uses the template, and has less arguments. | 13:51 |
| Saviq | karni, no it won't conflict | 13:52 |
| Saviq | karni, because I just merged trunk into new-scopes | 13:52 |
| Saviq | karni, so your change will be correct on top of new-scopes | 13:52 |
| Saviq | erm | 13:52 |
| Saviq | on top of trunk | 13:52 |
| === alan_g|lunch is now known as alan_g | ||
| karni | Saviq: 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 shows | 13:57 | |
| karni | heh, ignore me ;P | 13:58 |
| karni | Saviq: fixed. https://code.launchpad.net/~unity-team/unity8/new-scopes-fix-carousel5/+merge/203597 | 14:00 |
| Saviq | karni, cheers | 14:01 |
| karni | cheers! | 14:01 |
| elopio | good morning. | 14:11 |
| elopio | can I get a review here please? https://code.launchpad.net/~elopio/unity8/generic_preview/+merge/202793 | 14:11 |
| tsdgeos | elopio: sure, will do | 14:16 |
| elopio | tsdgeos: thank you. | 14:18 |
| tsdgeos | kgunn: Saviq: hangout or mumble today? | 14:20 |
| kgunn | tsdgeos: let's hangout | 14:21 |
| tsdgeos | okidokig | 14:22 |
| * tsdgeos fixes his hair | 14:22 | |
| tsdgeos | :D | 14:22 |
| kgunn | tsdgeos: you too huh ? | 14:23 |
| * dandrader looks for a tshirt to put on | 14:23 | |
| * dednick looks for his pants | 14:23 | |
| dednick | oh wait, dont need those | 14:23 |
| karni | Saviq: 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 |
| Saviq | karni, just switch the scope like you would on the phone | 14:25 |
| Saviq | karni, either via top bar or swipe left/right | 14:25 |
| karni | perfect | 14:25 |
| karni | If someone could PM me the hangout link, that'd be neat :) | 14:26 |
| karni | or, just add it to calendar event | 14:27 |
| mhr3 | they do mumble | 14:27 |
| karni | not today, I hear | 14:27 |
| mhr3 | oh wow | 14:27 |
| Saviq | mhr3, yeah, it's a full-blown bi-weakly hangout today ;) | 14:27 |
| karni | mhr3: I thought you were on this team, how confusing ;D | 14:28 |
| Saviq | karni, you got invite | 14:28 |
| mhr3 | karni, anyway, --scope-dir takes a dir which has subdir and inside it needs to have subdir.ini and libsubdir.so | 14:28 |
| mhr3 | subdir == scope_name usually | 14:28 |
| karni | Thanks :) | 14:29 |
| karni | mhr3: ah | 14:29 |
| mhr3 | so "--scope-dir /foo" if you have /foo/bar + /foo/bar/bar.ini + /foo/bar/libbar.so | 14:29 |
| greyback_ | dammit google | 14:30 |
| karni | mhr3: helpful! | 14:30 |
| tsdgeos | yeah i need to swtich g+ identities too | 14:30 |
| tsdgeos | nope | 14:31 |
| karni | hrm. unity-scope-tool working for folks? http://paste.ubuntu.com/6838096/ | 14:31 |
| Saviq | karni, restart scope-registry should help | 14:32 |
| * karni tries | 14:32 | |
| Saviq | kgunn, you joining? | 14:32 |
| karni | Saviq: ha :> thanks | 14:33 |
| Saviq | kgunn, lost you on hangout... | 14:34 |
| elopio | Saviq: is the standup on hangout today? | 14:36 |
| Saviq | elopio, yes, sorry, let me paste you the link | 14:36 |
| elopio | :) | 14:36 |
| Saviq | greyback_, not happening for you? | 14:42 |
| greyback_ | Saviq: no I only stay connected for 15-20 seconds then loose connection | 14:42 |
| Saviq | greyback_, nice... | 14:43 |
| greyback_ | Saviq: no good, I can't even connect now | 14:45 |
| Saviq | greyback_, nw, just put your stuff in the doc | 14:45 |
| greyback_ | Saviq: ack | 14:45 |
| karni | Saviq: 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 |
| karni | re: my question, just heard that trunk would only be a bit behind demo-stuff builds | 14:49 |
| sil2100 | Saviq: hi! About unity-notifications, could you use CITrain for landing this maybe? Would be much of a problem? | 14:52 |
| Saviq | thostr_, ↑ | 14:53 |
| thostr_ | sil2100: I'm having a big timing problem tbh. So, if you could get landed I'd be more than happy | 14:54 |
| Cimi | mzanetti, http://bazaar.launchpad.net/~unity-team/unity8/trunk/revision/558 | 15:01 |
| elopio | Saviq: are you landing branches into a staging branch before landing them to trunk? | 15:01 |
| === dandrader is now known as dandrader|afk | ||
| Cimi | mzanetti, did you forget to add the ogg test files? | 15:01 |
| Saviq | elopio, no, that was a plan we had, but decided against it | 15:01 |
| mzanetti | Cimi: need to check, one sec | 15:02 |
| Saviq | Cimi, I don't think we wanted test files like that - the Audio { } component is overridden for the tests | 15:02 |
| elopio | Saviq: 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 |
| Saviq | elopio, 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 advantages | 15:03 |
| MacSlow | kgunn, Saviq: so first action (for the in-call notification of 2nd call) is to get latest feedback from the sdk-folks | 15:03 |
| Saviq | elopio, other than having a branch into which we merge before actually merging to trunk | 15:03 |
| Saviq | macSlow, there's two of you... | 15:04 |
| MacSlow | kgunn, Saviq: yeah... I've not logged out of the other machine | 15:04 |
| Saviq | MacSlow, truth is first of all we need to get design to stabilize | 15:04 |
| kgunn | MacSlow: Saviq ...2 things really, design stable & sdk support.... | 15:04 |
| Saviq | MacSlow, because they're changing all the buttons and their layouts to save space | 15:04 |
| karni | mhr3: 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 |
| Saviq | kgunn, yeah | 15:04 |
| elopio | Saviq: so you land them on trunk one by one, running all the tests and making exploratory on each merge? | 15:05 |
| MacSlow | Saviq, sure... I'll continue with the visual snap-dec. queue for the moment... until we have the new design and the sdk-component | 15:05 |
| Cimi | Saviq, so why having the uri specified then? :) | 15:05 |
| Saviq | elopio, I'm not doing it, CI Train is | 15:05 |
| Saviq | elopio, CI Train is building a PPA with all of the branchs | 15:06 |
| Cimi | anyway fine, I'll do without | 15:06 |
| Saviq | elopio, I add that PPA and run the test plan on that | 15:06 |
| elopio | Saviq: ah, ok, after all of it is merged, only once. | 15:06 |
| karni | elopio: there's a temporary ppa build before stuff gets merged | 15:06 |
| karni | yes | 15:06 |
| Saviq | elopio, yeah, it's all merged together and put in the PPA | 15:06 |
| elopio | I get that. | 15:06 |
| Saviq | elopio, so yeah, staging branch was just more work with no particular advantage | 15:07 |
| Saviq | elopio, we also reduced the per-MP checklist a lot | 15:08 |
| Saviq | elopio, https://wiki.ubuntu.com/Process/Merges/Checklists/Unity8#preview | 15:08 |
| Saviq | elopio, while the landing TestPlan is more extensive https://wiki.ubuntu.com/Process/Merges/TestPlans/Unity8 - but that's what you do with the PPA | 15:08 |
| elopio | Saviq: ok. Sounds reasonable. I'm still missing to have trunk and a stable branch, instead of trunk being the stable one. | 15:09 |
| elopio | but I suppose we can work with this. | 15:09 |
| elopio | I'll tell the autopilot guys about your process for them to copy the nice things. | 15:09 |
| Saviq | elopio, we've been doing stable trunk since the beginning, really | 15:10 |
| Saviq | elopio, or at least have been trying to | 15:10 |
| Saviq | elopio, 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 machinery | 15:11 |
| elopio | right. 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 |
| Saviq | not a lot will change conceptually, but the underlying code is changing a lot | 15:12 |
| elopio | Saviq: 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 |
| Saviq | elopio, 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 desirable | 15:13 |
| Saviq | elopio, [...] well, that's what happens with CI Train, only there isn't a branch, but a set of MPs that get manually certified | 15: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 | |
| elopio | right, 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 |
| Saviq | mhr3, see karni's comment ↑↑ | 15:15 |
| tsdgeos | can i convince someone to do https://code.launchpad.net/~aacid/unity8/delegateRangeBugfix/+merge/203498 ? | 15:15 |
| tsdgeos | Saviq: you? | 15:15 |
| karni | mhr3: artShape's are present in proper aspect-ratio, artImage within are completely gone (not shown) | 15:15 |
| Saviq | tsdgeos, findChild(genericScopeView, "dashCategory0") != undefined; | 15:16 |
| Saviq | tsdgeos, it's null now | 15:16 |
| Saviq | tsdgeos, but anyway | 15:16 |
| Saviq | tsdgeos, better to do typeof == "object" instead | 15:16 |
| tsdgeos | Saviq: != still works ;) | 15:16 |
| Saviq | tsdgeos, yeah, because it will never == ;) | 15:16 |
| Saviq | tsdgeos, which means you won't actually wait for the category, will you ;) | 15:17 |
| tsdgeos | Saviq: not really, since it's != and not !== it does that javascript magic | 15:17 |
| Saviq | tsdgeos, right, but still | 15:17 |
| tsdgeos | i'd say it does wait, i don't think the test would pass otherwise | 15:17 |
| tsdgeos | Saviq: want me to change the whole file? it's full of those that you forgot to update when you changed the return value | 15:18 |
| === dandrader|afk is now known as dandrader | ||
| mhr3 | karni, it's only about checking a settings/dconf value, we just didn't hook it up cause the default for that is not yet known | 15:19 |
| mhr3 | karni, i could give you an envvar where you'd specify comma-separated scope names | 15:20 |
| * karni consults | 15:20 | |
| Saviq | tsdgeos, ok, leave it be now, and we'll fix it whole later | 15:20 |
| tsdgeos | Saviq: i'll propose a branch on top of it with the fixes | 15:20 |
| elopio | hey 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 |
| Saviq | tsdgeos, ok | 15:21 |
| Saviq | elopio, who uses daily-build any more ;) | 15:21 |
| elopio | you can downgrade to the one in trusty, and things will work. | 15:21 |
| elopio | me and two more people who have just said that things broke everywhere :) | 15:22 |
| elopio | Saviq: shouldn't I be using it? | 15:22 |
| karni | mhr3: yes, we would very much like to know that envvar | 15:22 |
| tsdgeos | Saviq: 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 |
| Saviq | elopio, I don't think so, CI Train omits daily-build | 15:22 |
| karni | mhr3: so that we can get rid of unwanted subscopes for demoing at MWC (until we have a better solution) | 15:22 |
| elopio | ok, I'll remove it. | 15:23 |
| mhr3 | tsdgeos, feel free to approve, i'm confident in the tests :) | 15:23 |
| Saviq | tsdgeos, checked the coverage? ;) | 15:23 |
| mhr3 | karni, will make a branch, but lunch first :) | 15:23 |
| Saviq | mhr3, want me to check the coverage? :D | 15:23 |
| Saviq | elopio, FWIW it was never "safe" as in stuff from there might never have ended up in distro | 15:24 |
| karni | mhr3: thank you | 15:24 |
| mhr3 | Saviq, of course i do! | 15:24 |
| tsdgeos | Saviq: only visually, didn't run the coverage tool tbh | 15:24 |
| tsdgeos | visually == it has a test of the new stuff | 15:24 |
| Saviq | tsdgeos, ah so your delegateRangeBugFix fixes the "only 9 apps visible" issue? | 15:26 |
| tsdgeos | hmmmm | 15:26 |
| tsdgeos | maybe? | 15:26 |
| tsdgeos | i never understood that bug | 15:26 |
| karni | mhr3: 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 |
| tsdgeos | but it can cause that only some apps are displayed yes | 15:26 |
| karni | mhr3: Enjoy your lunch now, though :) | 15:26 |
| mhr3 | karni, yea, looking at it now | 15:27 |
| tsdgeos | Saviq: i.e. people was teorizing about how you needed to disconnect from wifi or something | 15:27 |
| tsdgeos | but maybe they did just not know how to break it properly | 15:27 |
| karni | mhr3: 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 |
| Saviq | tsdgeos, yeah | 15:27 |
| mhr3 | karni, i don't want dconf yet cause the default would be wrong | 15:28 |
| karni | mhr3: ack, let's go envvar then :) | 15:28 |
| Saviq | mhr3, just three holes in collectors.cpp, I'd say I'm happy ;) | 15:31 |
| === alan_g is now known as alan_g|tea | ||
| mhr3 | Saviq, btw QDesktopService::openUrl crashes when you don't have QGuiApplication :P | 15:31 |
| Saviq | mhr3, on xcb that is? | 15:32 |
| mhr3 | yea | 15:32 |
| Saviq | sounds like a bug | 15:32 |
| Saviq | tsdgeos, 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 today | 15:34 |
| Cimi | Saviq, you see the power of @uiteam ^^ | 15:34 |
| dandrader | Saviq, no | 15:34 |
| * greyback_ listens for all: | 15:34 | |
| mterry | Saviq, https://code.launchpad.net/~unity-team/unity8/unity8.test_nested_mir/+merge/203088 | 15:34 |
| MacSlow | Saviq, nope | 15:34 |
| greyback_ | nope | 15:34 |
| Saviq | mterry, it's approved, so will be part of it, yeah | 15:34 |
| mterry | Saviq, ah OK | 15:34 |
| karni | Saviq: checking | 15:34 |
| kgunn | i like @uiteam...."all" way too many false+ | 15:34 |
| Saviq | Cimi, fill in checklist ↑↑ please | 15:34 |
| * mzanetti listens for "all:" too | 15:35 | |
| mterry | Saviq, unreviewed, but I'd like to see https://code.launchpad.net/~mterry/unity8/hide-greeter-on-focus-request/+merge/201817 land | 15:35 |
| Saviq | kgunn, Cimi @unity? | 15:35 |
| Saviq | mterry, Cimi, please fill in checklist for test_nested_mir | 15:35 |
| kgunn | Saviq: +1 adding now | 15:35 |
| mterry | oh right the checklist. | 15:35 |
| Saviq | mterry, Cimi, as per https://wiki.ubuntu.com/Process/Merges/Checklists/Unity8#preview | 15:35 |
| karni | Saviq: new-scopes-fix-carousel5 and unity8-fix-filtergrid I care most about | 15:35 |
| mzanetti | karni: note the ":" at "all:" | 15:35 |
| mzanetti | kgunn: ^^ | 15:35 |
| mzanetti | sorry karni | 15:35 |
| karni | mzanetti: np | 15:35 |
| Saviq | karni, former is merged already, latter is in queue | 15:35 |
| kgunn | added both | 15:36 |
| Saviq | tsdgeos, Cimi, dandrader, elopio, kgunn, greyback_, mterry, mzanetti, karni, MacSlow, dednick, please add "@unity" to IRC highlight words | 15:36 |
| Saviq | @unity | 15:36 |
| tsdgeos | Saviq: with the @ ? | 15:36 |
| Saviq | tsdgeos, ye | 15:36 |
| Cimi | someone to test pls | 15:37 |
| Saviq | tsdgeos, so that you don't get highlighted for unity, but @unity | 15:37 |
| dandrader | Saviq, I'm agains it | 15:37 |
| Cimi | works fine | 15:37 |
| Cimi | thx | 15:37 |
| elopio | roger | 15:37 |
| dandrader | Saviq, it will make it easier to polk people | 15:37 |
| tsdgeos | @unity | 15:37 |
| tsdgeos | didn't work | 15:37 |
| Saviq | dandrader, if it gets abused, we won't hold you accountable | 15:37 |
| mzanetti | works here | 15:37 |
| dandrader | Saviq, there's a value in having to autocomplete every single person | 15:37 |
| Saviq | tsdgeos, @unity | 15:37 |
| MacSlow | tsdgeos, worked | 15:37 |
| tsdgeos | Saviq: stop saying my name, that breaks the test :D | 15:37 |
| Saviq | @unity | 15:38 |
| Cimi | ahah | 15:38 |
| Saviq | lol | 15:38 |
| MacSlow | @unity | 15:38 |
| tsdgeos | Saviq: ok, worked now | 15:38 |
| tsdgeos | guess doesn't highlight myself | 15:38 |
| mzanetti | dudes! I've got work to do :P | 15:38 |
| Saviq | dandrader, let's see how it works | 15:38 |
| Cimi | tsdgeos, even @cimirocks works for you I imagine :P | 15:38 |
| Saviq | kgunn had enough | 15:38 |
| dandrader | @unity makes poking more impersonal | 15:38 |
| * mzanetti adds @cimirocks | 15:38 | |
| Saviq | dandrader, yeah, but it's only meant for that | 15:38 |
| Cimi | ahaha | 15:38 |
| === alan_g|tea is now known as alan_g | ||
| Saviq | dandrader, to poke the whole team | 15:39 |
| kgunn | one more time please | 15:39 |
| MacSlow | @unity | 15:39 |
| Saviq | @unity | 15:39 |
| Saviq | dandrader, if wanted to poke any one in particular, you would type their name | 15:39 |
| kgunn | hmmm...manager probs | 15:39 |
| tsdgeos | dednick: there? | 15:39 |
| mzanetti | @unity: shouldn't it be @unity8 ? | 15:39 |
| mzanetti | :P | 15:39 |
| MacSlow | mzanetti, now you're being german :) | 15:40 |
| Cimi | ahaha | 15:40 |
| mzanetti | :D | 15:40 |
| * karni adds hilight | 15:40 | |
| Saviq | mzanetti, we'll be unity soon... ha ha ha.... HA Ha ha... MUAHAHAHAHAHA | 15:40 |
| karni | *highlight | 15:40 |
| dandrader | @unity | 15:40 |
| mzanetti | Saviq: define "soon" | 15:40 |
| Saviq | mzanetti, whenever you implement window management for destkop | 15:40 |
| mterry | hmm, no highlight for me | 15:40 |
| Saviq | desktop, even | 15:40 |
| dandrader | could someone say "@unity". testing my setting :) | 15:40 |
| Cimi | @unity | 15:40 |
| MacSlow | @unity | 15:40 |
| mterry | Saviq, OK, checklist added to both branches I mentioned | 15:41 |
| Saviq | @unity | 15:41 |
| MacSlow | @cimirocks | 15:41 |
| dandrader | didn't work | 15:41 |
| Saviq | mterry, cheers | 15:41 |
| mzanetti | dandrader: really? you missed the 300 times it happened already= | 15:41 |
| karni | MacSlow: hahah | 15:41 |
| Cimi | MacSlow, got that :P | 15:41 |
| * elopio leaves the channel :) | 15:41 | |
| Saviq | ;) | 15:41 |
| Cimi | aha | 15:41 |
| kgunn | no workie for me either | 15:41 |
| * MacSlow ROTFLBTC | 15:41 | |
| MacSlow | kgunn, which IRC-client? | 15:41 |
| kgunn | MacSlow: Xchat | 15:41 |
| MacSlow | kgunn, hm... the moment I added @ unity (without the space) it worked | 15:42 |
| tsdgeos | ouch | 15:42 |
| dandrader | maybe restarting xchat does the trick | 15:42 |
| tsdgeos | i broke an autopilot test ^_^ | 15:42 |
| MacSlow | dandrader, didn't have to | 15:42 |
| tsdgeos | and it got merged | 15:42 |
| karni | for irssi folks: /hilight -line -channels #ubuntu-unity -regexp @all | 15:42 |
| mzanetti | @unity (for dandrader) | 15:42 |
| karni | d'uh | 15:42 |
| tsdgeos | the joy of having broken autopilot | 15:42 |
| karni | not @all, heh | 15:42 |
| dandrader | mzanetti, lol | 15:42 |
| mzanetti | damn | 15:43 |
| kgunn | MacSlow: that's what i added...in both exact words & nicknames to always highlight | 15:43 |
| * mzanetti -> stupid | 15:43 | |
| kgunn | and restarted | 15:43 |
| mzanetti | @unity (for daniel) | 15:43 |
| Saviq | ;D | 15:43 |
| * tsdgeos works on a fix fix | 15:43 | |
| dandrader | mzanetti, didn't work.... anyone using xchat here? | 15:43 |
| MacSlow | @unity (for kgunn) | 15:43 |
| kgunn | aha | 15:43 |
| karni | works here :) | 15:43 |
| mzanetti | MacSlow: stupid too :D | 15:44 |
| Cimi | hah | 15:44 |
| Cimi | MacSlow, don't add his name | 15:44 |
| MacSlow | kgunn, btw I'm using xchat-gnome... not plain xchat | 15:44 |
| kgunn | MacSlow: again...only minus kgunn :0 | 15:44 |
| MacSlow | mzanetti, Cimi: no... I like that "joke" :) | 15:44 |
| MacSlow | @unity | 15:44 |
| mzanetti | oh... so its a running gag already | 15:44 |
| kgunn | nope | 15:44 |
| MacSlow | mzanetti, of course | 15:45 |
| mzanetti | :D | 15:45 |
| MacSlow | kgunn, switch to xchat-gnome then ;) | 15:45 |
| mzanetti | or something with Q or K in the beginning :P | 15:45 |
| mterry | kgunn, you are using plain xchat too? Yeah, can't make it work for me | 15:45 |
| MacSlow | mzanetti, pfft :) | 15:45 |
| * MacSlow recommends xchat-gnome | 15:45 | |
| MacSlow | *cough* | 15:45 |
| mterry | MacSlow, we need xchat-unity ;) | 15:46 |
| MacSlow | mterry, give me two weekends ;) | 15:46 |
| elopio | quassel is the way. | 15:46 |
| mterry | dandrader, looks like we need to be using xchat-gnome | 15:46 |
| * MacSlow misses the times when it was vi vs. emacs | 15:46 | |
| dandrader | maybe I have to escape the "@" sign somehow in the setting.... | 15:46 |
| Saviq | didrocks, can we get a silo please? | 15:47 |
| dandrader | MacSlow, yeah, vim won :) | 15:47 |
| MacSlow | dandrader, yeah... thank god :) | 15:47 |
| karni | Saviq: changes will land in demo-stuff ppa first, then trunk (and image) next? | 15:48 |
| Saviq | karni, all goes directly into release | 15:48 |
| karni | ack | 15:48 |
| Saviq | karni, so demo-stuff loses its purpose soon | 15:48 |
| Saviq | karni, as trunk == release with CI Train | 15:48 |
| Saviq | or trunk <= release for a short moment | 15:48 |
| Saviq | karni, well, demo-stuff builds from new-scopes | 15:48 |
| * karni nods | 15:48 | |
| Saviq | karni, so it should be updated already | 15:49 |
| karni | wohoo | 15:49 |
| Saviq | karni, unity8 50 minutes ago | 15:49 |
| Saviq | Successfully built | 15:49 |
| Cimi | kgunn, did you manager to get it set? | 15:50 |
| Cimi | manage | 15:50 |
| kgunn | Cimi: nope..was googling | 15:51 |
| === mhall119_ is now known as mhall119 | ||
| Saviq | mhr3, 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 |
| mhr3 | Saviq, now that trunk isn't trunk... not much | 15:52 |
| Saviq | mhr3, so you're CI-Training them, too? | 15:52 |
| mhr3 | unfortunately :P | 15:53 |
| karni | Saviq: 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/203444 | 15:53 |
| karni | it's already merged | 15:53 |
| didrocks | Saviq: https://code.launchpad.net/~unity-team/unity8/unity8-fix-filtergrid/+merge/20365 doesn't exist | 15:53 |
| Cimi | kgunn, ?unity "might" work | 15:53 |
| karni | I mean we landed it without updating spreadsheet? | 15:53 |
| didrocks | probably a typo with the number | 15:53 |
| Saviq | karni, Status: | 15:54 |
| Saviq | Merged Edit status | 15:54 |
| Saviq | karni, it landed yesterday | 15:54 |
| karni | ok, that's all I care :) thanks | 15:54 |
| Saviq | karni, or well, today, but started yesterday | 15:54 |
| Cimi | kgunn, I read it should support wildcards, so ? should replace the char @ that does not get highlighted | 15:54 |
| Saviq | karni, row 9 in https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AuDk72Lpx8U5dFlCc1VzeVZzWmdBZS11WERjdVc3dmc&usp=drive_web#gid=0 | 15:54 |
| karni | ack | 15:55 |
| Saviq | or sil2100, could we get a silo for unity8 please? | 15:55 |
| Cimi | kgunn, otherwise take a screenshot of your settings page and share with us :) | 15:56 |
| sil2100 | Saviq: assigining! | 16:00 |
| Saviq | sil2100, thanks! | 16:00 |
| Saviq | sil2100, 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 |
| sil2100 | Saviq: I think we don't have that set-up for now, but it's a valid thing to think about | 16:02 |
| Saviq | sil2100, in lieu of that, apart you and Didier, who else has the power to do so? | 16:03 |
| === mterry_ is now known as mterry | ||
| mterry | Someone please try at-unity again | 16:05 |
| Cimi | @unity | 16:05 |
| mterry | Perfect! Thanks, Cimi | 16:06 |
| * mzanetti slaps Cimi | 16:06 | |
| sil2100 | Saviq: 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 too | 16:06 |
| mterry | kgunn, dandrader: switching to xchat-gnome was painless. It kept all my settings and even my new highlight setting from xchat | 16:06 |
| karni | @cimirocks | 16:06 |
| mzanetti | next one who uses this has to take notes in the standup for a week | 16:06 |
| karni | mzanetti: :DDD | 16:06 |
| mterry | mzanetti, I asked! | 16:06 |
| kgunn | mterry: ack | 16:06 |
| Saviq | sil2100, can I drop in one more MP? | 16:06 |
| karni | (uses without reason) | 16:06 |
| mzanetti | right... /me unslaps Cimi :D | 16:07 |
| sil2100 | Saviq: btw.! Make sure you type in the whole MPs in it! Had to correct one since it was missing some numbers! | 16:07 |
| Saviq | sil2100, oups | 16:07 |
| sil2100 | Saviq: yes, please add what you need, didn't assign it yet ;) | 16:07 |
| Saviq | sil2100, done | 16:07 |
| sil2100 | Saviq: ok, assigning | 16:08 |
| Saviq | sil2100, thanks! | 16:09 |
| Cimi | I'm downloading xchat source code :) | 16:09 |
| MacSlow | Cimi, :) | 16:09 |
| Cimi | indeed at-unity doesn't work here | 16:09 |
| sil2100 | Saviq: done! yw | 16:10 |
| dandrader | mterry, hmmm, good to know. I'm just not a big fan of its GUI. but will give it another try | 16:11 |
| Cimi | MacSlow, http://sourceforge.net/p/xchat/svn/HEAD/tree/src/common/inbound.c | 16:12 |
| dandrader | Cimi, found the bug? | 16:12 |
| Saviq | seb128, could we ask you for a packaging review on https://code.launchpad.net/~nick-dedekind/unity8/indicator.ubuntu-settings-components/+merge/199311 ? | 16:12 |
| Cimi | dandrader, I'm reading code | 16:13 |
| seb128 | Saviq, looking | 16:13 |
| Saviq | seb128, thanks | 16:14 |
| seb128 | Saviq, done, +1 from me | 16:16 |
| Saviq | seb128, thanks | 16:16 |
| seb128 | yw ;-) | 16:16 |
| Cimi | dandrader, the matching code is different between xchat-gnome and xchar | 16:17 |
| Cimi | dandrader, http://sourceforge.net/p/xchat/svn/HEAD/tree/src/common/inbound.c#l237 | 16:18 |
| Cimi | this is the broken function I believe | 16:18 |
| dandrader | Cimi, from the looks of it, seems a word has to have only alpha chars | 16:22 |
| Cimi | dandrader, indeed | 16:22 |
| dandrader | so the "@" is not considered part of a word | 16:22 |
| Cimi | dandrader, I was just looking at that | 16:23 |
| Cimi | dandrader, so, won't work | 16:23 |
| Cimi | unless you patch xchat | 16:23 |
| dandrader | Cimi, or we change the calling phrase :) | 16:24 |
| dandrader | s/phrase/word | 16:24 |
| Saviq | lol | 16:25 |
| karni | Saviq: FTR, comment on the doc from Katie "I removed art-mask height = header height for horizontal cards." | 16:26 |
| dandrader | Saviq, greyback_ (or whoever is more familiar with the side stage hacks): Does the QScreen have to have the side stage geometry | 16:27 |
| dandrader | ? | 16:27 |
| Cimi | Saviq, unless you want to fix xchat, let's change to unityui :) | 16:27 |
| dandrader | Could QScreen:geometry have the real deal and QScreen::availableGeometry the side stage area? | 16:27 |
| Cimi | or cimirocks | 16:28 |
| Cimi | cimirocks works | 16: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 dimensions | 16:29 |
| dandrader | greyback_, is there any benefit hiding it? the important thing is that the app window has a certain size and that it cannot grow | 16:31 |
| dandrader | by "hiding" I mean giving out fake info in QScreen | 16: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 everywhere | 16:32 |
| Saviq | greyback_, dandrader, I think it depends, what benefit does an app have to know the full screen geometry? | 16:34 |
| dandrader | Saviq, it can tell the virtual keyboard to be in landscape even though the app itself is in portrait :-D | 16:35 |
| Saviq | dandrader, 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 |
| dandrader | Saviq, 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 |
| dandrader | Saviq, it doesn't mean that its window is allowed to cover the entire screen, if that's the fear | 16:38 |
| Saviq | dandrader, 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 |
| dandrader | Saviq, it feels like you want to remove QScreen from the API when you deny it from giving out info | 16:40 |
| dandrader | giving out the info it's supposed to give | 16:41 |
| dandrader | it's like crippling it | 16:41 |
| dandrader | so, why are we crippling the API? | 16:42 |
| dandrader | </dramatic-effect> | 16:42 |
| Saviq | dandrader, I'm not saying we are | 16:43 |
| Saviq | dandrader, I just wonder what it's there for for 99% of apps (other than shells, really) | 16:43 |
| Saviq | dandrader, but no, I don't want to say don't expose it | 16:44 |
| Saviq | dandrader, I don't see it as a security concern or anything | 16:44 |
| Saviq | dandrader, 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 get | 16:45 |
| Saviq | and well, fullscreen in side stage will probably be prevented altogether... another good question | 16:45 |
| tsdgeos | mzanetti: i guess this is for you too https://code.launchpad.net/~aacid/unity8/bug1271676/+merge/203530 | 16:54 |
| tsdgeos | mzanetti: it's a bit hacky since i need to make sure it goes though that special cased if in GenericScopeView | 16:54 |
| Cimi | Saviq, https://code.launchpad.net/~unity-team/unity8/unity8.previews_audioPlayer/+merge/203387 | 17:13 |
| Cimi | Saviq, added tests, opinion? is the right way? | 17:14 |
| Saviq | Cimi, looks good, yeah, s/uri/source/ please, and length, I think, was in seconds, no? | 17:17 |
| Saviq | Cimi, could add a button or a selector for tryAudioPlayer, though | 17:17 |
| Saviq | Cimi, and drop the bottom ThinDivider | 17:17 |
| Saviq | Cimi, putting in review comment | 17:17 |
| Cimi | Saviq, it was string in musicpreview | 17:18 |
| Cimi | length | 17:18 |
| Cimi | I can do int | 17:18 |
| Saviq | Cimi, no if it was string then let it be it | 17:18 |
| Saviq | mhr3, length in Audio widget - string or int (seconds?) | 17:18 |
| sil2100 | Saviq, 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 it | 17:19 |
| Cimi | Saviq, tryAudioPlayer? | 17:19 |
| mhr3 | Saviq, int seconds | 17:19 |
| Saviq | Cimi, ↑ | 17:19 |
| Saviq | Cimi, when you go make tryAudioPlayer, you just get a blank page | 17:19 |
| Saviq | Cimi, we try to add controls so that you can actually interact with the component to verify it works | 17:20 |
| Saviq | manually | 17:20 |
| Cimi | Saviq, weird it's an int,... in music preview I see model.length.length > 0 | 17:20 |
| Saviq | Cimi, yeah, but let's go for seconds now, easier to apply locale | 17:20 |
| Saviq | UI-side | 17:20 |
| === dandrader is now known as dandrader|lunch | ||
| Cimi | Saviq, so why we had that .length > 0 ? :) | 17:22 |
| Saviq | Cimi, OLD | 17:22 |
| Cimi | ok fine | 17:22 |
| Saviq | Cimi, you're doing the new thing | 17:23 |
| Saviq | Cimi, in which we're fixing broken things like that | 17:23 |
| === boiko_ is now known as boiko | ||
| Saviq | sil2100, done | 17:26 |
| Saviq | sil2100, can we have a silo please | 17:26 |
| Saviq | sil2100, unity8 is through testing, BTW | 17:27 |
| Cimi | Saviq, which thin dividers to drop? | 17:27 |
| Saviq | Cimi, the last one | 17:27 |
| Cimi | why only the last one0 | 17:27 |
| Cimi | ? | 17:27 |
| Saviq | Cimi, because that's between preview widgets, will be handled outside of the widgets, I think | 17:28 |
| Saviq | Cimi, if there's a "first" one, too - drop as well | 17:28 |
| Cimi | Saviq, it was the separator | 17:28 |
| Cimi | Saviq, between tracks | 17:28 |
| Saviq | Cimi, ah ok, it's only between them? | 17:28 |
| Cimi | I'd leave them both for now | 17:28 |
| Cimi | yes | 17:28 |
| Cimi | well, it's the last | 17:29 |
| Cimi | as well | 17:29 |
| Cimi | there's one at the top, all in between, one at bottom | 17:29 |
| Cimi | Saviq, I can drop the first one and last one using index and model count | 17:29 |
| Saviq | Cimi, sec, looking at visuals | 17:29 |
| Saviq | Cimi, you should, too, btw: https://drive.google.com/a/canonical.com/#folders/0B-a_7E3tDxOgWDlCSWxkUGZlcnM | 17:30 |
| sil2100 | Saviq: super awesome! Assigning - and we already saw unity8 and published it, it's migrating | 17:30 |
| Saviq | Cimi, just drop them altogether https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgMnV2ZU1IamJoaEU/edit | 17:30 |
| Cimi | Saviq, qt alternative to this? http://stackoverflow.com/questions/6312993/javascript-seconds-to-time-with-format-hhmmss | 17:34 |
| Cimi | if you know | 17:34 |
| Saviq | Cimi, Qt.formatTime | 17:35 |
| Saviq | Cimi, can't find the new docs, http://qt-project.org/doc/qt-4.8/qml-qt.html#formatTime-method | 17:37 |
| Cimi | Saviq, but this is time | 17:37 |
| Cimi | Saviq, I just want length in readable format | 17:37 |
| Saviq | Cimi, that's the only thing available | 17:38 |
| Saviq | Cimi, you should be able to use it fine | 17:38 |
| Saviq | Cimi, you probably need "if length > 3600" to change the format to include hours, too | 17:38 |
| Saviq | sil2100, did you disable upstream merger for unity-notifications, though? | 17:39 |
| sil2100 | Saviq: yes, but CI might have kicked in by accident | 17:40 |
| sil2100 | (I redeployed it on jenkins so it should be OK) | 17:40 |
| Saviq | sil2100, yeah, looks fine | 17:40 |
| mhr3 | karni, https://code.launchpad.net/~mhr3/unity-scopes-shell/non-dconf-dash-pages/+merge/203806 | 18:01 |
| Cimi | Saviq, might be ugly though | 18:06 |
| Cimi | Saviq, I'd have to use a static date as reference | 18:06 |
| Cimi | Saviq, add seconds to it | 18:06 |
| Cimi | is this ok to you? | 18:07 |
| Cimi | I googled a bit and cannot think of anything else js-only | 18:07 |
| Saviq | Cimi, the static date is 1.01.1970 | 18:07 |
| Cimi | yeah I know | 18:07 |
| Cimi | if you're fine counting from it.. ok | 18:07 |
| Saviq | Cimi, you just ignore it anyway | 18:08 |
| Cimi | indeed | 18:08 |
| Cimi | cause I'll just print time | 18:08 |
| Cimi | but still I hoped I could have done just new QTime in js | 18:08 |
| Cimi | not possible | 18:08 |
| Saviq | sil2100, 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 |
| karni | mhr3: looking | 18:11 |
| sil2100 | Saviq: ;) Will do in a moment, but it's not required for it to be approved before landing - it's just for people cooperating with eachother | 18: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 |
| dednick | mzanetti: ping | 18:13 |
| mzanetti | dednick: hey | 18:13 |
| karni | mhr3: 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 |
| mhr3 | karni, we'll try to land it tomorrow | 18:14 |
| karni | mhr3: shweet, sir! | 18:14 |
| mhr3 | karni, but you need to find approver, i can't approve my own branch :P | 18:14 |
| * karni opens directory, haha | 18:15 | |
| dednick | mzanetti: if you can take a look quick. https://code.launchpad.net/~nick-dedekind/unity8/plugin-qml/+merge/203810 | 18:15 |
| dednick | mzanetti: fixes the plugins not showing qml in qtcreator | 18:15 |
| mzanetti | dednick: oh really. cool! | 18:16 |
| * mzanetti tries | 18:16 | |
| Saviq | dednick, mzanetti https://code.launchpad.net/~mzanetti/unity8/indicators-qml-to-cmake/+merge/203351 hmm? | 18:16 |
| karni | mhr3: 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 |
| mzanetti | Saviq: I'll drop it if dednick's branch above works | 18:16 |
| dednick | Saviq: yeah, that's the nasty way to do it :) | 18:16 |
| Saviq | ok | 18:16 |
| mhr3 | karni, Saviq, mzanetti, tsdgeos, dednick... there's lots of approvers around ;) | 18:17 |
| karni | mhr3: ah, I thought it had to be someone from Unity APIs team | 18:17 |
| Saviq | mhr3, pfft, don't you have your own team? :D | 18:17 |
| karni | hahahahaha | 18:17 |
| Saviq | man I thought for a second tsdgeos is still here... that'd be the end of the WORLD! | 18:17 |
| mhr3 | Saviq, i'm on the edge, i can steal your people! :) | 18:18 |
| karni | Saviq: :D | 18:18 |
| karni | all your base are belong to us | 18:18 |
| mhr3 | karni, i managed to rep the art issue with a test case | 18:19 |
| mhr3 | now only to figure out why doesn't it work | 18:19 |
| karni | mhr3: 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 |
| Saviq | seb128, think you should own ubuntu-settings-components landings or should we? | 18:27 |
| seb128 | Saviq, those are shared widgets/components right? I would say the sdk team should own that :p | 18:28 |
| === rsalveti_ is now known as rsalveti | ||
| Saviq | seb128, well, they're shared between you and us | 18:28 |
| Saviq | seb128, any of them that should be shared with other apps should just move to the sdk | 18:29 |
| seb128 | Saviq, well, you guys did most of the work there so far so I would say it makes sense for you to ask for landing | 18:29 |
| seb128 | Saviq, we can "trade" later if needed | 18:29 |
| seb128 | ;-) | 18:29 |
| Saviq | seb128, ok | 18:30 |
| seb128 | Saviq, 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 |
| Saviq | seb128, yeah | 18:30 |
| Saviq | didrocks, if still around, unity-notifications can be published now | 18:48 |
| didrocks | Saviq: done | 18:51 |
| Saviq | didrocks, thanks | 18:51 |
| Saviq | dednick, ouch, forgot to add ubuntu-settings-components to the build script :/ | 18:52 |
| Saviq | mhr3, "OnlineMusicQuery.cpp:20:32: fatal error: unity/scopes/Reply.h: No such file or directory" that expected? | 18:54 |
| mhr3 | Saviq, yes, old onlinemusic | 18:55 |
| Saviq | mhr3, ah, recipe failed for saucy | 18:55 |
| mhr3 | sigh... of course it did | 18:56 |
| Saviq | mhall119, sorry for the late reply, but it should now be possible to build lp:unity8 with ppa:unity-team and ppa:ubuntu-sdk-team enabled | 18:57 |
| karni | Saviq: landed means landed in unity8 - this is unrelated to release of trusty-proposed image, right? | 18:59 |
| karni | well, demo-stuff ppa should suffice anyway | 19:00 |
| Saviq | karni, yes, it's in archive, but the image is a few hours away still, but you can make your device writable | 19:00 |
| Saviq | karni, and apt-get update/upgrade | 19:00 |
| * karni nod and thanks :) | 19:01 | |
| Saviq | karni, demo-stuff will only get updated now that I merged trunk into new-scopes and pushed | 19:02 |
| karni | Saviq: roger that | 19:03 |
| Saviq | daaaamn LP ;/ | 19:03 |
| Saviq | oh! only 15 tries to request a build on a recipe... | 19:03 |
| davmor2 | Saviq: 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 |
| Saviq | davmor2, man, south? I barely see sun here, how the heck am I to know where's south! | 19:05 |
| davmor2 | Saviq: throw together a compass app for the phone :) | 19:07 |
| mhr3 | karni, https://code.launchpad.net/~mhr3/unity-scopes-shell/fix-json-merging/+merge/203820 | 19:08 |
| * karni looks | 19:09 | |
| mhr3 | karni, oh, but you are in unity-team, so you're free to top-approve | 19:13 |
| === dandrader|lunch is now known as dandrader | ||
| karni | mhr3: top-approved non-dconf-dash-pages | 19:14 |
| karni | mhr3: you take it from here? | 19:14 |
| mhr3 | yep | 19:15 |
| karni | mhr3: also, trying to wrap my head aroud that second fix, very much appreciated | 19:15 |
| cwayne | karni, make sure we document that envvar pls | 19:15 |
| mzanetti | dednick: your branch works fine. I'll wait for ci to work before doing the review stuff | 19:15 |
| cwayne | should be easy enough to set | 19:15 |
| karni | cwayne: I'll keep you in the loop | 19:15 |
| cwayne | karni, kthxbye | 19:16 |
| karni | haha | 19:16 |
| mzanetti | davmor2: I have acompass app... was just waiting for the compass to actually work on the platform | 19:16 |
| mzanetti | to polish for the platform | 19:16 |
| davmor2 | mzanetti: haha | 19:17 |
| mzanetti | :D | 19:17 |
| karni | Saviq: 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 |
| karni | mhr3: I only don't understand line 56 of https://code.launchpad.net/~mhr3/unity-scopes-shell/fix-json-merging/+merge/203820 | 19:23 |
| karni | you're comparing a string to "results for: \"metadata\"" which looked suspicious to me | 19:23 |
| karni | that does not look like a key or value | 19:23 |
| mhr3 | that's the title of the result the scope provides | 19:24 |
| mhr3 | karni, http://bazaar.launchpad.net/~mhr3/unity-scopes-shell/fix-json-merging/view/head:/tests/data/mock-scope/mock-scope.cpp#L56 | 19:26 |
| karni | mhr3: and it should really be "result for \"metadata\"" instead of "metadata" in the test? | 19:27 |
| karni | because the RoleArt is compared against QString("art") | 19:28 |
| karni | that's why RoleTitle compared to this stange string confused me | 19:28 |
| mhr3 | because most of the time the tests are doing empty searches (/surfacing) and comparing all the titles against "" is error prone | 19:29 |
| karni | mhr3: so the RoleTitle ends up literally with a string saying 'result for: "metadata"' ? :) | 19:30 |
| mhr3 | yes | 19:30 |
| karni | that's a strange title, but let it be, if that's the case. | 19:30 |
| karni | ok! | 19:30 |
| karni | mhr3: top-approved. can I leave it with you for whatever merging/landing that requires? (I honestly have no clue where that flows) | 19:31 |
| mhr3 | sure.. and no worries... noone has these days :) | 19:31 |
| karni | haha | 19:32 |
| karni | mhr3: much thanks for prompt action, Michal | 19:32 |
| mhr3 | i planned to write some docs in the afternoon... so that's why there are still none :P | 19:34 |
| Saviq | karni, apt-cache policy unity8? | 19:35 |
| karni | Saviq: http://paste.ubuntu.com/6839700/ | 19:35 |
| Saviq | karni, 20140128 | 19:36 |
| Saviq | karni, apparently 0129 didn't reach your archive yet | 19:36 |
| Saviq | karni, but anyway, since you want newscopes, you need to wait for 0129 to get built in demo-stuff | 19:36 |
| karni | I see. I guess I rushed things. | 19:36 |
| * karni nods | 19:36 | |
| karni | Sorry, I'm like a kid waiting for presents these days. | 19:37 |
| dandrader | Saviq, still there | 19:38 |
| dandrader | ? | 19:38 |
| Saviq | karni, you can always look at https://launchpad.net/~unity-team/+archive/demo-stuff/ to see when's the last thing happened | 19:38 |
| Saviq | dandrader, wassup? | 19:38 |
| karni | Saviq: yeah, there right now ^ ^ | 19:38 |
| karni | *looking there | 19:38 |
| mterry | mzanetti, 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 tests | 19:39 |
| dandrader | Think I found a reason for having QScreen::geometry holding the side stage dimensions: QScreen::orientation concept seems to be tied to it | 19:39 |
| mzanetti | mterry: ah ok. thanks | 19:39 |
| dandrader | not QScreen::availableGeometry | 19:39 |
| dandrader | thus if geometry is landscape but availableGeometry is portrait, the orientation will still be landscape even though the window actually has a portrait aspect ration | 19:40 |
| dandrader | ratio | 19:40 |
| dandrader | seems like the whole orientation thing is designed with fullscreen (or "maximized") windows in mind | 19:41 |
| Saviq | dandrader, TBH I think that's correct | 19:41 |
| Saviq | dandrader, orientation of the screen is landscape | 19:42 |
| Saviq | even if orientation of the window is not | 19:42 |
| karni | new-scopes-fix-carousel5 was merged into new-scopes, but is not listed in the changelog. | 19:42 |
| dandrader | Saviq, for instance, the definition of QWindow::contentOrientation is tied to the orientation of its QScreen | 19:43 |
| mhall119 | Saviq: thanks, mhr3 told me that last week and I already have it built :) | 19:43 |
| Saviq | karni, http://bazaar.launchpad.net/~unity-team/unity8/new-scopes/revision/584 | 19:43 |
| Saviq | mhall119, well, I broke it in the mean time ;) | 19:43 |
| Saviq | mhall119, and well, we broke it today again... | 19:43 |
| * karni shuts up ;) | 19:44 | |
| dandrader | Saviq, yeah, makes sense indeed. problem is the content layouts are tied to the orientation and not to the window aspect ratio.... | 19:44 |
| Saviq | mhall119, but now it's good, only you need to manually install qtdeclarative5-ubuntu-settings-components | 19:44 |
| mhr3 | Saviq, why was it broken btw? | 19:45 |
| dandrader | so a window in landscape orientation but with a portrait aspect ratio will end up having a squeezed landscape layout | 19:45 |
| mhall119 | Saviq: so what you're saying it that it builds when you're on holiday, and breaks when you're at work? | 19:45 |
| Saviq | mhall119, isn't that the better way? | 19:45 |
| mhall119 | Saviq: is that new package in the PPA? | 19:45 |
| Saviq | mhall119, getting built now, yes | 19:45 |
| Saviq | mhr3, I built from lp:ubuntu/*, and that apparently isn't a good thing to do for packages that have an upstream | 19:45 |
| mhall119 | Saviq: and will you update the build script to pull it in too? | 19:46 |
| Saviq | mhall119, https://code.launchpad.net/~saviq/unity8/packaging-and-cleanup/+merge/203829 | 19:46 |
| mhr3 | Saviq, but it worked a few times | 19:46 |
| mhall119 | cool | 19:46 |
| mhall119 | so there's nothing I need to update on the docs | 19:46 |
| Saviq | mhr3, and then it stopped ;) | 19:46 |
| Saviq | mhall119, nope, should be sorted tomorrow | 19:46 |
| mhall119 | thanks | 19:46 |
| mhr3 | lol.. ok | 19:46 |
| mhr3 | well.. now trunk == distro anyway | 19:47 |
| bregma | hey 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 |
| bregma | and, does anyone want to try it out? | 19:51 |
| karni | phablet-config writable-image used to reboot the phone. that is no longer the case? | 20:22 |
| cwayne | karni, it should | 20:25 |
| Cimi | Saviq, I think I have my date in local time, cannot find reference to get UTC | 20:38 |
| Cimi | .UTC() doesn't seem to work | 20:39 |
| greyback_ | bregma: I'd be interested in trying it out. Have you instructions somewhere? | 20:46 |
| === salem_ is now known as _salem | ||
| Cimi | greyback_, you have any idea how to get utc time? | 20:47 |
| Cimi | greyback_, Qt.formatDateTime(new Date(modelData["length"] * 1000), "H:m:ss") adds 1 hour to me | 20:48 |
| greyback_ | Cimi: have you seen this: http://qt-project.org/doc/qt-5.0/qtqml/qtqml-cppintegration-data.html#qdatetime-to-javascript-date | 20:49 |
| greyback_ | the QDateTime class has a toUTC() method that might be what you need? | 20:50 |
| Cimi | greyback_, how do I use it in qml? | 20:51 |
| Cimi | I cannot see this toUTC() | 20:53 |
| Cimi | there is supposed to be a UTC() in js reference, but it's not processed by qml | 20:53 |
| bregma | greyback_, https://launchpad.net/~unity8-desktop-session-team/+archive/custom | 20:54 |
| bregma | works best with a touchscreen :) | 20:54 |
| greyback_ | bregma: cool, thanks, will give it a shot | 20:55 |
| greyback_ | Cimi: aha, this looks more like it: http://qt-project.org/doc/qt-5.0/qtqml/qml-qtquick2-date.html | 20:56 |
| greyback_ | The QML Date object extends the JS Date object with locale aware functions. | 20:57 |
| Cimi | greyback_, but the command above is already locale aware, maybe | 20:57 |
| greyback_ | Cimi: I'm not sure. the JS Date object isn't. The QML one is. So hopefully you just need to swap | 20:58 |
| Cimi | read with Qt.formatDateTime(new Date(30 * 1000), "H:m:ss") | 20:58 |
| Cimi | should be 0:0:30 | 20:58 |
| greyback_ | new Date() looks like JS | 20:58 |
| Cimi | instead is 1:0:30 | 20:58 |
| greyback_ | else use the JS: http://qt-project.org/doc/qt-5.0/qtscript/ecmascript.html#date-objects | 20:59 |
| karni | demo-stuff: Failed to build: amd64 armhf i386 | 21:11 |
| karni | dpkg-buildpackage: error: debian/rules build-arch gave error exit status 2 | 21:12 |
| === dandrader_ is now known as dandrader|afk | ||
| karni | Saviq: failed on tests/plugins/DashViews/horizontaljournaltest.cpp:28 private/v8.h: No such file or directory (missing include o_O) | 21:17 |
| Saviq | karni, that's on saucy | 21:21 |
| Saviq | karni, fixed in https://code.launchpad.net/~saviq/unity8/packaging-and-cleanup/+merge/203829 | 21:22 |
| karni | Saviq: oh. do we expect a trusty build kicks off as well? | 21:22 |
| Saviq | karni, checking, something failed in the recipe | 21:23 |
| Saviq | karni, kicking it again | 21:23 |
| * karni dances \o\ /o/ \o\ | 21:24 | |
| Saviq | karni, 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 wrong | 21:24 |
| karni | Saviq: noted! | 21:24 |
| karni | lp time outs on me :D how dare you, lp! | 21:25 |
| karni | Saved the link, anyway :) | 21:25 |
| === seb128_ is now known as seb128 | ||
| Saviq | mterry, humpf :/ https://code.launchpad.net/~unity-team/unity8/unity8.test_nested_mir/+merge/203088 is broken on x86 | 21:29 |
| Saviq | Cimi, ↑ | 21:30 |
| Saviq | https://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/2383/?#showFailuresLink | 21:30 |
| * mterry looks | 21:30 | |
| mterry | curious it would be arch-specific | 21:30 |
| Saviq | mterry, it's probably not arch-specific, but unity8-as-shell vs. unity8-in-window | 21:32 |
| mterry | Saviq, hmm... I'm guessing that network-manager is not something we can assume is installed? | 21:35 |
| Saviq | mterry, it definitely is installed on those machines | 21:35 |
| karni | If I want ssweeny to test my locally built unity8, is the unity8 deb sufficient, or unit8-private required as well? | 21:35 |
| mterry | ok | 21:35 |
| Saviq | karni, -private, too | 21:35 |
| karni | tnx | 21:35 |
| * ssweeny takes note | 21:35 | |
| Saviq | karni, ssweeny, btw unity8 cross-builds nicely again | 21:36 |
| ssweeny | yay | 21:36 |
| Saviq | karni, ssweeny, sbuild + ccache and I can cross-build it within a few minutes for armhf | 21:36 |
| === dandrader|afk is now known as dandrader | ||
| karni | nice! | 21:36 |
| Saviq | check out https://wiki.ubuntu.com/CrossBuilding and https://wiki.ubuntu.com/SimpleSbuild if interested | 21:37 |
| karni | I will! | 21:37 |
| karni | I tried pbuilder some time ago, but didn't spent enough time on it. | 21:37 |
| mterry | Well the error is a basic "Not authorized", not that the policykit file isn't installed or working or whatever | 21:37 |
| * karni notes | 21:37 | |
| mterry | The policy we ask about would only be rejected if the session that unity8 was in were "inactive" from a logind sense | 21:37 |
| mterry | Saviq, "WARNING: Could not get session id for session. Check that logind is properly installed and pam_systemd is getting used at login." | 21:39 |
| Saviq | mterry, they are saucy machines IIUC, does that matter? | 21:40 |
| mterry | Saviq, 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 way | 21:40 |
| mterry | Saviq, I don't think so... It looks like the machine is just misconfigured | 21:40 |
| Saviq | mterry, yes, the test just run inside a standard unity7 session | 21:40 |
| mterry | fginther, heyo! | 21:41 |
| mterry | fginther, Cimi added a new AP test that checks if the unity session is active from a logind perspective | 21:41 |
| mterry | fginther, but tests are showing that sometimes that's not true, when I would expect it to be. See above ^ | 21:41 |
| mterry | fginther, any ideas? | 21:41 |
| mterry | or nicks of who I should actually bother? :) | 21:41 |
| Saviq | mterry, I believe fginther's in London this wek | 21:43 |
| Saviq | week | 21:44 |
| mterry | Saviq, do you know a good CI backup? | 21:44 |
| * mterry hops in ci-eng | 21:44 | |
| Saviq | mterry, and please pay more attention to CI failures next time ;) | 21:46 |
| Saviq | mterry, as we got the warning here https://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/2304/? | 21:47 |
| Saviq | which is the last otto run on that branch | 21:47 |
| mterry | Saviq, yup, that was my fault | 21:47 |
| mterry | I was in ignore-otto mode because of the persistent failures | 21:47 |
| Saviq | mterry, yeah, me too :/ | 21:48 |
| Saviq | karni, uploaded to ppa and is building | 21:48 |
| karni | Saviq: thanks, I was just sharing a local build with Scott :) Nevertheleess, that's great news! | 21:49 |
| veebers | Saviq: Would you know where the unity ppa get's its version of libautopilot-qt? Or who to ask at least? | 21:49 |
| Saviq | veebers, which unity ppa? | 21:50 |
| veebers | Saviq: 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/consoleFull | 21:50 |
| Saviq | veebers, D09add_ppa~ubuntu-unity~daily-build | 21:51 |
| veebers | Saviq: hah, right just found that in the log :-) | 21:51 |
| Saviq | veebers, daily-build is enabled on that job (and if you ask me - it shouldn't be) | 21:51 |
| veebers | Saviq: ah ok. I'm now trying to figure out where that ppa gets it's libautopilot-qt code from | 21:53 |
| veebers | there may be an issue | 21:53 |
| Cimi | mterry, you still here? | 23:29 |
| mterry | Cimi, sure | 23:30 |
| Cimi | mterry, what does Qt.formatDateTime(new Date(modelData["length"] * 1000), "H:m:ss") give you? | 23:30 |
| Cimi | sorry | 23:30 |
| Cimi | mterry, Qt.formatDateTime(new Date(30 * 1000), "H:m:ss") | 23:30 |
| Cimi | just on a label | 23:31 |
| mterry | in qml? let me see | 23:31 |
| Cimi | yeah | 23:31 |
| mterry | Cimi, 19:0:30 | 23:32 |
| Cimi | mterry, yeah | 23:32 |
| Cimi | mterry, it's not UTC | 23:32 |
| Cimi | mterry, would like it to be 0:0:30 | 23:32 |
| Cimi | I have no ideas | 23:33 |
| Saviq | Cimi, just write a formatter yourself | 23:34 |
| Cimi | Saviq, it's what I am doing now | 23:34 |
| Cimi | Saviq, but before giving up, wanted to understand why | 23:34 |
| Saviq | Cimi, although it's weird that it would assume the time stamp to be UTC... | 23:35 |
| Cimi | Saviq, on qt docs they say that there might be issues with that | 23:35 |
| Saviq | OTOH maybe a timestamp needs to be UTC, as that's the only way to be nonambiguous | 23:35 |
| Cimi | no that as for dst | 23:36 |
| Saviq | Cimi, please take on the simpler widgets next, like image / text etc., leave the more extensive ones for later | 23:36 |
| Saviq | on that note o/ | 23:37 |
| Cimi | ok | 23:37 |
| Cimi | Saviq, ok pushed a formatter | 23:45 |
| Cimi | good night guys | 23:45 |
| karni | Saviq: you still here, hah :) FYI Card carousel card-size only affects first card (lolwut :) ) | 23:46 |
| karni | night Cimi | 23:46 |
| * karni also EODed recently | 23:46 | |
| Cimi | night karni | 23:46 |
| karni | s/recently/moments ago | 23:46 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!