=== salem_ is now known as _salem === chriadam|away is now known as chriadam [01:31] how am I supposed install the ubuntu touch emulator? === jkeyes0_ is now known as jkeyes0 === _salem is now known as salem_ === salem_ is now known as _salem === tsegat is now known as tsega === tsega is now known as tsegat === tsegat is now known as tsega === chriadam is now known as chriadam|away [07:10] good morning [07:17] dpm: hi [07:18] morning mzanetti [07:18] dpm: FYI: also the accounts plugin seems broken [07:18] dpm: can't log in to reminders any more [07:18] dpm: I quickly debugged it, and the Accounts {} item always says 0 accounts, even though there is one in system settings [07:18] mzanetti, argh, with which image? [07:19] latest promoted [07:19] which one is that? I'm on #28 [07:19] dpm: I tend to think its either accounts or the evernote accounts plugin [07:19] dpm: let me check [07:19] on #28 it works, but I'm not sure I'm using the latest [07:20] dpm: Its 30 or 31 [07:21] good morning all :-) [07:21] mardy, any ideas of known issues with online accounts on image #30 or #31? ^^ [07:21] it seems Reminders is no longer being able to access accounts [07:21] or any API changes? [07:21] dpm: given that the Ubuntu one account still works, its most likely the evernote-plugin [07:22] ok, let me upgrade the image to check myself [07:22] dpm, mzanetti: my fault, I forgot about reminders app -- I need to make a branch and propose that for merging [07:23] dpm: ah cool if you know the solution already :) [07:23] dpm, mzanetti: no API changes, but a big behavioural change: in order to be able to use an account, the app first has to request access to it [07:23] err. mardy ^^ [07:23] mardy, would it be possible to do that today? Evernote are reviewing the app with promoted images, and if they use the latest, they'll find out that the app is broken [07:23] ah, I see [07:23] dpm, mzanetti: I did that for friends-app: https://code.launchpad.net/~mardy/friends-app/app-access2/+merge/211717 [07:24] for reminders app the diff should be smaller, I'll start working on it immediately [07:24] awesome, thanks mardy [07:24] mzanetti, dpm: FYI, existing accounts will continue to work; it's the new accounts, who won't work [07:25] ack, thanks [07:25] mardy: yep. that's why it took us so long to figure its broken :) [07:28] mzanetti, regarding the issues with the new header (blank toolbars, Reminders tab page content behind the header), what do you think we should do? Do you know if there is a quick workaround we can use to still use the toolbars, or should we migrate to the new headers immediately? [07:28] dpm: I still haven't tried [07:28] mzanetti, I'm thinking of the quickest solution for Evernote not to end up with an app that they can't use [07:28] ok [07:28] dpm: . ok. will take a look today [07:29] t1mp, do you have any suggestions on what we can do? I think you looked at Reminders yesterday too ^ [07:29] thanks mzanetti [07:53] elopio, around? [07:53] dpm: o/ [07:54] hey! :) [07:54] elopio, thanks for the quick response. So I'm happy to approve your branch and it seems to make Jenkins happy too. [07:55] elopio, is that all that will be needed to unblock landings? [07:55] dpm: yes, your branch should be good now. [07:56] mine and carla's still miss thrift. [07:57] Good morning all; happy Friday, and happy Endangered Species Day! :-D [07:57] JamesTait: you should have said happy ubuntu naming day :p [07:57] elopio, yes. Let's have a look at it after we get the app itself back in shape. For now the first step is to get CI working again to land the (unrelated, but critical) Online Accounts fix [07:57] or ubuntu anymal day [07:58] *animal [07:58] I was just thinking if I were an endangered species I probably wouldn't be very happy.... [08:01] hahaha :p [08:01] I would, they would encourage me to have sex all the time :D [08:02] Well that's a different way of looking at it! :) [08:03] always look on the bright side of life :D [09:02] dpm, mzanetti: https://code.launchpad.net/~mardy/reminders-app/app-access2/+merge/219798 [09:02] oh, I forgot to translate a string, adding one commit now... [09:06] dpm: you mean w.r.t. the account problem that I had? For that I have no idea [09:07] mardy, awesome, thanks! So a few questions: 1) Does this work at all on the desktop? 2) Does this add the functionality of going back to the app after having set up the account in System Settings? [09:07] t1mp, what account problem? [09:09] dpm: for reminders, I couldn't use it on the phone. It always said I needed an account, but I made an evernote sandbox account and configured it on the phone [09:09] mzanetti and rpadovani confirmed it yesterday and were looking into it [09:11] t1mp, ah, yes, we discussed it this morning and it was due to a change of behaviour with online accounts. Could you try this branch? It should fix the issue: https://code.launchpad.net/~mardy/reminders-app/app-access2/+merge/219798 [09:11] dpm: I tested it on the desktop only, as I wrote in the MP description [09:12] dpm: the UI flow on unity8 is not perfect yet, that will come when bug 1230091 is fixed [09:12] bug 1230091 in unity-mir "[enhancement] Trusted Session surface management (required for appstore app trust model), modal subwindows" [High,Triaged] https://launchpad.net/bugs/1230091 [09:13] mardy, yes, I know, but from the description I cannot figure out whether it loads online accounts on the desktop or not [09:13] I'm building the branch now, but I wanted to have some context on what to expect [09:13] dpm: ah, well, on the desktop it works if you have ubuntu-system-settings-online-accounts installed [09:14] dpm: actually, I forgot to add a dependency in debian/control [09:14] yes, that's exactly what I was going to ask :) [09:15] dpm: is that an acceptable dependency? Do you expect people to be running reminders-app on the unity7 desktop (beside developers)? [09:15] mardy, I very much so expect people to be running any core apps to test convergence, not only developers, but early adopters [09:16] how many extra deps does it pull? The whole of unity? [09:16] 8 [09:16] dpm: no, just a few online-accounts related stuff [09:18] ah, cool [09:18] dpm: I just pushed one more commit to add the dependency [09:18] mardy, thanks. And about question 2) ? [09:53] dpm: what exactly is the issue with the empty toolbars? I just built latest trunk and it seems fine [09:54] mzanetti, I cannot longer reproduce it on #32. And also I cannot reproduce the content behind header, which is good [09:54] t1mp: hey it seems you fixed the tabBarStyle warnings at https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/fix-warning-TabBarStyle/+merge/219687 .. nice :) [09:54] dpm: I think the content behind the header was fixed with the new design branch. We enabled clipping on the listview [09:55] zsombi: Hi, Does your layouts-visibility branch have the enabled fix we talked about before? [09:56] mzanetti, the issue was still there using trunk and image #28, but it only affected the reminders tab. In any case, it seems it's fine now on image #32 [10:04] nik90_: yes. I don't know why the warning started to appear, but it became annoying [10:13] did ItemSelector change API? [10:14] It looks like it used to have a selectedIndex [10:14] but now that doesn't exist? [10:14] no, hang on [10:31] nik90_: are you working on weather-app? [10:31] t1mp: nope..me the clock guy [10:31] nik90_: I saw your name here ;) http://bazaar.launchpad.net/~ubuntu-weather-dev/ubuntu-weather-app/trunk/changes [10:32] t1mp: :) .. m-b-o is the main weather app guy [10:32] nik90_: ok, thanks [10:32] m-b-o: hello [10:33] t1mp: hi [10:35] I updated the ubuntu-ui-toolkit recently to fix this bug - https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1317902 [10:35] Ubuntu bug 1317902 in Ubuntu UI Toolkit "[pageStack] PageStack shows the title of a page not pushed into the stack" [Critical,Fix committed] [10:35] m-b-o: but my fix breaks weather-app [10:35] m-b-o: I don't see a way to change the toolkit to have that bug fixed and to make weather app work without changes [10:35] m-b-o: luckily, there is a very easy change in weather-app that makes it all work :) [10:36] t1mp: tell! :) [10:36] m-b-o: give me a few seconds, I'll push the change to lp [10:38] m-b-o: https://code.launchpad.net/~tpeeters/ubuntu-weather-app/no-tabs-in-page/+merge/219808 [10:38] m-b-o: is there a reason you need a Page there? [10:40] t1mp: can't remember, tbh... [10:41] maybe the structure used to be different and it was needed before [10:41] the app seems to work fine for me now with the change :) [10:41] t1mp: Yes, it relats to Tabs inside. There's still a workaround for creating the Tabs dynamically [10:42] t1mp: I guess there's now a way to add and remove tabs, isn't? Last time I checked the docs I found nothing about, saw just Bugs about [10:44] m-b-o: but now, your tabs code works without that Page right? I mean with the Item in my MR it works [10:44] m-b-o: no, unfortunately not. We had dynamic tabs, but it turned out that it broke a lot of other stuff, so we removed it again [10:45] m-b-o: dynamic tabs are still planned, but postponed until we finish some other urgent features.. unfortunately [10:46] t1mp: ok, tabs are working, so I can wait! :) [10:46] t1mp: the label inTab header sticks not to left, when newly added [10:46] t1mp: when adding two new locations [10:47] m-b-o: what do you mean? [10:47] t1mp: is it related to "file:///usr/lib/x86_64-linux-gnu/qt5/qml/Ubuntu/Components/Themes/Ambiance/TabBarStyle.qml:120: TypeError: Cannot read property of null" [10:47] t1mp: will make screenshot [10:48] m-b-o: that's a warning from the toolkit that I fixed, it is in our staging but still needs to land [10:48] ah, ok [10:48] t1mp: http://imgur.com/fLpGwPb [10:49] (12:55:03 PM) nik90_: zsombi: Hi, Does your layouts-visibility branch have the enabled fix we talked about before? [10:49] t1mp: had the same problem, as I once tried to build the tabs with Repeater [10:49] m-b-o: weird. I added amsterdam and bejing, and it worked fine [10:49] nik90_: it fixes the bugs which are attached to the branch.... I guess we were talking about those ones... [10:49] m-b-o: is that tabs thing a new bug with my change only? === MacSlow is now known as MacSlow|lunch [10:50] t1mp: I can't say, but I would guess rather not [10:50] m-b-o: adding useDeprecatedToolbar: false to your MainView fixes it for me ;) [10:51] zsombi: yeah..but at the end of the day, during my testing I informed you that in the conditional layout, the touch inputs are disabled..you said you fixed it but did not push it to launchpad since you were working on a test..I was referring to that. [10:51] zsombi: but I guess they are in launchpad now [10:51] but I need to do something about the color of the icons on a non-white background [10:51] t1mp: hehe, you're right! :) [10:51] t1mp: +100 ^^ [10:51] nik90_: ah, sorry, the visibility-related MR is ready for review! :) [10:51] kalikiana_: ^ [10:52] nik90_: kalikiana_: https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/layouts-visibility/+merge/218916 [10:53] looking [10:53] m-b-o: can we get this change in the weather-app? https://code.launchpad.net/~tpeeters/ubuntu-weather-app/no-tabs-in-page/+merge/219808 [10:53] m-b-o: we like to land a new ubuntu-ui-toolkit without breaking weather-app :) [10:53] mardy, so I'm testing your branch and it works well on the device, but it fails on the desktop: [10:53] file:///tmp/build-app-access2-Desktop-Default/src/app/qml/reminders.qml:326:17: Cannot assign to non-existent property "applicationId" [10:53] applicationId: "com.ubuntu.reminders_reminders" [10:53] ^ [10:53] t1mp: yes, tests are running :) [10:54] mardy, although it seems I've got /usr/share/accounts/applications/com.ubuntu.reminders_reminders.application installed [10:54] m-b-o: great, thanks. [10:54] hey m-b-o! How are you doing? :-) [10:54] m-b-o: let me know if it works, or if something breaks [10:54] t1mp: approved! [10:55] dpm: I'm fine! [10:55] dpm: :) [10:56] m-b-o: awesome :) that was fast! [10:56] m-b-o, nice. Let us know how you're getting on with your Go adventure ;) [10:58] dpm: I will. The thing works on shell, but have to make an ui now. PLaying songs from last.fm with spotify tracks.... [10:59] m-b-o: how is the weather-app project configured? will it now automatically land in trunk and then in an image? or the other way around? [10:59] t1mp: I can't say. but popey will know [11:00] nice! m-b-o, I see in the end we'll have to recruit you for the music app :) Let the SDK guys know how you're doing and if you're encountering any blockers. I'm sure they'll appreciate any feedback on Go apps [11:00] popey: ^ do I need to ask you to land a weather-app update? [11:00] t1mp: no, popey or balloons has to push the trunk to the store. Only then does it land in the image.. they are the gatekeepers [11:00] t1mp: once something lands in trunk we then decide what to land on the image. [11:01] t1mp: which is bascially us pulling a .click from jenkins, then QA (ap tests on device) it and push to the store [11:01] t1mp: just ping us if there's a specific rev you want landing in the store and we can make it happen [11:01] nik90_: hmm... seems the MR fixes the bug #1298886 as well... [11:01] bug 1298886 in Ubuntu UI Toolkit "Layout incorrectly sizes the children in the default layout when using Components{}" [Critical,In progress] https://launchpad.net/bugs/1298886 [11:02] popey: I need this https://code.launchpad.net/~tpeeters/ubuntu-weather-app/no-tabs-in-page/+merge/219808 in the image before we can proceed with our current ubuntu-ui-toolkit landing [11:03] popey: I made changes in uitk that break the weather-app otherwise. And the uitk changes fix some bugs so I like to avoid reverting them === seb128_ is now known as seb128 [11:03] dpm: I will. I'm a noob regarding to Go, so I guess to get it built will be the major issue. [11:03] ok :) [11:04] t1mp: ok, as soon as that lands in trunk I'll pull a click and push to the store [11:04] m-b-o, do ask around here if you encounter any issues and someone can jump in to help [11:05] popey: great, thanks :) [11:05] popey: can you let me know when it happens? [11:05] of course t1mp [11:05] it failed, can you set a commit message please [11:07] zsombi_: ah yeah I remember the bug now [11:07] zsombi_: testing now [11:08] t1mp: ^ [11:08] ok [11:09] popey: done [11:09] m-b-o: can you top-approve https://code.launchpad.net/~tpeeters/ubuntu-weather-app/no-tabs-in-page/+merge/219808 again? I set a commit message [11:09] zsombi_: yup it seems to fix it. [11:10] nik90_: the Layouts take int oaccount those items from the default layout, which were created at the time the Layouts got completed. Other components which are marked to be layd out but loaded after that point will be omitted. [11:10] nik90_: so in case we need such a functionality, we have to add that [11:11] nik90_: yeah, I thought so... so I can link the bug to the same MR then [11:11] zsombi_> nik90_: the Layouts take int oaccount those items from the default layout, which were created at the time the Layouts got completed. Other components which are marked to be layd out but loaded after that point will be omitted [11:11] zsombi_: hmm would this be a common use case? [11:11] zsombi_: I am trying to think of scenarios where this might be hit by app devs [11:12] nik90_: I can imagine that there can be such a case: use Loader to load some components later depending on some input [11:13] nik90_: if those components loaded by the Loader are in the default layout, and those contain items marked to be layd out, I'd say it woudl be expected to work. ATM it won't, because the laid out items are only collected on Layouts default completion [11:14] zsombi_: ok [11:14] zsombi_: I will create a bug report about this just to ensure we dont forget it. [11:15] nik90_: good, and I'll move the other one to teh layouts-visibility MR [11:15] zsombi_: ok [11:21] zsombi_: https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1320190 [11:21] Ubuntu bug 1320190 in Ubuntu UI Toolkit "Layouts does not display elements that were loaded after the the creation of the layouts" [Undecided,New] [11:22] zsombi_: feel free to add more information in case I missed something [11:23] nik90_: thx, I made it a wishlist [11:24] nik90_: yet it is not so important, getting it working is not a big deal, especially now that we simplified the default layout handling [11:24] t1mp popey: there have to land another mp first [11:24] zsombi_: true, but I imagine you have other high priority stuff in your plate [11:25] t1mp popey: https://code.launchpad.net/~martin-borho/ubuntu-weather-app/fix-1318676-flaky-tests/+merge/219329 [11:25] nik90_: like always :D [11:25] which works, but can be made better [11:25] zsombi_: :D [11:25] m-b-o: t1mp failed 91.189.93.70:8080/job/generic-mediumtests-trusty-weather-app/111/? [11:25] m-b-o: ohh :( do you know when that will happen? [11:26] m-b-o: ^ are those failures because of the flaky tests that you are fixing in the other MR? [11:26] t1mp: yes. [11:26] kalikiana_: btw you should join https://plus.google.com/communities/111350780270925540549 to see app devs using the SDK and your work. [11:27] t1mp: the mp fixes the tests. I think it can be approved. And I will refactor it the next days. [11:31] nik90_: I'm already in there :-D though g+ being g+ whether I see the good posts in all the flood is up to chance :-P [11:32] kalikiana_: lol :D [11:34] mzanetti, rpadovani is https://code.launchpad.net/~rpadovani/reminders-app/setDefaultFontSize/+merge/219092 ready to land? [11:34] dpm: no [11:35] dpm: while it fixes on issue, it doesn't fix all of them and introduces a new one [11:35] dpm: we agreed that I'll have a look and while fiddling around I decided to do the transition to the new WebView first [11:35] m-b-o: okay. When do you think the fixes will land? [11:36] mzanetti, ok, so do you think we should mark it as Rejected or Work in progress until the new WebView lands? [11:36] dpm: yes. probably even reject it [11:37] t1mp popey has a look at on it. next 30-60 minutes. will make a bug report for the refactoring, so I think this would be ok for qa [11:37] ok [11:43] zsombi_: the tests *Visibility are somewhat cluttered… I wonder if there's a way to simplify it. like by using _data. though [11:44] m-b-o: approved https://code.launchpad.net/~martin-borho/ubuntu-weather-app/fix-1318676-flaky-tests/+merge/219329 [11:44] kalikiana_: I could split them in smaller ones, and test one thing at a time, like is the parent the proper one, is the size preserved, is the visible property of one restored, etc... [11:44] kalikiana_: but we cannot really use the data there :( [11:46] popey thanks! will you top-approve it? [11:47] m-b-o: done [11:47] popey thanks, t1mp wil be pleased :) [11:52] zsombi_: hmm you may be right. though the "blocks" of tests all are very similar that's why I thought of data [11:52] kalikiana_: well.... in a sense yes, as the only difference is the layout that is activated... [11:54] zsombi_: and even the qml is the same for the nested tests [11:54] popey, are you around? [11:54] renato: for you, always [11:54] :D [11:54] t1mp: the fix for the tests is merged now [11:55] popey, could you test the CalManger branch with qtpim package from here: https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-proper/+packages [11:55] kalikiana_: ok, so eventually I can merge the tests with a data then..., will do an update on it [11:55] renato: yup! [11:56] renato: https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-proper/+files/qtpim5-dev_5.0~git20140515~29475884-0ubuntu1~test7_armhf.deb [11:56] oops [11:56] renato: which deb? [11:57] the dbg one? https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-proper/+files/qtpim5-dbg_5.0~git20140515~29475884-0ubuntu1~test7_armhf.deb [11:57] the best is add the ppa and do a upgrade [11:57] ok [11:58] zsombi_: appreciated! btw nice to see you using UbuntuTestCase more [11:59] kalikiana_: I made a promise to myself that whenever I have to touch the test cases I will use it! [12:00] m-b-o: ok. :) will you top-approve this one again? https://code.launchpad.net/~tpeeters/ubuntu-weather-app/no-tabs-in-page/+merge/219808 [12:18] renato: left a comment on https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/CalManagement/+merge/213355 [12:19] tl;dr: no crashes, very slow. [12:20] popey, yes calendar app is using a lot of "sync" functions and querying for events unnecessary [12:22] right [12:23] would be good to get the calendar management branch landed, then work on the performance === MacSlow|lunch is now known as MacSlow [12:55] dpm: do you already have a PPA for reminders, or do you prefer me to create one for the OA dependencies? [12:56] rpadovani, mzanetti, so I've had a chat with mardy, and it seems that https://code.launchpad.net/~mardy/reminders-app/app-access2/+merge/219798 introduces a new dependency on an utopic package [12:57] and this is bad or good? :-) [12:57] mardy, we've got the https://launchpad.net/~ubuntu-touch-coreapps-drivers/+archive/daily PPA that we use for that purpose. I can give you upload permissions if you want to upload directly there. [12:57] rpadovani, it means after that branch lands you'll need an utopic desktop to develop and run reminders [12:58] so we're thinking of backporting and putting those dependencies in the core apps PPA [12:58] so that you can install them from 14.04 and continue developing without having to upgrade to utopic [12:59] dpm, this sounds good, I don't want to have to partition to develop (and on Utopic there isn't steam) :-) [13:00] mardy, ok, you should now have permissions to upload to the core apps PPA [13:01] dpm: OK, will now try :-) [13:01] awesome, thanks :) [13:46] mardy, excellent! That's fine, I only need the amd64 version to test it works on the desktop [13:46] let me give it a go [14:01] Morning everyone [14:02] I've come across a minor issue that I hope someone here can help me with [14:03] Somkit: feel free to explain [14:06] Great! My title bar says qmlscene: before the actual title of the program [14:07] I have narrowed it down to the use of "id" in MainView, but removing id breaks my program [14:09] Its not a large issue, but its easy to replicate, if you add "id:" to any UbuntuSDK genterated qml app it wil change the title to qmlscene: [14:09] mardy, the backport works really well, thank you! [14:11] Sorry, if you add "id:" under MainView...it causes the issue. Otherwise it works as expected. [14:11] dpm: cool! :-) [14:19] fginther, on Reminders for 14.04, we've had to backport some Online Accounts packages to the core apps PPA for the latest branch to work on the desktop. I'm guessing Jenkins will just pick them up, as it's using the core apps PPA already? Or is there anything that needs to be changed in the jobs' config? See more details on my last comment on https://code.launchpad.net/~mardy/reminders-app/app-access2/+merge/219798 [14:22] dpm, yes, jenkins is already configured to use that PPA, but looking at the last test failure from that MP, it looks like more may be missing [14:23] dpm, is this the package that's been backported? qtdeclarative5-online-accounts-client0.1 [14:23] dpm, ahh, I see it now [14:23] fginther, yep, that one and the ubuntu-system-settings-online-accounts [14:23] but the Jenkins failure seems to be unrelated? [14:24] "Autopilot Package Version: 1.4+14.04.20140416-0ubuntu1 [14:24] could not import package reminders: No module named 'reminders'" [14:24] I think that was fixed in the tests in a separate MP that landed afterwards [14:24] dpm, that's the result of it not failing on the earlier install failure [14:25] ah, ok [14:25] dpm, it justed passed the trusty test [14:25] nice! [14:30] ok popey, balloons, can we set the machinery in motion to upload the latest from Reminders trunk (r143) to the store? [14:31] dpm: ok, just triggered a build of r143 in jenkins [14:32] \o/ [14:32] Somkit: take a look at some of the core apps like calculator / calendar - they did this too [14:35] @popey will do [14:36] lol, don't remember the irc commands [14:36] ☻ [14:39] I'll upload popey [14:40] balloons: I'll test [14:40] I'll push weather too.. I saw the earlier conversation with martion [14:44] nope [14:44] uh, wrong channel [15:19] popey, only one filemanager MP to go \o/ :) http://people.canonical.com/~dpm/core-apps-sponsoring/ [15:31] woah! [15:31] thats a nice way to start the weekend! [15:31] under 20 [15:31] * popey sets his sights on nik90_ [15:33] * davmor2 hands nik90_ a bulletproof vest so popey can't shoot him [15:35] :) === gatox is now known as gatox_lunch [16:29] * nik90_ is grateful for the armour davmor2 === gatox_lunch is now known as gatox [16:53] popey, do you have a link to the Reminders click package generated by jenkins? [16:53] sure [16:54] dpm: http://popey.com/~alan/com.ubuntu.reminders_0.5.143_armhf.click [16:54] awesome, thanks [16:54] np === _salem is now known as salem_ [18:49] "Juju Charm School Vagrant Workflow" starting in 10 minutes https://www.youtube.com/watch?v=qLNPn2rQynM === cff is now known as CodePulsar === salem_ is now known as _salem