| === _salem is now known as salem_ | ||
| === salem_ is now known as _salem | ||
| === jono is now known as Guest53113 | ||
| === iahmad_ is now known as iahmad|afk | ||
| mhr3_ | pstolowski, just me who dropped out? | 08:44 |
|---|---|---|
| === iahmad|afk is now known as iahmad | ||
| === iahmad is now known as iahmad|afk | ||
| nic-doffay | Saviq, got anything new for me to look at? | 09:05 |
| Saviq | nic-doffay, what's the deal with filters? | 09:06 |
| nic-doffay | Saviq, waiting on fginther to sort out an autopilot issue with the multi selector landing. | 09:07 |
| === alan_g is now known as alan_g|afk | ||
| Saviq | nic-doffay, there's a conflict in multi selector branch | 09:21 |
| Saviq | dednick, pete-woods, so, we're ready to pull the plug | 09:21 |
| nic-doffay | Saviq, another one? | 09:21 |
| dednick | Saviq: um? plug? | 09:21 |
| nic-doffay | Saviq, I solved a conflict yesterday evening. | 09:21 |
| Saviq | dednick, network indicator | 09:21 |
| Saviq | nic-doffay, maybe you didn't push? | 09:21 |
| Saviq | nic-doffay, or maybe something merged in the mean time | 09:21 |
| Saviq | dednick, pete-woods I'll build unity8 and indicator-network packages with: | 09:22 |
| Saviq | https://code.launchpad.net/~saviq/unity8/drop-network-agents/+merge/189009 | 09:22 |
| Saviq | https://code.launchpad.net/~indicator-applet-developers/indicator-network/secret-agent/+merge/182898 | 09:22 |
| dednick | Saviq: yeah, assuming it all works :) | 09:24 |
| === alan_g|afk is now known as alan_g | ||
| nic-doffay | Saviq, looks like something new was merged. | 09:26 |
| Saviq | dednick, you should proof-check that my cleanup there is not too trigger-happy | 09:26 |
| dednick | Saviq: doing now. | 09:27 |
| Saviq | dednick, pete-woods, and then I'd like both of you (Mirco isn't here) to test on a newest image | 09:27 |
| * Saviq builds | 09:27 | |
| dednick | Saviq: You sure we want to remove the IndicatorModel::indicatorData? We may need it later. | 09:28 |
| Saviq | dednick, I think we shouldn't need it | 09:28 |
| Saviq | dednick, and if we don't have it - there's more pressure to make it right | 09:28 |
| Saviq | s/make/do/ | 09:29 |
| dednick | Saviq: ok. | 09:29 |
| dednick | my funeral i guess | 09:29 |
| Saviq | dednick, and if need be - we can always bring it back - it's in the history | 09:29 |
| dednick | indeed | 09:29 |
| Saviq | dednick, on that note - feels like IndicatorsModel should be a singleton now | 09:30 |
| Saviq | but let's leave that for another cleanup | 09:30 |
| tsdgeos | pstolowski: soo https://codereview.qt-project.org/#change,67214 | 09:30 |
| tsdgeos | Saviq: ↑↑↑ | 09:30 |
| Saviq | tsdgeos, did I tell you you ROCK, sir? | 09:31 |
| tsdgeos | thank you | 09:31 |
| tsdgeos | i kind of cheated | 09:31 |
| tsdgeos | since the same bug is in there | 09:31 |
| tsdgeos | is in a few other places | 09:32 |
| tsdgeos | in the same qsortfilterproxymodel | 09:32 |
| tsdgeos | and since we use it extensively | 09:32 |
| tsdgeos | i should problable fix those too | 09:32 |
| tsdgeos | but let's get this in first | 09:32 |
| tsdgeos | and then get the others | 09:32 |
| tsdgeos | Saviq: tbh i am not sure it is *the* fix, but looks quite like it i'd say | 09:35 |
| Saviq | tsdgeos, yup, good | 09:35 |
| tsdgeos | problem is | 09:35 |
| tsdgeos | the only guy that knows about that told me he'd be drunk until next week :D | 09:35 |
| tsdgeos | since oktoberfest and whatnot | 09:35 |
| tsdgeos | so we aint going to get a review today nor tomorrow | 09:35 |
| Saviq | tsdgeos, right ;) | 09:36 |
| Saviq | tsdgeos, we'll distro-patch, as usual | 09:36 |
| * tsdgeos gets onto creating the bug | 09:36 | |
| Saviq | tsdgeos, thanks | 09:38 |
| Saviq | tsdgeos, want me to prep a MP with the patch in place in the mean time? | 09:38 |
| tsdgeos | Saviq: that'd be great | 09:40 |
| tsdgeos | Saviq: btw the other one you created for the delegate range patch | 09:40 |
| tsdgeos | you should mark it as merged or something | 09:40 |
| tsdgeos | since the patch is already in | 09:40 |
| Saviq | tsdgeos, is not? | 09:40 |
| tsdgeos | i think not | 09:40 |
| Saviq | tsdgeos, will do | 09:40 |
| Saviq | tsdgeos, the new one's qtbase, right? | 09:42 |
| tsdgeos | yep | 09:42 |
| Saviq | tsdgeos, sorry, have the QTBUG handy? | 09:45 |
| tsdgeos | nope | 09:45 |
| tsdgeos | just the codereview | 09:45 |
| Saviq | tsdgeos, it's https://codereview.qt-project.org/#change,66585 ? | 09:48 |
| Saviq | no | 09:48 |
| tsdgeos | Saviq: https://codereview.qt-project.org/#change,67214 | 09:48 |
| Saviq | tsdgeos, that? didn't we want the thing for empty scope views? | 09:48 |
| Saviq | tsdgeos, or is that handled already? | 09:48 |
| tsdgeos | that's already integrated | 09:49 |
| Saviq | aah | 09:49 |
| tsdgeos | and Mirv's working on it | 09:49 |
| tsdgeos | integrated in qt | 09:49 |
| tsdgeos | upstream | 09:49 |
| Saviq | yeah | 09:49 |
| Saviq | I thought that's what we were after | 09:49 |
| tsdgeos | Mirv: told me he was working in the distropatch yesterday | 09:49 |
| tsdgeos | Saviq: no this one is one needed for pstolowski's https://code.launchpad.net/~stolowski/unity8/category-reordering | 09:50 |
| Saviq | tsdgeos, yeah yeah, I know what it is | 09:50 |
| tsdgeos | ok :-) | 09:50 |
| tsdgeos | Mirv: https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+bug/1234603 | 09:51 |
| ubot5 | Ubuntu bug 1234603 in qtbase-opensource-src (Ubuntu) "[Patch] Fix rowsInserted not being emmited in some cases in QSortFilterProxyModel" [Undecided,New] | 09:51 |
| Mirv | tsdgeos: yep, it's in unapproved queue now, after tested by me today and chris etc. during the night | 09:53 |
| tsdgeos | ook | 09:53 |
| pstolowski | tsdgeos: awesome, thanks! | 09:53 |
| tsdgeos | Mirv: there's another one we'd need https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+bug/1234603 | 09:53 |
| ubot5 | Ubuntu bug 1234603 in qtbase-opensource-src (Ubuntu) "[Patch] Fix rowsInserted not being emmited in some cases in QSortFilterProxyModel" [Undecided,New] | 09:53 |
| dednick | Saviq: slightly under zealous. http://pastebin.ubuntu.com/6187471/ | 09:54 |
| Saviq | dednick, ah! | 09:54 |
| dednick | Saviq: otherwise all good | 09:58 |
| Saviq | dednick, ok, packages coming right up | 09:58 |
| Saviq | or well, in 10 mins | 09:58 |
| Mirv | tsdgeos: ok.. | 09:59 |
| Saviq | tsdgeos, Mirv https://code.launchpad.net/~saviq/ubuntu/saucy/qtbase-opensource-src/add-proxymodel-patch/+merge/189028 | 10:06 |
| Saviq | tsdgeos, please do a sanity-check, there was a small conflict | 10:07 |
| tsdgeos | Saviq: something's wrong | 10:07 |
| Saviq | tsdgeos, no, it's right | 10:07 |
| tsdgeos | you are adding the whole qsortfilterproxymodel.cpp file? | 10:08 |
| Saviq | tsdgeos, yeah, quilt | 10:08 |
| Saviq | tsdgeos, look at the paths | 10:08 |
| Mirv | Saviq: thanks, I'm really busy at the moment with other integration | 10:08 |
| tsdgeos | Saviq: ok, no clue how quilt works tbh | 10:08 |
| tsdgeos | Saviq: where's the patch supposed to be? | 10:09 |
| Saviq | tsdgeos, at the bottom ;) | 10:09 |
| Saviq | tsdgeos, the changes to the actual files | 10:09 |
| tsdgeos | i can't find it :D | 10:09 |
| Saviq | tsdgeos, but launchpad won't display it | 10:09 |
| Saviq | tsdgeos, launchpad crops at 5k lines | 10:09 |
| tsdgeos | doh | 10:09 |
| tsdgeos | ok | 10:09 |
| Saviq | tsdgeos, will paste you just the patch, sec | 10:09 |
| tsdgeos | there's a -1 line arrangement needed | 10:10 |
| Saviq | tsdgeos, http://paste.ubuntu.com/6187511/ | 10:10 |
| tsdgeos | yep, that's right | 10:10 |
| Saviq | tsdgeos, quilt - long story short, it's a poor-man's version control - it stores all modified files before patching | 10:11 |
| Saviq | tsdgeos, but is really nice in letting you rewind, drop a patch, replay | 10:11 |
| tsdgeos | i see | 10:15 |
| === alan_g is now known as alan_g|afk | ||
| Saviq | dednick, ok, I broke something... | 10:34 |
| dednick | Saviq: ? | 10:34 |
| Saviq | dednick, looking at unity8 logs | 10:35 |
| Saviq | dednick, but https://people.canonical.com/~msawicz/ext-snaps/ | 10:35 |
| dednick | Saviq: hm. i cant get to that address | 10:35 |
| Saviq | dednick, huh? | 10:35 |
| Saviq | dednick, ah, perms | 10:35 |
| dednick | hm. the s in https | 10:36 |
| Saviq | dednick, weird, it's a+rw | 10:36 |
| Saviq | dednick, ah, interesting | 10:36 |
| dednick | Saviq: unity-notifications ? | 10:37 |
| Saviq | dednick, right! | 10:37 |
| Saviq | dednick, wait | 10:37 |
| Saviq | dednick, that's released isn't it | 10:37 |
| dednick | Saviq: no | 10:37 |
| Saviq | dednick, k, building | 10:37 |
| dednick | Saviq: unless you did a manual | 10:37 |
| === iahmad|afk is now known as iahmad | ||
| Saviq | dednick, but anyway, that's unrelated, as I'm just not getting any indicators | 10:40 |
| Saviq | dednick, indicator pages, that is | 10:40 |
| Saviq | dednick, I do get indicators, but not their pages | 10:41 |
| Saviq | nor titles | 10:41 |
| Saviq | must've removed too much | 10:42 |
| === alan_g|afk is now known as alan_g | ||
| dednick | hm. | 10:44 |
| dednick | i ran it up on local build. seemed to be ok. | 10:45 |
| Saviq | dednick, yeah, building locally now, too | 10:45 |
| dednick | Saviq: you get the icons though? | 10:47 |
| dednick | oh right, you did say | 10:47 |
| Saviq | dednick, yes | 10:47 |
| dednick | trying a run_on_device. | 10:47 |
| dednick | Saviq: is their title coming in empty? or "indicator-XXX" | 10:49 |
| Saviq | dednick, indicator-XX | 10:49 |
| Saviq | heh... I cleaned my device, so now installing everything from scratch... | 10:50 |
| dednick | Sounds like the IndicatorPage not being able to load. | 10:51 |
| === _salem is now known as salem_ | ||
| Saviq | dednick, pushed unity-notifications to people.c.c | 10:54 |
| === salem_ is now known as _salem | ||
| dednick | Saviq: file:///usr/lib/arm-linux-gnueabihf/unity8/qml/Unity/Indicators/MenuItemFactory.qml:22:1: module "Unity.Indicators.Network" plugin "IndicatorsNetworkQml" not found | 10:55 |
| Saviq | dednick, right | 10:55 |
| Saviq | dednick, I didn't build with that gone, hoped it's gonna be fine ;) | 10:56 |
| Saviq | dednick, /me fixes | 10:56 |
| Saviq | dednick, btw, we should've bumped some versions here and there... | 10:56 |
| Saviq | dednick, plugin.qmltypes doesn't make sense there now either | 10:58 |
| dednick | Saviq: still getting an error. Cant load Indicators.IndicatorModel now. :( | 11:00 |
| dednick | *IndicatorsModel | 11:00 |
| Saviq | dednick, I'm at 270/451 on the device... | 11:00 |
| dednick | i just edited the lib qmldir manually | 11:01 |
| dednick | that's really odd. i cant even create the Shell now. | 11:02 |
| dednick | oh. that's just because I'm retarded. | 11:03 |
| Saviq | dednick, I'm getting Binding loop detected for property "indexActive" a lot | 11:07 |
| Saviq | dednick, yeah, it works apart from ↑ | 11:07 |
| dednick | Saviq: yeah, i've seen that. it wont actually cause an problem. just an stupid thing in the MenuContentActivator. | 11:08 |
| Saviq | *almost* worked, need to install notifications | 11:08 |
| dednick | Saviq: i installed. not popping up a notification though :( | 11:11 |
| Saviq | dednick, did here? | 11:11 |
| Saviq | just didn't show the text entry, 'cause notifications plugin was old | 11:11 |
| dednick | hm maybe i missed something | 11:11 |
| Saviq | dednick, you rebooted? | 11:11 |
| dednick | yeah | 11:11 |
| dednick | might have multiple network indicators installed | 11:12 |
| dednick | oh. didnt install new network indicator. dpkg error | 11:14 |
| Saviq | dednick, yeah, works fine here, will have a new unity8 pkg in a bit to retest | 11:18 |
| dednick | er. 'm having network issues :) cant download new package... | 11:19 |
| === hikiko is now known as hikiko|lunch | ||
| Cimi | mhr3_, hey michal, how do I know when a plugin is disabled? | 11:28 |
| Cimi | Saviq, you know? ^ | 11:41 |
| Cimi | here's the branch https://code.launchpad.net/~cimi/unity8/unity8.dash-plugins-renderer/+merge/189055 | 11:51 |
| Saviq | Cimi, I think the icon source | 11:51 |
| Cimi | Saviq, icon source? | 11:51 |
| Saviq | Cimi, or not... | 11:52 |
| Saviq | Cimi, maybe a hint then? | 11:52 |
| Saviq | Cimi, just print out all the data you get from the scope ;) | 11:52 |
| Cimi | Saviq, tried printing model.hints | 11:52 |
| Cimi | Saviq, how do I print the whole thing? | 11:52 |
| Cimi | console.log model? :D | 11:52 |
| Saviq | Cimi, yeah, it's there in the metadata | 11:53 |
| tsdgeos | Saviq: so that crash doesn't give us much :-/ | 11:53 |
| tsdgeos | http://paste.ubuntu.com/6187816/ | 11:53 |
| Saviq | tsdgeos, right | 11:54 |
| Saviq | tsdgeos, apport-bug it | 11:54 |
| Saviq | tsdgeos, and I'll try and apport-retrace it | 11:54 |
| Cimi | Saviq, how do I print the whole model? | 11:54 |
| Saviq | Cimi, don't | 11:54 |
| tsdgeos | 0x42a65fe0 is inside /android/system/lib/libc.so and 0x4702d798 is inside no lib | 11:54 |
| Saviq | Cimi, it's "scope_disabled" in the hints | 11:54 |
| Cimi | Saviq, so how do I know the right rule? | 11:54 |
| tsdgeos | wonder what it is, v8 autogenerated code¿? | 11:55 |
| Saviq | Cimi, hints['scope_disabled'] == 1 | 11:55 |
| Saviq | Cimi, or something like that | 11:55 |
| Saviq | Cimi, make that metadata['scope_disabled'] == 1 | 11:56 |
| Saviq | Cimi, libunity-tool -g is your friend :) | 11:56 |
| Cimi | Saviq, model.hints['scope_disabled'] is undefined | 11:57 |
| Saviq | Cimi, read | 11:57 |
| Saviq | Cimi, metadata | 11:57 |
| Saviq | Cimi, not hints | 11:57 |
| Saviq | Cimi, if you'd look at categoryresults.{h,cpp} first, you'd know | 11:57 |
| Saviq | dednick, updated the packages on people.c.c | 11:58 |
| Saviq | Cimi, mhr3_, what's "toggled"? | 11:59 |
| Cimi | Saviq, the rendererhint | 12:00 |
| Cimi | Saviq, dunno why it's called that way | 12:00 |
| Saviq | Cimi, I get it... but that seems not descriptive enough TBH... | 12:00 |
| Cimi | Saviq, yeah | 12:00 |
| === hikiko|lunch is now known as hikiko | ||
| Saviq | dednick, it's worky! | 12:02 |
| Saviq | pete-woods, you around? | 12:02 |
| pete-woods | Saviq: yep, been a little busy this morning, though | 12:03 |
| Saviq | pete-woods, wanna merge snap decisions? | 12:03 |
| Saviq | pete-woods, I just have one question - will we not break desktop? | 12:03 |
| === alan_g is now known as alan_g|lunch | ||
| pete-woods | Saviq: (replied to your MR question), indicator-network isn't part of desktop | 12:03 |
| Saviq | pete-woods, ah | 12:04 |
| Saviq | pete-woods, nm-applet still? | 12:04 |
| pete-woods | Saviq: yep | 12:04 |
| pete-woods | Saviq: it's the only non-converged indicator, I think | 12:05 |
| Cimi | Saviq, model.metadata['scope_disabled'] is undefined for me | 12:05 |
| Saviq | Cimi, is metadata undefined, too? | 12:05 |
| Cimi | Saviq, no it's a qvariant | 12:05 |
| Saviq | pete-woods, so then, indicator-network should start depending on unity8, or maybe unity-notifications at a version > something (we'll bump unity-notifications version) | 12:05 |
| Saviq | Cimi, go into scope.cpp to debug, then | 12:06 |
| om26er | jibel, Saviq bug 1234670 | 12:06 |
| ubot5 | bug 1234670 in unity8 (Ubuntu) "Cannot launch apps anymore" [High,New] https://launchpad.net/bugs/1234670 | 12:06 |
| Saviq | Cimi, erm, categoryresults.cpp | 12:06 |
| Saviq | om26er, thanks, we're reworking the whole focus machinery now, especially with mir around the corner | 12:07 |
| Saviq | om26er, could you try the same under mir? | 12:07 |
| om26er | Saviq, ok, sure. | 12:07 |
| pete-woods | Saviq: whichever one you want, should probably be unity8, as that's what is actually exporting the dbus interface | 12:07 |
| Saviq | pete-woods, btw, x-canonical-ext-snap-decisions isn't there anymore | 12:08 |
| Saviq | pete-woods, x-canonical-private-menu-model is the only capability to advertise support for that | 12:08 |
| jibel | om26er, Saviq I have something very similar under Mir | 12:09 |
| jibel | but I had to close the calculator, then it doesn't reopen | 12:09 |
| Saviq | jibel, could you try with unity-mir from https://code.launchpad.net/~gerboland/unity-mir/add-fail-and-resume-focus-support/+merge/188040/comments/432495 too | 12:09 |
| Saviq | ? | 12:09 |
| pete-woods | Saviq: okay, does that mean I just delete the line that sets this? notificationHints["x-canonical-ext-snap-decisions"] = "true"; | 12:10 |
| Saviq | pete-woods, yeah | 12:11 |
| Saviq | pete-woods, depend on unity8 >= 7.82, too | 12:11 |
| mhr3_ | Cimi, Saviq, it's in the icon actually, but unity8 doesn't have annotated icon | 12:14 |
| Saviq | mhr3_, hmm, libunity-tool shows it in metadata? | 12:15 |
| mhr3_ | but well... yea it's in the metadata too | 12:15 |
| pete-woods | Saviq: update pushed | 12:15 |
| Saviq | pete-woods, cheers | 12:15 |
| Saviq | mhr3_, can we use it for now? | 12:15 |
| === jhodapp|afk is now known as jhodapp | ||
| Saviq | dednick, https://code.launchpad.net/~saviq/unity-notifications/bump-to-0.1.1/+merge/189059 https://code.launchpad.net/~saviq/unity8/bump-to-7.82/+merge/189060 | 12:24 |
| Saviq | pete-woods, is there a review needed on the indicator-network branch? | 12:25 |
| Saviq | didrocks, help? | 12:25 |
| Saviq | didrocks, unity8 will stop providing the network agent (password entry), indicator network will start (through unity8's notifications) | 12:25 |
| pete-woods | Saviq: yep, it should be fairly tight, though | 12:25 |
| Saviq | pete-woods, can anyone on your team do that? | 12:26 |
| Saviq | didrocks, so we need to make sure that they both get upgraded at the same time, or not at all - how do we do that? | 12:26 |
| kgunn | dandrader: so we ended up merging a slightly mod'd version of what you mp'd y'day | 12:26 |
| kgunn | https://code.launchpad.net/~robertcarr/mir/1233944-addendum/+merge/188983 | 12:26 |
| dandrader | kgunn, yeah, I saw it | 12:26 |
| kgunn | cool | 12:27 |
| dandrader | reviewed even | 12:27 |
| kgunn | dandrader: so y'day when you tested it, were you able to run an autopilot test ? | 12:27 |
| Saviq | pete-woods, can you add a version bump to indicator-network so that we can Breaks: indicator-network < $the_new_version | 12:28 |
| Saviq | pete-woods, I think that will make sure we upgrade both unity8 and indicator-network at the same time | 12:28 |
| dandrader | kgunn, didn't do a full blown test. just checked that the fake (uinput) devices created by those autopilot tests did get loaded by android-input | 12:28 |
| pete-woods | Saviq: sure, will do that now | 12:28 |
| kgunn | dandrader: ah...i ask, b/c we may stil not have a data point | 12:28 |
| dandrader | kgunn, data point? | 12:29 |
| kgunn | dandrader: lots of struggling with build/packages and then poor thomi ended up testing for a while but w/o mir enabled...doh | 12:29 |
| dandrader | kgunn, but i'll try it today, as it might be that the input device created by autopilot is not fully recognized by android-input | 12:29 |
| pete-woods | Saviq: what should the new version be, like 0.5.1-0ubuntu1, or do I need to do something more complicated? | 12:29 |
| kgunn | dandrader: data point as in...does it realy fully enable the AP's | 12:29 |
| Saviq | pete-woods, bump the upstream version | 12:30 |
| kgunn | dandrader: thanks to try.. | 12:30 |
| Saviq | pete-woods, so 0.5.1 should be enough - the upstream merger will do the rest | 12:30 |
| Saviq | pete-woods, assuming 0.5.1 is higher than what's there currently ;) | 12:30 |
| pete-woods | Saviq: it's 0.5.0+13.10.20130918-0ubuntu1 atm | 12:31 |
| Saviq | pete-woods, yeah, 0.5.1 is good | 12:31 |
| pete-woods | okay, thanks, I have no idea about our versioning strategy | 12:32 |
| Saviq | pete-woods, there is none ;D | 12:33 |
| Saviq | pete-woods, with the upstream merger, no one cares anymore ;) | 12:33 |
| Saviq | kgunn, we'll be ASKing for the extended snap decisions | 12:38 |
| Saviq | kgunn, anything you have to hitch a ride? | 12:38 |
| kgunn | Saviq: i don't think so...but thanks for asking | 12:45 |
| pete-woods | I have a trivial MR for the camera app if anyone does't mind helping: https://code.launchpad.net/~pete-woods/camera-app/minimum-infographic/+merge/188995 | 12:49 |
| Saviq | pete-woods, drop the -0ubuntu1 | 12:50 |
| Saviq | pete-woods, and it needs to be UNRELEASED for upstream merger | 12:50 |
| pete-woods | Saviq: thanks (I have no idea what I'm doing with the numbers) | 12:50 |
| pete-woods | (to make the merger happy that is) | 12:50 |
| tsdgeos | Saviq: hmmm | 12:50 |
| Saviq | pete-woods, nw | 12:50 |
| tsdgeos | Saviq: can't apport-bug the report | 12:50 |
| Saviq | tsdgeos, why? | 12:51 |
| tsdgeos | in the pc says unity8 has changed since the crash | 12:51 |
| tsdgeos | and in the phone i get *** Error: Invalid problem report | 12:51 |
| Saviq | tsdgeos, yeah, you won't be able to do it from desktop | 12:53 |
| Saviq | tsdgeos, ok, ignore | 12:53 |
| Saviq | tsdgeos, it's not like we'd get more symbols anyway :/ | 12:53 |
| Saviq | tsdgeos, let's see if it repeats | 12:54 |
| tsdgeos | oka | 12:54 |
| Saviq | sil2100, got a minute? | 12:56 |
| sil2100 | Saviq: what's up? | 12:56 |
| Saviq | sil2100, so, the situation is: we're moving the secret agents for wifi passwords from unity8 to indicator-network | 12:57 |
| Saviq | sil2100, so we need to upgrade both at the same time | 12:57 |
| Saviq | sil2100, or none | 12:57 |
| Saviq | sil2100, I've added Breaks: indicator-network (<< 0.5.1) to unity8, so they will not get installed together | 12:58 |
| Saviq | https://code.launchpad.net/~saviq/unity8/drop-network-agents/+merge/189063 | 12:58 |
| Saviq | sil2100, and Requires: unity8 (>> 7.82) to indicator-network, 'cause the password entry now goes through unity8's snap decisions | 12:58 |
| Saviq | sil2100, sounds sane? | 12:58 |
| Saviq | sil2100, https://code.launchpad.net/~indicator-applet-developers/indicator-network/secret-agent/+merge/182898 | 12:58 |
| Saviq | sil2100, make that (>= 7.82) | 12:59 |
| Saviq | pete-woods, I think you need to wrap in parens ↑ in debian/control, sil2100, that right? | 12:59 |
| dednick | Saviq: can confirm the indicator connection working. | 13:00 |
| dednick | Saviq: although it could probably use a "show password" field. | 13:01 |
| Saviq | dednick, right, it's not there in the spec | 13:01 |
| dednick | i c | 13:01 |
| Saviq | dednick, I did raise that with Mirco yesterday | 13:01 |
| davmor2 | Guys Music scope. if you do a search that results in one artist if you try clicking on it if move around the wheel but doesn't play, however if you turn the wheel to it when all are on display it plays when you click it | 13:01 |
| Saviq | dednick, but well, it's not in the spec, so... | 13:01 |
| Saviq | dednick, we'll be able to fix when the time comes | 13:02 |
| dednick | Saviq: yeah, i might just raise that with design today while i'm in the office | 13:02 |
| Saviq | dednick, sure | 13:02 |
| === alan_g|lunch is now known as alan_g | ||
| dednick | Saviq: functionally it's working though, so +1 on getting it in. | 13:03 |
| Saviq | dednick, ok, you're it - you'll help me get this in, provided we get sil2100's ACK on packaging / dependencies | 13:04 |
| sil2100 | Saviq: yes, that sounds sane and looks ok - but you need the parents in the version in the indicator-network as you mentioned | 13:04 |
| Saviq | sil2100, cool! | 13:04 |
| Saviq | dednick, ok, you can start reviewing: | 13:04 |
| Saviq | dednick, https://code.launchpad.net/~saviq/unity-notifications/bump-to-0.1.1/+merge/189059 | 13:04 |
| sil2100 | Saviq: please add that in the Landing Asks and noting that both are required to be released at the same time | 13:04 |
| Saviq | sil2100, already there | 13:04 |
| Saviq | dednick, https://code.launchpad.net/~saviq/unity8/drop-network-agents/+merge/189063 and https://code.launchpad.net/~saviq/unity8/bump-to-7.82 | 13:05 |
| Saviq | (prerequisite) | 13:05 |
| Saviq | sil2100, and well, in theory the deps should handle us releasing only one or the other ;) | 13:05 |
| tsdgeos | Saviq: any idea why https://bugs.launchpad.net/unity8/+bug/1198282 appears in https://launchpad.net/~aacid/+upcomingwork if it's marked as dupe? | 13:06 |
| ubot5 | Ubuntu bug 1200374 in Unity 8 "duplicate for #1198282 Categories in Dash should only animate their height if visible" [Medium,Fix committed] | 13:06 |
| Saviq | tsdgeos, probab;y 'cause it's stupid | 13:07 |
| tsdgeos | makes sense :D | 13:07 |
| tsdgeos | can you unassign me? i don't seem to be able to | 13:07 |
| Saviq | tsdgeos, nope! | 13:08 |
| Saviq | tsdgeos, I can't... | 13:08 |
| tsdgeos | ok, let's just hope it goes away | 13:08 |
| Saviq | yeah | 13:08 |
| Saviq | tsdgeos, btw, I noticed yesterday... | 13:08 |
| Saviq | tsdgeos, expand dash plugins, swipe to top, swipe back down | 13:09 |
| Saviq | tsdgeos, HUGE pause :( | 13:09 |
| Saviq | tsdgeos, ah, you need some apps running | 13:09 |
| tsdgeos | HUGE pause on the scrolling? | 13:10 |
| tsdgeos | or? | 13:10 |
| Saviq | tsdgeos, yeah | 13:10 |
| Saviq | like 4 apps running, so that the category gets scrapped | 13:10 |
| tsdgeos | well that's where the "delegate range" thing comes in, no? | 13:10 |
| Saviq | something's *not* async | 13:10 |
| Saviq | tsdgeos, right! | 13:10 |
| Saviq | forgot | 13:10 |
| tsdgeos | Saviq: it's not about asyncness | 13:10 |
| tsdgeos | well it is | 13:10 |
| tsdgeos | but not everything can be async :D | 13:11 |
| Saviq | tsdgeos, yeah, let's see how your stuff helps | 13:11 |
| Saviq | tsdgeos, at least you have a way to test now | 13:11 |
| Saviq | tsdgeos, TBH I expect UShape to be breaking the asyncness | 13:11 |
| tsdgeos | e.g. everything in there is using the ubuntu shape and once i calculated that it took a good chunk of ms just to run | 13:11 |
| tsdgeos | yep | 13:11 |
| Saviq | s/expect/suspect/ | 13:11 |
| tsdgeos | i once killed the UShape and everything was so much more better | 13:12 |
| === iahmad is now known as iahmad|afk | ||
| Saviq | more better! | 13:13 |
| Saviq | much more better! | 13:13 |
| Saviq | that's like REALLY GOOD lD | 13:13 |
| dednick | Saviq: why are there so many package req for unity-notifications? qtdeclarative5-unity-notifications-plugin | unity-notifications-impl ? | 13:14 |
| Saviq | tsdgeos, sorry, I just had a proper espresso on empty stomach... | 13:14 |
| Saviq | dednick, soo | 13:14 |
| Saviq | dednick, unity-notifications-impl == *an* implementation of unity-notifications interface | 13:14 |
| tsdgeos | :D | 13:14 |
| Cimi | mhr3_, so how can I see if it's disabled or not? | 13:14 |
| Saviq | dednick, unity-notifications-impl-2 == *an* implementation of unity-notifications interface at version 2 of that interface | 13:14 |
| Saviq | Cimi, metadata['scope-disabled'] for now | 13:15 |
| mhr3_ | Saviq, no plans for annotated icon at this point, right? | 13:15 |
| Saviq | mhr3_, sure plans | 13:15 |
| mhr3_ | so yea.. metadata ^ | 13:15 |
| Saviq | mhr3_, not for 13.10 | 13:15 |
| Saviq | dednick, qtdeclarativet5-unity-notifications-plugin - *the default* implementation of unity-notifications interface | 13:15 |
| Saviq | dednick, we need that last one 'cause we need at least one non-virtual dependency | 13:15 |
| mhr3_ | Cimi, iirc it will be defined only for results that actually are disabled | 13:16 |
| Saviq | dednick, so what this all means: unity8 depends on an implementation of unity-notifications interface at version 2, of which the default is qtdeclarative5-unity-notifications-plugin | 13:16 |
| Saviq | dednick, sounds ~sane? | 13:16 |
| Saviq | dednick, the idea is that we can swap out the implementation, provided it implements the same interface | 13:16 |
| dednick | I thought the whole point of virtual was that you didnt require the concrete. | 13:17 |
| didrocks | Saviq: just ensure it's listed on the landing entry (both project) and mention we need both with what to test | 13:17 |
| Saviq | dednick, you need it if you want that dep to actually be pulled | 13:17 |
| didrocks | Saviq: then, we'll take care of that | 13:17 |
| dednick | Saviq: but it's taken care of in seed isnt it? | 13:17 |
| dednick | oh, right, for apt | 13:18 |
| Saviq | dednick, not if you apt | 13:18 |
| Saviq | dednick, yeah | 13:18 |
| Saviq | didrocks, done | 13:20 |
| Saviq | dednick, but that's the thing - if the seed provides a different implementation of said interface, apt won't get the default one | 13:20 |
| didrocks | Saviq: thanks! | 13:22 |
| dednick | Saviq: yup. ok, well the version bumps are fine and I've tested the drop agent, so I'm approving all. Ok to do now? | 13:22 |
| Saviq | dednick, yeah | 13:23 |
| Saviq | dednick, we'll need to wait for indicator-network review from the api guys then before releasing | 13:24 |
| Saviq | dednick, but we can merge all those, yeah | 13:24 |
| Saviq | dednick, ah right, and let me know when you approve unity-notifications, I need to trick jenkins there | 13:24 |
| Saviq | fginther, ping | 13:25 |
| dednick | Saviq: if i approve something with a prerequisite, it wont merge before it's parent will it? | 13:26 |
| Saviq | dednick, nipe | 13:26 |
| Saviq | nope, even | 13:26 |
| dednick | Saviq: ok, all done | 13:27 |
| Saviq | dednick, 'fraid we need to wait for fginther for unity-notifications, will un-approve | 13:27 |
| fginther | Saviq, yo | 13:27 |
| Saviq | fginther, yo! | 13:27 |
| Saviq | fginther, we need to land unity-notifications without unity8 tests (chicken'n'egg) | 13:27 |
| Saviq | fginther, I tried removing the test suite, but then it complains no mediumtests were run | 13:27 |
| fginther | Saviq, let me take a look | 13:28 |
| Saviq | fginther, would it be possible that mediumtests only run when there are test suites? | 13:28 |
| Saviq | fginther, here's the merge https://code.launchpad.net/~saviq/unity-notifications/bump-to-0.1.1/+merge/189059 | 13:28 |
| Saviq | fginther, unity8 doesn't yet "know" about the new unity-notifications-impl-2 | 13:28 |
| Saviq | fginther, and requires unity-notifications-impl-1 | 13:29 |
| Saviq | fginther, so we need *something* to land first :) | 13:29 |
| Saviq | fginther, and sorry to be throwing that on you without a morning coffee :) | 13:29 |
| fginther | Saviq, first cup is already gone | 13:30 |
| Saviq | fginther, oh good! | 13:30 |
| seb128 | ChrisTownsend, hey | 13:31 |
| === _salem is now known as salem_ | ||
| Saviq | Cimi, standup? | 13:32 |
| seb128 | ChrisTownsend, I found https://bugs.launchpad.net/ubuntu/+source/xorg/+bug/1234178 which might interest you (basically blank screenshot issues caused by an intel driver bug), just as fyi since I saw you were trying to figure if it's a compiz issue | 13:32 |
| ubot5 | Ubuntu bug 1234178 in xserver-xorg-video-intel (Ubuntu) "Capturing screen only returns black when resumed before" [Undecided,Incomplete] | 13:32 |
| Saviq | Cimi, notes! | 13:32 |
| fginther | Saviq, the job updates are in progress, as soon as the current -ci and -autolanding jobs finish, those will be updated to remove the unity8 AP testing | 13:34 |
| Saviq | fginther, you can cancel those - they'll both fail | 13:34 |
| fginther | Saviq, ack | 13:35 |
| Saviq | fginther, for next time, could it be that we'd have unity8 there by default, but if we rebuild without a test suite, it will just omit mediumtests? | 13:36 |
| Saviq | fginther, 'cause we'll have that every time we bump the API version there | 13:36 |
| Saviq | fginther, but it's good to have unity8 tests there, to make sure they don't break | 13:36 |
| === dandrader is now known as dandrader|afk | ||
| fginther | Saviq, https://bugs.launchpad.net/jenkins-launchpad-plugin/+bug/1234724 | 13:41 |
| ubot5 | Ubuntu bug 1234724 in jenkins-launchpad-plugin "Allow graceful way to omit or pass mediumtests if no test_suite is specified" [Wishlist,Confirmed] | 13:41 |
| Saviq | fginther, yup! | 13:42 |
| === alan_g is now known as alan_g|afk | ||
| om26er | mhr3_, who is the PoC for the video scope in the Dash ? | 13:43 |
| mhr3_ | om26er, let's say i am | 13:43 |
| Saviq | nic-doffay, here's one you could take over from Mirco https://bugs.launchpad.net/unity8/+bug/1233411 | 13:44 |
| ubot5 | Ubuntu bug 1233411 in Unity 8 "Clicking on a snap decision action button is causing the click to hit the application underneath the notification area" [Critical,Triaged] | 13:44 |
| om26er | mhr3_, webm videos are not showing up in Dash | 13:44 |
| mhr3_ | om26er, gstreamer issue then | 13:44 |
| Saviq | nic-doffay, and there are two more assigned to you in https://launchpad.net/unity8/+milestone/ubuntu-13.09 | 13:44 |
| om26er | mhr3_, I have a .webm file in ~/Videos but its not showing up there. It does play with mediaplayer directly | 13:44 |
| mhr3_ | om26er, open a bug, attach mediascanner log | 13:46 |
| om26er | mhr3_, bug for unity8 ? | 13:47 |
| om26er | and where to get mediascanner log | 13:47 |
| mhr3_ | om26er, for mediascanner | 13:47 |
| mhr3_ | om26er, ~/.cache/upstart/mediascanner.log | 13:47 |
| nic-doffay | Saviq, those have been done. | 13:48 |
| nic-doffay | Saviq, will take a look at that one though ta | 13:48 |
| === alan_g|afk is now known as alan_g | ||
| Saviq | nic-doffay, if something's been done, why is it not marked so? | 13:49 |
| mhr3_ | om26er, and perhaps also zip ~/.cache/mediascanner | 13:50 |
| mhr3_ | om26er, and attach | 13:50 |
| Saviq | dednick, something pretty high up there in terms of priority for you https://bugs.launchpad.net/unity8/+bug/1234736 | 13:52 |
| ubot5 | Ubuntu bug 1234736 in Unity 8 "indicators_client tests need to run under a unity8 session" [Critical,Triaged] | 13:52 |
| Saviq | dednick, if you say "there's virtually no valid test left in indicators_client now", let's just disable them for now | 13:52 |
| nic-doffay | Saviq, I've marked fix committed for the one. | 13:53 |
| nic-doffay | The other still needs a review last I checked. | 13:53 |
| Saviq | nic-doffay, what about https://bugs.launchpad.net/unity8/+bug/1224552 and https://bugs.launchpad.net/unity8/+bug/1150151 | 13:54 |
| ubot5 | Ubuntu bug 1224552 in Unity 8 "[Dash] Category expansion transition has varaible speeds" [High,Triaged] | 13:54 |
| ubot5 | Ubuntu bug 1150151 in Unity 8 "[SIDESTAGE] GU threshold for enabling sidestage should exclude 7" tablets" [Medium,Triaged] | 13:54 |
| dednick | Saviq: i woudld say to disable. They are pretty much useless at the moment. | 13:54 |
| Saviq | dednick, can you make it so, please? | 13:54 |
| dednick | Saviq: ok# | 13:54 |
| === dandrader|afk is now known as dandrader | ||
| Saviq | tedg, damn, too fast, man! | 13:59 |
| tedg | Saviq, ? | 13:59 |
| Saviq | tedg, the network indicator review ;) | 13:59 |
| * Saviq crunches fix-genericpreview | 13:59 | |
| tedg | Heh | 13:59 |
| om26er | mhr3_, how do you force the media scanner rescan. for me it seems it only shows the videos that I pasted the first time after flashing the device. Adding/removing more videos have no effect in the videos scope even after a reboot | 13:59 |
| tedg | Saviq, It was bad, pete-woods was making me talk about HUD if I didn't review it. | 14:00 |
| Saviq | rotfl | 14:00 |
| pete-woods | :D | 14:01 |
| mhr3_ | om26er, it should rescan on each boot | 14:02 |
| mhr3_ | om26er, you can try `restart mediascanner-service` | 14:02 |
| Saviq | om26er, mhr3_ it should even rescan on added - but then you need to search for something to force the dash to refresh | 14:02 |
| Saviq | until we land scope-isactive, that is... | 14:02 |
| mhr3_ | om26er, nope, no -service | 14:02 |
| om26er | mhr3_, yeah, that didn't help | 14:03 |
| === mterry_ is now known as mterry | ||
| mhr3_ | om26er, anything interesting in the log? | 14:03 |
| kgunn | Saviq: moreover to that error on the AP tests...i think they kill the real shell, then try to launch the fake one..and that may be where the failure is | 14:03 |
| ChrisTownsend | seb128: Ah, thanks for pointing that out. | 14:03 |
| mhr3_ | Saviq, it installs monitors, so additions should be fine | 14:04 |
| Saviq | mhr3_, yeah, that's what I meant | 14:04 |
| Saviq | kgunn, that should work | 14:05 |
| om26er | mhr3_, attached to bug 1234726 | 14:05 |
| ubot5 | bug 1234726 in mediascanner (Ubuntu) "webm videos don't show up in Dash" [Medium,New] https://launchpad.net/bugs/1234726 | 14:05 |
| Saviq | kgunn, the only problems there I know of - unblanking and stale socket (what thomi mentioned) | 14:05 |
| om26er | Just now | 14:05 |
| Saviq | kgunn, but those errors are different | 14:05 |
| Saviq | kgunn, I *think* I remember having the fb one when there was an app running and holding the device somehow | 14:06 |
| kgunn | Saviq: i'm gonna try to get alf or someone to look at it... | 14:06 |
| mhr3_ | om26er, btw do you have anything besides webm videos? | 14:08 |
| om26er | mhr3_, I have many formats here. | 14:08 |
| om26er | a long list actually | 14:08 |
| mhr3_ | om26er, fwiw your index is empty | 14:09 |
| === salem_ is now known as _salem | ||
| om26er | mhr3_, that's the problem then. I have 7 videos in the ~/Videos dir on my phone | 14:13 |
| om26er | none of them show up in the Dash | 14:13 |
| om26er | I just rebooted and even tried to search in the dash | 14:14 |
| Saviq | mhr3_, btw, if you say fix-genericpreview looks great, did you see switching-previews? | 14:20 |
| mhr3_ | Saviq, :) i did, but it must look even better with fix-genericpreview ;) | 14:20 |
| Saviq | mhr3_, tru dat! | 14:24 |
| * Saviq needs to reboot... stupid btrfs | 14:24 | |
| Saviq | biab | 14:25 |
| === _salem is now known as salem_ | ||
| mhr3_ | Saviq, is there some problem with reading VariantMap in qml? | 14:57 |
| mhr3_ | getting just undefined | 14:57 |
| mhr3_ | Saviq, got | 14:59 |
| mhr3_ | QVariant(QVariantHash, QHash(("scope-id", QVariant(QString, "applications-scopes.scope") ) ( "content" , QVariant(QVariantHash, QHash(("scope_disabled", QVariant(uint, 1) ) ) ) ) ) ) | 14:59 |
| mhr3_ | neither metadata["scope-id"], not ["content"] work | 14:59 |
| === salem_ is now known as _salem | ||
| tsdgeos | mhr3_: i'm not sure qvarianthash gets automagically translated to qml | 15:08 |
| mhr3_ | tsdgeos, what does then? | 15:08 |
| tsdgeos | a QVariantMap | 15:08 |
| tsdgeos | tbh without a deeper look | 15:11 |
| tsdgeos | i've no clue of why one would get translated and the other not | 15:11 |
| === mpt_ is now known as mpt | ||
| mhr3_ | tsdgeos, how expensive is it to turn the hash into a map? | 15:19 |
| mhr3_ | tsdgeos, it's pretty much creating a copy of the whole thing, right? | 15:19 |
| tsdgeos | mostly yeah | 15:19 |
| tsdgeos | where the qvarianthash come from? | 15:19 |
| mhr3_ | gvariant -> qvariant | 15:19 |
| mhr3_ | so from dee-qt actually | 15:20 |
| mhr3_ | pstolowski, any reason why we're creating a QHash and not a QMap for a{sv}? | 15:22 |
| pstolowski | mhr3_: yeah, that hit me too in dee-qt; it's been there for quite a long time already. I would have change that for onlinemusic scope, but it can break other client code afaict | 15:24 |
| mhr3_ | well... oh one hand it's good, cause metadata should be a black box for the shell and it shouldn't touch it ever :) | 15:26 |
| mhr3_ | on the other we need it now | 15:26 |
| pstolowski | mhr3_: and that's why I do a stupid qvariantmap <-> qvarianthash conversions forth and back in onlinemusicscope... | 15:26 |
| pstolowski | mhr3_: and the reason it could break existing code if changed is if anyone does value.toHash(), it will return empty hash if there's a qvariantmap inside | 15:28 |
| mhr3_ | pstolowski, do we know who might be doing that? | 15:29 |
| mhr3_ | there's not too many dee-qt users | 15:29 |
| mhr3_ | although... i was surprised when doing the transaction stuff in dee-qt, scopes are definitely not the only ones | 15:30 |
| pstolowski | mhr3_: I've no idea, but I can imagine with phone stuff, there might be quite some clients | 15:32 |
| mhr3_ | so... since metadata doesn't work in qml, and it shouldn't expose everything... and we don't have annotatedicon, i'd go for providing just the scope_disabled in the metadata (and create a VariantMap with just that) | 15:32 |
| mhr3_ | Saviq, objections^? | 15:32 |
| mhr3_ | tsdgeos, maybe you as well^? | 15:33 |
| pstolowski | mhr3_: perhaps it would be a good idea to open a bug for dee-qt, and fix it in new cycle | 15:33 |
| pstolowski | and see what breaks ;) | 15:33 |
| kgunn | Saviq: mind joining a meeting...check your calendar....as a greyback stand in | 15:33 |
| kgunn | sorry it was last minute | 15:33 |
| mhr3_ | kgunn, leave him alone, i want him :P | 15:34 |
| mhr3_ | although he's ignoring me anyway | 15:34 |
| kgunn | mhr3_: oh to be so popular | 15:34 |
| mhr3_ | i think he's hiding under the table from all the paparazzis | 15:34 |
| tsdgeos | mhr3_: well the question is what a a{sv} is supposed to be | 15:34 |
| mhr3_ | tsdgeos, well theoretically with hash you're losing order of the container, right? | 15:35 |
| tsdgeos | yep | 15:35 |
| mhr3_ | tsdgeos, so it should be map | 15:35 |
| tsdgeos | well with map you're losing it too | 15:36 |
| mhr3_ | then again, everyone uses it as a dict | 15:36 |
| mhr3_ | oh? | 15:36 |
| tsdgeos | it gets sorted with the key | 15:36 |
| mhr3_ | i thought map would keep the insert order | 15:36 |
| tsdgeos | i.e. it sorts itself | 15:36 |
| mhr3_ | so it's not like std::map | 15:36 |
| tsdgeos | it is like std::map | 15:37 |
| tsdgeos | std::map also sorts | 15:37 |
| pstolowski | tsdgeos, mhr3_ : "When iterating over a QHash, the items are arbitrarily ordered. With QMap, the items are always sorted by key." (qt docs) | 15:37 |
| tsdgeos | exactly | 15:37 |
| tsdgeos | interesting | 15:38 |
| tsdgeos | the dbus spec says | 15:38 |
| tsdgeos | { -> "Entry in a dict or map" | 15:38 |
| mhr3_ | well then it doesn't really matter | 15:38 |
| tsdgeos | :D | 15:38 |
| mhr3_ | anyway, i don't feel like changing dee-qt now | 15:40 |
| mhr3_ | so back to my workaround for unity8 | 15:40 |
| fginther | Saviq, ping | 15:45 |
| fginther | Saviq, http://10.97.2.10:8080/job/autopilot-testrunner-otto-saucy/752/console (unity8 test) is failing because the new unity-notifications is not in the archive yet | 15:46 |
| Saviq | fginther, isn't unity-notifications in the same stack? shouldn't it use the local archive? mbs? | 15:47 |
| Saviq | kgunn, sorry, just managed to kill my system - and reinstalled it | 15:47 |
| Saviq | kgunn, still going? | 15:48 |
| kgunn | Saviq: awesome! nope we just ended | 15:48 |
| kgunn | Saviq: are you having one of those days ? | 15:48 |
| fginther | Saviq, ugh that's right. looks like a bug in the job. | 15:48 |
| tsdgeos | mhr3_: yeah i mean if it has 2 contents | 15:48 |
| tsdgeos | should be that hard | 15:48 |
| Saviq | kgunn, kind of | 15:48 |
| mterry | mzanetti, you still looking at that 40% bug? | 15:54 |
| mterry | I was busy yesterday, but started looking at reproducing it again this morning | 15:55 |
| mterry | mzanetti, using the cargo-culting formula of reboot+photo+gallery+wait, I'm able to get it 2/12 times | 15:55 |
| mterry | (I figured percent numbers would be good for testing any potential fixes, so we know when we have confidence it is fixed) | 15:56 |
| mterry | mzanetti, intriguingly, I also note that once you hit it, you keep hitting it | 15:56 |
| mterry | (in the same session) | 15:56 |
| mhr3_ | tsdgeos, sorry didn't get that | 15:57 |
| mhr3_ | "2 contents"? | 15:57 |
| tsdgeos | mhr3_: i mean converting from QVariantHash to QVariantMAp | 15:57 |
| tsdgeos | shouldn't be that expensive if it only has two items inside | 15:57 |
| * mterry has to grab lunch now, will be back | 15:57 | |
| mhr3_ | tsdgeos, but the second item is another varianthash | 15:58 |
| mhr3_ | tsdgeos, and i just realized that we can't simply convert it, cause then activation breaks (cause it expects the original hashes) | 15:58 |
| tsdgeos | ouch | 15:59 |
| mzanetti | mterry: yep... pretty much the same as I've seen. but 2/12 times is kinda low... | 16:01 |
| mterry | mzanetti, after hitting it initially, I also noticed that manually pressing power will still make it happen (in the same session). I assume that's true for first-bug-hit too, so maybe that's a faster way to test | 16:02 |
| mterry | mzanetti, how often are you seeing it? | 16:03 |
| mzanetti | mterry: well, I always have it after waiting a couple of hours. but I never got it straigt away after rebooting | 16:03 |
| mterry | mzanetti, oh interesting. Both mine were straight away | 16:04 |
| mterry | mzanetti, so you have times where greeter will show without bug, but a while later, the greeter will evince the bug? | 16:04 |
| mzanetti | mterry: always after using it... but not real pattern | 16:04 |
| mzanetti | mterry: it's not that it starts on its own when the device is sleeping | 16:05 |
| mterry | mzanetti, hrm. Well, I have to run for food, will keep testing after | 16:05 |
| mterry | mzanetti, right, ok | 16:05 |
| paulliu | mzanetti: Just add a small comment. https://code.launchpad.net/~unity-team/unity8/fix-genericpreview/+merge/188355 | 16:05 |
| paulliu | Saviq, mzanetti: I think it already looks good? Will that be merged today? | 16:05 |
| === dandrader is now known as dandrader|lunch | ||
| Saviq | paulliu, needs a review which I didn't manage to do today | 16:09 |
| Saviq | paulliu, but yeah, soon | 16:09 |
| * Saviq failed at dogfooding today... no incoming call sound *or* missed call notifcation - just a blue icon :( | 16:10 | |
| Saviq | larsu, ↑ | 16:10 |
| mzanetti | Saviq: hey, you got a blue icon! I don't get that one either | 16:11 |
| mzanetti | sometimes people tell me that they called me and I don't see anything at all on the phone | 16:11 |
| paulliu | Saviq: ok. thanks. | 16:14 |
| Saviq | mzanetti, well, blue icon is good... nothing in the messaging menu is not... nor is no incoming call signal :/ | 16:14 |
| Saviq | mzanetti, I *think* I messed mine up with the settings app | 16:15 |
| Saviq | seb128, possible ↑? | 16:15 |
| Saviq | fginther, you'll let me know when we can approve again? | 16:27 |
| fginther | Saviq, yes | 16:27 |
| Saviq | fginther, thanks | 16:27 |
| Saviq | Cimi, you couldn't access model.metadata values through []? | 16:29 |
| Saviq | mhr3_, Cimi that should work fine.... | 16:29 |
| Cimi | Saviq, it didn't work | 16:29 |
| Cimi | Saviq, tried model.metadata['scope_disabled'] | 16:30 |
| Cimi | Saviq, with " and ' even :D | 16:30 |
| mhr3_ | he just started to read the log, let him continue :) | 16:30 |
| Saviq | Cimi, it was probably just missing a .toMap() for RoleMetadata | 16:30 |
| Cimi | Saviq, with content before ['content']['scope_disabled'] | 16:30 |
| seb128 | Saviq, larsu is off, national holiday | 16:31 |
| Cimi | mhr3_, hah ok | 16:31 |
| Saviq | seb128, right | 16:31 |
| seb128 | Saviq, gema reported similar bugs with segfaults of the indicator-messages-service | 16:31 |
| seb128 | does it segfault for you? | 16:31 |
| Saviq | seb128, checking | 16:31 |
| dednick | Saviq: i can't seem to run the autopilot tests with 'make autopilot' anymore. make[2]: *** No rule to make target `CMakeFiles/install.dir/all', needed by `CMakeFiles/autopilot-indicators_client.dir/all'. Stop. | 16:31 |
| dednick | the targets have a install dependency, but it seems to screw with it. | 16:32 |
| Saviq | dednick, make install first | 16:33 |
| dednick | Saviq: i did | 16:33 |
| Saviq | dednick, hmm, then we've broken something | 16:33 |
| Saviq | seb128, no crash in /var/crash, it seems running still | 16:34 |
| seb128 | Saviq, just for the record | 16:34 |
| seb128 | https://bugs.launchpad.net/bugs/1234673 | 16:34 |
| ubot5 | Ubuntu bug 1234673 in indicator-messages (Ubuntu) "indicator-messages crashes and entry call is missed for the user" [High,New] | 16:34 |
| seb128 | https://bugs.launchpad.net/bugs/1234680 | 16:34 |
| ubot5 | Ubuntu bug 1234680 in indicator-messages (Ubuntu) "indicator-messages blue even though there are no messages" [Undecided,New] | 16:34 |
| seb128 | Saviq, did that start today? | 16:34 |
| Saviq | seb128, I doubt it, mzanetti reported it happen to him some times | 16:34 |
| Saviq | seb128, but anyway - my question to you was more about the incoming call sound | 16:35 |
| Saviq | seb128, I can select the sound in settings app and can hear it, but when a call comes - silent | 16:35 |
| seb128 | Saviq, what image do you run? | 16:35 |
| seb128 | Saviq, ringtone was buggy, it got fixed in 78 today | 16:35 |
| Saviq | seb128, k, upgrading to 79 then | 16:36 |
| seb128 | Saviq, https://launchpad.net/ubuntu/saucy/+source/qtmultimedia-opensource-src-touch/5.1.1+git20130920+5b12abb862-0ubuntu3 | 16:37 |
| seb128 | Saviq, is the fix | 16:37 |
| seb128 | Saviq, make sure you have that version, I'm not sure in what image it landed | 16:37 |
| Saviq | seb128, k thanks | 16:37 |
| Saviq | "Install & Restart" BOOM | 16:37 |
| Saviq | could use a confirmation :) | 16:37 |
| * om26er wonders if holding unity8 from daily release is a good idea. we are getting less testing | 16:42 | |
| dednick | Saviq: worked when I sudo make install | 16:46 |
| Saviq | dednick, huh? installed to / or ./builddir/install? | 16:46 |
| dednick | ./builddir | 16:46 |
| dednick | ./builddir/install | 16:46 |
| Saviq | dednick, maybe you did that before and install as user failed? | 16:46 |
| Saviq | dednick, sudo rm -R builddir/install and try again? | 16:47 |
| dednick | Saviq: hm. thats weird. worked when i cleaned builddir. might have tried make autopilot before make install, then no longer works | 16:48 |
| dednick | Saviq: oh.. i removed the dep :) | 16:49 |
| dednick | try again | 16:49 |
| mhr3_ | Saviq, could you look at the conflicted test in lp:~unity-team/unity8/scope-isactive if you have a sec? | 16:49 |
| Saviq | mhr3_, I will... as soon as... | 16:49 |
| mhr3_ | nw | 16:49 |
| dednick | Saviq: shoul i remove the indicator_client tests, or just remove it from autopilot target? | 16:50 |
| Saviq | dednick, remove them altogether, when we get back to them, we'll bring parts of them back - but they need a rework | 16:50 |
| dednick | ok | 16:51 |
| dednick | Saviq: ok, i cant get make autopilot to work at all when the install dep is there. | 16:53 |
| Saviq | dednick, you mean the package?! | 16:53 |
| Saviq | dednick, ah, the install dep for autopilot? | 16:53 |
| dednick | Saviq: the install dep for the autopilot-X targets in cmake macro | 16:54 |
| Saviq | dednick, mhm | 16:54 |
| Saviq | dednick, worked fine here.. | 16:57 |
| dednick | Saviq: from a clean build? | 16:57 |
| === _salem is now known as salem_ | ||
| Saviq | dednick, trying a clean build now, maybe indeed something was installed before | 16:58 |
| dednick | Saviq: https://code.launchpad.net/~nick-dedekind/unity8/remove-indicators-client-autopilot/+merge/189134 | 16:59 |
| Saviq | dednick, thanks! | 16:59 |
| Saviq | dednick, yup, works just fine here | 17:00 |
| dednick | thats weird | 17:01 |
| === alan_g is now known as alan_g|EOD | ||
| dednick | working from office = leave on time + get home an hour later... sigh. it's a hard life | 17:06 |
| === jhodapp is now known as jhodapp|lunch | ||
| om26er | Saviq, unity8 under Mir is pretty laggy, is that something being investigated or should I keep the hopes down. A few weeks ago it was pretty fast and responsive under Mir | 17:10 |
| Saviq | om26er, galaxy? | 17:10 |
| om26er | Saviq, Nexus 4 | 17:10 |
| Saviq | om26er, that should be perfectly smooth | 17:10 |
| Saviq | om26er, top? | 17:10 |
| om26er | Saviq, except its really not :/ | 17:11 |
| om26er | Saviq, top looks reasonable its not eating alot of cpu | 17:11 |
| mhr3_ | Saviq, btw objections to http://paste.ubuntu.com/6188689/ ? | 17:16 |
| Saviq | om26er, any particular place it's laggy or just a general feeling? | 17:16 |
| Saviq | mhr3_, yes | 17:17 |
| mhr3_ | it's pretty awful, but i'm afraid we have to do it | 17:17 |
| Saviq | mhr3_, let's just fix the metadata | 17:17 |
| om26er | Saviq, yeah, its slow in general. i.e. app opening animation, navigation in dash as a whole | 17:17 |
| Saviq | om26er, tried restarting unity8? | 17:17 |
| mhr3_ | Saviq, there are dee-qt users that already expect the variants to be hashes | 17:17 |
| mhr3_ | Saviq, we'd break them | 17:17 |
| Saviq | mhr3_, or we just request a map instead | 17:18 |
| Saviq | mhr3_, leave the default behavior as is | 17:18 |
| om26er | Saviq, a number of times. did a lot of reboots. | 17:18 |
| mhr3_ | Saviq, converting hash to map means copying everything | 17:18 |
| Saviq | om26er, then something's wrong on your device, I'm not seeing it here | 17:18 |
| mhr3_ | Saviq, and if we change the metadata to be map it'll break activation and previews cause those expect hashes | 17:19 |
| === dandrader|lunch is now known as dandrader | ||
| om26er | it couldn't be. I flash daily and over the last week I know for certain things are slow. | 17:19 |
| Saviq | mhr3_, it's not like we don't own those ;) | 17:19 |
| Saviq | mhr3_, and copying some strings about is still better than what we have above | 17:19 |
| Saviq | mhr3_, and if other places use the hashes, why can't we here? the other are cpp-only? | 17:20 |
| mhr3_ | Saviq, we're already talking about converting it back and forth | 17:20 |
| mhr3_ | sucks | 17:20 |
| Saviq | mhr3_, why back'n'forth? | 17:20 |
| mhr3_ | Saviq, hash to map when reading, map to hash when activating | 17:21 |
| Saviq | mhr3_, not necessarily, we don't create the result for activation in QML do we? | 17:22 |
| mhr3_ | Saviq, if you want a real fix, fix qt to be able to read hashes :P | 17:22 |
| Saviq | mhr3_, sure, that's another thing | 17:22 |
| mhr3_ | Saviq, but ultimately, imo the models should have rendererHints | 17:23 |
| mhr3_ | and that should contain all renderer-specific properties | 17:23 |
| mhr3_ | the ones you found and noted in the big renderer table, that aren't common for all | 17:23 |
| mhr3_ | how does that sound ^? | 17:25 |
| Saviq | mhr3_, sure, as long as they're a map ;P | 17:25 |
| mhr3_ | sure :) | 17:25 |
| mhr3_ | Saviq, but you fix qt in 14.04 cycle | 17:25 |
| mhr3_ | to do hashes too | 17:26 |
| Saviq | mhr3_, TBH I imagine there's a reason why they're not exposed to QML, and we'd be taught it when we tried... | 17:26 |
| Saviq | mhr3_, what are the consumers of dee-qt assuming hashes, btw? and is there a reason for using hashes and not maps? | 17:27 |
| Saviq | mhr3_, also, why are they double-wrapped in there? | 17:27 |
| Saviq | mhr3_, and why couldn't we pass a hint to DeeListModel::data() to get us a map and not a hash - without copying it? | 17:29 |
| Saviq | we could even template it! ;) | 17:29 |
| mhr3_ | Saviq, the thing is, it doesn't matter for dbus, but apparently converting from hash to map in qt isn't exactly cheap | 17:34 |
| mhr3_ | Saviq, and pls stop that c++ non-sense :P | 17:35 |
| Saviq | mhr3_, sure, but we don't need to convert, if we create a map straight away | 17:35 |
| Saviq | mhr3_, and sure, converting it back into hash for activation would be stupid, but would we actually do it? | 17:35 |
| mhr3_ | Saviq, right, i'm just saying the original implementation did hashes, and now consumers do .toHash() on the returned variants | 17:36 |
| Saviq | mhr3_, I agree, and that should remain | 17:36 |
| Saviq | mhr3_, but there's no reason why we couldn't *add* a way to talk maps | 17:36 |
| Saviq | mhr3_, where the default would still be hash | 17:36 |
| mhr3_ | Saviq, right now it wouldn't, the qvariant -> gvariant conversion doesn't do maps | 17:36 |
| mhr3_ | cause nothing produced them, right? :) | 17:36 |
| Saviq | mhr3_, and why couldn't we add *that*? ;) | 17:37 |
| mhr3_ | we could | 17:37 |
| Saviq | mhr3_, but again, would we really need that? | 17:37 |
| Saviq | mhr3_, is activation happening on QML side at all? are we actually passing the whole QHash back? | 17:37 |
| mhr3_ | Saviq, well you want it changed to QMap so we'd have to | 17:38 |
| mhr3_ | but otherwise yea, hashes are just passed through qml and it works fine | 17:38 |
| Saviq | yeah, you just can't access them, really - which I agree is stupid | 17:38 |
| mhr3_ | yep | 17:39 |
| Saviq | mhr3_, I think there's one important thing though | 17:39 |
| Saviq | mhr3_, QVariantMap gets *converted* to a JSArray | 17:39 |
| Saviq | mhr3_, and back | 17:39 |
| mhr3_ | ouch | 17:39 |
| Saviq | mhr3_, when you pass it QML <-> C++ | 17:39 |
| mhr3_ | so i guess we shouldn't do it too much :) | 17:39 |
| Saviq | mhr3_, so there's no way to know that something is supposed to be a QVariantHash and not QVariantMap | 17:40 |
| Saviq | mhr3_, well, QVariants are generally copied back and forth anyway | 17:40 |
| Saviq | it's not like we're passing pointers to them... well, we try to pass refs, so that's good | 17:40 |
| Saviq | but I'm sure we're copying them here and there | 17:40 |
| mhr3_ | i kinda hoped they're refcounted | 17:41 |
| Saviq | mhr3_, that may be - /me doesn't know enough about the internals there | 17:42 |
| mhr3_ | Saviq, but anyway, for now - adding rendererHints to the model, keeping it as untyped variant in most cases, populating with scope-disabled for dash-plugins (and using VariantMap), k? | 17:43 |
| Saviq | mhr3_, you mean to CategoryResults? | 17:43 |
| mhr3_ | yes | 17:43 |
| Saviq | mhr3_, and what would you put there? just a pre-set list of valid hints? or the whole of metadata? or metadata['contents']? | 17:45 |
| Saviq | mhr3_, good thing about Q...Models is that if you don't access the role, data() won't get called for it | 17:45 |
| mhr3_ | Saviq, pre-set list | 17:45 |
| mhr3_ | Saviq, yep, that's nice | 17:45 |
| Saviq | mhr3_, so you're fine with (hopefully shallow-copying) that? | 17:46 |
| Saviq | mhr3_, copying QStrings should be fine - those are refcounted for sure | 17:46 |
| mhr3_ | Saviq, well, it'll look only for scope-disabled right now, so just a simple lookup in a hash... super fast | 17:46 |
| Saviq | mhr3_, yeah, +1 | 17:47 |
| mhr3_ | coolio | 17:47 |
| Saviq | mhr3_, still a hack, but a better one | 17:47 |
| mhr3_ | better hacks... should be my middle name :P | 17:47 |
| mhr3_ | Saviq, adding it to https://code.launchpad.net/~mhr3/unity8/expose-renderer-hint/+merge/188848 | 17:47 |
| mhr3_ | since ap doesn't like you :) | 17:47 |
| Saviq | mhr3_, lol | 17:48 |
| Saviq | good timing | 17:48 |
| Saviq | mhr3_, and well, that lookup we'd have to do regardless, if we had support for QHash in QML ;) | 17:48 |
| mhr3_ | indeed | 17:48 |
| mhr3_ | Saviq, shall i map the key name to a qt-ish "scopeDisabled"? | 17:49 |
| Saviq | mhr3_, no, just pass as is in the map | 17:49 |
| mhr3_ | k | 17:49 |
| Saviq | mhr3_, we'll still access it with rendererHints['scope-disabled'], correct? | 17:49 |
| mhr3_ | yea | 17:49 |
| mhr3_ | well... underscore | 17:49 |
| Saviq | mhr3_, yeah, camelCase in strings isn't really Qt-ish at all ;) | 17:50 |
| mhr3_ | it's kind of a prop name | 17:50 |
| Saviq | but not really :) | 17:52 |
| === jhodapp|lunch is now known as jhodapp | ||
| mhr3_ | Saviq, pushed | 17:57 |
| Saviq | mhr3_, o. | 17:57 |
| Saviq | o/ | 17:57 |
| mhr3_ | and you get to explain to thostr_ why the apps scope isn't done yet :P | 17:57 |
| * mhr3_ hides | 17:58 | |
| Saviq | mhr3_, right... and when we need to add something to this list - it's you that'll go and make it more generic than that ;P | 17:59 |
| Saviq | mhr3_, you could've gone with some QStringList for it at least ;P | 17:59 |
| mhr3_ | Saviq, you mean i prepare a patch with another if? :) | 17:59 |
| Saviq | mhr3_, yeah, that | 18:00 |
| mhr3_ | Saviq, it's not like we're changing everything scope-related right? | 18:01 |
| mhr3_ | oh wait | 18:01 |
| Saviq | mhr3_, :D | 18:01 |
| * Saviq needs to make a unity-scopes API ASAP | 18:03 | |
| Saviq | then you can swap things out under me however you like | 18:03 |
| Saviq | fginther, sorry to be breathing down your neck.... but any update? | 18:10 |
| fginther | Saviq, nothing yet, I'm working on it right now | 18:11 |
| Saviq | fginther, ok, won't be bothering you again... | 18:11 |
| thostr_ | so, mhr3, the apps scope is done by now? :) | 18:12 |
| mterry | mzanetti, I have success with disabling the infographic | 18:18 |
| mterry | mzanetti, no reproductions in 10 runs | 18:18 |
| mzanetti | mterry: mhm... | 18:20 |
| mterry | mzanetti, I wish I could see what was happening... I wonder if the infographic is out of control animating | 18:21 |
| mterry | I guess I'll add output comments | 18:22 |
| mzanetti | mterry: I have put prints in every animation and timer I could find and none printed anything when it happened | 18:22 |
| mterry | mzanetti, :( | 18:22 |
| mzanetti | mterry: but there's still lots of stuff open I guess... some sort of binding loop for example | 18:22 |
| mzanetti | mterry: but if its really the infographics and only happens after e.g. taking a picture, it must be those bubbles | 18:23 |
| mzanetti | because only that changes when taking a picture for example | 18:23 |
| mterry | fair | 18:24 |
| mzanetti | mterry: hmm... would be intersting to see if it stops when switching to another info source | 18:24 |
| mterry | mzanetti, next time I reproduce it, I'll try that | 18:24 |
| mterry | oh wait, it still is reproducing | 18:25 |
| mterry | uh... what's another source besides telephony? | 18:25 |
| mterry | mzanetti, ^ | 18:25 |
| mzanetti | mterry: otp... one sec | 18:26 |
| mzanetti | mterry: I've seen pictures, phone calls (incoming/outgoing are different) and sms | 18:28 |
| mterry | mzanetti, nope... I received a call, and regardless of which is up, they both use up CPU | 18:34 |
| kgunn | Saviq: sorry...my brain is slow...so, i'm pestering dandrader, but earlier you mentioned about the killing unity8 to load | 18:38 |
| kgunn | the fake one kind of made sense....do we already know what needs to be changed ? | 18:38 |
| kgunn | Saviq: hoping to save dandrader some debug if possible | 18:39 |
| Saviq | kgunn, nothing, in theory - everything should behave as usual | 18:39 |
| Saviq | kgunn, with the addition of https://code.launchpad.net/~nick-dedekind/unity8/remove-indicators-client-autopilot/+merge/189134 | 18:39 |
| Saviq | kgunn, that were outdated and half-disabled and we just decided to get rid of them and rework from scratch | 18:39 |
| Saviq | kgunn, if testing unity8 - you need to stop unity8 - if testing anything else - you need to start unity8 - that's about it, no brainer | 18:39 |
| kgunn | Saviq: ok, so its worth it for dandrader to dig into why the fake unity8 doesn't launch after the real one is murdered | 18:48 |
| Saviq | kgunn, I'd say there's one potential reason: is it started with correct QT_QPA_PLUGIN? | 18:49 |
| Saviq | kgunn, so - is it using ubuntumirserver? | 18:49 |
| dandrader | kgunn, I'm still debugging the steps from https://bugs.launchpad.net/mir/+bug/1233944. Now investigating why a tap (made with the fake device) doesn't seem to have any effect | 18:50 |
| ubot5 | Ubuntu bug 1233944 in Mir "Unity8/Mir is unable to open autopilot uinput devices" [Critical,Fix committed] | 18:50 |
| dandrader | taking from where I stopped before we had that bug fix | 18:50 |
| dandrader | kgunn, that message "[WW, android-input] [InputReader] Touch device 'autopilot-finger' did not report support for X or Y axis! The device will be inoperable." | 18:52 |
| kgunn | Saviq: i think that's what thomi's UPA plugin was meant to solve (along with hardcoded geom)...it was specific to mir | 18:54 |
| dandrader | kgunn, so after that InputReader discards all events coming from that device | 18:54 |
| dandrader | kgunn, and no wonder a tap from autopilot doesnt do anything | 18:54 |
| kgunn | dandrader: so you're trying to run autopilot for ubuntu toolkit ?? | 18:55 |
| dandrader | yes | 18:55 |
| dandrader | kgunn, ^ | 18:55 |
| kgunn | dandrader: did you unlock the screen mannually, e.g. make sure you're showing dash not greeter | 18:55 |
| dandrader | kgunn, I unlocked it manually | 18:55 |
| dandrader | kgunn, so the fake input device autopilot creates has to be properly configured otherwise android-input won't take it seriously :) | 18:56 |
| Saviq | kgunn, specific to mir is one thing, specific to running a mir *server* is another | 18:56 |
| dandrader | kgunn, now I gotta find if the bug is in uinput or in the way autopilot uses it | 18:58 |
| kgunn | Saviq: yeah...wondered about that...like, would seem wherever in the AP stack the call to kill /restart unity8 is...there should also be ref to mir server | 18:58 |
| kgunn | dandrader: cool | 18:58 |
| kgunn | dandrader: wondering...how did plars run notepad and friends ok....hmmmm | 18:58 |
| dandrader | gonna ask him | 18:59 |
| Cimi | Saviq, saw your branch | 19:06 |
| Saviq | Cimi, yeah, I'm simplifying the thing slightly | 19:07 |
| Cimi | Saviq, I left the image inside the item in case was needed later to align labels and description | 19:07 |
| Saviq | Cimi, you can align without that | 19:08 |
| Saviq | Cimi, I'm taking the Label out of the Item as well | 19:08 |
| Cimi | ok.. | 19:08 |
| Cimi | Saviq, you can simplify maybe tileStyle then | 19:08 |
| Saviq | Cimi, yeah, Label out of Item as well there | 19:08 |
| === _salem is now known as salem_ | ||
| Saviq | Cimi, just two Shapes and a Label | 19:08 |
| Saviq | Cimi, and in case of FlatTileStyle - Image, Shape and a Label | 19:09 |
| Cimi | sure | 19:09 |
| Saviq | Cimi, pushed - you can have a look if you want | 19:11 |
| Cimi | looking | 19:11 |
| Cimi | Saviq, do you think it's possible to set properties of delegates from outside? | 19:12 |
| Saviq | Cimi, not sure what you mean? | 19:12 |
| Cimi | Saviq, I wanted in the beginning of creating an alias in GenericFilterGrid pointing at style of its delegate | 19:13 |
| Cimi | like property alias delegateStyle: delegate.style | 19:13 |
| Cimi | or tile.style | 19:13 |
| Saviq | Cimi, nah, that's not gonna work | 19:13 |
| Cimi | both don't work | 19:13 |
| Saviq | Cimi, 'cause delegate is a component | 19:13 |
| Saviq | Cimi, not an object | 19:14 |
| Cimi | ah I see | 19:14 |
| Cimi | would have saved the duplicated code | 19:14 |
| Saviq | Cimi, yeah, there's still some cleanup possible | 19:15 |
| Saviq | mzanetti, still there? | 19:15 |
| mzanetti | Saviq: yeah | 19:17 |
| Saviq | mzanetti, do you know who to talk to about some pendrives you're supposed to give out next week? | 19:17 |
| mzanetti | Saviq: greyback | 19:18 |
| karni | mzanetti: The ones with Ubuntu preinstalled on a VM, I've read somewhere. I thought it's a great idea for the Deview workshop around 15th Oct | 19:18 |
| Saviq | karni, ↑↑ | 19:18 |
| karni | Saviq: mzanetti: Thank you, guys!! | 19:18 |
| Saviq | karni, cheers\ | 19:18 |
| fginther | Saviq, I have the changes made to pull from the local archive. | 19:24 |
| Saviq | fginther, wow, that took you longer than expected... the otto job never supported it yet? | 19:24 |
| fginther | Saviq, no, it was an oversight | 19:25 |
| Saviq | fginther, so we can approve? | 19:25 |
| fginther | Saviq, yes | 19:26 |
| Saviq | fginther, \o/ thank you | 19:26 |
| fginther | Saviq, you're welcome, sorry for the blocker | 19:29 |
| karni | mzanetti: Do you know if your presentation will be streamed? I'd love to see it! | 19:34 |
| mzanetti | karni: no idea | 19:34 |
| karni | ack | 19:34 |
| karni | mzanetti: last question. can you tell me what sort of app will you guys show off during the live demo? I guess it'll be something simple. | 19:36 |
| mzanetti | karni: not sure yet tbh | 19:37 |
| karni | I see :D | 19:37 |
| thomi | kgunn: https://bugs.launchpad.net/unity8/+bug/1234901 | 19:38 |
| ubot5 | Ubuntu bug 1234901 in Unity 8 "Need DBus application manager interface" [Undecided,New] | 19:38 |
| mzanetti | Saviq: do you know if there's a way to generate some infographic data with the command line? | 19:38 |
| karni | mzanetti: we have infographics if you want | 19:38 |
| karni | mfisch: pointers for mzanetti re: populating infographics? ↑ | 19:39 |
| mzanetti | karni: I'd need to increase the counter for taken pictures for example | 19:39 |
| mzanetti | doesn't really matter what. just populate some data | 19:39 |
| mzanetti | at runtime | 19:40 |
| karni | mzanetti: ah. what we have is populate some sensible data, so you get the "flower" effect | 19:40 |
| karni | I don't know the details, you'd have to wait for mfisch or cwayne. or ssweeny might know? | 19:40 |
| ssweeny | karni, mzanetti libusermetrics-tools has a commandline tool to populate it | 19:42 |
| mzanetti | ssweeny, karni: cool, thanks | 19:42 |
| mzanetti | ssweeny: what is $DATASOURCE_ID? | 19:44 |
| ssweeny | mzanetti, i think that's like "camera-app" or "twitter" or some identifying string | 19:45 |
| mzanetti | ssweeny: and the format_string? % desn't seem to work | 19:48 |
| mzanetti | oh... %1 it is | 19:49 |
| ssweeny | there you go | 19:51 |
| mzanetti | ssweeny: can I also change the day somehow? | 19:52 |
| ssweeny | mzanetti, i *think* if you call it 30 times you'll get 30 days worth of data | 19:54 |
| ssweeny | at least that's how it works if you use the api | 19:54 |
| mzanetti | ssweeny: hmm, no it just overwrites the current one | 19:54 |
| ssweeny | hm | 19:54 |
| ssweeny | it may not be possible with the commandline then | 19:55 |
| mzanetti | ah, found it, just add arguments | 19:55 |
| ssweeny | oh neat | 19:56 |
| karni | mzanetti: If you're still here, and I'm not being a PITA at this hour, can you tell me what is that point about? "3rd Party offering with full Qt/QML support" | 20:02 |
| mzanetti | karni: hmm... seems to lack a "developer | 20:03 |
| mzanetti | " | 20:03 |
| mzanetti | 3rd Party developer offering... | 20:03 |
| karni | mzanetti: oh | 20:03 |
| karni | mzanetti: so basically, anyone can do what we're doing - that is the essense? | 20:03 |
| mzanetti | karni: what do you mean with "what we're doing"= | 20:03 |
| mzanetti | ? | 20:03 |
| karni | mzanetti: we canonical. "1st party" ;) | 20:03 |
| mzanetti | well, probably not... | 20:04 |
| mzanetti | but there's full Qt/QML support :) | 20:04 |
| karni | mzanetti: ah. gotcha :) | 20:04 |
| karni | mzanetti: I've added that missing 'developer' in your presentation | 20:04 |
| mzanetti | karni: thanks | 20:04 |
| mzanetti | karni: do you use the final version or the drag still? | 20:05 |
| karni | mzanetti: Not sure if you saw my message - you might want to remove a couple of "confidential" headers from the presentation | 20:05 |
| karni | mzanetti: I'm looking at the fianl | 20:05 |
| karni | *final version | 20:05 |
| mzanetti | karni: no. didn't look at it at all today (bank holiday here) | 20:05 |
| karni | mzanetti: sorry I'm disturbing your holiday.. :)! | 20:06 |
| mzanetti | no worries. as long as you fix my presentation ;P | 20:07 |
| karni | mzanetti: for my workshop, I've added a little bit about Html5, removed the porting part as I'm not familiar with it (but can probably answer some questions), and will merge two slides (portfolio/ app design). your presentation is really helpful. | 20:07 |
| karni | mzanetti: ^ ^ | 20:07 |
| karni | mzanetti: I'll delete those few "confidential" footers for ya | 20:07 |
| mzanetti | heh :D | 20:07 |
| mzanetti | karni: yeah, the html5 stuff is not that interesting for the QtDevDays | 20:07 |
| mzanetti | we probably could quickly mention it somewhere tho | 20:08 |
| karni | mzanetti: I can imagine ;D | 20:08 |
| Saviq | fginther, hrmpf https://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-saucy/770/console ? | 20:09 |
| Saviq | fginther, "No test report files were found. Configuration error?" | 20:10 |
| fginther | Saviq, 770 was in-flight before the changes were finished. The one you want is 773 | 20:11 |
| Saviq | fginther, k, thanks | 20:11 |
| fginther | Saviq, actually 773 was one I restarted as a second test. 774 was triggered by autolanding of https://code.launchpad.net/~mhr3/unity8/expose-renderer-hint/+merge/188848 | 20:13 |
| Saviq | fginther, k! | 20:14 |
| Saviq | fginther, does this look like it's running http://10.97.2.10:8080/job/autopilot-testrunner-otto-saucy/774/console ? | 21:12 |
| fginther | Saviq, no, that's stuck | 21:13 |
| Saviq | fginther, ugh, not our lucky day, is it... | 21:14 |
| fginther | Saviq, shouldn't it be depending on unity-notifications-impl-2? | 21:14 |
| Saviq | fginther, that didn't land yet | 21:15 |
| Saviq | fginther, it's one of the other approved branches | 21:15 |
| Saviq | fginther, https://code.launchpad.net/~saviq/unity8/bump-to-7.82/+merge/189060 | 21:15 |
| === seb128_ is now known as seb128 | ||
| fginther | Saviq, which MP needs to land first? | 21:18 |
| Saviq | fginther, doesn't matter | 21:19 |
| Saviq | fginther, only https://code.launchpad.net/~saviq/unity8/drop-network-agents/+merge/189063 has a prerequisite | 21:19 |
| Saviq | fginther, the rest should land just fine in whatever order | 21:19 |
| fginther | Saviq, then I'm unclear on what provides unity-notifications-impl-1, it's not in the local archive | 21:20 |
| Saviq | fginther, http://bazaar.launchpad.net/~unity-api-team/unity-notifications/trunk/revision/175 | 21:21 |
| fginther | Saviq, qtdeclarative5-unity-notifications-plugin provides unity-notifications-impl, unity-notifications-impl-2 | 21:21 |
| Saviq | fginther, yeah, but the one in saucy provides -impl-1 | 21:21 |
| Saviq | fginther, so both should work just fine | 21:21 |
| Cimi | Saviq, this one should land before https://code.launchpad.net/~mhr3/unity8/expose-renderer-hint | 21:23 |
| Saviq | Cimi, sure, it's a prerequisite | 21:23 |
| Saviq | fginther, ok that has a prerequisite, too https://code.launchpad.net/~unity-team/unity8/dash-filtergrid-renderers/+merge/189160 | 21:23 |
| Saviq | fginther, but the other three should land in whatever order | 21:24 |
| Cimi | Saviq, didn't really understand this https://bugs.launchpad.net/unity8/+bug/1231937 | 21:24 |
| ubot5 | Ubuntu bug 1231937 in Unity 8 "Video & music renderers are not to spec" [High,Triaged] | 21:24 |
| Cimi | Saviq, mounted means cropped with black frames top&bottom? | 21:25 |
| Saviq | Cimi, yes | 21:25 |
| fginther | Saviq, http://paste.ubuntu.com/6189916/ | 21:25 |
| fginther | Saviq, unity-notifications-impl-1 can't be installed? | 21:25 |
| Saviq | fginther, it's virtual | 21:26 |
| Saviq | fginther, you can install qtdeclarative5-unity-notifications-plugin=0.1.0+13.10.20130905.2-0ubuntu1 | 21:26 |
| fginther | Saviq, but can apt figure that out? | 21:26 |
| Saviq | fginther, I thought it would | 21:26 |
| Saviq | fginther, I was told it would | 21:27 |
| fginther | Saviq, i'll admit to not knowing myself | 21:27 |
| Saviq | fginther, but maybe it doesn't | 21:27 |
| Saviq | fginther, it would handle upgrades gracefully, but it might be that when installing a non-newest version, it craps out | 21:27 |
| Cimi | Saviq, there's also this one to approve if you're on fire https://code.launchpad.net/~cimi/ubuntu-ui-toolkit/panel_export-position/+merge/186727 :) | 21:27 |
| Saviq | fginther, in which case https://code.launchpad.net/~saviq/unity8/bump-to-7.82/+merge/189060 would need to land first | 21:28 |
| fginther | Saviq, lets try that | 21:28 |
| Saviq | Cimi, you need to add an entry to CHANGES | 21:29 |
| Cimi | changes? | 21:29 |
| Saviq | Cimi, and update the API description | 21:29 |
| fginther | Saviq, it's now next in line | 21:29 |
| Cimi | Saviq, ok... | 21:29 |
| Saviq | Cimi, and add docs, probably | 21:29 |
| Saviq | fginther, thanks, and sorry about that - was really under the impression apt would figure it out | 21:30 |
| Cimi | Saviq, it's an internal plugin, we don't need docs | 21:30 |
| Saviq | Cimi, ah | 21:30 |
| Cimi | Saviq, like, a communicator between toolbar and shell | 21:30 |
| Cimi | Saviq, as more secret as possible :) | 21:30 |
| fginther | Saviq, no worries, I thought it was more likely that there was still an issue with the job | 21:31 |
| Cimi | Saviq, so probably don't need CHANGES | 21:31 |
| Saviq | Cimi, probably - shouldn't it set the property on construction, too? | 21:32 |
| Cimi | Saviq, it's 0 in the communicator | 21:33 |
| Cimi | Saviq, default value | 21:33 |
| Cimi | c++ | 21:33 |
| Saviq | Cimi, does dbus know about it? | 21:33 |
| Cimi | Saviq, the shell registers the service | 21:33 |
| Cimi | Saviq, the communicator in the sdk uses the service | 21:33 |
| Saviq | Cimi, ok | 21:33 |
| === salem_ is now known as _salem | ||
| Cimi | Saviq, I need to make the inputfilterarea smaller when the toolbar is shown | 21:34 |
| Saviq | that's so bad, btw... this whole thing... | 21:34 |
| Cimi | Saviq, yeah, better ideas? :) | 21:34 |
| Cimi | Saviq, how can the app tell the shell? | 21:34 |
| Saviq | Cimi, yeah, we need a private channel of communication between app in focus and shell | 21:34 |
| Saviq | Cimi, I'd say through Mir, even | 21:35 |
| Cimi | Saviq, in general, +1 | 21:35 |
| Cimi | Saviq, but for the moment... | 21:35 |
| Saviq | Cimi, yeah, of course | 21:35 |
| Cimi | Saviq, we already had that service for the hud to hide the toolbar on opening the hud | 21:35 |
| Cimi | Saviq, I added this property since we needed it | 21:35 |
| Saviq | Cimi, I know | 21:35 |
| Cimi | ok | 21:35 |
| Saviq | Cimi, I was there when the decision was made - it was the lesser evil anyway | 21:36 |
| Saviq | Cimi, was hoping it'd go away by now ;) | 21:36 |
| Saviq | fginther, shall we kill the stuck one? | 21:37 |
| fginther | Saviq, ack | 21:37 |
| Saviq | fginther, done | 21:37 |
| fginther | Saviq, bump-to-7.82 based otto test | 21:54 |
| fginther | s/based/passed/ | 21:55 |
| Saviq | fginther, \o/ | 21:59 |
| Saviq | halfway there :) | 21:59 |
| === _salem is now known as salem_ | ||
| === salem_ is now known as _salem | ||
| Saviq | fginther, uh, it's the second time we got that https://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/1963/console | 23:13 |
| Saviq | fginther, I'm worried it's gonna hit us again | 23:14 |
| Saviq | fginther, it's installing unity8-autopilot and then decides there's no test results | 23:15 |
| Saviq | fginther, weird is that maguro ran fine | 23:15 |
| veebers | Saviq: hey query: Do you know what's happening with the indicators_client tests in the unity8 autopilot tree? There are 2 there and the are both skipped (@unittest.skip("Battery indicator has been removed.")) | 23:16 |
| Saviq | veebers, yeah... https://code.launchpad.net/~nick-dedekind/unity8/remove-indicators-client-autopilot/+merge/189134 | 23:16 |
| * veebers looks | 23:16 | |
| Saviq | veebers, we need to re-do them from scratch | 23:16 |
| Saviq | veebers, they never got the needed love, and would make it difficult to run under Mir | 23:17 |
| Saviq | veebers, so we decided to scrap them for now | 23:17 |
| veebers | Saviq: hah nice. That MR makes my fix a lot easier | 23:17 |
| Saviq | veebers, they didn't do anything anymore | 23:17 |
| Saviq | veebers, ;) | 23:17 |
| veebers | Saviq: I was just going to go in and clean it up (or give it the love as you put it :-) ) | 23:17 |
| veebers | Saviq: so what's the plan for getting autopilot test coverage for the indicators? | 23:18 |
| Saviq | veebers, we need a .desktop file for it and start again | 23:18 |
| Saviq | veebers, just treat indicators_client as any app (although the .desktop would have NoDisplay=true) | 23:18 |
| Saviq | veebers, so we'd split the tests out into a separate suite | 23:18 |
| veebers | Saviq: cool makes sense. Who and when is working on that? (just trying to get my understanding correct :-) ) | 23:19 |
| Saviq | veebers, you, and whenever you want ;P | 23:19 |
| Saviq | veebers, but for real - no plans yet | 23:20 |
| veebers | Saviq: oh right, well I presume I should talk to dednick and probably alesage re: the indicators tests then? | 23:21 |
| Saviq | veebers, yup | 23:21 |
| veebers | Saviq: coolio, thanks for clarifying | 23:22 |
| Saviq | veebers, cheers, glad we didn't waste your time | 23:22 |
| veebers | Saviq: ^_^ | 23:22 |
| Saviq | fginther, yeah, -maguro is broken for unity8 tests ;( | 23:38 |
| Saviq | or wait, it was -mako before :? | 23:38 |
| * Saviq no get it... | 23:40 | |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!