[00:16] daker: sure [00:20] mhall119: we can have it tomorrow, sleeping... [00:27] daker: sure, PM me any time === _salem is now known as salem_ === chihchun is now known as chihchun_afk === chihchun_afk is now known as chihchun === salem_ is now known as _salem === chriadam is now known as chriadam|away [07:15] good morning [07:15] good morning :) [07:56] morning mardy, could you give us a hand with bug 1351041 to help unblock image promotion? I've added more information in the comments, and I've noticed we're not getting the token. I did as you mentioned in your comment, but /var/log/syslog didn't show anything related to accounts, and I wasn't sure how to run dbus-monitor [07:56] bug 1351041 in Ubuntu Reminders app "App is unable to retrieve notes" [Critical,Triaged] https://launchpad.net/bugs/1351041 [08:03] dpm: I added a comment [08:03] dpm: what I find more suspicious is this: file:///usr/share/click/preinstalled/com.ubuntu.reminders/0.5.203/qml/reminders.qml:143: TypeError: Cannot read property 'HostName' of undefined [08:04] dpm: actually, that line happens if you get an access token [08:07] mardy, generally when we get an access token, it's output on the app's log, that's what made me think we're not getting the token. Should we be using dbus-monitor to see if it gets it? [08:08] dpm: I'm quite sure you get it: see http://bazaar.launchpad.net/~reminders-app-dev/reminders-app/trunk/view/head:/src/app/qml/reminders.qml#L143 [08:08] dpm: this is the error line, in the logs [08:08] yes, that's the line I'm looking on Qt Creator [08:09] dpm: try to print accountService.authData [08:09] dpm: you probably need to use JSON.stringify() [08:11] mardy, ok, will take me a while, I'm upgrading now to image 158 to reproduce the issue [08:15] mardy, something like this? -> http://pastebin.ubuntu.com/7968339/ [08:16] dpm: change line 15 with console.log('params: ' + JSON.stringify(accountService.authData.parameters)) [08:17] ok [08:23] dpm: thx for changing teh status of teh statesaver bug for trusty :) [08:23] s/teh/the [08:24] zsombi, np ;) [08:24] dpm: no idea why it was grayed out for me... [08:24] zsombi, I think you need to be a member of the bug squad to change some statuses of bugs in the distro [08:25] if they have been milestoned [08:25] dpm: aha... ok :) [08:30] mardy, ok, so it seems we're getting the token, but we're not able to read any params, that's the issue. See lines 31 and 32 [08:33] dpm: you probably forgot to link to the pastebin :-) [08:34] mardy, argh, it was the excitement about finding a hint for the cause of the bug :-) There you are: http://pastebin.ubuntu.com/7968430/ [08:35] dholbach: fyi, for core apps, we're treating them like any other app, so can be processed in the store like 3rd party apps. [08:35] popey, gotcha [08:35] dpm: weird. If the app can authenticate, it should also be able to read that setting [08:36] dpm: can you please paste the .provider and .service files (not from the source tree, but from the installed locations) [08:36] ok [08:36] dpm: also check if you have something in /usr/share/accounts vs. ~/.local/share/accounts [08:39] mardy, here are the contents of the two locations -> http://pastebin.ubuntu.com/7968500/ [08:42] dpm: OK. and what are the contents of the evernote files there? [08:42] mardy, and here are the contents of the .provider and .service files: http://pastebin.ubuntu.com/7968518/ [08:42] dpm: all looks fine [08:43] dpm: in the older image (where it works), are you using the same version of reminders app, or an older one? [08:43] mardy, the same version of reminders: 203 [08:43] weeeird [08:44] dpm: and if you change accountService.authData to just authData, does it behave differently? [08:50] mardy, unfortunately, same result, same log [08:51] dpm: anything interesting in the syslog? [08:52] mardy, I could not see anything related to accounts in the syslog, even after I set the logging to verbose for signond [08:52] but I can try again [08:53] dpm: no, no need === hatchetation_ is now known as hatchetation [08:54] dpm: I see that the authData can return an empty variant map if the underlying libaccounts's AccountService is NULL [08:54] dpm: and that's the only case [08:54] dpm: that could happen if you change the accountService object (for instance by changing its objectHandle property) [08:55] dpm: or if the object referred to by the objectHandle property dies [08:57] mardy, I don't think we do the first, but not sure if the later happens. In any case, if I understand it correctly this is something we have no control over in the app, do we? [08:57] dpm: can you please add some debug message on the onObjectHandleChanged inside AccountService? [08:58] dpm: you have only one evernote account, right? [09:00] mardy, yes, I only have one account. Here's what I'm going to try now, I've added the debug message on object handle changed: http://pastebin.ubuntu.com/7968611/ [09:00] dpm: nope, you need {} around that code [09:02] argh, typed too fast [09:05] mardy, no luck, it seems that that callback is not triggered: http://pastebin.ubuntu.com/7968658/ [09:07] dpm: I'm running out of ideas [09:07] mardy, can we add a bug task for the relevant project and leave it in your hands? I'm not sure what else we can do in the app itself. [09:17] dpm: yes, but how urgent is it? I've my hands full, and I have to prioritize another work, before bugfixing [09:20] mardy, right now this is blocking promotion and we're entering TRAINCON-0 from what I see on #ubuntu-ci-eng. I guess the priority depends on your other tasks and what the QA folks say [09:21] mardy, in any case, regardless of the priority, what would be the best project to add the bug task for? [09:24] dpm: accounts-qml-module [09:25] dpm: one more thing: can you please move the printing of authData inside the onObjectHandleChanged handler? [09:26] mardy, sure, but will that make a difference? It seems that the handler is not called [09:26] dpm: it is called once [09:26] dpm: or you wouldn't authenticate :-) [09:27] oh, then I missed it in the outpug [09:27] *output [09:27] dpm: line 18 of http://pastebin.ubuntu.com/7968658/ [09:27] ok, cool, yes, let me do that [09:31] Mirv: when you get a moment, could you please re-upload latest sudoku from jenkins to the store, current upload is borked. thanks! /cc dholbach [09:31] mardy, progress :) -> http://pastebin.ubuntu.com/7968816/ [09:36] popey: uploaded. weirdly, different behavior from before (too "quick"), so check also this one [09:36] or maybe something on the server side has just been made faster [09:36] dpm: well, then at least we have a workaround: you can do "EvernoteConnection.hostname = accountService.authData.parameters["HostName"];" right before calling authenticate() [09:37] thanks Mirv [09:39] mardy, awesome, thanks! [09:40] dpm: let's keep the bug open anyways, I'll have a look at it when I can [09:48] ok, thank you mardy, I've updated the bug description with the discussion this morning [09:48] hi rpadovani, popey, could you review/test https://code.launchpad.net/~dpm/reminders-app/workaround-for-1351041/+merge/229753 ? [09:49] * popey looks [09:49] * rpadovani is on i [09:49] *it [09:49] dpm: got a click? [09:50] yes, let me upload it [09:51] thanks [09:54] popey, http://people.canonical.com/~dpm/click/com.ubuntu.reminders_0.5.204_armhf.click [09:57] ta [09:58] dpm: yup, that works [09:59] dpm: \o/ we might have notes again \o/ [10:00] excellent :) [10:04] dpm, just to confirm, works also for me :-) [10:05] ship it! [10:05] rpadovani, popey, in this case, could you guys approve/top-approve? [10:05] i have approved [10:05] rpadovani: can you top-approve pls? [10:05] and popey, start the process to publish the click package from Jenkins? [10:06] yup, doing as soon as it merges [10:06] done [10:06] * dpm high-fives rpadovani and popey [10:06] \o/ [10:06] :-) [10:14] popey: ping [10:14] yo [10:16] popey: how can we upgrade Calendar to qt 5.3 , as per comment https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1351024 [10:16] Launchpad bug 1351024 in Ubuntu Calendar App "Date & Time picker is not working on device." [Critical,Triaged] [10:16] mihir: the device _is_ on 5.3 [10:17] meaning we ship with qt5.3 on the image, and it's in the utopic archive. [10:17] popey: okay ,so what should we conclude from comments , should we use DatePicker instead of panel [10:17] ? [10:20] mihir: you running utopic? [10:20] not now , but yes i do have on VM. [10:20] so on utopic do you see the date picker? [10:21] i need to check that. [10:22] so if it does , then it should work on device that's what you meant to say ? [10:24] well, trying to figure out where the issue lies [10:24] t1mp: did you unassign yourself because you dont believe uitk to be at fault for date picker? [10:26] popey: he unassigned himself because he is already assigned to https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1349366 which is the bug he linked to which causes the issue I guess [10:26] Launchpad bug 1349366 in Ubuntu UI Toolkit "[PageStack] Pages does not get correct anchored in the PageStack area" [Critical,Confirmed] [10:27] ah, so he did [10:27] mihir: ^ ☻ [10:27] popey: yeah , i did look at that now, will try tonight and then revert back [10:38] popey, davmor2 I unassigned myself because I convinced zsombi to take it (I won't be around for long today) [10:39] popey: zsombi has the bug now in the uitk project, but I cannot assign people in uitk (ubuntu) [10:39] zsombi: ^ https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1351024 [10:39] Launchpad bug 1351024 in Ubuntu Calendar App "Date & Time picker is not working on device." [Critical,Triaged] [10:39] davmor2: the other bug that I linked, I thought it might be related, but it seems not [10:39] t1mp: gotcha, thanks! [10:52] oSoMoN: kalikiana: please review https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/statesaver-path/+merge/229762 [10:54] zsombi, will do [10:54] oSoMoN: thx === MacSlow is now known as MacSlow|lunch [10:57] zsombi: looking [10:58] kalikiana: thx 4 U 2 :) [11:05] zsombi: should we unit test this? though it's kinda "obvious" if we had one place that checks all paths might be nice [11:08] brb [11:18] * nik90 sighs.. its TRAINCO mode :/ the indicator-datetime silo for enabling alarm vibrations is blocked [11:22] bzoltan: https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1351024 [11:22] Launchpad bug 1351024 in Ubuntu Calendar App "Date & Time picker is not working on device." [Critical,Triaged] [11:30] hey nik90, could sergiusens help in any way? [11:31] dholbach: I think he got busy yesterday to work on it. [11:31] ah ok [11:31] dholbach: he said he would work on it 2h+ later, but I didnt see any reply from him [11:32] probably got too busy with cocktails with those small umbrellas by the pool :-P [11:33] zsombi: hi, you are back. How was your vac? [11:34] kalikiana: well, maybe we could have unit test for that, but for all paths the toolkit uses, not for this particular one. OTOH there's a test which checks whether the state file is in teh proper place, that's why I have the test updated as well [11:34] nik90: yeah, I'm back, and I'm on 1338697 [11:35] nik90: bug #1338697 [11:35] bug 1338697 in Ubuntu UI Toolkit "Alarm sound url read from a saved alarm is always empty" [Critical,In progress] https://launchpad.net/bugs/1338697 [11:35] nik90: the vacation was awesome, relaxing, after the first 2 weeks I forgot where I've parked my car in teh airport :D [11:35] zsombi: lol [11:35] nik90: it took me ~20 mins to find it :D [11:36] nik90: but back to the bug [11:36] nik90: it is QOrganizer who eats it... :/ [11:36] zsombi: oh..would that require a patch upstream then? [11:36] nik90: damn it :( it doesn't get saved into the occurence... [11:37] nik90: perhaps... [11:39] zsombi: oh also I added some crash logs to https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1337405 which causes the clock app to crash. It used to happen since the old clock app times. [11:39] Launchpad bug 1337405 in Ubuntu Clock App reboot "Clock App crashes frequently when UCAlarmModel::clear() is called" [Critical,Triaged] [11:39] nik90: I'll check that too [11:40] zsombi: I realise you will be busy with the datetime picker crash for calendar app. So bug 1337405 can go later if you wish [11:40] bug 1337405 in Ubuntu Clock App reboot "Clock App crashes frequently when UCAlarmModel::clear() is called" [Critical,Triaged] https://launchpad.net/bugs/1337405 [11:41] nik90: ok, seems it was a false alarm... the URL gets converted from QTodoItem to RawData, but then in the alarms << alarm it get lost... [11:41] nik90: which is weird [11:41] ok [11:41] nik90: well, the crash is as bad as the DatePicker [11:45] nik90: OH DEAR!!!! :s [11:46] * nik90 waits for zsombi to explain [11:46] nik90: in organizerEventFromAlarmData() teh sound is saved as QOrganizerItemAudibleReminder... [11:47] nik90: but in alarmDataFromOrganizerEvent() there's alarm.sound = QUrl(event.description()) :/ [11:47] nik90: so the fix is obvious :/ [11:48] zsombi: so why :/ then? [11:48] mismatched variable types [11:48] nik90: because this wouldn't had have to wait me to fix it :) [11:48] :P [11:48] nik90: however the event.description is also set to sound's URL, so it should work! [11:50] nik90: so then we have a problem: why teh detail was missing from it? [11:50] dpm: hi! I won't be able to attend the meeting about the evernote plugin [11:50] renato___: ping [11:51] dpm: but as David wrote, it's possible to have the plugin shipped by the click package, just for trusted third parties [11:51] mardy, that's fine, thanks for the heads up. We devised a plan after Pat's last e-mail. It seems we can ship everything (app+plugin) in one same click package, which would address all our problems. I had meant to cancel the meeting, so let me do that. [11:52] dpm: cool [11:52] Hi all [11:52] popey ping [11:53] qtros: heya [11:54] popey can you push new version of Shorts to store?) As far as I know you are doing it manually) [11:55] sure can [11:55] zsombi, hi [11:56] renato___: I have a problem with the alarm sounds, seems the todos do not save the description, that's where the sound is saved too. [11:56] zsombi, let me check, I think I have a unit test for that [11:57] Mirv: could you please also upload shorts? http://s-jenkins.ubuntu-ci:8080/job/rssreader-app-click/lastSuccessfulBuild/artifact/out/com.ubuntu.shorts_0.2.281_all.click [11:57] renato___: kewl@ [11:57] popey maybe update version to 0.3 too, because after 0.2 we fixed a lot) [11:58] tricky, you'd need to do that in trunk, lets do that another time. [12:01] renato___: huhh???! QUrl(QString) gives me an empty invalid URL!!! [12:02] zsombi, what do you mean? [12:02] renato___: I have an alarm.sound = QUrl(event.description()) conversion [12:02] popey: done [12:02] renato___: if I print out the event.description() that has the file:///..... URL [12:03] thanks Mirv [12:03] renato___: but the QUrl() ends up to be invalid [12:03] qtros: there you go, latest pushed and approved [12:03] zsombi, humm, yes that is strange [12:06] popey I'll test it soon) Thanks! [12:08] renato___: ok, it is not the conversion... when I'm storing the AlarmData in the QList the QUrl sound content is getting lost [12:11] zsombi, yeah I did some tests and I can store and restore the description value, and parse it using QUrl [12:11] renato___: because I do not copy that field in the copy constructor of AlarmData :D [12:12] renato___: me stupid!!! [12:12] nice, that you found the problem [12:12] renato___: nik90: the fix for bug #1338697 seems a one-liner :D [12:12] bug 1338697 in Ubuntu UI Toolkit "Alarm sound url read from a saved alarm is always empty" [Critical,In progress] https://launchpad.net/bugs/1338697 [12:13] renato___: nik90: and we don't have any tests on that because we didn't knew how the sounds will get stored in the system, whether will it be indexed or simply stored as a URL [12:15] * zsombi lunch, brb [12:18] Saviq, Kaleo , do you know if we have a fix for the ubuntushape already? [12:18] renato___, you have along tail today ;) [12:18] renato___, the batching fix landed, yes [12:19] Saviq, the ubuntushape size fix? [12:19] nice this was quick :D [12:19] renato___, size? [12:20] Saviq, yes the address book is broken, the ubuntu shape is not respecting the size [12:20] renato___, not sure about that, no [12:20] Saviq, http://imgur.com/EnFvIP8 [12:20] renato___, easy to check https://code.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk [12:21] I just want to know who is working for fix that [12:21] I want to try the patch [12:21] renato___, is there a bug? [12:21] renato___, in any case, not me, or anyone in my team [12:21] I think so kaleo and bzoltan was discussing that yesterday [12:23] renato___: I can also confirm that that ubuntushape bug is also affecting other apps like music-app, my personal app etc. [12:23] nik90: renato___: would you please elaborate what bug are you talking about? [12:24] bzoltan, this one: http://imgur.com/EnFvIP8 [12:24] that ken report yesterday [12:24] bzoltan: it seems like the width and height assigned to UbuntuShape is not respected [12:25] Kaleo: ^ [12:26] renato___: are you also aware of https://imgur.com/lkEvmRm [12:26] nik90, no this is new :D [12:27] renato___: I reflashed my phone to make sure it is not some remanants from an older image or something. I will create a bug report for this [12:27] nik90, this should be the same problem with ubuntu shape [12:28] renato___: there are 2 issues in that screenshot. the first is the ubuntu shape. The second is the bottom edge tab is missing, or is that intentional? [12:28] nik90, the bottom edge indicator should disappear as soon as you start to drag the page [12:29] but it should be visible in the bottom [12:29] renato___: ah ok. Yeah I noticed it missing only when dragging it up. [12:30] renato___, talking about this issue? https://bugs.launchpad.net/address-book-app/+bug/1353420 [12:30] Launchpad bug 1353420 in messaging-app "Autopilot tests that use the bottom edge failing since #174" [Undecided,New] [12:30] elopio, here's the merged branch plus tests. Thanks. https://code.launchpad.net/~jonas-drange/ubuntu-system-settings/call-fwd-wait-dual-sim/+merge/229774 [12:31] zsombi, popey: Looks like indirectly we might have alarm snooze being implemented [12:32] zsombi, popey: The indicator-datetime will provide a snooze button which on clicked will hide the alarm notification and show it against after x minutes. [12:32] * ogra_ looks for some QML advise about bindings ... i have some code like: http://paste.ubuntu.com/7970031/ ... how do i define the Binding to actualy en/disable the element if the securityPrivacy.securityType property changes (since securityPrivacy.securityType is an enum i guess i need to call a function or some such instead of dirctly binding property to property) [12:33] nik90: nice! [12:35] brendand, yes this bug is because the ubuntu shape change the object type [12:35] we need to update the autopilot test [12:36] renato___, how did it get past CI testing? [12:36] brendand, I do not know, maybe because both changes land together [12:37] brendand, I do not know which change land first [12:37] renato___, where did Ubuntu Shape change? which project? [12:37] brendand, SDK [12:38] renato___, can you identify the package it belonged to? the sdk team don't seem to think they had anything to do with it [12:38] brendand, this is a SDK change that cause it to fail for sure [12:39] renato___, these are the changes : http://people.canonical.com/~ogra/touch-image-stats/174.changes [12:39] renato___, do you see a culprit in there? [12:41] brendand, the package is: qtdeclarative5-ubuntu-ui-toolkit-plugin [12:41] I do not know when this land [12:41] renato___, 173 [12:42] renato___, so i guess the problem is that the test/release cycle is not tight enough [12:42] renato___, so maybe you did your test on #172 or earlier [12:44] brendand, probably, something out of the sync, I remember that we approved the silo one day and it was released in another day [12:46] brendand, but anyway boiko has branches to fix that already, he will be online soon === _salem is now known as salem_ [12:58] ogra_: were you able to fix your binding issue? [12:58] nik90, heh, yeah [12:58] found a way easier solution (which turned the whole code into a one liner) [12:58] ogra_: ok, I was just about to suggest a solution. But glad to know its fixed [12:59] lol nice [12:59] https://code.launchpad.net/~ogra/ubuntu-system-settings/dev-mode-password-binding/+merge/229780 [13:00] ogra_: hmm I not so sure if securityPrivacy.securityType changes it would actually also change the developer mode switch value [13:00] nik90, it does [13:01] works just fine ... tested it [13:01] ah ok [13:01] secirityPrivacy properly emits its changed state so the "enabled" property just picks it up [13:02] no need for extra bindings or anything [13:02] ogra_: I was more going for http://paste.ubuntu.com/7970232/, but your solution works as well [13:03] * nik90 brb..goes to get some lunch [13:03] ah, yeah [13:03] nik90: do you have a bug report for that UbunuShape width-height problem? [13:04] bzoltan: not that I am aware of. I will be back in 15 mins to write a sample app and report a bug. [13:04] nik90: OK [13:14] nik90: I filed this bug https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1353493 and Kaleo is working on it already. [13:14] Launchpad bug 1353493 in Ubuntu UI Toolkit "UbunuShape does not respect width/height" [Critical,Confirmed] [13:15] renato___: Not sure if you have heard, but you should not use the UbuntuShape as an object type when selecting an object with autopilot. Just simple select the object name. [13:28] renato___, btw did you know about the visual problem with the bottom edge? [13:28] renato___, it might be a different issue to the one causing the AP failures [13:30] bzoltan: ack [13:45] nik90: and we have the fix for the size problem [13:46] bzoltan: that was fast [13:47] nik90: I will give you a PPA with the .deb to verify [13:47] bzoltan: sure [13:48] brendand, the visual is caused by the bug on UbuntuShape [13:49] renato___: once I get the PPA from bzoltan I will update you on whether it fixes the address book and dialer app issue [13:49] renato___, ok [13:50] bzoltan, nik90 great [13:51] nik90: it will be available in this PPA https://launchpad.net/~ubuntu-sdk-team/+archive/ubuntu/testing in 30 minutes [13:51] renato___: nik90: that is only for the sizing problem [13:52] bzoltan: ok, I think dialer, address have the issue due to the sizing problem,but we will see [13:52] nik90: OK [13:53] DanChapman, popey: looks like some trojita tests are failing again: https://launchpadlibrarian.net/181654609/buildlog_ubuntu-utopic-amd64.trojita_0.4.1%2B2-0~201408060805.gita0e7ec1~ubuntu14.10.1_FAILEDTOBUILD.txt.gz [13:53] brendand, boiko has the fix for the autopilot [13:54] link to build records: https://code.launchpad.net/~ubuntu-touch-coreapps-drivers/+recipe/trojita-daily [14:00] rpadovani, are you up for the reminders meeting today? I think mzanetti is on holiday, so he probably won't join us [14:00] dpm, if there is something to talking about, I'm in :-) [14:00] rpadovani, sure [14:01] ok then :-) [14:02] rpadovani: after the meeting, would have have few minutes to review a MP? [14:04] nik90, link it to me, I'll do this evening, no time now, sorry [14:04] rpadovani: not a hurry, so that's fine [14:05] rpadovani: https://code.launchpad.net/~nik90/ubuntu-clock-app/add-vibration-support/+merge/229327 [14:09] nik90, saved, I'll take a look asap [14:24] nik90: kalikiana: alarm sound lost fix: https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/alarm-sound-fix/+merge/229800 [14:25] and now, DatePicker's turn [14:26] zsombi: sweet! Onward to custom alarm sounds in the clock app :D [14:29] bzoltan, jdstrand, lool: do you know of any documentation we have about frameworks, ie what the definition means, differences between frameworks or how to target a framework different from what you're running? [14:30] just asking, so we can write this up at some stage, but any pointers to code, examples, notes or any kind of docs would be appreciated :) [14:30] dholbach: I know as much as visible in the lp:click click/chroot.py and in the SDK frontent. The frameworks are just simple list of packages without versions to be installed on the click chroot. [14:31] bzoltan, for an app developer they will mean a bit more than that, right? what I can think of is: support time frame, which frameworks are used on which devices, which APIs are supported where, how do I stay on the LTS but still test/develop for a more recent framework, etc [14:32] bzoltan, that's at least what I could think of for now which might be interesting [14:32] nik90: ok, so the PickerPanel in the toolkit - on desktop at least - seems to work [14:32] zsombi: it is a calendar app issue, but what I heard was that the PickerPanel doesn't only work on the device [14:33] zsombi: In the clock app, I am using the Picker instead of the PickerPanel [14:33] nik90: so the device is wrong there... ok, we can check that as well [14:33] nik90: PickerPanel yet works only with teh DatePickers [14:34] nik90: ah, you meant DatePicker not the PickerPanel :) [14:34] dholbach: I know little about the frameworks... so far I did not find an API to request the supported frameworks... [14:34] * dholbach nods [14:34] zsombi: ok. I remember you telling that the PickerPanel is meant to be used to replace the OSK (it is placed there). However our clock designs wanted that as a component in the UI instead. [14:35] dholbach: https://wiki.ubuntu.com/Click/Frameworks has the most docs I believe [14:35] dholbach: we could extend to cover more [14:35] * nik90 again refers to DatePicker [14:35] thanks bzoltan, lool [14:35] nik90: np, that's why we have a separate component as well [14:35] nik90: I had a feeling that "always in OSK area" will mean yeah, sometimes emberred in teh app UI :D [14:36] yup [14:37] dholbach: https://wiki.ubuntu.com/Click/Frameworks is what I know of [14:37] thanks jdstrand [14:38] nik90: oh dear, you shoudl look after these logs: (qml/qqmlcomponent.cpp:1143, QQmlComponent_setQmlParent) - QQmlComponent: Created graphical object was not placed in the graphics scene [14:38] nik90: this happens when I open the "New Event" [14:39] zsombi: I don't work on the calendar app ;) [14:39] mihir: ^^ [14:39] nik90: I saw the code has some createQmlObject() calls, and seems these are having troubles [14:39] mihir: ^^ [14:40] dholbach: also, 'click framework list' shows available frameworks [14:40] jdstrand, ah, great! [15:19] popey: I updated https://code.launchpad.net/~nik90/ubuntu-clock-app/add-custom-sound-backend/+merge/225812 by merging trunk into it and then commiting it. But then on bzr push it says branches have diverged and that it won't push it. [15:19] popey: is there any way to resolve that? [15:23] and if you bzr merge from trunk again does it show any differences? [15:23] nope [15:23] it says nothing to merge [15:24] hmm. [15:24] I suppose I will have to copy this over to a new branch :/ [15:24] let me see [15:25] i see 1 conflict in app/alarm/EditAlarmPage.qml [15:25] i just did bzr branch lp:~nik90/ubuntu-clock-app/add-custom-sound-backend [15:25] cd add-custom-sound-backend/ [15:25] I fixed that conflict and did bzr resolve. It resolved the conflicts and the I did bzr merge [15:25] bzr merge lp:ubuntu-clock-app/reboot [15:26] hmm, dunno. need a bzr expert. [15:26] * popey pokes dholbach [15:26] it isn't a huge branch thankfully, so wouldn't mind just restarting a new branch [15:26] popey, what what? [15:26] dholbach: I am getting bzr branches diverged error [15:27] dholbach: which we are unable to solve [15:27] nik90, ubuntu-clock-app/reboot trying to merge which other branch? [15:27] bzr branch lp:~nik90/ubuntu-clock-app/add-custom-sound-backend && cd add-custom-sound-backend/ [15:27] bzr merge lp:ubuntu-clock-app/reboot [15:28] dholbach: when you try merging you will get a conflict. You can fix the conflict by changing the conflicted code to http://paste.ubuntu.com/7971240/ [15:28] nik90, if you have two branches which make changes to the same file and the changes are not contained in both branches in the same way, you can have conflicts [15:28] nik90, edit the file, resolve the issue [15:28] then run "bzr resolve " [15:28] after resolving and bzr merging it says branches diverged [15:28] hum [15:28] i did bzr resolve [15:28] let me see [15:30] elopio: can you have a look at this? it looks like j is trying to run a non-existing -autopilot package; it was intentionally removed https://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/3082/console [15:31] nik90, I ran 'bzr commit' after resolving it and that worked [15:31] nik90, shall I push the change somewhere, so you can grab it? [15:31] dholbach: did you then do bzr push? [15:31] dholbach: for me bzr commit after resolving worked as well [15:31] nik90, where would you like to push it? [15:32] dholbach: back to lp:~nik90/ubuntu-clock-app/add-custom-sound-backend [15:32] I can't push it there [15:32] oh yeah [15:32] I'm not nik90 - don't have his good looks, etc :-P [15:32] lol :P [15:32] lp:~dholbach/ubuntu-clock-app/add-custom-sound-backend [15:32] that would do [15:32] rock on [15:32] popey, ^ [15:33] ♥ [15:33] dholbach: so do I merge your branch into mine? [15:34] ok that worked. I merged your branch into mine and then pushed it [15:35] brilliant [15:35] thnx [15:46] nik90: the 1.1.1179+14.10.20140804-0ubuntu2 uitk is ready for testing === gatox is now known as gatox_lunch [15:46] bzoltan: testing now [15:55] nik90: popey: any idea when the DatePicker stopped working on the Calendar app? [15:56] bzoltan: Yup that works! [15:56] bzoltan: address book and dialer work as expected [15:58] zsombi: according to balloons there wasn't any AP test to say when it starting failing [15:58] balloons: any thoughts may be? ^^ [15:58] nik90: but manually, noone checked it recently, whether it works? [15:58] balloons: ^ [15:59] zsombi: no, not really, sorry. [15:59] popey: :( [15:59] popey: seems something more serious - and I suspect Qt5.3 [16:00] popey: I cannot open neither a popover nor a panel from the PickerPanel singleton object :/ [16:00] zsombi, yes due to https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1328600, the tests using the datepicker had to be disabled in the interim [16:00] Launchpad bug 1328600 in Ubuntu UI Toolkit "Autopilot lacks support for large timestamps" [Critical,Confirmed] [16:00] so sometime after June 10th [16:00] nice one balloons [16:00] popey: nik90: balloons: on desktop everything works fine, but not on teh device... [16:00] yeah [16:00] nik90, I thought clock worked on the device? [16:01] you confused me, I thought this was a calendar bug, not a toolkit issue [16:01] balloons: clock uses Picker, not DatePicker [16:01] balloons: nik90: I cannot open any popover, not just the datepicker one [16:01] my guess is it broke at somepoint in Mid July then [16:01] when did qt 5.3 come? [16:01] zsombi: wait you cant open any popovers? [16:01] balloons: way earlier... but DatePicker had been modified last time then [16:01] nik90: from PickerPanel, no [16:03] zsombi the other thing is I worked on the ui toolkit tests in relation to that bug.. they still work [16:03] how can we say it's a toolkit bug then, what's different? [16:04] in other words, the pickers have test coverage, despite the app itself not having tests using the pickers [16:10] renato___: the PPA bzoltan provided fixed the dialer and addressbook visual issues. [16:10] nik90, nice [16:10] nik90: Cool, I start the landing procedure [16:11] renato___: btw the header section in the address book looks really nice ':D [16:13] nik90, yes timp did a good job [16:14] zsombi: btw I tested your alarm sound fix branch with my own branch. It works. Will comment on the MR [16:14] nik90: thx [16:18] nik90, I will work on your fast scroll branch today [16:19] renato___: thnx :-) [16:40] popey, dpm: Any further info on the RTM branching that needs to be done for RTM? Frankly it shouldn't affect the clock app since the reboot is indeed for RTM specifically. === gatox_lunch is now known as gatox [16:44] balloons, ping [16:45] ahayzen, pong (pretend this occurs 15 mins from now) [16:46] balloons, hah :) ... actually may not need u...just gonna left jenkins have another go as the new ms2 package has appeared here http://packages.ubuntu.com/utopic/mediascanner2.0 [16:46] ahayzen, I can kick something off.. [16:46] balloons, i'm just gonna top approve again... [16:47] balloons, the mp u just commented on is/was failing because it needs the new ms2 package [16:47] balloons, i had just assumed it had landed into the repo when i top approved earlier...but obviously not lol [16:47] balloons, so i'm gonna try again.. [16:49] t1mp, ping [17:04] bzoltan: zbenjamin: I'm getting the following click package validation error: ERROR: Could not find scope INI file 'ubuntucommunityscope/com.ubuntu.developer.mhall119.ubuntucommunity.ini' [17:04] I don't know where it's getting the .ini name from, because mine is called ubuntucommunityscope.ini [17:05] ${CMAKE_BINARY_DIR}/src/com.ubuntu.developer.mhall119.ubuntucommunity_ubuntucommunityscope.ini [17:05] it's like it left off the _ubuntucommunityscope part [17:06] and when I install the package in my emulator, it fails with: http://paste.ubuntu.com/7971969/ [17:07] I can confirm that the first .so path exists on the emulator [17:07] this may be unrelated to the .ini error [17:21] elopio, not sure you had the latest diff when you reviewed? [17:21] elopio, the test failures in https://code.launchpad.net/~jonas-drange/ubuntu-system-settings/call-fwd-wait-dual-sim/+merge/229774 is also due to an unrelated issue we are trying to resolve [17:35] jgdx: oh, I might have looked at the wrong one. I'm sorry. [17:36] could be I forgot to push actually [17:37] jgdx: you still have things like [17:37] 641 ->>>>>>> MERGE-SOURCE [17:39] elopio, I'm not sure why that is there. [17:43] jgdx: oh, nevermind. I'm still looking at the old review. [17:43] wow, I need some vacations. I'm slow today :) [17:44] :) [17:47] jgdx: we have helpers for switches, so you won't have to check if they are checked before clicking them. [17:48] http://bazaar.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk/view/head:/tests/autopilot/ubuntuuitoolkit/_custom_proxy_objects/_checkbox.py [17:49] you use them like this: self.wait_select_single(ubuntuuitoolkit.Checkbox, [17:49] objectName="callWaitingSwitch") [17:52] also one for textfields: http://bazaar.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk/view/head:/tests/autopilot/ubuntuuitoolkit/_custom_proxy_objects/_textfield.py [17:52] jgdx: you could do [17:52] text_field = self.wait_select_single(ubuntuuitoolkit.TextField, objectName='destNumberField') [17:52] text_field.write('...') [17:52] it will take care of focusing the field, of using the right input device and of checking that it was written. [17:57] elopio, thanks! [18:01] jgdx: I think you are not asserting the calls to the mocks anywhere. [18:02] I got to go for a while. I'll leave this and a couple of other comments on the branch. [18:02] thanks again jgdx. Nice work with this. [18:03] elopio, thanks!:) [18:19] nik90, I posted a comment on your fast scroll branch [18:19] renato___: looking now [18:20] renato___: can't believe I missed that :P [18:26] nik90, this will avoid the fast scroll letters to apear behind the list items: http://paste.ubuntu.com/7972496/ [18:27] renato___: ah you reordered it after the listview [18:27] nik90, yes [18:28] renato___: I noticed this https://imgur.com/xmAEkIi in the address book where the section header separator appears below the fastscroll instead of reducing its width. Is that part of the new design? [18:28] renato___: I still need to sync your changes from the fit-n-finish branch for these components [18:28] yes [18:29] the designers ask for that, the fast scroll should not push the items to the left [18:29] ok [18:29] thnx a lot for your help. I am almost done with fastscroll and custom swipe delete then :-) [18:30] welcome [19:59] mhall119: PM ? [20:01] daker: give me a few minutes, on a call [20:01] ok tyt [20:07] Letozaf_: can I merge your branch now? [20:07] nik90, yes [20:07] Letozaf_: awesome [20:07] nik90, :-) [20:07] balloons: hey, we just got 3 AP tests for the new clock app :D [20:34] How do I inform qmltestrunner of the path to my custom module.. The app finds the module just fine, but all the tests fail because they can't find it. [21:12] nik90, awesome ;-) [21:14] balloons: I am going to break it soon enough :P..but I guess who ever breaks it fixes it :P [21:20] that's right [21:20] :p [21:33] ahayzen: did you figure out the AP tests for the custom listitem actions? [21:34] balloons, o/ the errors todo with https://code.launchpad.net/~vthompson/music-app/fix-swipe-delete-test/+merge/229718 are because of the toolbar not being shown [21:34] balloons, now suffering the same isssues with https://code.launchpad.net/~andrew-hayzen/music-app/listitem-actions-take-2/+merge/229536 [21:34] ahayzen, that was my first stop.. and right you are [21:34] I know the bug.. d'oh! [21:34] nik90, yep just got them 'working' except the toolbar isn't shown [21:34] ahayzen: and also the upstream ListItemWithActions.qml file was updated upstream. The changes mainly involve around a new multiselection mode. [21:35] https://bugs.launchpad.net/bugs/1353420 yes? [21:35] Launchpad bug 1353420 in messaging-app (Ubuntu) "Autopilot tests that use the bottom edge failing since #174" [Undecided,New] [21:35] nik90, hah so i've gotta do my first 'pull'... i put // CUSTOM next to everything i changed :) so should be able to figure it out [21:35] balloons, no... [21:35] balloons, the toolbar should be shown when in wideaspect as default [21:35] balloons, but for whtever reason the toolbar is hidden something somewhere has broken it [21:36] ahayzen: Check https://code.launchpad.net/~renatofilho/address-book-app/release-2014-08-04/+merge/229528 for those changes. If they benefit you, then update. If not leave it [21:37] balloons, basically to get the music-app working again we need to solve the ap tests and get... https://code.launchpad.net/~vthompson/music-app/lookup-file/+merge/229612 https://code.launchpad.net/~vthompson/music-app/prevent-incorrect-no-music-screen/+merge/228972 ... https://code.launchpad.net/~vthompson/music-app/fix-swipe-delete-test/+merge/229718 or https://code.launchpad.net/~andrew-hayzen/music-app/listit [21:37] em-actions-take-2/+merge/229536 landed .. with one of the last two landing first as they block the others [21:37] nik90, awesome thanks :) [21:37] yw [21:37] nik90, i assume this same component will eventually be part of the sdk? [21:38] balloons, oh and.. https://code.launchpad.net/~vthompson/music-app/remove-white-error-labels-1353173/+merge/229713 [21:38] ahayzen: well may be not as it since they might need to generalise it a bit more but more or less yes [21:38] balloons, both mediascanner2 and uitk have broken us.... [21:38] balloons, nik90 oh yeah the new UbuntuShape may have broken us as well .... u guys seen this before? https://bugs.launchpad.net/music-app/+bug/1353229 [21:38] Launchpad bug 1353229 in Ubuntu Music App "Music app's Now Playing queue has stretched and squished cover art" [Undecided,New] [21:39] chaos! lol [21:39] ahayzen: that's been fixed today [21:39] ahayzen: let me grab the link for you [21:39] nik90, oh sweet! [21:40] ahayzen: https://code.launchpad.net/~fboucault/ubuntu-ui-toolkit/shape_reliable_geometry_changes/+merge/229790 [21:40] nik90, awesome i'll mark it as a dup of that bug [21:41] ahayzen: yeah. I tested the music app against that MP. It is fixed :D [21:41] nik90, awesome thanks :) [21:43] ahayzen, whoa.. that's alot.. How can I help? [21:44] I must admit, I'm confused because of the many branches [21:44] balloons, figure out the toolbar not being shown then should just be matter of approve approve approve done :) lol [21:44] balloons, remember jenkins is wideAspect and device isn;t [21:49] ahayzen, ok, so I can work on the failures I see in em-actions-take-2/+merge/229536 [21:49] and you are not sure why, except that the toolbar doesn't appear [21:49] yes? [21:49] balloons, yeah that one of the swipe-delete one... i suspect my listitem-actions will require further review [21:49] balloons, so probably best to figure out this one https://code.launchpad.net/~vthompson/music-app/fix-swipe-delete-test/+merge/229718 [21:50] balloons, i'll just double check why that was failing again as there are so many mps lol [21:50] balloons, yeah resolve ^^ mp [21:50] balloons, u can see it clicking offscreen (as that is where the toolbar is) lol [21:54] ahayzen, got it.. ok, let's see [21:54] balloons, i'm gonna try and figure too...but i don't have an environ to test with [21:54] balloons, ping me if u need any help with the logic i can tell u what it should be doing === salem_ is now known as _salem [21:58] balloons, unless the Panel has changed with the new UITK [21:59] balloons, place to start is to check if onWideAspectChanged {} is run in the musicToolbar ~L150 [21:59] balloons, and ensure showToolbar() is called [22:06] nik90, u still around? [22:06] ahayzen: what's up? [22:07] nik90, if u have something listening to the onChanged ... will the onChanged get called when it is first set .. i always forget if it does(n't) [22:08] ahayzen: it does [22:08] nik90, and with the listitemactions any idea if they plan to add reordering? [22:08] nik90, ok so thats not the issue then ;) [22:08] ahayzen: regarding the reordering no idea [22:09] nik90, cool my reordering works...but it limited to the view u can see... ie u cannot scroll yet be dragging the item near the top/bottom [22:09] ahayzen: ah [22:09] nik90, but it does the job for now.... it has always been like this [22:10] nik90, i did attempt scrolling at one point but then with our view with listitems of different sizes the calcs becomes super complex [22:10] nik90, but then again we have taken out the different sizes in this merge...maybe i can reinvestigate in a second iteration [22:11] ahayzen: yes I suppose [22:12] ahayzen: did you try it in a sample app with constant listitem heights? [22:12] nik90, not yet that was in my old version like start of this yr ... need to reimplement it [22:13] nik90, do u know if the Panel has changed recently? [22:13] ahayzen: afaik no [22:14] ....something must have happened to make the toolbar just disappear (stay offview)... [22:14] hmmm actually u can just see the progress bar...suggesting it is not in wideAspect? ... maybe the width of the jenkins window has changed? [22:15] hmm but i think that would be shown even it is in wideaspect grrr [22:15] balloons, u got any further?