=== shiznix_ is now known as shiznix === _salem is now known as salem_ === salem_ is now known as _salem [08:03] Cimi: just letting you know we landed a fix for the broken settings API last night [08:08] Saviq, hey, can we land stuff today? the v4 api would be nice to get in the image [08:23] mhr3, yeah, wanted to confirm it's "static"? [08:23] Saviq, meaning? [08:24] mhr3, like did we stop changing it by now? ;) [08:24] Saviq, i'd certainly hope so :) [08:25] Saviq, also, status of overview? [08:25] mhr3, to be confirmed today [08:25] mhr3, wanna clean the merge queue a bit first [08:25] see silo 1 [08:26] ouch :) [08:26] mhr3, adding v4 to it, too [08:26] Saviq: so the fixes i have for dash-as-app [08:26] just add it to overview? [08:26] or want a separate merge? [08:26] Saviq, k, plugin part here - https://code.launchpad.net/~unity-team/unity-scopes-shell/sort-order/+merge/229190 [08:27] tsdgeos, for "does not have i18n" you mean that .pot wasn't updated? [08:27] Saviq: means it is not loading the catalog [08:27] tsdgeos, ah know what you mean [08:27] tsdgeos, yeah, separate MP please, will push it in with silo 1 then [08:28] mhr3, could use a review... [08:28] Saviq, being done right now [08:28] mhr3, and if we push that, we need the alt nav, too [08:28] right pete-woods? ^ :) [08:28] which needs a review, too [08:29] so I'm thinking the next train ride [08:29] Saviq, we don't really, just the api compatibility part of it [08:29] and tsdgeos did split it up i think [08:29] kind of yes [08:29] he did, might wanna look that it's good [08:29] can do [08:29] think it didn't get updated after we changed v4 5 times [08:29] mhr3: yeah, review happening :) [08:29] branch? [08:30] mhr3: you have a merge conflict with trunk, btw [08:30] pete-woods, ah, fixing [08:30] mhr3, https://code.launchpad.net/~unity-team/unity8/scopes-v4/+merge/229224 [08:30] mhr3, but I'm looking actually [08:35] pete-woods, pushed [08:35] Saviq: https://code.launchpad.net/~aacid/unity8/dash-app-fixes/+merge/229745 [08:36] tsdgeos, thanks [08:39] Saviq: any idea about that autopilot failure we have everywhere? [08:40] tsdgeos, yeah, should be gone with a more recent image [08:40] ok [08:40] tsdgeos, we didn't depend on the fixed qtmir [08:41] tsdgeos, yeah, 174+ should be fine [08:42] tsdgeos, see! http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/3778/console [08:42] yeah modeminfo succeeded [08:42] and we have 3 machines running qmluitests again [08:42] that's starting to look *much* better http://s-jenkins.ubuntu-ci:8080/job/unity-phablet-qmluitests-utopic/ [08:42] finally [08:43] ah no, just two machines still... we were down to 1 though [08:46] Saviq: the icons of view-expand not landed yet? [08:46] tsdgeos, we'll land them with expandables [08:47] which makes the whole thing hard to test [08:47] or well, I can land them now [08:47] since i don't want to lose 3 hours runing optipng [08:48] Saviq: that'd be cool [08:48] tsdgeos, export NO_PNG_PKG_MANGLE=1 [08:48] tsdgeos, just add that to your .profile and be done with it ;) [08:50] tsdgeos, added to silo [08:50] great :) [08:52] Saviq: i have that in ~/.sbuildrc instead =) $build_environment = { 'NO_PKG_MANGLE' => '1', 'DEB_BUILD_OPTIONS' => 'parall\ [08:52] el=12', 'HOME' => '/build/' }; [08:52] Saviq: not working (the export) [08:53] anywy [08:53] i'll wait for the silo package [08:55] tsdgeos, been working for me, did you actually get it into your env? [08:55] Saviq: well i just exported it in the same shell i ran dpkg-buildpackage [08:55] tsdgeos, hmm interesting [08:59] dednick, prio for this morning https://code.launchpad.net/~unity-api-team/unity8/modeminfo/+merge/225159 please [09:02] tsdgeos, lol, why did you go Abstain on your own branch? ;) [09:03] tsdgeos, ah, it's actually doing svg optims, not png ones [09:05] Saviq, hm, the icons on the default app page use the magic @ thing, right? [09:06] mhr3, no [09:06] no? [09:06] mhr3, nope [09:06] mhr3, we're not shipping any icons any more [09:06] mhr3, so unless the app does, we're not doing anything to it [09:06] Saviq: so it shows in https://code.launchpad.net/~aacid/+activereviews [09:07] tsdgeos, craazy ;) [09:07] Saviq, ah, right, theme icons [09:07] Saviq: yeah [09:07] Saviq, so that explain the slowness to load them [09:07] mhr3, only some of them [09:07] tsdgeos, it'll clear up a bit today [09:08] Saviq, same thing as with the http thing - image provider -> everything serialized [09:08] there is hope! [09:08] mhr3, right, but only some of them are themed [09:08] mhr3, like 3 or something [09:08] need to see why seeLessStuckBottom qmluitests fail :/ [09:08] and they shouldn't be, we need to put the actual PNG icons to the projects [09:10] Saviq, like pretty much all the top 6? :) [09:10] mhr3, maybe ;P [09:11] which you're supposed to see all the time [09:12] tsdgeos, http://people.canonical.com/~msawicz/themes.tar.xz [09:12] mhr3, we'll most probably load the whole dash page to memory when you're focused on one, just defer image loading [09:12] tx [09:12] Saviq, meaning, it's not going to change one bit? [09:14] mhr3, for apps page we might even load them all [09:15] mhr3, but when you're scrolling around in a scope, once it loads the image, it won't drop it until you unfocus that scope [09:18] Saviq, oh, i see [09:18] Saviq, or if the model changes persumably? [09:18] mhr3, well, of course [09:22] mzanetti, hey man [09:22] Cimi, he's only back tomorrow [09:22] Cimi, quick one for you: https://code.launchpad.net/~saviq/unity8/fix-pageheader-colours/+merge/229698 [09:22] Saviq, ok [09:23] Cimi, easy to verify with tryDash [09:23] Saviq, and code makes sense [09:23] Cimi, yeah, that's just a bonus [09:23] Saviq, I decided I'll go for loaders in genericscopeview [09:23] mentally feels cleaner and nicer [09:24] for preview/settings/whatever [09:24] might be easier to replace in the future with stack [09:24] Saviq, on dash activity indicator, remember that we needed to fix autopilot tests with searches? [09:25] since we removed the items [09:25] Cimi, oh indeed did I not do that [09:25] Cimi, will do ASAP [09:27] tsdgeos, can you please sanity-check https://code.launchpad.net/~unity-team/unity-api/scopes-v4/+merge/228346 and make sure your v4 branch is updated to taste please === shiznix_ is now known as shiznix [09:28] dednick, hey, did you see my request for prio this morning on https://code.launchpad.net/~unity-api-team/unity8/modeminfo/+merge/225159 ? [09:28] it even passed CI! [09:30] Saviq: i was having some issues with it yesterday evening. gsm wasn't showing connected when it should have been [09:30] Wellark: ^ did you get to the bottom of it? [09:31] Saviq: want me to top approve https://code.launchpad.net/~unity-team/unity-api/scopes-v4/+merge/228346 then? [09:34] Saviq, tsdgeos new bug for dash as app [09:35] wtf [09:35] let me reproduce... [09:35] tsdgeos, yeah [09:35] Saviq: scopes-v4 in unity8 looks sensible to me [09:36] but someone should review [09:36] since i've done parts of it too :D [09:36] Saviq: why is unity-scopes-shell in silo 1? or is it just a "general landings" silo? [09:36] dednick, why not? ;) [09:37] Saviq, tsdgeos full left to right swipe to dash does not go to app scope [09:37] dednick, I took it over to land unity8... but now we're in TRAINCON-0 again :| [09:37] so run any app from launcher [09:37] swipe to right [09:37] Saviq: i thought we do silos on a more functional basis. since now the gsm issue may hold up the rest of the landing. [09:37] Cimi: i didn't do any dash-as-app, you know as much as i do [09:37] I am saying [09:37] dednick, if it does, we just kill it from the silo for now, no problem [09:37] change to music or video scope [09:38] run app from the launcher [09:38] Cimi: open a bug or fix it, saying it here will make it get lost 99% probbly [09:38] swipe to right [09:38] it doesn't go back to app scope [09:38] Wellark: ping [09:38] dednick, it's the same problem when they're in separate silos, really, as one of them blocks the other [09:38] Saviq: i c [09:38] Saviq, can it be due to app suspension ^ ? [09:39] Cimi, unlikely, we're not suspending the dash yet... [09:40] I will file a bug then [09:45] Saviq, tsdgeos https://bugs.launchpad.net/unity8/+bug/1353374 [09:45] Launchpad bug 1353374 in Unity 8 "Regression - swipe to right does not switch back to app scope" [Undecided,New] [09:45] Cimi, tx [09:46] as said index changed, but not visually [09:46] I might have a look around since I am touching those files [09:46] but if you have idea throw it :) [09:47] Cimi: so listview index is changing but not the listview contents? [09:47] sounds like something we know? [09:47] tsdgeos, if you flick after you got back to dash [09:47] tsdgeos, it starts from app scope [09:47] so it feels like the index changed [09:47] Saviq, mir issue? [09:47] Cimi: are you totally up to date? [09:48] tsdgeos, 175 [09:48] Cimi, can you actually interact with the "incorrect" scope? [09:48] Cimi: which qml version is there? [09:48] tsdgeos, qtdeclarative got in 174 [09:48] http://people.canonical.com/~ogra/touch-image-stats/174.changes [09:49] :/ [09:49] Saviq, ok is mir [09:49] indeed [09:49] Cimi, so frame not posted? [09:49] Saviq, yes [09:49] Saviq, if I switch back [09:49] Saviq, tap the screen [09:49] Saviq, I am touching the app scope [09:49] Cimi, yeah, sounds like mir didn't get us the latest frame from the dash [09:49] I switched to video scope (so tapping should open a video) [09:50] instead I managed to open amazon webapp :D [09:50] yups [09:51] Cimi, comment on the bug please, and we need to let Gerry know [09:51] ok [09:51] it'll probably be between mir and qtmir [09:52] lupus in fabula [09:52] Saviq: also it looks like the initial guide can be swiped out of the way on the initial swipe and it leaves you on the apps scope and nothing opens [09:52] davmor2, checking [09:53] greyback, last frame sometimes is not rendered [09:53] greyback, see [09:53] https://bugs.launchpad.net/unity8/+bug/1353374 [09:53] Launchpad bug 1353374 in Unity 8 "Regression - swipe to right does not switch back to app scope" [Undecided,New] [09:53] Saviq: I'm just digging back to find when it landed [09:53] davmor2, I can tell you [09:53] Cimi: ack [09:53] davmor2, yesterday, dash as app [09:53] 90% that's the reason [09:54] commented [09:54] Saviq: possibly I was already on a fresh image when the silo was added so it wasn't tested till I needed to do a fresh install for silo testing today [09:57] davmor2, yeah, probably [10:01] tsdgeos, v4 http://pastebin.ubuntu.com/7969014/ [10:01] ah damn [10:01] he added refresh [10:01] give me 4 sec [10:03] Saviq, can you add affected project to the bug? [10:04] Saviq: did a blind commit that should fix it, rebuilding everything to prove it now [10:05] tsdgeos, let me [10:05] tsdgeos, you please have a look and understand https://ci-train.ubuntu.com/job/landing-001-1-build/164/console [10:05] no idea how DashNavigation.qml could conflict... [10:05] yes yes [10:06] makes sense [10:06] it's a renamed file [10:06] ah ok [10:06] probably conflicts with https://code.launchpad.net/~aacid/unity8/dashSepartmentsListTweaks [10:06] tsdgeos, please confirm and rebase if that's the case [10:06] yes [10:06] tsdgeos, +1 on v4 [10:09] Saviq: so it conflicts with ../dashSepartmentsListTweaks/ and unstable_departments_test [10:09] i'll merge them [10:10] and resubmit? [10:12] tsdgeos, +1 [10:12] Saviq: but resubmit based on which of the two? :D [10:13] tsdgeos, v4 after [10:13] tsdgeos, ah with both [10:13] tsdgeos, you need to chain them... [10:13] meh [10:13] ok [10:18] Saviq: ok, so you need to re-review https://code.launchpad.net/~aacid/unity8/unstable_departments_test/+merge/229757 and https://code.launchpad.net/~unity-team/unity8/scopes-v4/+merge/229758 [10:19] tsdgeos, testDash fails in v4 btw [10:19] hmmm [10:19] does it? [10:19] tsdgeos, it needs updates for visible vs. not [10:20] damnit [10:20] will fix it in a sec, let me finish fixing test_narrow_delegate_ranges_expand in seelessstuck [10:20] don't want to forget the fix in my head [10:22] sure [10:24] Cimi, https://code.launchpad.net/~saviq/unity8/fix-pageheader-colours/+merge/229698 ? [10:28] Saviq, testing [10:29] Saviq, do I need something updated? [10:29] Cimi, uitk [10:29] Cannot assign to non-existent property "foregroundColor" [10:29] ok [10:44] Saviq, can you reproduce the issue of the app scope on other devices not mako? [10:45] Cimi, let me see [10:45] Cimi, yeah [10:45] ok [10:48] tsdgeos, we need to bump dep to -4 [10:48] why [10:48] tsdgeos, since otherwise we'll get the plugin with old API [10:48] there's renames and such [10:48] if there's only additions is when we can not bump straight away [10:49] right [10:50] mhr3: Saviq: pushed === Estilanda_ is now known as Estilanda [10:51] cardcreator fails in master [10:51] trunk [10:51] somename [10:51] someday we'll get the CI to pickup its failiures [10:51] D: [10:51] tsdgeos, yeah, when fginther's back [10:51] i'll prepare a fix [10:51] tsdgeos, but wtf, it doesn't report a failure in the .xml for me [10:52] or my grep is wrong [10:52] or does it even run on qmltests? [10:52] it does here [10:52] ah it doesn't run with qmltests [10:53] qmluitests [10:53] stupid ubuntu pastebin doesn't let me paste it [10:53] error http://paste.kde.org/pnedmybtd === MacSlow is now known as MacSlow|lunch [10:56] Saviq: https://code.launchpad.net/~aacid/unity8/cardCreatorTest7Update/+merge/229763 [10:58] Saviq, tested [10:58] Saviq, works better in app scope, (make tryDash), shows issues maybe in the tests on Ubuntu scope [11:00] dednick: sorry, I'm in a long hangout [11:00] tsdgeos, maybe tweak the cmake module to name the tests testFooFoo.xml instead, this way they should get picked up already? [11:01] Saviq: ok, let me try that [11:02] tsdgeos, huh, testPreview segfault :| [11:02] wow [11:02] tht's new [11:02] which branch? [11:02] or the silo? [11:02] tsdgeos, v4 [11:02] weird i just ran them all and worked, let me try again [11:02] not sure not there in trunk, though [11:02] tsdgeos, yeah, most probably intermittent, I'm running them parallel [11:02] besides v4 should not be changing previews at all [11:03] indeed === Malsasa_ is now known as Malsasa [11:05] Saviq: i looped it, no crash in like 100 runs :S [11:06] it's a very fast test :D [11:06] tsdgeos, https://code.launchpad.net/~aacid/unity8/cardCreatorTest7Update/+merge/229763 [11:07] Cimi: er what? [11:07] tsdgeos, yeah, good [11:07] tsdgeos, https://code.launchpad.net/~aacid/unity8/cardCreatorTest7Update/+merge/229763/comments/557582 [11:08] yes [11:08] why are you reverting the code instead of fixing the test? [11:08] tsdgeos, that branch should get merged today [11:09] tsdgeos, tests pass on it [11:09] sure [11:09] tsdgeos, and the code is better [11:09] you reverted good code [11:09] no [11:09] code is worse [11:09] tsdgeos, we should not look for cardData anymore [11:09] 25 - model: cardData && cardData["attributes"] || undefined; \n\ [11:09] 26 + model: cardData["attributes"]; \n\ [11:09] tsdgeos, there's no need to check for cardData["attributes"], it will always be an object [11:09] tsdgeos, saviq told me cardData and cardData attributes is defined [11:09] yep [11:10] Saviq: i'm mostly sure it was there because it created zillions of warnings [11:10] but ok, whatever [11:10] tsdgeos, it did, way before [11:10] tsdgeos, in the past [11:10] tsdgeos, now that code won't even be used if attributes are not declared [11:10] tsdgeos, i.e. it won't even be there [11:10] * Saviq needs a break, /food [11:12] Saviq, https://code.launchpad.net/~saviq/unity8/fix-pageheader-colours/+merge/229698 approved [11:12] tx [11:27] mhr3, tsdgeos, FWIW I'd rather it be a 2-value enum, but won't press [11:27] Saviq, the interface is quite internal, we can change it anytime we want [11:28] holas [11:28] o/ [11:28] mhr3, I know, it's just a future thing [11:28] Saviq, fwiw i'd expect the second nav to go away completely before design comes up with a 3rd nav :P [11:29] ;) [11:29] tsdgeos, didn't push qmltypes did you? [11:29] /reallyfood [11:30] Saviq: now [11:30] sorry [11:35] Saviq: know what happened to the app scope on desktop shell run? [11:37] Saviq: nevermind. just pulled latest [11:54] dednick_, yeah, it's a separate window now [12:02] Saviq, Had problem building qtmir-gles in silo-7 (due to mircommon packaging error). I removed mircommon from debian/control and started another build but it seems to not really build from scratch, but just report the previous failure. See the last couple of builds at https://ci-train.ubuntu.com/job/landing-007-1-build ... Have you seen this before? [12:02] camako, you can't rebuild in PPAs with the same version, you need to bump the ubuntu? part in the version [12:03] camako, it never really uploaded to the PPA, it rejected it because there is the same version there already [12:04] Saviq, I see... Do i just manually increase it? or do I use dch somehow? [12:04] camako, you can use the same dch -v command [12:04] camako, just use ubuntu2 instead of ubuntu1 [12:04] ok [12:04] camako, or just dch -i "" [12:04] thanks Saviq [12:04] should do the same [12:04] camako, you can always see whether something actually happened in the PPA in https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/landing-007/+sourcepub/4336117/+listing-archive-extra [12:05] camako, there is truth that the train won't let you know that the upload failed, 'cause it's an async thing - you get an email back from PPA [12:06] or well [12:06] not you but PS Jenkins Bot [12:06] Saviq, is there a way to get to that link from the silo? [12:06] camako, from https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/landing-007 [12:06] camako, you can already see that qtmir-gles was published 1 hour ago [12:07] * camako looks [12:07] camako, and clicking at "View package details" you can get more [12:07] camako, and clicking on arches there you get more info about the builds and suhc [12:10] Saviq, I see.. Why are other packages reported to have failed? They all succeeded (except for qtmir-gles).. [12:10] on the right margin of https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/landing-007 [12:10] camako, they failed on some architectures, which is expected [12:11] ah I see [12:11] camako, basically those packages don't explicitly say which arches they build on [12:11] camako, and they fail to build for arches where mir (or some other dependency) are unavailable [12:11] camako, but that's a warning rather than an error [12:11] Right... Thanks Saviq... That really helps.. [12:11] camako, bug #1353451 btw, if you can mark it confirmed :) [12:11] bug 1353451 in Canonical Upstream To Distro "build job in train should check whether built version is higher than in silo" [Undecided,New] https://launchpad.net/bugs/1353451 [12:12] * camako looks [12:13] Saviq, done! [12:13] thanks :) [12:25] Saviq, how about emblems branch? [12:25] Cimi, did you get a review yet? ;) [12:25] Saviq, no [12:26] Cimi, then you should probably pester someone to get it [12:26] Saviq, https://code.launchpad.net/~cimi/unity8/card_emblems_2/+merge/229046 [12:26] :D [12:27] Cimi, 'fraid there's a queue :P [12:27] Saviq: what is 'overlay color' [12:28] oh nm [12:28] cwayne, https://f966f709-a-c881af26-s-sites.googlegroups.com/a/canonical.com/unity8dash/scopes/dash-overview/01_Dash_scopes_nav_full_dark_v7.jpg?attachauth=ANoY7cptkkW9cy57jSgVaA-CdCSQJ_lN1VLPcjiBV67pFaWz6FIdyM_V15vfNg_kr73C39O71IInJ8II9Zin3XV5sYe3VZ2yfEGy-OetN2lpfzDlXpGkQ8uFCM5yGcgKAehj0sbtY_1_T_lZDbUMKpJLmO8UPuWjxC8ytAtj6m5aUspGWMFWkNsTK1dfpw0TW45d0aQ2dqiFUJ5XUpUjw5Tc7NaOrTAggtjgeyTLZeRH91OLGUfed0NF-APSpX70z_1g8qULJY7UHpMR2gA01BchUDn3RTFRXQ%3D%3D&at [12:28] tredirects=0 [12:28] ah right [12:28] cool [12:42] Cimi, it's not really a FIXME [12:42] Cimi, because that's the desire, to have this hardcoded [12:43] Saviq, so we're not gonna change this? [12:44] Cimi, no [12:44] Cimi, I know, this comment should probably just begone === _salem is now known as salem_ [12:52] mterry, can I ask you https://code.launchpad.net/~saviq/unity8/fix-edges-demo/+merge/229778 [12:52] Saviq, yup [12:53] thanks [12:53] mterry, FWIW, I can't make edges demo work on desktop unless I modify EdgeDemo.qml, that expected? [12:54] Saviq, ... no? How do you modify the file? [12:54] Saviq, oh! [12:55] mterry, nvm how I modify the file, how do I get the mock AccountsService to return the real value? ;) [12:55] Saviq, with the mocks? Yeah, they disable the edge demo (for the same reason we disabled password prompting when testing -- it's really annoying) [12:55] mterry, yeah, ok, so expected [12:55] Saviq, if I want to test the edges specifically, I edit the mock. We probably should have a nicer way, yes [12:55] mterry, is fine [12:57] mterry, https://code.launchpad.net/~aacid/unity8/collapsingPreviews/+merge/228857/comments/557634 btw [12:58] tsdgeos, ^ [12:58] mhr3: yeah i fixed what you askd, no? [12:58] mterry, he knows [12:58] k [12:59] er [12:59] mhr3 -> mterry in ↑↑↑ [13:01] tsdgeos, mhr3, having a look at the current dash... I'm starting to feel like we're going to drop see all/see less if stuff goes in that direction [13:01] (which I'd be quite happy with TBH) [13:01] Saviq, what direction? [13:01] mhr3, everything opening in a new scope page [13:02]