/srv/irclogs.ubuntu.com/2015/02/26/#ubuntu-app-devel.txt

=== _salem is now known as salem_
=== salem_ is now known as _salem
zsombinik90: alarm, ring-ring06:38
kalikianat1mp: a pop over review for you https://code.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/paperChase/+merge/25103807:46
zsombibzoltan_: nik90: I found the cause of teh Alarm regression, might take few hours to get a clean solution07:54
bzoltan_zsombi:  rock baby, rock!07:55
zsombibzoltan_: it's an ugly one...07:55
bzoltan_zsombi: I believe07:55
zsombibzoltan_: we wouldn't have this if we would not use the QtOrganizer stuff...07:56
zsombicharles: we need to talk about the Toolkit alarms vs. datetime indicator common backend!07:57
bzoltan_zsombi:  yeps .. we knew that it is a dangerous track08:02
nik90zsombi: ooh nice08:02
zsombinik90: nice to see you08:03
zsombinik90: this organizer is killing me :D08:03
nik90zsombi: same...once this bug is fixed I need to look at why the alarm switch is not working in vivid alone08:04
zsombinik90: if you don't use the alarm.date in a Label binding or any binding to show the date, the date saving works08:05
davidcalleGood morning08:05
zsombinik90: if you use it in a binding, it doesn't08:05
nik90zsombi: oh08:06
zsombinik90: and this because the alarms are cached, but the individual field changes are not synced to teh cache until the save() is called08:07
zsombinik90: but, the date change causes the property to be changed, however the items using teh AlarmsModel fetch the data from the cache, and that restorews the original date08:08
nik90Ah ok08:08
zsombinik90: so I have to update the cache as well on each existing alarm update...08:09
zsombinik90: I bet the same goes with the switch...08:10
nik90zsombi: would that have any performance implication or would the user not notice since it is done in the background?08:10
dholbachgood morning08:11
zsombinik90: the cache update is a pretty fast operation, however you will still need to save() in order to get the backend synced08:11
zsombinik90: so teh model will see the changes real time, but the save will do its job async08:11
nik90zsombi: ok08:12
zsombinik90: and this situation was not catched as we did not have the binding kind of test yet...08:13
nik90zsombi: well I need to also have an edit alarm test added to clock ap qml test suite08:14
rpadovaninik90, zsombi sorry, what's the bug you're talking about? Because this morning I found a critical bug: yesterday night, before going to sleep, I rebooted the phone and all alarms disapperead from the indicator, but I didn't see it so this morning I didn't wake up :/08:16
rpadovanialso now alarms are listed in the clock app but not in the indicator08:16
rpadovanirtm r1808:17
rpadovani(the last one, right?)08:17
nik90rpadovani: we are talking about a bug in the alarms back end in vivid alone08:17
rpadovaninik90, so the bug I found isn't known?08:18
nik90rpadovani: although in your case if indicator date time loses its alarms then it won't ring since it is the service that does the job08:18
nik90rpadovani: I am seeing a silo assigned for indicator date time in rtm related to a crash08:18
nik90May be that might fix your issue08:19
nik90Let me check08:19
dpmdavidcalle, you might have seen that already, but if not, check this out: https://forums.oneplus.net/threads/wip-ubuntu-touch-for-oneplus-one.266170 :)08:22
davidcalledpm, yes, seen that last night, this is great :)08:24
nik90rpadovani: https://bugs.launchpad.net/ubuntu/+source/indicator-datetime/+bug/142496608:27
ubot5Launchpad bug 1424966 in indicator-datetime (Ubuntu) "/usr/lib/arm-linux-gnueabihf/indicator-datetime/indicator-datetime-service:5:reset_timer:unity::indicator::datetime::LiveClock::Impl::on_timerfd_cond:g_main_dispatch:g_main_context_dispatch:g_main_context_iterate" [Critical,In progress]08:27
davidcalledholbach, I'm thinking of getting a phone for Estelle, but every time I tried to propose her something different for her laptop, things went not so great. (Last time was an EeePC with a finicky Nvidia and 1Gb of ram, it didn't help ^^)08:27
nik90rpadovani: not sure if this is the same bug that you face as well although it looks like a critical crasher that started only recently08:27
dholbachdavidcalle, right - it'll take some getting used to - on the other hand: phones have a long history of everyone trying to be different, with the result that everything worked differently on every phone I had in the last 15 years :)08:29
rpadovaninik90, could be. Unfortunately I'm at uni atm without usb, so I cannot read the logs. I'll take a look tonight, if my log is different I report a new bug! Thanks for the head up :-)08:29
dholbachso "some getting used to" will probably be all right :)08:29
nik90rpadovani: yw08:29
davidcalledholbach, heh, maybe :)08:29
rpadovanidavidcalle, I did a fast read of the porting guide, as soon as I have time I want to try a porting for the Galaxy S4. As first feedback, I think the step "Add your device specific git repositories under phablet/device" needs some more information (like where I can find repo (yes, I see you linked them at the start of the guide, but I think is good if you report them also in the specific step) and how I add a new repo there (git08:34
rpadovanisubmodules or what?)) :-)08:34
davidcallerpadovani, true, thanks!08:46
rpadovanidavidcalle, thanks to you for the guide! I'll report more feedbacks as long I do the porting (not soon, I fear, unfortunately)08:48
davidcallerpadovani, no worries, in the meantime, it will keep improving ;-)08:49
=== oSoMoN_ is now known as oSoMoN
seb128qml question, is there a way to do some computation before an object is built?09:04
seb128like I would like to sort a list before using it in a grid repeater09:04
zsombinik90: I think I have the fix09:05
seb128ok, found a way, I use model: function(09:08
seb128)09:08
dpmdholbach, sorry I got distracted by something else, but I'm testing the help app branch now. So it builds as expected, and I'm looking at the missing links part. One thing I don't understand is this syntax:09:12
dpm[Take me to the FAQ!]({filename}faq.md)09:12
dpmwhy is the {filename} prepended to the actual file name of the .md file?09:13
dpmdholbach, I've reviewed the branch and added a comment re: the broken links09:32
dpmgood luck at the dentist :)09:32
JamesTaitGood morning all; happy For Pete’s Sake Day! :-D09:51
nik90dpm: hi, quick question, how does one copy translations from one branch to another? For clock I have the rtm branch which is used for click packaging atm, but there is also the vivid branch which I intend on switching to once the platform itself switched in vivid in 2-3 weeks time09:51
dpmnik90, I guess if you just push to the vivid branch the .po files should be added as any other file in the branch. In any case, once translations are enabled for the vivid branch in LP then they'll be autocommitted (and overwritten)09:53
nik90dpm: ah ok09:53
popeyanyone (ogra?) know what triggers this disk space indication? http://people.canonical.com/~alan/screenshots/device-2015-02-26-095318.png09:54
seb128popey, ciborium09:57
popeythanks seb12809:57
seb128popey, yw09:57
popeythat icon is a bit nasty09:57
ogra_popey, coborium IIRC09:57
ogra_hmm, that "clear all" looks kind of weird in this usecase09:58
seb128popey, http://bazaar.launchpad.net/~phablet-team/ciborium/trunk/view/head:/cmd/ciborium/main.go#L25609:58
seb128errorIcon                 = "error"09:59
seb128I guess they need a better icon to use :-)09:59
popeyya10:02
popeymzanetti: ping!10:11
mzanettipopey, pong10:11
popey\o/10:11
mzanettiyeah, this is good, thank10:12
mzanettis10:12
popeynp10:12
popeyElleo: if I manually delete podcasts from /home/phablet/.local/share/com.mikeasoft.podbird/podcasts - will podbird get "confused"?11:03
Elleopopey: possibly, it'll still have the file listed in its database, so will try to play that instead of streaming11:06
Elleopopey: file a bug and I'll add a check for missing files11:06
popeywell, might be more useful to have a "get rid of shows older than "n days"11:06
popeybecause I'm manually deleting stuff to recover space11:07
Elleoshould do both really11:07
dholbachthanks dpm11:36
dholbachdpm, hum... I thought we came to an agreement regarding en_US?11:39
dholbachdpm, what is your objection or what else do you propose?11:39
dpmdholbach, I was not objecting against it, I just meant I wasn't sure myself what the best solution is.11:40
dholbachah ok11:40
dholbachright, me neither :)11:40
dholbachhah, I think I found the issue11:50
dholbachdpm, the links issue is fixed now - thanks a lot for the review11:59
dpmdholbach, re-reviewed and commented, thanks12:09
dholbachdpm, the links were added automagically by pelican - I'll see what we can do and file a separate bug if that's ok12:22
dholbachdpm, I think we're now at the point where we can start adding content, fixing the styling and everything else12:23
=== MacSlow is now known as MacSlow|lunch
dholbachdpm, shall I fix it in the same branch or in a separate branch?12:34
dpmdholbach, yeah, let's do a separate branch12:43
dholbachok cool12:43
dholbachI'll land the other one, ok?12:44
dpmdholbach, yeah. Also daker mentioned last night that he'd be looking at the app12:44
dholbachnice!12:44
* dpm hugs dholbach13:00
dpmgood work, thanks for keeping at it!13:00
=== rmescandon is now known as rmescandon|lunch
* dholbach hugs dpm back13:06
dholbachdpm, https://code.launchpad.net/~dholbach/ubuntu-devices-help/1425924/+merge/251076 :)13:10
=== _salem is now known as salem_
dakerdpm: the only i can think of is to have an index.html that will call some .js file, the .js file will have detect the locale and redirect to the appropriate index file13:19
dholbachdaker, I filed bug https://bugs.launchpad.net/ubuntu-devices-help/+bug/142040613:19
ubot5Launchpad bug 1420406 in Help for Ubuntu for devices "Set up an independent index.html" [High,New]13:19
dholbachmaybe we can re-use that bug?13:19
dakeryes13:19
dholbachdaker, oh and I brought your package to the post office today :)13:19
dakerwow :)13:20
dholbachlet's see when it makes its way over to you :)13:21
dpmdaker, yeah, that's what I was mentioning on bug 1425025, but I wasn't sure if it's a hack, and I wasn't sure of the proper way to do this redirect (or loading) of another page13:22
ubot5bug 1425025 in Help for Ubuntu for devices "Automatically load translated pages according to the user language" [High,Triaged] https://launchpad.net/bugs/142502513:22
dpmdholbach, yeah, I think we're talking about the same thing, sorry for having missed your bug. So we might need to have an index.html + translations after all13:23
* dpm goes really really for lunch now13:24
dholbachdpm, enjoy!13:30
=== MacSlow|lunch is now known as MacSlow
dholbachballoons, we're soon in business with the help app :)13:44
renatucharles, hi, any progress on the alarm bug?13:56
renatucharles, I have some time now to help you13:56
dholbachdpm, are you working on the bug already? shall I add a stop-gap index.html for you, where you can just then add your javascript magic?14:05
=== rmescandon|lunch is now known as rmescandon
dholbachdpm, did we already check if the translations settings of help are already set correctly?14:15
snizzomhall119: My nexus died, you know when will it be approximately the next flash sale for bq? I'd love to keep developing/testing apps on real hardware :/14:19
mcphailsnizzo: there's one on just now14:22
mhall119snizzo: hurry and see if there are still some available to buy!14:28
dpmballoons, popey, how are you getting on with the testing bugs?15:02
popeydpm: i have a few more to file15:03
dholbachballoons, maybe you can give lp:ubuntu-devices-help a go as well and let me know how it goes?15:04
dholbachdpm, next I'll start work on enabling a second build for online use15:06
dpmpopey, do you have a tag that you've been using?15:06
dpmdholbach, great15:06
popeydpm: needs-autopilot-test15:07
dpmpopey, so essentially, this is the list of bugs, correct? https://bugs.launchpad.net/ubuntu-phone-coreapps/+bugs?field.tag=needs-autopilot-test15:08
popeyits a start :)15:08
dpmwe should probably add dekko in there too15:08
dpmi.e. as part of the ubuntu-phone-coreapps superproject15:09
zsombinik90: ready for a check? https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/alarm-regression-fix/+merge/25109615:12
zsombikalikiana: t1mp: anyone of you up for an alarm regression review? https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/alarm-regression-fix/+merge/25109615:13
balloonsdholbach, sure thing15:19
dholbachcool, thanks15:19
dholbachand there's another MP up for review15:20
balloonsdpm, do you want to make dekko an official member of the lp team?15:20
dholbachdpm, shall we give the themes names? I'm not sure "online" and "offline" will be distinct and self-explanatory enough - maybe "phone" and "web" or something? :)15:20
balloonsI had not planned to review it15:21
dpmdholbach, sounds good to me15:23
dholbachcool15:23
dpmballoons, yeah, that's what I was thinking, popey, thoughts on adding dekko to the ubuntu-phone-coreapps superproject?15:23
popeydpm: ask DanChapman, it's his project :)15:24
popey(not being obtuse, but it technically is)15:24
kalikianazsombi: I'll take it15:33
dpmpopey, if he doesn't see this ping, could you have a chat with DanChapman to see what he thinks?15:33
popeysure15:33
nik90zsombi: what do you mean by "The returned object is not guarantied to remain valid, it should not be used in property bindings." Does this mean in the "Edit Alarm" page I cannot set the label to be alarm.name and so on?15:35
balloonsdholbach, just a minor tweak otherwise it looks good and works for me15:53
dholbachballoons, fixed15:56
dholbachballoons, thanks for the testing - I think we can mark the "set up app infrastructure" as "closed" now :)15:57
balloonswoot!16:00
=== chihchun_afk is now known as chihchun
dholbachballoons, the index.html will come back16:02
dholbachin fact I pushed a small branch to start work on bug 142502516:02
ubot5bug 1425025 in Help for Ubuntu for devices "Automatically load translated pages according to the user language" [High,Triaged] https://launchpad.net/bugs/142502516:02
dholbachcan I merge the prerequisites branch now?16:02
balloonsdholbach, yep go for it16:03
* dholbach hugs balloons16:03
dpmthanks popey16:03
nik90dpm: for bug https://bugs.launchpad.net/ubuntu-clock-app/+bug/1354466, I got the basic work done I think...I just need to now manually transfer the 200+ cities from xml into the new format now..16:06
ubot5Launchpad bug 1354466 in Ubuntu Clock App "Cities and Countries are not translatable in the timezone selection dialog" [High,In progress]16:06
nik90dpm: however before I start, I wanted to inform you that the city list will be defined in a .cpp file and I used the tr() function call to mark it for translation16:06
* dpm is on the phone, but will read the backlog16:07
nik90dpm: however after building it, I don't notice the city or country name appearing in the pot file...do I need to do anything special to gettext to make it look through the cpp files as well?16:07
nik90dpm: np, just let me whenever you are on your computer :) ... I will start after I get the assurance that the city names are indeed getting translated before starting the huge conversion process.16:08
dholbachballoons, dpm: new MP: https://code.launchpad.net/~dholbach/ubuntu-devices-help/1425984/+merge/251116 :)16:27
balloonsgo go go!16:29
dholbachballoons, it's quite beautiful how easily pelican can handle different themes/output directories, etc16:29
dholbachthe pain we had to go through beforehand seems to pay off16:29
balloonswow, look at that html16:32
dholbachballoons, all stolen ;-)16:32
popeydpm: you around for reminders hangout?16:32
mzanettirpadovani, ^16:33
dpmpopey, conflicting call, sorry16:33
dholbachthanks balloons!16:39
nik90dpm: nevermind I figured it out...the pot file now shows the city and country names ready for translation :)16:47
dpmawesome, good job nik90 - happy to review the branch if you want me to16:48
balloonsmihir, I had a test fix for kunal's branch but I placed it against trunk now since I haven't seen him merge it: https://code.launchpad.net/~nskaggs/ubuntu-calendar-app/fix-infloop-ap-trunk/+merge/25112216:49
dpmdholbach, it seems I should do phone calls more often, the two pings I got got solved while I was afk :)16:49
nik90dpm: I still need to do some code cleanups..will let you know when its rdy for review16:49
nik90dpm: thnx16:49
dpmgreat16:49
mihirballoons: okay , i'll try to catch him and ask him to merge your branch.16:50
dholbachdpm, one still outstanding: translations.launchpad.net/ubuntu-devices-help16:51
balloonsmihir, well I was thinking it would be better at this point to just fix it, so I did a new branch against trunk so it's not depending on his16:51
balloonsif it merges to trunk it'll fix his issues too :-)16:52
mihirballoons: okay ,  have you withdrawn your older branch which merge to his branch ?16:52
balloonsI will now16:52
mihirokay , great :)16:53
=== rmescandon is now known as rmescandon|afk
renatucharles, nik90, I think I have a fix for the alarm bug, could you guys help me to test it? https://code.launchpad.net/~renatofilho/qtorganizer5-eds/fix-1424924/+merge/25112917:32
nik90renatu: Should I test this on rtm mako? Or on vivid?18:16
ahayzen_Hey guys, So if I have a click with the arch of 'all' it should appear in the x86 store right? or does it have to be 'any'? ... note it is a pure QML app (the music-app :P )18:18
nik90ahayzen_: for flashback, the manifest file shows "all" and it appears in the x86 store18:23
ahayzen_nik90, cool thanks ... maybe jouni just hasn't checked in the store and assumed we needed a fat click ...18:24
nik90I gues18:24
nik90s/gues/guess18:24
nik90yw18:24
renatunik90, vivid18:41
zsombinik90: uhm... that's not entirelly true... I have to remove that comment as I realized there was a different thing that was causing the regression!19:20
zsombinik90: thx for pointing it out!19:20
zsombinik90: ehh, the MR got merge into staging, so I will correct it later19:25
zsombi+d19:25
balloonsmhall119, can we talk about fixing the mass mailings for ubuntu core apps drivers teams?20:10
mhall119what's the matter, not getting enough?20:12
* mhall119 subscribed balloons to more mailing lists20:12
mhall119*subscribes*20:12
balloonsmhall119, hah.. there's some folks not wanting to be spammed with so much email. I want to try and solve it for them and there's a proposed solution20:12
mhall119I haven't actually done it20:12
mhall119balloons: what's the proposal?20:12
balloonsscroll down and have a read of #11 and #12 https://bugs.launchpad.net/launchpad/+bug/81022920:13
ubot5Launchpad bug 810229 in Launchpad itself "no way to be both official-reviewer on a project and not recieve email (without using an external mailing list service)" [Low,Triaged]20:13
balloonsmhall119, so it *seems* like just setting a contact address for the team could end the mass emailings20:14
balloonsmhall119, so my thoughts are to maybe set up a reviewers mailing list that we can set to catch all this noise20:15
balloonsmhall119, so if you are "ok" with that, I will create the team and mailing list and edit the drivers team so the mailings stop20:22
mhall119balloons: I don't want to make these changes without popey's input, I don't want emails to just suddently stop for him or the coreapp devs and them not know what's going on20:27
mhall119but otherwise I'm fine with the proposed changed20:27
popeyso long as it's possible to opt back in, i dont mind20:31
balloonsright, I created https://launchpad.net/~ubuntu-touch-coreapps-reviewers which has a mailing list also20:38
balloonsI'm going to set things to ubuntu-touch-coreapps-reviewers@lists.launchpad.net20:39
balloonsk popey mhall119, I'll finish making the changes. Core app devs who are direct members of the project should still get mailed directly, but otherwise the review spam email should cease. I'll let the core-apps list now as well20:46
balloonsahayzen_, you about?21:04
balloonspopey, mhall119 so I removed test writers from all the devs teams, and they now get access via the new reviewers team. Everything seems to be working. The next step is to remove the drivers team, and swap the admin status to the new reviewers team instead.21:13
* mhall119 notes the irony that balloons's effort to cut down on main spam has caused a flood of emails to his inbox21:33
nik90mzanetti: hey, I added you as a reviewer to https://code.launchpad.net/~nik90/ubuntu-clock-app/predefined-world-city-translation-fix/+merge/251180, can you review just the new c++ subclass I added..I implemented it as we discussed it in the afternoon. Just around 30-40 lines of code.21:33
balloonsmhall119, popey should be complete now.. the spam now goes here: https://lists.launchpad.net/ubuntu-touch-coreapps-reviewers/maillist.html21:35
balloonsofc, you guys are directly subbed to some dev teams, so you'll continue to get those mails as a direct sub21:36
ahayzen_balloons, yo whats up?21:47
nik90balloons: stop sending me so many emails :P21:48
kenvandinelol21:48
kenvandinenik90, i was just deleting those :)21:48
nik90kenvandine: hehe :)21:48
balloonskenvandine, nik90 I hope those are the last of the mails; I just sent what I was up to, to the list. In short, the mass emails should be gone now21:49
balloonsahayzen_, I was going to have you test out the changes to the teams to stop the mass emails, but I found someone else. Thanks though!21:49
ahayzen_balloons, haha cool :)21:49
nik90balloons: so I am the admin of the ubuntu-clock-developers team, would I still get emails about MPs, bug reports in the clock app?21:50
balloonsnik90, yep you'll still get everything about clock. And I suspect you are directly subbed to some other teams also21:50
nik90balloons: ok...so whichever team I am directly part of, I will receive emails from them21:51
balloonslook at https://launchpad.net/~/+participation to see how it works out21:51
balloonsmaybe I should have mailed that.. if you don't see anything in the 'via' column you will get emails still21:51
balloonslooks like weather, calendar and clock for you nik9021:52
nik90yup21:52
balloonsahayzen_, nik90 while I have you two however, popey and I are on a mini crusade of filing bugs for testing needs. Can you make sure your apps have bugs for any needs you have?21:54
balloonsI use needs-autopilot-test or needs-qml-test tags21:54
nik90balloons: sure..on it21:55
ahayzen_balloons, yup will do :)21:55
=== salem_ is now known as _salem
nik90balloons: quick question..I see milestones like ww09-2015..do we follow those same milestones for apps as well? I am pretty pickky about keeping clock up to date in those aspects as you may have noticed with the amount of milestones I created for clock app.21:57
nik90balloons: also I take it we have shipped OTA-1 with the insiders event21:58
balloonsnik90, I suppose you could say that, since the insiders have there devices :-) I was going to say until we have devices in the wild, there's no OTA :-)22:02
nik90balloons: ok, I will correct the milestone names22:03
balloonsso barring the insiders, after folks get there devices they can certainly expect an OTA. As far as core apps, I wouldn't say it's impossible to get a bug in a milestone that requires you to work it before the milestone, but thus far you've been free of such things22:03
balloonsnik90, are you trying to correlate your milestones for clock is that it?22:03
nik90balloons: yeah it helps me focus on the most important for that milestone and postpone others to later milestones22:04
balloonsif so then right, what's happening now will end up being an OTA update22:04
balloonsnik90, ahh I see you have ota-1, ota-2, etc22:05
nik90balloons: yup, although post-rtm needs to be renamed to something like backlog22:06
balloonsnik90, I would say you could stay more or less independent in your milestone naming. Whether or not OTA1 is called as such is irrelevant. Consider every big push to the image with clock changes to be your own milestone I'd say22:10
balloonsor if you do want to sync, follow along with ci on ubuntu-phone.. the milestone will be three weeks apart22:11
nik90balloons: ah ok..ack22:11
balloonsI hope that made sense, heh.. I'm guessing you could even target bugs if you wish into there milestones22:13
nik90yup I understand. I am just contemplating if I should stick with ota or rename them to somethiing that makes sense within the clock app.22:19
juezdreddhello23:55

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