=== chriadam|away is now known as chriadam [02:08] Is there a way to run the phone emulator from the SDK? [02:09] Nevermind, found it [06:14] balloons: any idea, why it is failing Jenkins https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/MutiEventOnSameSlot/+merge/216507 [06:24] jdstrand: ping, i saw the recent devel-proposed already has a debug policy. I just tried it and i can run and connect to the gdb server , but the app itself is rejected by the Mir server: "QUbuntu: Could not create application instance [06:24] " [06:35] Good Morning all [06:38] morning DanChapman! [06:41] DanChapman: dpm Good Morning :) [06:41] hey mihir :) [06:41] dpm: Jenkins getting failed on Calendar any idea [06:42] mihir, I know balloons investigated it and came up with some conclusions, and I'm trying to figure out what needs fixing [06:42] I'm looking at this right now [06:42] dpm: okay thanks !! [06:46] hey dpm, mihir :-) [07:35] mihir, would you be able to do 2 very trivial reviews for calculator? -> https://code.launchpad.net/ubuntu-calculator-app/+activereviews [07:36] dpm: sure, i'll look into that by EOD [07:36] thanks! [07:39] dpm: i have approved it both of them. [07:39] \o/ [07:39] thanks mihir [08:44] mihir, quick question: do you know what this translatable message is supposed to mean? https://translations.launchpad.net/ubuntu-calendar-app/trunk/+pots/ubuntu-calendar-app/ca/5/+translate ? It's been causing some confusion to translators, and after even seeing it in the UI, I'm not sure I understand what it means [08:53] dpm: well that's interesting and odd [08:54] i think %1 is pulling in the event length argument, but I don't know what the E is [08:54] yeah, I think that string needs fixing, but I cannot figure out to what :) [08:55] dpm: actually this is for All Day event [08:55] so if there are 3 all day events on same date it shows on top 3E where %1 is an argument [08:55] yes, I see it appearing there [08:55] oh! [08:55] dpm: like this , text = i18n.tr("%1E").arg(events.length) [08:55] "3 events" [08:55] So "E" Should say "Events" [08:56] yeah [08:56] ok, I'll send a fix [08:56] popey: it wouldn't feet in weekview i guess, [08:56] oh of course [08:56] yes, i see that here, I have a 2E on a line [08:56] i dont like that ☻ [08:57] Hmm okay , i'll try to change that in 2 Events is that okay dpm,popey ? [08:57] well that wont fit [08:57] as you say in week view [08:57] Hmmm , yup. what would you suggest ? [08:58] well it's inconsistent with the events in the rest of the view [08:58] events are shown as bubbles with a blob in the corner [08:58] an all day event is just an event that happens to last all day - so has no time, but has a description and a blob [08:58] (thinking aloud here) [08:59] popey: how about same in Google calendar, stick those bubble on top ? [09:00] it makes sense, but you can't fit many in [09:00] and its hard to seleect them. [09:00] well, there's the popup for selecting them... [09:01] yup, when you tap on number there will be pop-up [09:01] dpm: what do you think, I'm not sure how to show that [09:02] can you file a bug to track the %1E issue so we can track it [09:03] rather than just an ad-hoc conversation between the three of us on irc [09:03] popey: dpm there is already one , https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1320009 [09:03] Ubuntu bug 1320009 in Ubuntu Calendar App "i18n issue" [Undecided,New] [09:03] awesome! [09:08] dpm: what is the PPA name you created for syncMonitor , Calenar app is carshing [09:10] mihir, popey, ok, I've added my suggestion to https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1320009 [09:10] Ubuntu bug 1320009 in Ubuntu Calendar App "i18n issue" [Medium,Triaged] [09:11] dpm: shall I push the Mr with same change? [09:12] mihir, it's the regular core apps PPA, and the package name is: qtdeclarative5-ubuntu-syncmonitor0.1 -> https://launchpad.net/~ubuntu-touch-coreapps-drivers/+archive/daily/+packages?field.name_filter=sync&field.status_filter=published&field.series_filter= [09:12] mihir, if you're happy with the suggestion, please go ahead and submit a MP [09:13] thanks dpm [09:13] that'll do for now. [09:13] dpm: sure, but popey if you get some idea about that let's disucss [09:13] yeah, will have a think. [09:13] been playing with calendar more recently now it's more stable and functional ☻ [09:15] popey: \m/ [09:15] popey: when you get time to test one MR, that would be great. [09:15] which one? [09:17] oh we've got scopes in the store already, cool! [09:17] popey: https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/1297739/+merge/221109 [09:17] the repetation options [09:19] dpm: i have updated with latest PPA , is there any moer dependency for Ubuntu.SyncMonitor ? [09:20] mihir, it didn't seem to need any more dependencies, but let me know if anything is missing [09:20] dpm: it says calendar.qml:71 SyncMonitor is not a type [09:21] dpm:sorry , calendar.qml:4 module "Ubuntu.SyncMonitor" is not installed [09:21] mihir, can you give me the output of this command? apt-cache policy qtdeclarative5-ubuntu-syncmonitor0.1 [09:22] dpm: http://paste.ubuntu.com/7600565/ [09:23] mihir, well, you don't have it installed :) [09:23] mihir, try sudo apt-get install qtdeclarative5-ubuntu-syncmonitor0.1 [09:24] dpm: thanks, i installed , sync-monitor only , it worked thanks :) [09:25] cool :) [09:26] dpm: pushed out meeting back as I have some dogfooding to do right now. [09:26] ok, no worries [09:31] dpm: is https://code.launchpad.net/~dpm/ubuntu-clock-app/fix-desktop-l10n-1325626/+merge/222287 ready to go? [09:31] nik90, yes :) [09:31] dpm: approved [09:31] \o/ [09:31] dpm: popey this how it looks if i use whole word Events , http://imgur.com/pk4X7yB [09:32] dpm: popey what do you think ? [09:32] I don't like it ☻ [09:33] mihir, I think I'd stick to the shorter form until we have a better solution. In any case, "Events" won't fit in the week view on a phone [09:33] dpm: yup , that looks more decent than this. [09:33] +1 [09:36] dpm: here is the MR, https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/1320009/+merge/222296 [09:37] popey: if you find any issue /suggestion with repeat options MR, let me know :) [09:42] k [09:47] mihir, reviewed [09:53] dpm: how many reminder options would you suggest, https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1322547 [09:53] Ubuntu bug 1322547 in Ubuntu Calendar App "'Remind me' time options are overly limited" [Medium,In progress] [09:54] dpm, popey: I am thinking of rejecting https://code.launchpad.net/~gang65/ubuntu-clock-app/ubuntu-clock-app-analog-clock-improve/+merge/219119 [09:55] dpm, popey: I will of course provide the reasons for doing so. Do you agree? [09:55] yes [09:56] mihir, I don't know, tbh, I thought the options we've got would be enough. I think if we add all of his suggestions, we'll end up with a really long option selector. I'd be tempted to ask him to come up with a proposal himself [09:57] nik90, sounds good to me [09:58] dpm: agreed. [10:00] dpm: should I move it in wishlist ? [10:05] mihir, yes, perhaps, but please add a comment [10:39] dpm, you mind checking over https://code.launchpad.net/~andrew-hayzen/music-app/fix-1325626/+merge/222303 [10:45] dpm, thanks for subscribing me to the blueprint ! [10:45] ogra_, ah, cool, thanks for attending! :-) [10:52] dpm, the clock .desktop has an _ before name, comment and keywords...whereas we don't ... and we have tr() around the name ... should i also change this to be the same as clock? [10:52] hi ahayzen, looking... [10:53] dpm, because this doesn't appear to totally work, i don't think, https://code.launchpad.net/~andrew-hayzen/music-app/fix-1325626/+merge/222303 [10:54] ahayzen, so your initial MP makes sense, and I'll approve it now. But to get translations loaded in the apps scope, you'll need to implement the same as either of the two branches in bug 1318008 do [10:54] bug 1318008 in gallery-app "Core apps .desktop files do not included translated strings" [High,Triaged] https://launchpad.net/bugs/1318008 [10:56] dpm, ok ... blimey they are bigger changes ;) [10:57] ahayzen, yeah, but actually not that big. What makes those diffs bigger is the update of the .pot file [10:57] and removal of the po/po.pro file [10:57] dpm, seem to be lots of cmake changes as well? ... my two favourite things cmake and translations in one merge lol [10:57] hahaha [10:58] yes, there are a few cmake changes [10:59] dpm, i'll have a go see how far i get :) [10:59] cool, thanks ahayzen! [10:59] need to learn about translations somehow [11:07] dpm: regarding that translation [11:08] hey [11:08] dpm: I don't think this even can be translated at that point [11:08] dpm: becuase a translator always needs the original string but we don't have that here [11:09] also, how would a translation look like in this case? say you want to change the order as you said in the comment [11:09] what would you do as a translator? [11:09] you get "%1 %2", would you just translate it as "%2 %1" ? [11:10] mzanetti, this is not different to other translations with parameters in other parts of Ubuntu. Yes, that'd be one option. Or in Catalan I'd just translated as "%1 de %2" (not changing the order, but adding a preposition) [11:10] dpm, what would you put for the keywords for the music-app? [11:11] dpm: hmm... fair enough... [11:12] mzanetti, but if there is a standard Qt Locale format that outputs "year, month" already, I'd prefer that too, which would mean less work for translators and would output the string arranged in the right way already [11:12] dpm: yes, I agree... [11:12] t1mp, Hi! can you comment about bug 1327171 ? [11:12] bug 1327171 in ubuntu-ui-toolkit (Ubuntu) "The back button in the header is a missed target, needs more area" [Undecided,New] https://launchpad.net/bugs/1327171 [11:12] ahayzen, perhaps "music;songs;play;tracks;player;tunes" ? [11:13] dpm, cool thanks, is that the things which helps the unity dash search? [11:13] ahayzen, yeah, I seem to remember they're for that [11:13] om26er: hmm.. I never had that issue, but I'm not using the phone a lot [11:13] can someone who uses the phone as their main phone confirm that bug? [11:13] popey: ^ ? [11:13] dpm, i'll put the ones you have suggested for now someone can always add more :) [11:14] great, thanks ahayzen! [11:16] dpm: https://code.launchpad.net/~dpm/reminders-app/translator-comments/+merge/222295/comments/532571 [11:17] mzanetti, ok, thanks. I'll try to come up with a fix that removes the need for translation. But I think in the meantime this will help with context for translators [11:17] dpm: couldn't find anything which would catch all the examples we just listed [11:18] so I guess this is indeed the proper thing to do [11:18] ah, ok [11:18] dpm: as an avenger, can you comment on https://bugs.launchpad.net/ubuntu/+source/ubuntu-ui-toolkit/+bug/1327171 ? [11:18] Ubuntu bug 1327171 in ubuntu-ui-toolkit (Ubuntu) "The back button in the header is a missed target, needs more area" [Undecided,New] [11:19] t1mp: which? the bug om26er posted? [11:19] popey: yes, the one that I just asked dpm to comment on [11:20] popey, t1mp seems the left edge might be conflicting with the back button [11:20] t1mp, I can. I can also confirm that something similar happens on the other side: right-side actions in the header are often difficult to tap: they either tend to be missed or the last action on the right tends to activate the right edge [11:20] dpm: ok, thanks. for the right side we have this bug https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1320863 [11:20] Ubuntu bug 1320863 in Ubuntu UI Toolkit "Rightmost icon in header needs a margin" [Critical,Confirmed] [11:21] i can't say I have noticed that much [11:21] popey, use with one hand or two ? [11:21] popey: me neither, so I'm asking for feedback from people who use the phone more [11:21] two because I have small ladyhands [11:21] om26er: maybe that's it, I also use two hands === MacSlow is now known as MacSlow|lunch [11:23] t1mp, i initially found that the back/tab button area was too far left, but once i knew it was further across than i thought i don't miss it often now [11:27] ok, thanks for the feedback [11:28] t1mp, how can you be using two hands to tap the back button? http://filocoaching.com/wp-content/uploads/2014/01/multitasking.jpg ? [11:28] ahayzen, popey I'll paste your comments on the bug report also [11:28] dpm, do i need to regenerate the *.pot files ... i've run click-buddy --dir . --no-clean [11:28] dpm: that dude doesn't even use his feet to type.. amateur ;) [11:28] lol [11:29] t1mp, indeed, and he's only got 2 monitors! [11:30] ahayzen, yes, you'll then need to copy the updated po/$SOMELONGNAME.pot file from the /tmp/tmp.* directory that contains the build into your source tree [11:37] dpm, i think i have done it right, could you check over? https://code.launchpad.net/~andrew-hayzen/music-app/fix-1318008/+merge/222315 [11:42] dpm, also i was just wondering how does the application name in the .desktop get translated as it does not appear to be added in any of the examples mps or mine? or is it already in the .pot somewhere? [11:43] ahayzen, awesome, on the phone atm, but will look at it in a few minutes [11:43] ahayzen, it gets put in the .pot [11:43] and then when building the click, its translations are put into the .desktop file [11:44] ah ok cool [11:52] zbenjamin: are there any apparmor denials? [11:53] zbenjamin: grep DEN /var/log/syslog [11:53] wait i need to start the app again [11:58] jdstrand: thats all i get [11:58] http://pastebin.ubuntu.com/7601299/ === MacSlow is now known as MacSlow|lunch === hunger_ is now known as hunger [12:32] zbenjamin: ok, those would be denied by DAC and a read-only image anyway, no? (ie, the user can't write out those files) [12:32] jdstrand: yeah i think thats not the issue right? [12:32] jdstrand: as long as gdbserver is not injected everything works. So the script works [12:33] zbenjamin: no. I'm going to mark the debug policy group bug as fixed then. if you see any apparmor denials, please file a new bug [12:33] jdstrand: any change we can get that for older policies too? [12:33] chance [12:34] I guess I could add it to the 14.04 (1.1) policy. people shouldn't be using the 13.10 policy (1.0) any more [12:34] jdstrand: yeah that would be awesome :) [12:35] jdstrand: any idea who i can ping about how to find out why Mir is rejecting the App? [12:36] zbenjamin: maybe wask in #ubuntu-mir? [12:36] zbenjamin: but hold on a sec [12:37] jdstrand: i wonder if this is aproblem because the app id , the upstart process pid (gdbserver) and the app pid are different, so upstart would report a different pid than the one connecting to Mir === chriadam is now known as chriadam|away [12:39] zbenjamin: ok, we don't have any explicit denials for mir sockets or anything (ie, things that would prevent logging the denial), so yeah, ask in #ubuntu-mir [12:39] zbenjamin: possibly? Saviq might be able to comment on that [12:39] jdstrand: ok thx [12:40] zbenjamin, yeah, that's definitely that [12:41] zbenjamin, http://bazaar.launchpad.net/~mir-team/unity-mir/trunk/view/head:/src/modules/Unity/Application/application_manager.cpp#L676 [12:41] zbenjamin, we authorize based on PID right now [12:41] Saviq: i feared that, any idea how i can make that work? === _salem is now known as salem_ [12:42] zbenjamin, quickest workaround would be to force --desktop_file_hint= on the command line [12:42] zbenjamin, longer term we'll have to transition to auth based on cgroups [12:42] Saviq: the point of the click launcher is to get rid of the desktop file hint [12:42] zbenjamin, I know, I know [12:43] zbenjamin, that's why I said workaround [12:43] zbenjamin, and anyway that would be internal to the launcher in gdb mode [12:43] zbenjamin, users wouldn't know [12:43] jdstrand, do you know if all our apps are being put in cgroups already? [12:44] Saviq: ok , but then we need to make sure the desktop file hint does not disappear before the cgroup is working [12:44] zbenjamin, there's one more approach mhr3 tried out https://code.launchpad.net/~mhr3/unity-mir/authenticate-via-apparmor [12:44] Saviq: I think that is what ted did with UAL to make sure lifecycle could handle the oxide sandbox, no? [12:45] jdstrand, makes sense, yes === MacSlow|lunch is now known as MacSlow [12:45] zbenjamin, so enabling auth per cgroup instead of PID should be doable somewhat now [12:45] zbenjamin, could you file a bug against unity-mir please [12:45] Saviq: i could :) and i will [12:45] zbenjamin, and mention that in the click launcher (and vice versa) [12:46] zbenjamin, pfft! [12:46] Saviq: you mean like # work around bug #12345 ? something like that? [12:46] bug 12345 in isdnutils (Ubuntu) "isdn does not work, fritz avm (pnp?)" [Medium,Fix released] https://launchpad.net/bugs/12345 [12:46] lol [12:46] zbenjamin, yeah ;) [12:47] Saviq: so i pass the desktop file hint to the app right? [12:47] just to make it work for now [12:48] zbenjamin, yes [12:48] zbenjamin, when launching under gdbserver [12:48] ok [12:48] zbenjamin, that will escape app lifecycle, though, just FYI [12:48] ahayzen, just had a look at https://code.launchpad.net/~andrew-hayzen/music-app/fix-1318008/+merge/222315 - according to LP, it seems there is a conflict that needs fixing, and I've added a comment for a small fix [12:48] zbenjamin, so it won't be suspended / killed as usual [12:49] Saviq: meh, so i cannot send a stop event to it with upstart? [12:49] dpm, cool thanks i must have branched before the other mp landed :).... inline comments \o/ [12:50] zbenjamin, that you can [12:50] zbenjamin, I mean that unity8+unity-mir won't [12:50] Saviq: another problem might be that gdbserver will receive the SIGSTOP when put into background ? [12:50] zbenjamin, that's the thing - it won't [12:50] Saviq: instead of the app right? [12:50] zbenjamin, 'cause that upstart job won't be considered as an app at all [12:50] Saviq: yeah lets say we have cgroups, and dropped the desktop hint [12:51] zbenjamin, only the app process, with unity-mir not caring why [12:51] zbenjamin, then the whole thing will be SIGSTOPed [12:51] zbenjamin, whole cgrouo [12:51] cgroup [12:52] dpm, updated please rereview when you have a moment [12:54] ahayzen, sure. I'm trying to figure out why the translations don't make it to the final .desktop file now [12:54] dpm, i've probably done something wrong somewhere i used the calculator mp as a template to my changes [12:55] Saviq: hm what about the authorize by apparmor thing? Would that behave different in that case? SIGSTOPPING gdbserver might be a problem because it has open sockets to the gdb running on the host.. [12:56] ahayzen, there needs to be a cmake rule that calls intltool-merge to put the translations in the final .desktop file [12:56] zbenjamin, no, we need to stop the whole cgroup [12:56] zbenjamin, and I don't think it would be a problem, you're debuggin after all, it will only be an issue after you time out? [12:56] ahayzen, I've not yet checked, doing several things at the same time atm :) [12:57] Saviq: yeah depending on how big or small the timeout is. Lets say you want to debug app resume [12:57] dpm, the change at line 55? [12:57] zbenjamin, then you only need 3s [12:57] zbenjamin, 'cause the app is stopped after 3s [12:57] zbenjamin, and for resurrection it'll be started again from scratch anyway, so any sockets will need to reconnect [12:58] Saviq: i thought it will just receive a SIGCONT? [12:58] zbenjamin, that's resume, resurrection is different [12:58] zbenjamin, resurrection is when it will first get SIGSTOP [12:58] zbenjamin, then SIGKILL through OOM [12:59] zbenjamin, and will get started again when you want it back [12:59] ahayzen, yeah, that should do it, but when I build the .click, the final .desktop file does not contain the translations [12:59] Saviq: will i get a application stopped signal from upstart in that case? [13:00] dpm, damn :/ [13:01] zbenjamin, the job will go away, yes [13:01] zbenjamin, with an "unexpected" exit status [13:02] ok , and also the small screenshot of the app in unity? [13:02] zbenjamin, we'll be screenshotting apps on suspend [13:02] Saviq: ok in that case QtC would assume the application has exited and close the debugging session [13:02] zbenjamin, then use those screenshots on app resurrection [13:03] zbenjamin, yeah, I think the only way to test resurrection would be to "persistently" make an app run under gdbserver [13:03] zbenjamin, and block on main() [13:03] zbenjamin, so that you can connect to it when it's resurrected [13:03] Saviq: what i'm doing is listening to : UbuntuAppLaunch.observer_add_app_stop(on_stopped, state), in the launcher. As long as that is fired when the app is killed we should be fine [13:04] zbenjamin, check out unity-mir http://bazaar.launchpad.net/~mir-team/unity-mir/trunk/revision/222 [13:04] zbenjamin, that reworked our upstart↔unity-mir comms [13:04] zbenjamin, should show you how we consider apps to behave [13:06] popey, do you mind if we move the apps upload meeting to 30 mins later? [13:08] sure [13:08] done [13:09] Saviq: does the SIGKILL happen always? or just if we need memory? === ara is now known as Guest42533 [13:10] zbenjamin, latter [13:10] zbenjamin, it's based on OOM [13:10] +killer [13:10] zbenjamin, we just set up scores (so that background apps are more likely to be killed than shell / foreground apps) [13:10] i see, thats why it never happend for me when i was testing with just one up :) [13:10] zbenjamin, and let the kernel do the rest [13:10] zbenjamin, yeah, try 15 or so ;) [13:11] zbenjamin, you can also mimic it by kill -SIGKILL the app when it's unfocused [13:11] zbenjamin, you will see it still there on the dash and tapping that will actually relaunch it [13:11] zbenjamin, only this case is special, SIGCONT apps won't (shouldn't) even really notice [13:12] ok good to know. The important thing however is that the upstart stop will be reported as stopped to the launcher. [13:12] upstart job i mean [13:18] ahayzen, hm, I cannot figure out why the translations are not merged into the .desktop file, but I have to go for lunch now. You can have a look at the calculator app to see if there is anything else missing. Running 'click-buddy --dir . --no-clean' and looking at the build directory will show you how the final .desktop file with translations would look like [13:19] Saviq: https://bugs.launchpad.net/unity-mir/+bug/1327216 [13:19] Ubuntu bug 1327216 in unity-mir "Use cgroups / apparmor profile to authorize application connections to MIR" [Undecided,New] [13:19] it seems as if the merge rule would not find the .po files where to read the translations to put into the .desktop file from [13:19] dpm, ok i'll have a look at the other two mps thanks for your help so far [13:20] zbenjamin, thanks [13:21] awesome, thanks! [13:21] thanks popey [14:09] Saviq: do you know if there is a way to query the desktop file path from appid? [14:09] zbenjamin, you should be able to just pass appid to --desktop_file_hint [14:09] zbenjamin, (we know, sneaky) [14:09] Saviq: ooo thats neat ;) [14:21] Saviq: works [14:22] zbenjamin, cool [14:22] Who tak I talk to about using QtLocation from C++? our doc site is non-existent :( http://developer.ubuntu.com/api/qml/sdk-1.0/QtLocation/qtlocation-module.html [14:23] ah, here it is: http://developer.ubuntu.com/api/qml/sdk-1.0/QtLocation.location-positioning-cpp/ [14:36] I'll take new translation for $100 alex! [14:37] שום שעון מתוחכם שמספק מתזמן, שעון עצר, התרעות ושעון עולמ [14:37] hows your hebrew? [14:37] we can do arabic if you like الوقت;وقت;الساعة;ساعة;مؤقت;ساعة إيقاف;ساعة ايقاف;منبه;تنبيه;إيقاظ;ايقاظ [14:38] עדיין עובד על זה [14:38] if that says "blame popey".... [14:38] ha, nice :) [14:38] popey, LOL [14:41] dpm, i still haven't figured out wht music-app doesn't translate i spotted a few differences and pushed them...any ideas? [14:41] ahayzen, just came back from lunch, let me try again [14:41] dpm, thanks ... i'll continue trying as well [14:43] ahayzen, ah, one of them is not necessary, the --add-comments argument in po/CMakeLists.txt is already specified, so you're adding it twice. But it should not affect the .desktop file translations [14:44] dpm, the calc/clock mps had it twice so i added it lol [14:44] ah :) [14:44] dpm, i'll remove it in the next commit i do [14:44] yeah, not too much of an issue [14:49] ahayzen, http://pastebin.ubuntu.com/7602186/ :) [14:49] evil braces [14:49] dpm: can you upload to the store? [14:50] popey, sure, do you have the click url? [14:50] dpm, ah damn lol how did it not explode with syntax errors lol [14:51] dpm: http://popey.com/~alan/com.ubuntu.clock_1.0.430_all.click [14:51] ahayzen, my thoughts too :) [14:52] dpm, dpm_ translations in my .desktop! woooo ... pushed please retest :) [14:53] \m/ [14:55] popey, https://myapps.developer.ubuntu.com/dev/click-apps/121/changerequest/ [14:56] approved [14:56] rock on [14:56] calculator now [14:56] r279 good dpm ? [14:56] dpm, once that is in is that all the translation work for now or is there anything else i need to do to music-app? [14:57] popey, yep, that's the one too [14:57] ahayzen, that should be pretty much it. I'm testing your latest changes now, we can probably get this into the store next [14:57] \o/ [14:58] k [15:01] ahayzen, top-approved [15:01] dpm, yey :D [15:02] dpm, do we poke popey with the click once it is merged? [15:02] ahayzen, he's aware we want to upload Music [15:02] sweet :) [15:03] he is [15:03] but we can poke him nevertheless, just for the fun of it ;) [15:03] \o/ [15:03] :) [15:04] balloons, did you see my reply to your e-mail re: calendar? What do you think we need to do next to get the landings back in shape? [15:04] dpm, I'm working on a branch at the moment to fix the test with a workaround [15:04] but no, I've not seen your reply [15:05] balloons, ah, nm, my mistake, you replied already but hadn't checked my e-mail since. So forget the question :) [15:05] dpm, heh :-) [15:07] balloons, ok, read your e-mail now, sounds good to me, thanks! [15:07] excellent [15:08] dpm: please upload http://popey.com/~alan/com.ubuntu.calculator_1.3.279_all.click [15:08] elopio, I need to come back to you about the reminders tests. Have you had the chance to look more into them? [15:08] dpm: not yet, sorry. [15:12] elopio, np, when do you think you might be able to? We're a bit blocked with it right now. Or could you give us some pointers on your findings so far, so that someone else can take over? [15:12] popey: ping [15:12] popey, https://myapps.developer.ubuntu.com/dev/click-apps/154/changerequest/ [15:12] dpm: you can run the tests on your machine and you will see the segfault. [15:13] checking the logs might give more information, but probably we will have to go back to mardy to understand what changed. [15:14] popey: after new header dropdown for sync monitor , does icons looks too big on device too like desktop , http://i.imgur.com/rNjHQB0.png ? [15:14] dpm_: approved [15:14] awesome [15:15] music ready or shall I do terminal instead? [15:16] popey, still running in jenkins i think [15:18] ok, terminal it is [15:18] popey, music has merged \o/ [15:18] lol [15:24] elopio, hm, I cannot get the reminders tests even to run. I've got evernote-sdk-python3 installed, but I'm getting these errors: http://pastebin.ubuntu.com/7602370/ - any ideas? [15:24] dpm: you need to install the evernote sdk from the apps ppa [15:25] kalikiana: I thought I could simply run the qml files in our unit test dir using qmlscene, like this http://pastebin.ubuntu.com/7602379/ [15:25] kalikiana: but it doesn't work while qmltestrunner works fine. Do you know how I get the Qt.test to work? [15:25] or to be ignore [15:25] elopio, that's where I installed it from, it's not in the archive [15:26] dpm: run the tests with autopilot3 run reminders [15:26] ah [15:30] elopio, I just ran the tests on my desktop - is this similar to what you're getting? -> http://pastebin.ubuntu.com/7602388/ [15:30] Hi all! Anyone know how to fix "" value in debugger? (Qt5.2.1 ubuntu-sdk, Qt Creator 3.0.1) [15:31] dpm: the same two error, but maybe you already had an evernote account [15:32] oh, I see the segfault now [15:32] Jun 6 17:26:52 avenc kernel: [277384.337262] reminders[1841]: segfault at 1b ip 00007fa19ca91034 sp 00007fff502f55a8 error 4 in libaccounts-qt5.so.1.2.0[7fa19ca81000+1a000] [15:33] I think that's the new thing, I suspect it's the cause of the failure. [15:34] dpm_: http://popey.com/~alan/com.ubuntu.terminal_0.5.73_armhf.click please upload [15:35] mardy, it seems that the Reminders app tests are failing on a libaccounts-qt5 segfault ^^ - could you give us a hand figuring out what's going on?. The full log: http://pastebin.ubuntu.com/7602388/ [15:41] popey, https://myapps.developer.ubuntu.com/dev/click-apps/157/changerequest/ [15:42] elopio: I commented on https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/closeActionsPopover/+merge/222261 [15:42] elopio: I'm not sure what to do with this test, but your comment makes sense [15:42] elopio: I'll have dinner now, we can discuss it afterwards (in 20 min) [15:43] dpm_: done, thanks [15:44] Is music ready? [15:48] t1mp: ok. [15:49] t1mp: ping me when you are back. If it's easier to test on autopilot than on qml, then we can just move it to a new suite. [15:59] elopio: I'm back [15:59] elopio: yes, this one seems much easier to test from autopilot than qml [16:00] t1mp: we have added a couple of regression tests for the widgets before, so that's ok. But I wouldn't like them to be mixed. [16:00] t1mp: what about making a module in ubuntuuitoolkit/tests/widgets and put it there? [16:00] or ubuntuuitoolkit/tests/components [16:01] elopio: yes, that makes sense [16:01] elopio: I guess apps that have their own components do the same? [16:01] elopio, +1. Best for the toolkit itself to keeps it's test seperate [16:02] elopio: sure. you are right that it doesn't belong with the cpo tests [16:08] popey, sorry, hadn't seen the question. Yes, music is ready [16:08] ok, will do now [16:08] dpm_: what rev? [16:09] popey, latest trunk, r484 [16:10] t1mp: actually, currently on the apps I have been mixing the tests for the helpers and the tests for the functionality. I should probably stop doing that. [16:12] popey, I need to go in a few mins. Do you have someone else that can do the .click uploads? [16:13] popey, actually, nevermind, appointment postponed, I'll still be around for a bit [16:13] yes [16:13] balloons can do it too [16:15] * balloons flexes [16:16] popey, it seems that phablet-screenshot is broken again with image 71 - do you know of any workaround? [16:17] yes [16:18] adb shell mirscreencast -m /tmp/mir_socket -n1 -f $MIRFILENAME [16:19] change /tmp/mir_socket to /var/run/mir_socket === gatox is now known as gatox_lunch [16:24] ahayzen: does mediascanner only scan ~/Music? (if I move my music to ~/_music - will mediascanner still find it?) [16:24] popey, err i think so try it :) [16:24] popey, i assume so as thats how the autopilot used to work [16:25] lets see [16:27] mzanetti, still around? [16:27] dpm_: yes sir [16:27] :) [16:28] mzanetti, something totally unrelated to reminders: so cwayne has pinged me to see if we could get some packages into the right-edge PPA for the MAE image [16:28] dpm_: what packages? [16:29] mzanetti, these fix some internationalization issues and they're already in image 78, but the MAE image is stuck at 58 [16:29] dpm_: for apps? or unity8? [16:29] mzanetti, for Unity8. The main one would be indicator-network. The latest version from the archive fixes the i18n issues [16:30] dpm_: uh... I'm not sure I want to open that can of worms now... we have a quite heavily modified version of unity8 in that ppa (after all that's the reason why its stuck at rev 58) [16:30] dpm_: if its a few simple commits we might be able to cherry pick them over [16:31] but we can't just upgrade unity8 in there atm [16:31] let me have a look at the code [16:34] mzanetti, I believe it'd be 2 commits: r334 and r340. The second one is trivial, not sure about the first one. http://bazaar.launchpad.net/~indicator-applet-developers/indicator-network/trunk.14.10/changes/342?start_revid=342 [16:35] dpm_: let me have a dry run with those commits. might take a bit [16:36] mzanetti, thanks a lot [16:38] dpm_: can you please upload http://popey.com/~alan/com.ubuntu.music_1.3.484_armhf.click [16:39] popey, https://myapps.developer.ubuntu.com/dev/click-apps/143/changerequest/ [16:40] done [16:41] dpm_, so so I supersede https://code.launchpad.net/~yohanboniface/ubuntu-calendar-app/DatePicker/+merge/212163 and land it? [16:42] balloons, sounds good. I guess you simply branched off it, applied your changes on top and you'll submit a new MP? [16:42] dpm_, yes I branched from it, fixed things up.. I can just do a new MP then [16:43] +1 === salem_ is now known as _salem [16:49] odd, I'm not seeing music 484 in the store on my phone [16:50] ahh, i have newer [16:52] popey, are you running the mediascanner2 test still? [16:52] i was [16:52] :) [16:53] click unregged [16:53] now i see the updates [16:54] 'Musique' yey :) [16:57] t1mp, ahayzen: is the bottom panel documentation the one shown at http://developer.ubuntu.com/api/qml/sdk-14.10/Ubuntu.Components.Panel/ ? [16:58] nik90, we use a panel yes i believe that is what you use [16:59] popey, dpm, i switched to french, the tested the music-app, then switched back to english and restarted...now my infographic for music is still in french? [17:00] "3 Morceaux joues aujourd'hui" [17:01] nik90, we converted our toolbar to use the Panel so you can look at what we have done if you get stuck [17:02] ahayzen: sweet..thnx [17:03] has the upstart-app-launch rename broken Run on Device from qtc? [17:03] doflaherty, yeah it did for me when i tried yesterday [17:04] doflaherty, i ended up building a .click then pushing to device and installing that [17:07] s/UpstartAppLaunch/UbuntuAppLaunch/g in qtc_device_applaunch.py seems to fix it [17:09] popey: cheeky Business Time :P [17:11] nik90, \o/ lol [17:11] oh, and found the bug report [17:11] nik90, have you seen youtube videos now play in the latest image? [17:13] ahayzen: yeah I read the ML..yet to try it out on the phone [17:14] nik90: :D === _salem is now known as salem_ [17:15] ahayzen: have you landed the different bottom panel states? === gatox_lunch is now known as gatox [17:29] nik90, different states? [17:30] nik90, we just have visible || not visible [17:30] nik90, we plan to link it to a page stack animation for the third state if that is what you mean? [17:30] ahayzen: I meant like swiping little and showing toolbar while swiping up completely shows the now playing page [17:30] ahayzen: yeah I was referring to that [17:30] ahayzen: so my question is if the third state an actual music playing page component? [17:31] nik90, yeah it will be a page on a pagestack [17:31] or just a item{} which shows the now playing stuff? [17:31] ahayzen: did you guys add a page inside the panel{} ? [17:32] nik90, no its in a pagestack somewhere [17:32] nik90, but the animations are WIP [17:32] ahayzen: then how do you transition from a panel item to a full page inside a pagestack? [17:32] nik90, and t1mp wasn't sure how everything will link yet [17:33] ahayzen: ah ok [17:33] nik90, https://bugs.launchpad.net/music-app/+bug/1247865 [17:33] Ubuntu bug 1247865 in Ubuntu UI Toolkit "Pages Stack misses animations" [High,In progress] [17:33] ahayzen, ah, strange, that looks to me like a bug in libusermetrics [17:34] dpm_, do you experience the same? [17:36] ahayzen, I'm experiencing the opppsite: I've never gotten the infographics to show translated [17:36] dpm_, ah [17:36] dpm_, it is weird only music is translated [17:37] dpm_, maybe it gets updated when the app sets something to it? [17:37] ahayzen, yeah, in my case I did a phone call, tried to play lots of music, take pictures... but it is stuck at showing the call info on the greeter [17:38] dpm_, if you double tab on them are some translated or are they all stuck? [17:42] oh [17:42] oh [17:42] I never realized you could double-tap! [17:43] dpm_, hah no way really? [17:43] dpm_, you learn something new every day :) [17:43] oh wow, indeed :) [17:45] dpm_, i'll have a play about after dinner to see if i can figure out exactly what is going on then make a bug, cyas [17:46] ok, cool === salem_ is now known as _salem === _salem is now known as salem_ [18:05] hi dpm_ ! I'm dual booting in my phone and after yesterday's android upgrade it doesn't seem to be working anymore. would you know something about? can give any advice/things to check? [18:08] ohh la_juyis you are about :-) [18:08] hi la_juyis, the best person to ask is ondra, but I'm not sure if he's around. How did you do the Android upgrade? If you didn't use the option from the dualboot app, then probably Android overwrote your recovery containing Ubuntu with the stock recovery and you might need to re-install Ubuntu [18:08] mwahahah [18:09] la_juyis, https://wiki.ubuntu.com/Touch/DualBootInstallation#If_dual_boot_replaces_the_recovery_partition.2C_can_I_do_an_Android_upgrade_with_dual_boot_installed.3F [18:10] i did use the dualboot option, and *also* tried reinstalling, with no luck so far. let's see your link ;) [18:10] oh, well, I did use that :/ [18:11] I'm atm trying with dualboot.sh channel, maybe that does the trick :) [18:11] la_juyis, hm, sorry, that's all I know :/ If all fails, see if you can get hold of ondra on #ubuntu-touch [18:12] dpm_, popey this should be all set. https://code.launchpad.net/~nskaggs/ubuntu-calendar-app/datepicker-test-fixes/+merge/222369. I'm concerned about the issues with trusty though. syncmonitor isn't found [18:13] balloons, could you update the description on https://code.launchpad.net/~nskaggs/ubuntu-calendar-app/datepicker-test-fixes/+merge/222369 ? Particularly removing the outdated info about the emulator not being available, and perhaps mentioning what you added on top of the superseded MP, etc [18:13] it's supposed to be in the ppa, right? I'll check and see if calendar includes the ppa [18:13] balloons, weird, it should find it. It worked for mihir this morning [18:13] and he's on trusty [18:14] dpm_, yes but I don't see the jenkins job as including the ppa [18:14] so we'll need to add it [18:14] hence, it's not finding it, not installing it, etc [18:14] balloons, oh, weird that it doesn't include the PPA, as I thought it was using the EDS plugin from there [18:15] I don't see it, but I'll have to confirm to be sure [18:16] hey mihir :-) [18:16] hey balloons [18:17] balloons: i installed sudo apt-get install qtdeclarative5-ubuntu-syncmonitor0.1 [18:17] mihir, so the longstanding branch should be ready to land.. I superseded it with some tweaks to the new_event test, and a workaround for the bug we found [18:18] dpm_, thanks :) [18:18] okay, so now jenkins should pass pass all the tests correct ? [18:20] mihir, yes and indeed they are; we're just trying to clear up the syncmonitor not being on trusty issue in jenkins [18:21] dpm_, I'm concerned that the package built in the builder, but not for the testrun.. odd eh/ [18:21] balloons: great !!! \m/ [18:21] fginther, I don't see an active vanguard, so you're up my friend. Can you ensure the core apps ppa is used for the trusty tests for calendar app? [18:22] balloons, sure, I'll give it a try in a moment [18:22] fginther, at the moment it's not able to find syncmonitor to install; and it comes from that ppa. It's kind of weird the build seems to work tho [18:22] note, it's in utopic, so just trusty needs it [18:28] ybon: we are done with changes, we'll land long awaited MR soon :) === dpm_ is now known as dpm-afk === salem_ is now known as _salem === _salem is now known as salem_ [19:35] mhall119: happy with the app dev sessions? [19:45] 18:57:41 < nik90> t1mp, ahayzen: is the bottom panel documentation the one shown at http://developer.ubuntu.com/api/qml/sdk-14.10/Ubuntu.Components.Panel/ ? [19:45] nik90: that's the only bottom-edge component we have now in UITK [19:45] nik90: it is quite basic, or shall I say 'flexible' :) [19:46] t1mp: :)...the reason I asked it was because it looked too generic in the wording used..it said it is the panel that can be swiped all edges and not the bottom edge in particular.. [19:46] t1mp: that spooked me a little..hence the enquiry [19:47] nik90: it is the parent class of the Toolbar that we used to have.. Panel was implemented before we deprecated the toolbar and gave it to the apps [19:47] nik90: but still it can be useful :) [19:47] nik90: you are right, perhaps we should update the wording... or we do that when we add more functionality to it [19:48] t1mp: true..I am personally fine with it. [19:48] t1mp: hey can you look at https://drive.google.com/file/d/0B-I2gOjaTc7dVzF3ZjI1NEctWmM/edit?usp=sharing ? [19:49] t1mp: I want to know how to implement the add city pull down animation thing. I mean I know it can be done easily using qml animations, but how do I make it go back up when the user lets go of it. [19:49] nik90: that looks sleek [19:49] t1mp: it looks quite similar to the pull down refresh functionality that zsombi was implemented [19:49] nik90: I wonder whether the pull-to-refresh offers something like that [19:50] t1mp: no..I asked zsombor if the pull-to-refresh allows app devs to change the thing shown when you pull it down [19:50] t1mp: he said you can only modify the string but not the visual effect itself [19:50] t1mp: just something to prick your brain during the weekend :P [19:51] s/prick/pick* [19:51] nik90: https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/pullToRefresh/+merge/219090 [19:51] nik90: yeah, it seems like the pulltorefresh does not offer the possibility to have a custom delegate [19:51] nik90: well.. you could change the text to "pull to add new city", but you won't have the clock visuals [19:52] t1mp: you think we both can convince zsombi to have a custom delegate :P [19:52] t1mp: the visuals are an important part of the clock app reboot :) [19:52] nik90: I think we can, but I doubt we can convince him to do it before RTM [19:52] t1mp: true.. I will have a look at the MP to see if I can borrow the code from that for now [19:53] nik90: you could do it with a custom style [19:53] by overriding PullToRefreshStyle, l. 367+ [19:53] t1mp: but I don't know how to use a custom style since there isn't documentation on how to do that [19:53] nik90: but that's the most complicated part of the pulltorefresh [19:54] nik90: you can add style: Theme.createStyleComponent("MyCustomStyle.qml", myComponent) to your component [19:55] uhm.. but probably you have to do something to the theme so that it can find MyCustomStyle.qml [19:55] best ask zsombi [19:55] * t1mp gotta go now [19:55] t1mp: will do..thnx [20:11] nik90: sessions look good, can we condense them all into just one room, or do we still need that second? [20:12] fginther, any luck with calendar? [20:15] balloons, the tests are running on trusty now, with 1 failure [20:16] balloons, looks like the same failure on utopic [20:18] mhall119, it looks like there is enough space if the one from each of the two double sessions on tuesday move to the spare slots on wednesday? assuming nik90 or anyone else doesn't need any more [20:22] fginther, a failure? everything's been showing as passing on utopic [20:23] balloons, http://91.189.93.70:8080/job/generic-mediumtests-utopic/413/testReport/calendar_app.tests.test_calendar/TestMainView/test_new_event_with_mouse_/ [20:24] balloons, looks like the failure crept in while the utopic system was down [20:26] fginther, ohh that's trunk.. yea, there's a toolkit bug and it doesn't pass === ken_ is now known as kenvandine [20:26] I'm wanting to land: https://code.launchpad.net/~nskaggs/ubuntu-calendar-app/datepicker-test-fixes/+merge/222369 which works around it [20:27] so anyways, if it's running now, we should be good to land [20:27] https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1326963 [20:27] Ubuntu bug 1326963 in Ubuntu UI Toolkit "PageStack.push() eats header action popover close" [Critical,In progress] [20:29] balloons, ahh, cool [20:31] fginther, thanks.. I'll land it now :-) [20:32] balloons, you're welcome [20:44] fginther, interesting to note it fails to import only on trusty.. ohh right, you need the sdk ppa for trusty too [20:44] fginther, honestly all the core apps ppa's will need this. The sdk is frozen in time in trusty's archive [20:45] balloons, then is there a point to building on trusty? [20:45] fginther, heh, I kind of always felt we would hit this point.. [20:45] :-) [20:45] if trusty wasn't an LTS this would be easier [20:46] balloons, right. I suppose we can always add the PPA and defer it a bit longer [20:47] fginther, in theory everything should continue to work with the ppa.. But yea, all of them will need it for trusty [20:47] it's probably the right decision to continue [20:48] it just makes for fun friday afternoons is all [20:48] balloons, those are the ones that never end (so you just have to start drinking anyway) [20:48] balloons, I'll add it and give your MP another build [20:49] balloons, this ppa correct? https://launchpad.net/~ubuntu-sdk-team/+archive/ppa [20:57] looking [20:57] fginther, it is, but I have a concern [20:57] popey: does the clock app works ? [20:58] fginther, if you have a look, the toolkit updates are in there which is good. However, the toolkit autopilot helper is not: https://launchpad.net/~ubuntu-sdk-team/+archive/ppa?field.series_filter=trusty [20:58] it will still fail without the helper [21:00] balloons, do you know what provides the helper? [21:00] fginther, yes, ubuntu-ui-toolkit-autopilot [21:01] popey: http://paste.ubuntu.com/7604169/ [21:02] fginther, that is the intended ppa [21:02] balloons, that is provided by the ubuntu-ui-toolkit source package, which is in there: https://launchpad.net/~ubuntu-sdk-team/+archive/ppa/+packages?field.name_filter=toolkit&field.status_filter=published&field.series_filter=trusty [21:02] popey: nik90 http://paste.ubuntu.com/7604179/ [21:02] fginther, so I'm just crazy then.. godo [21:03] fginther, heh, it's that time [21:03] so yea, go for it [21:04] building [21:14] balloons, that was a step backwards [21:14] balloons, trusty test is hitting an import error now [21:15] fginther, ugh.. link? [21:15] 91.189.93.70:8080/job/generic-mediumtests-trusty/2920/consoleText [21:15] or 91.189.93.70:8080/job/generic-mediumtests-trusty/2920/testReport/junit/unittest.loader.ModuleImportFailure.calendar_app/tests/test_calendar/ [21:15] ImportError: cannot import name pickers [21:17] fginther, right that's the error we are trying to solve [21:17] the new sdk helper should have it [21:18] balloons, is 0.1.46+14.10.20140520-0ubuntu1~0trusty2 new enough? [21:18] fginther, I don't see that it installed ubuntu-ui-toolkit-autopilo [21:19] fginther, I'd have to unpack it to check. I'm on utopic [21:19] let's see [21:20] fginther, yes indeed I don't see pickers in there [21:20] it's too old [21:21] balloons, I assume the sdk team pushes new versions manually? [21:21] fginther, to there ppa yes [21:21] afaik [21:22] I think this deserves larger discussion. However, can we disable trusty for calendar for the moment? [21:22] balloons, sure [21:22] the core apps ppa is to deal with a new depends that isn't in trusty either [21:22] this is becoming painful fast [21:24] balloons, building again [21:40] balloons, http://91.189.93.70:8080/job/ubuntu-calendar-app-ci/533/console [21:42] fginther, :-) excellent [21:42] I'll top approve and see if it will land now === salem_ is now known as _salem === _salem is now known as salem_ === salem_ is now known as _salem