bmatusiakanybody working on the proposed dev of ubuntu touch?02:03
TSHello World!03:29
mihirTS hey05:50
justCarakaso/ good morning all07:05
dpmmzanetti, hey, not bad, Tagger already translated into 15 languages ;) https://translations.launchpad.net/tagger07:28
justCarakaswhat does tagger do ?07:30
justCarakasbecause it isn't translated to dutch, I wouldn't mind helping with that07:31
nik90justCarakas: https://launchpad.net/tagger07:32
nik90app for QR code reader07:32
mihirdpm: you around ?07:44
mzanettidpm: nice :)07:52
mihirahh that's nice , at what stage the app is ?07:56
mihirnik90: ^^07:56
nik90no idea07:56
nik90I just read the project description07:57
mihirokay :)07:57
dpmmihir, I am, but I'm a bit busy for the next couple of hours. If you've got any question, please feel free to ask it directly instead of pinging first07:57
rpadovanimzanetti, hey, I'm reviewing your branch. Looks good, just some little type here and there, I'll leave inline comments. But one thing is broke the count() of tags: add a tag from website and assign to a note, refresh with pulldown, the new tag appear in the list, but the tag count reports 0, and I have to restart the app to have '1'07:58
mihirdpm: question was regarding splash-screen , are we unblocked , if we remove _X-Ubuntu-Splash-Title= ?07:58
dpmmihir, what do you mean by unblocked?07:58
mihirdpm: https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/calendar-splash/+merge/23731907:58
dpmmihir, I still don't understand07:59
mihirdpm: the splash merge was blocked because of bug https://code.launchpad.net/bugs/1377896 , translation issues07:59
rpadovanimzanetti, tldr; the tags.refres() doesn't refresh the count property07:59
rpadovaniwell, NotesStore.refreshTags();07:59
dpmmihir, that bug does not block the merge. Neither the current splash nor the splash in that MP are translated, but they will be once the fix for that bug lands08:00
dpmthat is, there is nothing blocking the MP to land08:00
mihirdpm: okay popey ^^08:00
mihirdpm: thank you :)08:01
dpmmihir, and afaik, we can remove _X-Ubuntu-Splash-Title= if it's the same as Name08:01
dpmbut that's the theory, I guess we'll have to try :)08:01
dpmpopey, music remix branches now on http://people.canonical.com/~dpm/core-apps-sponsoring/08:02
mihirdpm: okay thank you . i'll do that.08:02
mzanettirpadovani: ah, thanks for the testing!08:13
mzanettirpadovani: yeah, note things down in the MP08:13
dpmmzanetti, rpadovani, I'll be there in 2 minutes08:30
davidcallezbenjamin, hi08:41
zbenjamindavidcalle: hey08:42
davidcallezbenjamin, I know this is very fresh, but when can we expect the qtcreator "reviewtasks" fixes to be released?08:43
zbenjamindavidcalle: its in landing already, today or tomorrw08:44
davidcallezbenjamin, awesome :)08:44
JamesTaitGood morning all; happy Fire Prevention Day! :-D08:48
rpadovanihey oSoMoN, could you ping me when you have 5 minutes for a query, please? :-)09:12
oSoMoNrpadovani, ping09:16
nik90balloons, fginther: Will the jenkins focus issue for clock app be resolved today? (pretty pls :) )09:55
nik90rpadovani: hey, I remember you having university classes this afternoon, Can you help me with some code reviews when you are back home?09:56
rpadovaninik90, sure, after 18:00 I think09:56
rpadovaninik90, or 2night09:56
nik90rpadovani: cool, thnx..just give me a ping then09:57
mzanettidpm: hey, I approved the bouncing progress indicator10:16
mzanettidpm: can you test this one? https://code.launchpad.net/~rpadovani/reminders-app/multipleAccounts/+merge/23768210:16
dpmmzanetti, sure, I'll give it a go as soon as I've fixed UOA on my phone10:20
akiva-thinkpadmihir, ping10:49
DanChapmanhey, what does this 'ota' & 'ota-1' mean that i'm seeing in bugs and bug-tags? I presume it's a post-rtm thing?11:02
akiva-thinkpadDanChapman, hmmm good question11:03
DanChapmanpopey: ^^ care to enlighten me with your wordly knowledge :-)11:03
popeyover the air update post-RTM11:04
popeythings we expect to ship really soon after release11:04
DanChapmanpopey: ahh ok then, makes more sense now. Thanks !! :-)11:05
dpmhi zbenjamin, it seems I can no longer run Reminders on the emulator due to some click review tools check failures - I think you had a fix for this about to land IIRC? Do you know of any workaround in the meantime? http://i.imgur.com/jOitaqG.png11:31
zbenjamindpm: yes if you don't need debug output11:33
zbenjamindpm: are you on trusty or utopic?11:33
akiva-thinkpaddpm, I presume you know how to install them manually?11:33
akiva-thinkpaddpm, I know how to get around that; nik90 showed me11:33
zbenjamindpm: if you are on utopic, the silo7 contains your solution11:33
dpmzbenjamin, I'm on trusty on this PC11:33
akiva-thinkpaddpm, http://paste.ubuntu.com/8506701/11:34
zbenjamindpm: or that https://launchpad.net/~ubuntu-sdk-team/+archive/ubuntu/tools-development11:34
* akiva-thinkpad is not sure if that is what you need.11:34
zbenjamindpm: i think bzoltan pushed the newest fix to it11:34
dpmakiva-thinkpad, I know how to manually install click packages, but that's not what I'm trying to do - I appreciate the help, though11:35
zbenjamindpm: yeah the development ppa should have the fix11:35
akiva-thinkpadOkay sorry :P Its probably to you like me telling you how to use sudo apt-get! of course you know!!!11:35
dpmzbenjamin, hm, I thought I was using the development PPA already and I was up-to-date, let me re-check11:35
zbenjamindpm: the package is only 50 mins old11:36
* nik90 is on trusty and checks as well11:36
nik90zbenjamin: it is in11:36
dpmzbenjamin, do you know the cause of the failures, though? Why does it not recognize i386 as a valid arch, and where does that debug policy (which is not in my .apparmor file) comes from?11:36
dpmaha, yes, apt-get update shows some new qtcreator packages :)11:37
zbenjamindpm: the debug policy is injected by QtC so it can reroute stdout and stderr11:39
zbenjamindpm: the i386 arch i don't know, the review tools consider it as invalid arch11:39
dpmzbenjamin, ok, I've installed the new QtC, but it still does not launch the app. The only difference now is that 2 of the previous errors are demoted to warnings, but still it doesn't install and run the click11:40
zbenjamindpm: the problem with the debug policy is, it would be much nicer if i would inject it after the package has built. But its not so trivial and would make the app startup even slower11:40
zbenjamindpm: thats weird11:41
zbenjamindpm: how is the branch called again?11:41
dpmzbenjamin, bzr branch lp:~rpadovani/reminders-app/multipleAccounts11:42
nik90zbenjamin, dpm: Clock had the same error msg for the i386 issue, but after the update it works as expected. I got a dialog asking if I wanted to ignore the message and then it worked11:42
nik90dpm: did you see tht dialog?11:42
zbenjaminnik90: but you get the dialog because of the policy you use, there should be still a error listed11:43
dpmnik90, no, no dialog after updating QtC, closing it, restarting it and rerunning the app11:43
nik90zbenjamin: the i386 issue is listed as a warning, not a error anymore11:44
zbenjaminnik90:  ok, i had to filter that in the plugin ;)11:44
nik90the only real error is the use of "calendar" which is available only for vetted applications11:44
zbenjaminnik90: can you check the new devices page? does everything look alright to you?11:44
nik90(which clock is)11:44
dpmnik90, yeah, I saw that too, I get a warning now, but no dialog11:44
nik90dpm: and it still doesn't run on the emulator or phoen for you?11:45
zbenjamindpm: building11:45
nik90zbenjamin: wow you changed the device details page quite a bit11:45
zbenjaminnik90: yep :)11:45
dpmzbenjamin, also, the devices page has some black squares on my PC: http://i.imgur.com/71KKvb4.png11:46
zbenjaminnik90: do you see black squares also ?11:46
nik90zbenjamin: no I see no icon at all, https://imgur.com/5q1dToe11:46
nik90zbenjamin: so at first I didn't know they were clickable11:47
zbenjaminweird ... probably i should include the icon then11:47
dpmzbenjamin, oh, it lost the kit. I've got no kit assigned to my emulator. Let me see if that fixes it11:48
OrphisHey there!11:49
OrphisI'm working for Spotify, trying to improve the Linux client and implement proper media key support11:50
OrphisBut I'm hitting a bug in dbus-cpp documented here: https://bugs.launchpad.net/dbus-cpp/+bug/137880711:50
zbenjamindpm: it starts for me but i had to choose the reminders2 run configuration11:50
zbenjamindpm: since it creates a executable the default cmake plugin also creates a (sadly) invalid runconfig11:50
OrphisHas anybody any experience with that library? Knows the bug? Has any idea on how I could fix the issue?11:51
dpmseb128, perhaps you can point Orphis to the right person? ^^11:51
OrphisThat would be awesome11:51
ogra_Orphis, tvoss is your man :)11:52
dpmzbenjamin, yeah, I'm familiar with that bug. I seem to remember fixing that was blocked on not all project types using kits, but I'm not sure if I recall correctly. Are we in a better position now to fix that "extra run configs" bug?11:53
zbenjamindpm: no, fixing it would break all other cmake projects11:53
zbenjamindpm: or refactoring the QtCreator API11:54
zbenjamindpm: right now there is no way to choose tell the QtC API that the cmake runconfigs are not compatible with our kits11:54
zbenjamindpm: i probably can take another look at it later .. maybe i missed something before11:55
zbenjamindpm: nik90: do you get any errors on the console because of the icons?11:55
nik90zbenjamin: 1 sec, let me open qtc via console11:56
zbenjaminnik90: just paste the log somewhere i'll sort it out11:58
dpmzbenjamin, thanks. I think it'd be good to talk about it at the sprint. It currently makes for a bad developer experience, as running apps is a bit of a hit and miss. It happens to me all of the time, but because I know I have to switch run configs, I just do it. But the first time it happened to me I was quite confused and thought the SDK was broken. I could imagine most app developers don't know about the issue and are equally confused11:58
zbenjamindpm: agreed11:58
seb128dpm, Orphis: what library?11:58
zbenjamindpm: but most app devs will not hit it because we have only qml based templates, it happens only when you build your own main.cpp11:58
Orphisseb128: dbus-cpp; tvoss is on the issue already :)11:58
nik90zbenjamin: http://paste.ubuntu.com/8526422/11:59
zbenjamindpm: but it sucks that is true11:59
seb128Orphis, right, that's the right guy for your questions I think ;-)11:59
zbenjaminnik90: looks like https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/137150911:59
ubot5Ubuntu bug 1371509 in Ubuntu UI Toolkit "Icons black in gallery from krillin #45" [Critical,Fix released]11:59
dpmzbenjamin, well, I don't have stats as to how many devs write QML vs C++. I think QML still wins C++ at this time, but we might have a different picture when we start selling devices11:59
zbenjamindpm: agreed12:00
Orphisseb128: Yup, he gave me some awesome support already :)12:00
nik90zbenjamin: hmm that fix is going to be released into trusty though :/12:02
akiva-thinkpadhey check this out http://2buntu.com/touch/device-art-generator/12:02
zbenjaminnik90: yeah i most likely will have to work around by putting the icons into the qrc12:02
=== _salem is now known as salem_
zsombinik90: ping, are you destroying any Alarm data that you get using AlarmModel.get() function?12:18
nik90zsombi: destroy? Not that I can recall.. I don't remember calling alarm.clear() except for in the qml tests12:19
zsombinik90: alarm.reset() only clears the alarm data, does not delete it12:20
nik90zsombi: to delete an alarm I do, var alarm = alarmModel.get(i); alarm.cancel()12:20
zsombinik90: that's fine12:20
nik90zsombi: ok, other than the 2 above methods, I don't do anything to destroy alarm data12:20
nik90tbh I am not how to do it either way12:21
zsombinik90: :)12:21
zsombinik90: alarm.destroy() dould do the job.... however you shouldn't do that EVER!!! I'll put that in teh doc ;)12:21
nik90zsombi: lol12:21
nik90zsombi: if it should never be used EVER, why are you exposing it in the docs, or were you sarcastic about it?12:22
zsombinik90: did I say should NEVER be used?12:23
nik90zsombi: isn't shouldn't do that EVER and NEVER the same thing in this context? :P12:24
zbenjaminnik90: can you check if libqt5svg5 is installed for you?12:24
nik90zbenjamin: sure12:24
zsombi15:21 zsombi: nik90: alarm.destroy() dould do the job.... however you shouldn't do that EVER!!! I'll put that in teh doc ;)12:24
zsombinik90: from this I'd understand that I should never destroy the returned object...12:24
=== salem_ is now known as _salem
nik90zbenjamin: yup I have it, Installed: 5.2.1-112:24
zbenjamint1mp: ^12:24
t1mpnik90: and /usr/share/icons/suru/actions/scalable/view-expand.svg exists for you?12:25
* nik90 checks12:25
nik90t1mp: nope I don't have it12:26
zbenjaminnik90: is suru-icon-theme installed?12:26
nik90zbenjamin: yup12:26
nik90zbenjamin: I have the suru folder and the icons in it12:26
zbenjaminok then those files are new :/12:26
nik90but that particular icon I cannot find12:26
zbenjaminmeans i need to ship em12:26
t1mpzbenjamin: or the icon theme needs to be updated on trusty12:27
nik90zbenjamin: wouldn't backporting ubuntu-suru-icons be easier?12:27
nik90after all it is just a icon theme12:27
zbenjamint1mp: i would vote for that !12:27
zbenjamint1mp: but it requires a SRU still12:27
zsombinik90: otoh, a delegate can use the "model" role to access the Alarm object, so in cases where the actions are performed on the delegate, there's no need to use the get() function to retrieve the Alarm reference12:27
nik90zsombi: oh yeah, I could just have done model.cancel() and that would work as well?12:28
zsombinik90: and that is even documented12:28
zbenjamint1mp: except we backport it ourselfes in the ppa12:28
zsombinik90: yes12:28
nik90zsombi: tbh I don't remember why I did what I did there12:28
zsombinik90: perhaps because in the early phases teh "model" role wasn't there12:28
nik90I suppose12:29
nik90zsombi: I noticed I can access the alarm properties in a delegate by just doing "type, name, date" etc...couldn't I hence just do cancel() instead of model.cancel() ?12:30
zsombinik90: no, because a model can only export roles, cannot export slots12:31
nik90zsombi: ah .. ok12:31
zsombinik90: those you can access now are roles12:31
zsombinik90: for convenience the roles are named same way the Alarm properties12:31
zsombinik90: "model" role is a QML-is way to expose teh Alarm object itsemf ;)12:32
nik90ah yes true12:32
=== _salem is now known as salem_
=== salem_ is now known as _salem
dpmbfiller, have there been any issues with the camera recently that are pending landing? It used to work pretty well, but on the last couple of weeks, starting or using the camera on mako either freezes the whole UI or reboots the device12:39
=== rmescandon is now known as rmescandon-lunch
beunojdstrand, it is done.12:43
akiva-thinkpadjhodapp, oh hey the music-app designer is here12:50
akiva-thinkpadah shame ahayzen and vic arent online12:51
akiva-thinkpadjhodapp, question: Ahayzen and I were discussing the choice to not have a album label on the now playing page, and we decided that it would lead to a bug in cases where no bug was defined.12:52
akiva-thinkpadwhat say you?12:52
* DanChapman just sat here and did all the en_GB translations for dekko. "Apparently" you have to click "Save & Continue" after each page and not just "Next". 12:53
* DanChapman starts again12:53
akiva-thinkpadjhodapp, this is a comparison between really the two styles.12:55
akiva-thinkpadjhodapp, this one is better http://i.imgur.com/daNXXMQ.png12:57
dpmrpadovani, good work with the multiple accounts branch! I've reviewed and added a commend with some design suggestions: https://code.launchpad.net/~rpadovani/reminders-app/multipleAccounts/+merge/23768213:00
jdstrandbeuno: thanks!13:01
rpadovanidpm, thanks! I haven't freeze on desktop switching account. About the dropdown, I'm not sure to understand what you mean: you *need* to go to online account settings to authorize an account, for security reason13:02
rpadovaniit's impossible that an app could choose to use an unauthorized account13:02
dpmrpadovani, yeah, I understand that. What I mean is that when you tap on the option, it should go to online accounts, rather than using the Add account button for that13:03
rpadovanidpm, ah, understood!13:03
dpmsorry, that was perhaps not too clear on my drawing13:04
mivoligomzanetti: hi13:08
mzanettihi mivoligo13:08
mivoligomzanetti: just wanted to let you know I've added 5 new levels and sent a merge proposal13:09
mzanettimivoligo: cool, thanks13:09
mzanettiwill look at it tonight13:09
mivoligomzanetti: feel free to rearrange them :)13:10
=== rmescandon-lunch is now known as rmescandon
jdstrandbeuno, sergiusens, bzoltan: hi! do you guys know the status of fat packages these days? like, does the store handle them right? does the sdk? does installing on the device?13:21
beunojdstrand, the store does, yes13:22
jdstrandbeuno: what will the store do if it gets an i386 only package (assuming that the click-reviewers-tools says that is ok). it will be accepted but devices won't see it because the arch doesn't match?13:23
beunojdstrand, correct13:23
beunothe reverse case is common, where the emulator is i386 and a lot of packages aren't visible because they are armhf13:24
jdstrandbeuno: right now the scripts don't consider i386 as valid, but it was pointed out to me that makes it difficult for the sdk to deploy on the emulator, so I was thinking of adjusting that13:24
jdstrandyeah, true13:24
bfillerdpm: it's related to the location service13:24
* beuno nods13:24
jdstrand(and then that reminded me of fat packages in general)13:25
bfillerdpm: camera is trying to instantiate it at startup and it's either hanging or crashing13:25
bfillerdpm: working on a fix13:25
jdstrands/in general//13:25
beunojdstrand, a while back, I saw them in use, but had to be built manually13:26
sergiusensjdstrand: it is handled13:27
beunoI guess stats by arch is something I should add13:27
jdstrandsergiusens: and specifying 'multi' in the click manifest is the way, correct?13:27
sergiusensjdstrand: I have a long overdue task to make click buddy auto build for the aches in the manifest (creating a fat if necessary)13:28
* jdstrand is verifying the click-reviewers-tools13:28
sergiusensjdstrand: arch: ["i386", "armhf"]13:28
sergiusensjdstrand: or was it "architectures"? (but you get the point I hope :-)13:28
jdstrandI do13:29
dpmok, thanks for the update, thanks bfiller13:32
dpmkalikiana, how do I override the translation domain if I want to use something different than app id?13:33
dpmwhere and how should I specify i18n.domain?13:33
kalikianadpm: either set i18n.domain to set the default, for example in the MainView's Component.onCompleted13:33
kalikianaor use i18n.dtr("bla", "domain")13:34
dpmkalikiana, thanks. The second one is not an option, so I think the first should work. mzanetti ^13:34
dpmkalikiana, can i18n.domain be set in main.cpp too?13:34
kalikianadpm: UbuntuI18n* i18n = &UbuntuI18n::instance(); i18n->setDomain("domain");13:37
dpmjdstrand, it seems that the c-r-t have started throwing errors not detecting i386 as a valid architecture. As an example, this click: http://people.canonical.com/~dpm/click/com.ubuntu.reminders_0.5.latest_i386.click - is this a known bug?13:37
dpmzbenjamin, a question: does QtC injecting the debug policy in the click package mean that if I publish a click generated with QtC to the store, it will fail the upload? I.e. it will have the 'debug' policy in it, which is not allowed13:40
zbenjamindpm: thats why we have the publish page, if you create it there it will be clean13:40
dpmzbenjamin, I don't quite understand. Do you mean that QtC creates two types of click packages, depending on whether you press the Play button or use the Publish page?13:41
zbenjamindpm: thats what i mean13:41
jhodappakiva-thinkpad, I'm not the music-app designer :)13:48
jhodappakiva-thinkpad, I'm the backend guy for media13:49
akiva-thinkpadjhodapp, didn't you do the remix design?13:49
jhodappakiva-thinkpad, nope13:49
akiva-thinkpado_o sorry :P13:50
akiva-thinkpad... who the heck was it then...13:50
akiva-thinkpadI could have sworn it was you :P13:50
jhodappakiva-thinkpad, I forget, but ahayzen should be able to tell you13:56
renato___mihir, nik90 , ok we got approved from SDK guys (thanks zsombi ) https://code.launchpad.net/~renatofilho/ubuntu-ui-toolkit/save-alarms-in-local-time/+merge/23759813:57
renato___mihir, nik90 could you test eds branch again, and check if all still working: https://code.launchpad.net/~renatofilho/qtorganizer5-eds/fix-1311165/+merge/23714213:58
renato___nik90, you will need SDK branch to test alarms ^^13:58
zsombirenato___: could you confirm that eds backend does not send itemChanged() signal when the eds data is changed due to non-organizer initiated change?13:59
renato___zsombi, what do you mean? For example if I change the item with evolution app you do not get the change on QtOrganizer?]14:00
zsombirenato___: yes14:01
renato___zsombi, it should works (, I will test it14:01
renato___zsombi, after my current meeting14:02
zsombirenato___: ok, just drop me a mail, I14:02
zsombiI'll be away soon14:02
zsombirenato___: aaah, no, it does work!!!!!!!!!!!!14:03
zsombirenato___: awesome ;)14:04
mihirrenato___: :woot: awesome14:05
mihirrenato___: thanks a lot :)14:05
* mihir hugs renato___ :D14:05
=== mpt_ is now known as mpt
nik90zsombi, renato___ : Awesome. Please give me a an hour or two to thoroughly check both the MPs together with the clock app.14:17
fginthernik90, frantically working on it now14:31
nik90fginther: sry for the rush,14:32
fginthernik90, no worries, it needs to be fixed14:32
mhall119kenvandine: is there something blocking https://code.launchpad.net/~ken-vandine/content-hub/lp1373086/+merge/237459 from landing?14:37
kenvandinemhall119, yes... it FTBFS in the silo14:37
kenvandinebecause it now has translations... imported from LP :)14:37
kenvandineElleo has a branch14:37
kenvandinemhall119, so soon14:37
dpmnik90, is bug 1376513 something that needs to be fixed in the clock app or somewhere else on the platform?14:42
ubot5bug 1376513 in Ubuntu Clock App "Alarm volume should use the stream-restore dbus api available in pulseaudio" [High,Triaged] https://launchpad.net/bugs/137651314:42
nik90dpm: clock-app14:42
nik90dpm: I will try to get that implemented this weekend14:42
dpmok, thanks14:42
dpmawesome :)14:42
mhall119kenvandine: ah cool14:44
nik90zsombi, renato___: Works good. Tested clock app with both the MPs. I commented in the MPs.14:50
renato___nik90, thanks14:50
=== oSoMoN_ is now known as oSoMoN
fginthernik90, balloons, https://code.launchpad.net/~nik90/ubuntu-clock-app/sync-bottomedge-sep08/+merge/236604 passed \o/15:13
fginthernik90, balloons switched out the WM from fvwm to xmonad15:13
nik90fginther: woohoo yay..thnx15:13
balloonsfginther, awesome. Yea, I found xmonad worked great locally in tests, as it autofocused the window on launch15:14
nik90fginther, balloons: I presume the xmonad change is only for clock jenkins? or for jenkins as a whole?15:14
fgintherballoons, I wish I had discovered that fvwm was in use a few days ago, became a real simple update15:14
fginthernik90, this does apply to all testing at this time15:15
balloonsfginther, ohh.. heh, so a wm was already running, just not a useful one15:15
balloonseither way, this is good stuff15:15
nik90fginther: cool, this should make the testing environment more comparable to the user environment15:15
balloonsyep yep15:15
fgintherballoons, exactly :-), although I have some fondness for fvwm15:16
fgintherballoons, thanks for doing the investigation on the WM to use15:17
=== gatox is now known as gatox_lunch
mzanettipopey: hmm... this is interesting: http://i.imgur.com/EFBoFAH.png15:37
popeyi have had that15:37
popeyits submitted and you can go in and manually to the rest of the steps15:37
popeybueno ^15:37
m-b-oDanChapman: Hi, are you around?16:00
balloonsm-b-o, hey ,how are you!16:12
m-b-oballoons: fine, thanks and yourself?16:13
=== _salem is now known as salem_
m-b-oballoons: started the weather app with this new cpp extension for TimeZones. No way to get the app started out of QtcReator, do you know?16:15
balloonsm-b-o, what do you mean get the app started?16:17
balloonsm-b-o, this? https://code.launchpad.net/~ubuntu-weather-dev/ubuntu-weather-app/timezone-plugin/+merge/23761316:18
nik90m-b-o: You need to open the CMakeList.txt file in Qtc, and if the cmake file has been properly configured, you should be able to run it fine by qtcreator. Qtc will automatically build the plugin when you press the green run button.16:18
m-b-oballoons: wll, that didn't happen. I'Ve followed the steps in http://bazaar.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/utopic-3.0/view/head:/README.developers to get it run16:19
=== gatox_lunch is now known as gatox
rpadovaninik90, ping for review16:23
nik90rpadovani: hey, let me grab the MPs16:23
balloonsm-b-o, so you are a-ok now then?16:24
nik90rpadovani: https://code.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/utopic-3.0/+activereviews16:24
nik90rpadovani: You need to start from 10-alarm-change onwards..16:24
nik90rpadovani: ignore the top 216:24
m-b-oballoons: if there would be a way to star the app from QtC, it would be cool to do so16:25
rpadovaninik90, gotcha16:25
nik90rpadovani: Also you don't need to review the qmltest files in those MPs. They have already been looked at by elopio.16:25
rpadovanithanks elopio :D16:25
nik90rpadovani: and finally, these Mps are for the release tomrrow..so not required to complete the review tonight if you are busy16:25
rpadovaninik90, nah, itΕ› ok16:26
nik90cool, thnx mate16:27
=== karni is now known as karni-afk
nik90rpadovani: don't top approve yet, since I need to update the debian changelog and also merge trunk (trivial stuff) which I am doing one by one. I will top approve once I do it.16:34
rpadovaninik90, gotcha16:35
rpadovaninik90, https://code.launchpad.net/~nik90/ubuntu-clock-app/11-remove-utils/+merge/237047/comments/58329216:35
nik90rpadovani: cool, will fix16:35
=== salem_ is now known as _salem
=== karni-afk is now known as karni
nik90rpadovani: fixed the comment in the above MP ^^16:48
rpadovaninik90, https://code.launchpad.net/~nik90/ubuntu-clock-app/12-alarm-list-design-spec/+merge/23717716:51
nik90rpadovani: ah I did the code so since if the text is hidden, it still calls alarmUtils.get_time_to_next_alarm(model.date - localTime) which is unnecessary cpu wastage. hence when the alarm is disabled, I give it a static text.16:55
rpadovaninik90, makes sense16:56
rpadovaninik90, done17:05
nik90rpadovani: btw no need to code review https://code.launchpad.net/~nik90/ubuntu-clock-app/14-sync-listitem-actions-oct5/+merge/237189. It just syncs up with upstream code which was already reviewed. If you want you can just test on phone to see if it caused any regressions17:05
nik90rpadovani: cool, thnx a lot17:05
nik90rpadovani: I am merging them one by one as they are dependent on each other and so jenkins cant merge them all at once17:06
rpadovaninik90, you're welcome. I left a little comment on 13 about another way to implement an if, that IMO is better, but they are tastes :-) Anyway, I approved it17:06
nik90rpadovani: I am taking a look atm17:07
nik90popey: shall we start the clock meeting?17:08
popeynik90: sorry, didn't realise we had an overlap! lets17:08
DanChapmanm-b-o hey :-) have you managed to get it to run? sorry i havn't had a chance to get back to looking at dholbach's packaging changes. It was working before that just jenkins wouldn't have any of it.  I'll branch it now if you are still having issues17:10
m-b-odanChapman heyho! :) Yes, got it run. But not out of Qtc, though.17:11
akiva-thinkpadahayzen, ah how goes it?17:12
dpmm-b-o, what's the issue you're having with getting it to run? How are you trying to get it to run?17:12
ahayzenakiva-thinkpad, hey good thanks afternoon lectures today :) lol17:13
m-b-odpm: Using the TimeZone Extension in QML and run it from within QtC fails.17:14
m-b-odpm DanChapman " module "TimeZone" is not installed"17:14
DanChapmanm-b-o it's just run straight off the bat from QtC for me. Ahh... i still have the TimeZone plugin installed from last time.17:14
* DanChapman gets rid of that17:15
m-b-oI'Ve followed http://bazaar.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/utopic-3.0/view/head:/README.developers than and that works, but outside of QtC17:15
* dpm reads doc17:16
dpmm-b-o, ah, so the issue you're having is to run the weather app + plugin from QtC! Sorry, I thought it was the other way round. What's the error you are getting?17:21
m-b-odpm  " module "TimeZone" is not installed"17:21
ahayzenballoons, in response to your comments this mp was waiting for someone to review it... at the time my url-dispatcher had decided music:// should open facebook lol and victor was/is unable to use adt ... https://code.launchpad.net/~andrew-hayzen/music-app/add-url-dispatcher-tests/+merge/23384017:22
dpmm-b-o, ok, so it's not finding the module in the expected location. Are you running it on a desktop kit or on the emulator?17:22
m-b-odpm desktop17:23
DanChapmanm-b-o: dpm ok so looking at http://bazaar.launchpad.net/~ubuntu-weather-dev/ubuntu-weather-app/timezone-plugin/view/head:/plugins/CMakeLists.txt#L26 seeing as no -I plugin/path/TimeZone is being passed to qmlscene apart from the $@ then would it not be looking for the plugin in ${QT_IMPORTS_DIR}/TimeZone and not ${QT_IMPORTS_DIR}/qt5/qml/UbuntuWeather/TimeZone/17:23
dpmm-b-o, DanChapman, the other issue is that with reminders, terminal and file manager, where we've got this same setup, we have a rule in cmake that copies over the compiled plugin to the build directory, and adds the build directory to the list of locations where the plugin is looked for, so that you don't need to install it on the system. I'm not sure if that branch has that rule, I've not looked at it17:25
mzanettirpadovani: addressed all your comments17:30
rpadovanimzanetti, see, I'm on it. Nevermind for layout of delegate, let do it in another branch17:31
rpadovanimzanetti, about the message, I wrote the wrong thing: if you don't have any tag, and you go to the tab of tags, there isn't any message17:33
rpadovaniI think you have to add something here17:33
snizzomhall119: ping17:41
mhall119snizzo: pong17:42
snizzomhall119: I'd like to know if I'll receive my tshirt for being a pioneer :)17:43
snizzoI'm losing faith...17:51
mzanettirpadovani: done17:54
mihirpopey: you wanted me to revoew ubutu_title from splash MR , correct?18:18
popeymihir: no, i think it should still be there for now.18:20
mihirpopey: okay i recalled it, need to test from Calendar to Month view how does it go :|18:21
mihirsecurity_template_exists (apparmor-push.json): specified unsupported template 'ubuntu-push-helper' , need to install anything?19:54
jdstrandmihir: what version of click-reviewers-tools do you have? you need 0.1519:56
mihirjdstrand: how do i check that?19:56
jdstrandapt-cache policy click-reviewers-tools19:57
mihirjdstrand: http://paste.ubuntu.com/8528803/19:58
mihirokay i need to upgrade my click-reivew tools19:58
mihirthis one too regarding that correct, πŸ‘Ž error: security_template_push_helper(apparmor-push.json): template is not 'ubuntu-sdk' ?19:58
jdstrandif I understand your question, yes, you want to use the ubuntu-push-helper template in apparmor-push.json19:59
* mihir upgrades20:00
balloonsrpadovani, ping20:10
rpadovaniballoons, pong20:10
balloonsrpadovani, calculator doesn't seem to save calculations anymore between runs. It should be saving them right?20:11
rpadovaniballoons, after you swype them up, yes20:12
rpadovaniballoons, r329 works well on mako devel, where are you testing?20:13
rpadovaniballoons, do you have any output log?20:13
balloonsrpadovani, I'm playing with it on the desktop atm20:16
balloonsrpadovani, ahh I see.. heh20:16
* balloons notes we should have a test for that20:16
rpadovaniballoons, mhh, what's wrong? I'm able to save them also on desktop20:17
balloonsrpadovani, right.. I am too. I was just confused20:17
rpadovaniballoons, popey says that the app will be redesigned soon :-)20:17
balloonsfginther, ping20:18
balloonsfginther, check out the giant calculator :-)
mivoligomzanetti: I forgot to mention I didn't update enemies waves in level.json files but you probably noticed that :)20:19
balloonsfginther, so did you set the size for the window?20:20
balloonsfginther, testing locally I set the xserver size to 400x600, and that emulated the phone size properly. If I left it as 1024x768 you get weirdness with xmonad on some apps20:24
fgintherballoons, :-( Let me fix that20:26
nik90fginther, balloons: I saw it for clock as well, but it didnt affect the tests20:37
nik90balloons: it seems clock test is failing on https://code.launchpad.net/~nik90/ubuntu-clock-app/12-alarm-list-design-spec/+merge/23717720:37
nik90balloons: I didn't seem to make any change that part of the code. But I can reproduce the failure locally as well.20:38
balloonsnik90, ok, so how to fix?20:40
nik90balloons: yeah I am not sure what changed. I need to upgrade my VM after like 3 months and run autopilot vis to see what changed20:40
balloonsnik90, so you feel it's legit or not?20:41
nik90balloons: its certainly not jenkin's fault :)20:41
nik90balloons: but the annoying part is that it fails at the very last step of the test
balloonsnik90, ohh swipe to delete did change20:43
balloonsbut it's been some time20:43
nik90balloons: yeah but I am using custom swipe to delete (due to custom listitem with actions)20:43
nik90balloons: the function it fails at is https://bazaar.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/utopic-3.0/view/head:/tests/autopilot/ubuntu_clock_app/emulators.py#L51920:45
popeyballoons: you mentioned previously that someone (I don't recall who) might be able to help with AP tests for the new tab feature in terminal? https://code.launchpad.net/~flscogna/ubuntu-terminal-app/uri-and-tabs/+merge/23735320:52
balloonspopey, yes senan was going to work on it20:52
balloonsI've not seen him in a couple days20:52
mihirjdstrand: hey i am still getting that error while build it , i upgraded my system and it shows it has 0.15  click-review tools.20:57
mihirjdstrand: my bad error is different ,πŸ‘Ž error: security_template_exists (apparmor-push.json): specified unsupported template 'ubuntu-push-helper'20:57
jdstrandmihir: hrmm, try 'rm ~/.cache/click-reviewers-tools/apparmor-easyprof-ubuntu.json' and then try again20:58
mihirjdstrand: that works , thanks :)21:01
nik90balloons: quick question, I remember autopilot vis (long time) used to highlight an item in the UI when you click on the object tree items.21:04
nik90balloons: do you know how to enable it?21:04
balloonsnik90, use autopilot3 vis21:04
nik90ah thnx21:04
nik90I should update the readme docs21:04
balloonsyea, I get confused sometimes too21:04
balloonsit's correct in the readme21:05
balloonsbtw, I shamelessly modified it and using it for my calc mp :-)21:05
* nik90 looks silly21:05
nik90balloons: lol21:05
nik90balloons: I wrote that guide to help noobs like myself21:06
nik90balloons: if you got some time, I need help debugging this clock failure together21:10
balloonsnik90, I would love to help, but I need to finish this work on calc first21:10
nik90ok, I will try some things out meanwhile21:11
balloonsahayzen_, you about?21:11
ahayzen_balloons, yep21:11
balloonsahayzen_, well, heh, technically I'm playing with the old music app so you don't care :-)21:12
ahayzen_balloons, heh i have the job of syncing it with remix so i do care a little ;)21:12
ahayzen_balloons, so what are you working on the old app for?21:16
balloonsahayzen_, so I'm trying to fix https://bugs.launchpad.net/music-app/+bug/137948821:16
ubot5Ubuntu bug 1379488 in Ubuntu Calendar App "apparmor denial during test runs for /home/phablet/autopilot/fakeenv/*/.cache/QML/" [Undecided,New]21:16
balloonsahayzen_, as part of that, I'm going to be dropping support for patching home on the device21:17
balloonswell, I'd like to try and do that anyway21:17
ahayzen_yep ... balloons note i have branches up ready to put confinement back in (which is yet to be testing with autopilot ;) )21:17
ahayzen_balloons, ok so what would it do instead?21:17
balloonsahayzen_, nothing. IT would be up to the test runner to make sure the device is clean during test runs21:18
balloonsI'll leave basic mocking for the desktop for now21:18
ahayzen_balloons, so we would copy things directly into ~/Music ?21:18
balloonsahayzen_, yep21:18
ahayzen_balloons, so what would happen to all my music on the device?21:18
balloonsahayzen_, nothing. the tests would fail though as-is21:20
balloonsyou would need to run with adt-run to ensure you have a safe env setup21:20
balloonsahayzen_, we'll be talking more about this in a couple weeks at the sprint, but here's the metabug for it: https://bugs.launchpad.net/ubuntu-app-launch/+bug/137642321:21
ubot5Ubuntu bug 1376423 in Ubuntu Application Launcher "There is no easy and future-proof way of starting an app in a clean environment" [Undecided,Invalid]21:21
ahayzen_balloons, 'safe env setup' what does that involve does it create a new profile? or move all my music somewhere else or what?21:21
balloonsahayzen_, I'm making these changes now, or thinking about it, because it's causing issues again in CI21:22
ahayzen_oh yeah i remember seeing that bug21:22
balloonsahayzen_, I would rather not do anything until we have a better plan in place, but just wanted to give you the heads up21:22
ahayzen_balloons, is causing all those failures on utopic CI or is that something else?21:22
balloonsahayzen_, yes21:22
ahayzen_balloons, but RTM was fine at last check lol21:22
ahayzen_balloons, ok so cut to the chase... what do we need to do to music-app autopilot then?21:23
joehanneshey guys ... what's the normal way to create a db-driven ubuntu app with JS/HTML?21:26
balloonsahayzen_, I'll propose mp's you needn't worry about it21:26
balloonsahayzen_, we can keep the patch working by doing this: http://bazaar.launchpad.net/~nskaggs/ubuntu-calculator-app/precompiled-qml/revision/33321:26
balloonsbut I'm considering just gutting everything21:26
ahayzen_balloons, ok thanks :)21:27
balloonsif I can get it to work21:27
balloonsnik90, alright, I can help you out for a bit21:27
nik90balloons: ok so I am bit confused at the failure message I got21:28
nik90AttributeError: Class 'AlarmDelegate' has no attribute 'swipe_to_delete'.21:28
nik90when does that happen?21:28
nik90I thought it was timing error. So I added a sleep(10) in between that time to confirm it21:29
nik90but it is not a timing issue21:29
nik90and the MP I linked shows the code diff to AlarmDelegate.qml which is causing them failure. I am certain that it didn't change anything related to swipe delte21:30
balloonsnik90, it's not a timing issue. As it says it doesn't understand swipe_to_delete21:32
balloonsare you sure you have an instance of the object you think you do?21:32
nik90balloons: I think I found the issue (but do not know the syntax to fix it)21:33
nik90balloons: here is the old object tree in trunk https://i.imgur.com/L9uq0bx.png21:33
nik90balloons: here is the new one21:33
nik90do you see the difference?21:33
balloonscan you expand the new one? but yes my guess is yo udon't have the object you think you do :-)21:34
balloonsso just tweaking a select is probably all we need to do21:34
nik90yup the listitemwithactions object type got changed into AlarmDelegate object type21:34
balloonsthere you are ;-)21:34
balloonscool I'll assume you are sorted!21:34
nik90yup :-)21:35
fgintherballoons, how are you forcing 400x600 resolution? X doesn't appear to like this for me21:36
nik90balloons: quick question, in https://i.imgur.com/uuzd8wv.png, how do I refer to AlarmDelegate as a class type? Previously I did class ListItemWithActions(ubuntuuitoolkit.UbuntuUIToolkitCustomProxyObjectBase):21:39
balloonsfginther, you are using xvfb right? So -screen 0 400x600x2421:39
nik90balloons: however doing class AlarmDelegate(ubuntuuitoolkit.UbuntuUIToolkitCustomProxyObjectBase): doesnt seem to do it21:39
fgintherballoons, ahh, these tests have always been using X21:39
fgintherlet's figure this out21:40
nik90in the context of https://bazaar.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/utopic-3.0/view/head:/tests/autopilot/ubuntu_clock_app/emulators.py#L51921:40
balloonsfginther, so locally I'm running like so http://paste.ubuntu.com/8529327/21:42
fgintherballoons, thanks, that helps21:43
rpadovaniballoons, seems your copy and paste has to be improved :P21:44
balloonsfginther, I based that on autopilot3-sandbox-run.. not sure if using that script directly would be helpful or not.21:46
balloonsrpadovani, ack, thank you.. pushed the fix21:48
rpadovaniballoons, good, do you mind if I wait until tomorrow to test it on device? I'm a bit tired right now21:49
nik90rpadovani: btw I got an email about your webbrowser branch21:50
nik90rpadovani: did you check with osomon before?21:50
balloonsrpadovani, if you can approve without it, I've been asked to get this fixed asap. Obviously you needn't say you tested on the device :-)21:50
nik90rpadovani: last time I checked he said he wanted it to first land in the SDK before using it in the webbrowser app.21:50
rpadovaninik90, sure, I'm working on implementing some feature for history, and your bugs will be fixed21:51
rpadovaninik90, oh, well, we didn't talk of details of implementation, I'll ping him tomorrow21:51
rpadovaniballoons, ok, but if it will destroy the world is your fault, not mine :P21:52
rpadovaniballoons, you have a go21:52
balloonsrpadovani, ack21:55
nik90fginther: hope this is not a bad time to ask, but if you look at, after the WM fix, the build times have significantly increases from ~6mins to ~50mins. Was this expected?22:02
fginthernik90, In this case, multiple ubuntu-clock-app-ci jobs all started at the same time, but the only one autopilot job could execute at a time, so most of that time was just waiting22:04
nik90fginther: ah ok22:05
fgintherballoons, xvfb is setup now and passed the first few tests22:13
balloonsfginther, ack, awesome22:14
balloonsI'll try landing the calc mp22:14
balloonsrpadovani, any chance you are still around?23:24
rpadovaniballoons, maybe :-)23:24
balloonsrpadovani, one more quick calc mp.. I wanted to toss this in. It fixes the asserts so you get the proper result when there is more than one23:25
balloonsI want to get it merged and do a release so everything is fixed23:25
rpadovaniI'm on it23:25
balloonsrpadovani, I reverted the Screen.qml whitespace change hehe23:27
rpadovaniballoons, yes, your mr lgtm, and the implementation seems very elegant :-)23:28
rpadovaniballoons, ok, top approved, I think I'll go to sleep now, last 4 days were very intensive :D23:29
balloonsrpadovani, thank you much.. new calc for you in the morning ;-)23:29

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!