/srv/irclogs.ubuntu.com/2014/09/10/#ubuntu-app-devel.txt

=== _salem is now known as salem_
=== salem_ is now known as _salem
=== chihchun_afk is now known as chihchun
Akiva-Thinkpadmhall119, I didnt do shades for cool grey to reinforce the guideline that coolgrey, at least according to the webpage, is for typography. Otherwise; very cool; thanks03:31
=== chihchun is now known as chihchun_afk
=== _salem is now known as salem_
=== salem_ is now known as _salem
=== chihchun_afk is now known as chihchun
dholbachgood morning06:32
=== chriadam is now known as chriadam|away
zbenjaminogra_: any idea where this could come from? http://pastebin.ubuntu.com/8307166/07:47
zbenjaminogra_: ok nevermind, seems the app was already running07:49
mardydpm: hi! Could you please generate a new click package for your account-plugin branch?08:07
nerochiarozbenjamin: does qtcreator support building qmake based projects in chroot ?08:15
zbenjaminnerochiaro: no , sadly not08:16
nerochiarook, i'll build on device i guess08:16
zbenjaminnerochiaro: or migrate to a cmake project ;)08:16
nerochiarozbenjamin: it's not my code08:17
zbenjaminok08:17
nerochiarozbenjamin: have you seen the branch i suggest adding to your gallery branch ?08:17
zbenjaminnerochiaro: yeah, sorry i totally forgot about it yesterday, We had some discussions going on08:17
JamesTaitGood morning all; happy Swap Ideas Day! :-D08:19
nerochiarozbenjamin: what do you think then ?08:19
zbenjaminnerochiaro: should i just merge it into my branch?08:19
nerochiarozbenjamin: yes, that is what i suggest08:20
nerochiarozbenjamin: then i'll review your branch08:20
zbenjaminlet me see08:20
nerochiarozbenjamin: also i get this when i ask qtcreator to install devtools on the device: http://pastebin.ubuntu.com/8307355/08:22
nerochiaroon mako with rtm proposed08:22
zbenjaminnerochiaro: i guess that functionality is broken, but we can make it still work08:24
zbenjaminnerochiaro: there should be a file called /usr/share/qtcreator/ubuntu/scripts/device_developertools_packages08:24
zbenjaminnerochiaro: it contains all the required packages for the developer mode , some of them are deprecated i guess08:25
zbenjaminnerochiaro: just copy the packages and try to install it on the device08:25
zbenjaminnerochiaro: after you have ruled out the obsolete packages , qtc will still show that your device is not in developermode because of the missing packages, but you can use it anyway08:26
nerochiarozbenjamin: ok. the obsolete package is dh_make08:26
zbenjaminnerochiaro: ok just do not install it , then your device should be useable to build qmake08:27
zbenjaminprojects08:27
nerochiarozbenjamin: yep, trying that. thanks08:27
zbenjaminnerochiaro: ok i merged your branch08:29
nerochiarozbenjamin: what was your branch again ?08:29
zbenjaminnerochiaro: https://code.launchpad.net/~zeller-benjamin/gallery-app/sdkcompat/+merge/23388408:29
nerochiarozbenjamin: ok i'm testing it. regarding building on device, it seems that dh_make is actually needed. it's trying to use it to build08:38
karniguys, what was the switch to click install to avoid signature verification? can't find it :(08:39
zbenjaminkarni: --allow-untrusted08:40
karnizbenjamin: thanks!08:40
zbenjaminkarni: the SDK is doing all that for you btw ;)08:40
karnizbenjamin: that doesn't apply to managers just wanting to install a click ;)08:40
zbenjaminkarni: ok ;)08:41
victorppopey, ping08:42
popeyyo08:42
victorppopey, do you know how to turn off the signature checks on click packages?08:42
popeyi know what you're going to ask ☻08:42
popeywhat image you on?08:42
victorpis it that ? :)08:42
popeythat has been fixed AIUI08:42
* victorp checks08:43
zbenjaminnerochiaro: can you get dh_make from some other package?08:43
victorp2708:43
victorppopey, ^^ on rtm propose08:43
popeylemme test here, i just updated here too08:43
victorpit is the latest one08:43
zbenjaminnerochiaro: weird dh-make is still available on my host08:43
popeyvictorp: worked fine here, I'm on 2708:44
victorpwell, doesnt here..08:44
nerochiarozbenjamin: it is on my host too, but not on the phone08:45
victorppopey, updating using ota?08:45
popeyyes08:45
popeyvictorp: can you send me the click you're using?08:46
victorppopey, sure08:46
nerochiarozbenjamin: is dh_make available on your device ?08:47
zbenjaminnerochiaro: there seems to be a package called dh-make08:48
popeyvictorp: ok, fails for me too08:48
victorppopey, I got another 20 like that08:48
victorp:)08:48
popeyvictorp: pkcon --allow-untrusted install-local foo.click08:49
brendandpopey, victorp - what's wrong? signed clicks don't install or unsigned clicks don't install?08:50
popeyi think he's sorted now08:51
brendandpopey, needs to work ootb though08:51
popeypreviously pkcon didn't have that parameter, click did08:51
popeyit does ☻ if you specify the parameter08:51
brendandpopey, oh so it's specifically to do with running pkcon from cli?08:52
brendandpopey, not impacting the click scope08:52
popeyyes08:53
victorpbrendand, unsigned, I didnt know we needed to add that flag. no probs08:53
victorpI couldnt see an email on ubuntuphone about that08:53
nerochiarozbenjamin: back to testing your branch: it all works great. except that when i rename the package in the manifest.json.in and rebuild, it builds fine but when it tried to isntall on device it complains that the package is untrusted and refuse to install it08:53
zbenjaminnerochiaro: update your qtcreator08:54
zbenjaminnerochiaro: one question if you "adb shell" in are you still root?08:55
nerochiarozbenjamin: yes08:55
nerochiarozbenjamin: but i haven't reflashed since yesterday08:56
zbenjaminnerochiaro: and which channel do you use again?08:56
nerochiarozbenjamin: ubuntu-touch/ubuntu-rtm/14.09-proposed08:56
zbenjaminogra_: ^^^^ nerochiaro is on rtm proposed and his adb is executed as root. Didn't that land there as well?08:57
nerochiarozbenjamin: ogra_: i last flashed yesterday08:58
victorppopey, yes, that work. Thanks! back to doing some development09:00
popey\o/09:00
nerochiarozbenjamin: ok, all green on your branch now that i updated qtcreator09:09
nerochiarozbenjamin: approved09:09
zbenjaminnerochiaro: awesome thx :)09:10
ogra_nerochiaro, zbenjamin not in rtm yet09:10
ogra_silo will be ready today for it though09:10
zbenjaminogra_: ah ok09:13
popeymihir: PING! (I get the chance to ping you today) :D09:20
popeymihir: https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1367654 i filed that because the tests seem to be failing in jenkins terribly today09:20
ubot5Ubuntu bug 1367654 in Ubuntu Calendar App "autopilot tests failing mako 235 ImportError: No module named 'address_book_service_testability'" [Undecided,New]09:20
mihirpopey: pong, I am honored :)09:20
popeyheh09:21
mihirpopey: is it failing on device ?09:21
popeyyes09:22
mihirpopey: i believe module name has been changed of address_book_service_testability09:23
mihirpopey: i can see this in https://code.launchpad.net/~nskaggs/ubuntu-calendar-app/fix-1359167/+merge/23338709:24
mihirpopey: it has been replaced by address-book-service-testability09:25
sturmflut-workI noticed that the PickerPanel QML component does not support the mouse wheel out of the box on the Desktop. Is this intentional? Do I have to add a MouseArea somehow?09:25
popeyhmm, so did that merge break it?09:25
mihirpopey: this can be issue, as it passes Jenkis on MRs but not on device ?09:25
mihirpopey: nope, i had issues but balloons asked me to install this package and it did work well09:26
popeyaddress-book-service-testability isn't installed on device...09:27
popeyso yes, I can see how that would fail.09:27
mihirpopey: hmmm09:27
mihirpopey: does that required any action on code side ?09:29
* mihir looks for debian lists 09:30
popeywell, i don't quite understand this, does this mean we need address-book-service-testability on the phone?09:30
popeywe don't install other application specific autopilot related packages09:30
mihirpopey: may be yes..i installed this on my machine and then it APs works, otherwise it failed because of same reason..not sure on device too.09:31
popeyI'll poke balloons when he wakes, thanks ㋛09:31
mihirpopey: np :)09:31
mihirnik90: ping09:38
dpmsturmflut-work, I think this might rather be a bug than intentional09:45
sturmflut-workdpm: I thought so09:49
sturmflut-workpopey: Since you're always looking for things to talk about on the Ubuntu UK Podcast, I would suggest more Ubuntu Touch news. The Meizu MX4 was announced a week ago, it might be the right time.09:56
popeysturmflut-work: interesting ☻09:58
sturmflut-workpopey: And maybe an interview with an Ubuntu Touch developer?09:58
popeysmack we interviewed nik90 some time back, probably due to interview more actually... good call09:59
Akiva-Thinkpadmorning10:05
sturmflut-workdpm: I think that in the end Ubuntu.Components.Pickers.Picker has no support for the mouse scroll wheel. Should i file a bug somewhere?10:23
dpmsturmflut-work, yes, on https://bugs.launchpad.net/ubuntu-ui-toolkit, please10:25
nik90mihir: pong10:42
mihirnik90: i am not sure what kind of issue I am facing for running CMAkelist10:54
mihirnik90: any idea , http://i.imgur.com/2F15gAb.png10:56
jppiiroinenhowdy, about the ubuntu-sdk meta-package. I have a clean 14.04.1 installation and when I try to install the ubuntu-sdk, it wants to remove modemmanager. is that a bug or feature?10:58
sturmflut-workjppiiroinen: Hm, I'm on 14.10 and have both installed. It might be because of ofono11:05
nik90mihir: can you perhaps try on your phone by creating a 14.10 armhf kit perhaps11:06
nik90zbenjamin: could you help mihir..after the cmake change, he is unable to run calendar app on his desktop. He sees https://i.imgur.com/2F15gAb.png11:07
nik90mihir: why do you have unresolved dependencies in your update manager?11:08
zbenjaminnik90: mihir: try to remove the old builddirectory and user file11:09
mihirzbenjamin: i cleaned whole directory and pulled project from bzr11:11
zbenjaminmihir: hmm let me try again11:11
nik90zbenjamin: it works for me on 14.1011:11
zbenjaminnik90: ok11:12
zbenjaminmihir: your QtC is up 2 date?11:12
zbenjaminmihir: i added lots of fixes to support core apps last week or so11:12
mihirzbenjamin: Hmm may be, i haven't run any update manager since long , may be i can try updating my VM and will try again11:13
zbenjaminmihir: you use the SDK ppa on trusty right?11:13
zbenjaminmihir: always stay up 2 date on this , we have releases almost every week11:14
mihirzequence: i am using utopic11:16
nik90zbenjamin: he is running utopic on vm which is why it is not probably up to date. I tend to forgot to update mine as well sometimes11:18
zbenjaminok11:18
=== MacSlow is now known as MacSlow|lunch
=== chihchun is now known as chihchun_afk
rpadovanidpm, unfortunately, tomorrow I'm not at home, so I can't join the weekly hangout12:05
dpmrpadovani, thanks for the heads up. Would Friday work for you?12:06
dpmhi bzoltan, if I import Ubuntu Components 1.1 in an app, will the app run in trusty desktop?12:07
rpadovanidpm, will be better, yes12:07
=== MacSlow|lunch is now known as MacSlow
rpadovanithanks dpm :-)12:09
nik90mihir_: did it woek?12:09
nik90dpm: no you can't launch in trusty desktop if you import 1.112:10
mihir_nik90: i forgot my external today , so will do it once i reach home :)12:12
nik90ah ok12:12
bzoltandpm: no, it will not12:13
dpmbzoltan, nik90 thanks. Is there any plan to backport 1.1 components? Note I'm not asking for a backport, just trying to find out what the plan is12:13
nik90dpm: as I understand, 1.1 requires qt 5.3 which is not being planned to be backported..so my guess is no. However I will let bzoltan confirm that.12:14
bzoltandpm: No, and the backport become impossible since we moved on to Qt5.3 as the 1.1 components need features from the 5.3 what are not available in 5.212:14
dpmok, thanks guys12:14
bzoltannik90:  you, you ... you are faaast12:14
nik90bzoltan: lol12:15
nik90bzoltan: I have irc sentence templates (j/k)12:15
bzoltannik90:   We should improve that to an app template ... you type import ... hit tab-tab and the killer app is ready12:15
nik90:)12:16
mivoligomzanetti: ping12:27
mzanettimivoligo: hey12:28
mivoligomzanetti: I've pushed first five level backgrounds to launchpad12:29
* mzanetti looks12:29
mzanettimivoligo: awesome, can you propose it for merging?12:30
mivoligomzanetti: for now it's just svg files without level description files12:30
mzanettiin here: https://code.launchpad.net/~mpredotka/machines-vs-machines/level-backgrounds-sand12:30
mzanettiyou can find a link "Propose for merging"12:30
mivoligook12:30
mzanettimivoligo: then add a commit message in the next page and optionally a description12:30
mzanettithe rest should be fine as is12:30
mivoligodone12:31
mzanettimivoligo: perfect12:32
mzanettimivoligo: now, please always start with a new branch if you do some changes12:32
mivoligomzanetti: ok12:32
mzanettimivoligo: the reason for that is simple: if for some reason I would reject a branch, its getting a mess to remove that if you already have built on top of it12:32
mzanettithat's why its better to work with parallel instead of chained branches12:33
mivoligomzanetti: but should I revert back to trunk, or just make some changes and push to new branch?12:34
mzanettimivoligo: best thing is if you just do a fresh bzr branch lp:machines-vs-machines each time12:34
mzanettimivoligo: then you'll have multiple dirs locally12:35
mzanettionce the are merged, you can delete them again from your hard dist12:35
mzanettidisk12:35
mzanettimivoligo: if you don't want to download the whole thing each time, you can also do something like this:12:35
mzanettionce do: bzr branch lp:machines-vs-machines machines-vs-machines-clean-checkout12:36
mzanettiand then just do: bzr branch machines-vs-machines-clean-checkout my-new-feature-or-bugfix-branch12:36
mzanettiso you just branch things around locally without the need of downloading stuff12:37
mzanettimivoligo: as always, feel free to ping me for help if you run into troubles12:37
mivoligomzanetti: so switching between brunches does not work like in git at all?12:38
mzanettimivoligo: no. bzr doesn't support multiple branches within one directory12:38
mzanettimivoligo: there is a unsupported plugin which I've been told works fine12:38
mzanettibut by default you need a new directory for each branch12:38
mzanettimivoligo: http://doc.bazaar.canonical.com/plugins/en/colo-plugin.html12:39
mivoligoI need to think the bzr way ;)12:40
kalikianarenatu: ping12:42
renatukalikiana, hi12:42
kalikianarenatu: in the design hangout removing the border of text fields came up. contacts is doing that when you add a new one it seems12:43
kalikianacan you tell me how you do it?12:43
kalikianado you use a custom entry?12:43
renatulet me check12:44
kalikianamaybe it makes sense to have an official API like borderStyle for example if that makes it easier12:44
renatukalikiana, http://bazaar.launchpad.net/~phablet-team/address-book-app/staging/view/head:/src/imports/ContactEdit/TextInputDetail.qml#L7612:45
mivoligomzanetti: about the background sizes: the „playable” area is 10 units wide and 5 units high, on the left and right side of that area there's 1 unit wide margin. Over and below there's 2 units margin. Does it make sense to you?12:49
kalikianarenatu: you're customizing style: TextFieldStyle is that all you're doing? I think that could go in the docs, which don't really demonstrate styling tweaks12:49
kalikianaso no ugly hacks there12:49
mzanettimivoligo: hmm... not really12:49
mzanettimivoligo: so the thing is we can't define fixed sizes. device screen sizes might differ12:50
renatukalikiana, I think Kaleo  help me with that12:50
kalikiana^^ zsombi look at that TextFieldStyle customization12:50
kalikianarenatu: yeah digging that out must've been a bit of a pain12:50
mzanettimivoligo: we can define an aspect ratio and stretch that to fill the available space12:50
mivoligomzanetti: right, but you probably can make it so the playable area area adapts to the available space, and the rest is either on the sides or over and below that area12:52
mzanettimivoligo: yep, that's what I meant with the last sentence12:53
mzanettimivoligo: other than that, design wise you're the boss12:53
zsombikalikiana: whazzup?12:53
mzanettimivoligo: I just preserve the right to veto stuff I really dislike12:53
mzanetti:)12:53
mivoligomzanetti: :)12:53
kalikianazsombi: observe http://bazaar.launchpad.net/~phablet-team/address-book-app/staging/view/head:/src/imports/ContactEdit/TextInputDetail.qml#L7612:53
kalikianato disable the border and change the color12:53
zsombikalikiana: yeah, I see it...12:54
kalikianaI thnik that's a nice example of tweaking styling for a custom use case12:54
zsombikalikiana: that's correct...12:54
kalikianazsombi: should we just add that to our docs? as we all know we need those but never get around to finding examples…12:55
zsombikalikiana: that was the way we suggested to get rid of the UbuntuShape without the need to create a custom theme12:55
zsombikalikiana: well, yes, once we will have docs for theming :/12:55
zsombikalikiana: but you know, that we are planning to tweak teh theming with subTheming and co12:55
kalikianazsombi: as jounih was asking about how to do this properly12:55
zsombikalikiana: meanwhile we can add this to the Theme engine documentation12:56
kalikianazsombi: well "docs for themeing" is big. this is a specific use case for text fields12:56
kalikianathere's nothing genric12:56
zsombikalikiana: yup12:56
kalikianaso imho we can just add it there12:56
kalikianaso instead of gawking at "style: Component" you can see an example ;-)12:56
kalikianatho it depends on whether qdocs is happy to override the property per component12:57
mivoligomzanetti: what I mean is you have to somehow calculate and specify the playable area, right?12:57
kalikianazsombi: here I mean http://developer.ubuntu.com/api/qml/sdk-14.10/Ubuntu.Components.TextField/#style-prop12:57
zsombikalikiana: well, in this way we can specify these in every component :)12:58
mzanettimivoligo: yeah... so far I just stretch it12:58
zsombikalikiana: 'cause custom style is valid for all12:58
sturmflut-workmzanetti, mivoligo: machines-vs-machines is great. And it even has particle effects!12:58
mzanettihehe12:58
mivoligo:)12:58
mzanettisturmflut-work: yeah... some spare time project I hack on every once in a while12:59
zsombikalikiana: BUT, remember, the Style API is not yet complete, so first we must make sure we publish all APIs, then we can base on these examples in teh doc12:59
mzanettisturmflut-work: the levels aren't good yet. But it reaches feature completeness and we can start designing proper levels soon12:59
kalikianazsombi: it is already used in apps, practically speaking it's api we already support12:59
mzanettimivoligo: so yeah... I guess we should  just define a width/height ratio13:00
kalikianazsombi: ideally I agree we would have an example for every component13:00
zsombikalikiana: well, we kinda do, it's just we must make it "official" :)13:00
kalikianazsombi: what does that really mean for you?13:00
zsombikalikiana: it means that we have to include the API in teh Ubuntu.Components.Styles13:01
zsombikalikiana: and document it13:01
zsombikalikiana: whatever is not documented is not meant to be stable API13:01
zsombikalikiana: that's why we must come up with a complete set of style APIs, documented13:02
kalikianazsombi: so goes the fairy tale. until the wolf eats you :-P13:02
zsombikalikiana: yet we have ~3 style APIs13:02
zsombikalikiana: well, we promised this for 14.10.... so we must do it13:02
zsombikalikiana: actually that was sthing I wanted to start, but me alone won't be able to finish it in time :(13:03
zsombikalikiana: so, buddy, are U up for an API freeze? ;)13:03
kalikianazsombi: what I see is app devs (at least one, renatu) pulling their legs out, to find out how to tweak theming… part of me things the "big docs" won't ever be done in one go13:04
kalikianazsombi: technically we are in freeze :-]13:04
* kalikiana likely fairy tales13:05
zsombikalikiana: no, true, but we can have an effort to do the style APIs and then we can have docs in each component13:05
nik90kalikiana: actually calendar app also uses the styles tweak at https://bazaar.launchpad.net/~ubuntu-calendar-dev/ubuntu-calendar-app/trunk/view/head:/NewEventEntryField.qml#L2913:06
zsombikalikiana: all we need to do is to get the API from the styles we have and move into a base component. The API to be published should be more or less what the target component uses + some the style really depends on.13:06
zsombikalikiana: so even if we are in API freeze, we don't really bring new API, we just "freeze" it :)13:07
mivoligomzanetti: ok, so you know now what I'm trying to say with these sizes?13:08
mzanettimivoligo: yes... I'll have a look when I import your background images13:08
mzanettimivoligo: hopefully tonight. right now I'm busy with unity13:08
mivoligomzanetti: remember this screen-ratio image? https://spideroak.com/browse/share/michal-here/1n3k34izs8asdfjazjkdas89e03/mix/13:09
sturmflut-workIs there a way to list all apps published for Ubuntu Touch? Maybe a way to access the API the store itself uses?13:09
mzanettimivoligo: ah perfect!13:09
=== chihchun_afk is now known as chihchun
mivoligomzanetti: I made the background in the way the green and red is seamless13:09
mzanettisturmflut-work: I'm afraid there isn't atm... popey, please correct me if I'm wrong13:09
mzanettimivoligo: yep. saw that... will make it a bit more tricky in the code, but no problem. I'll figure something. Most important thing is to make level design easy13:10
popeysturmflut-work: mzanetti https://search.apps.ubuntu.com/api/v1/search?q=architecture:armhf&size=1000&page=113:11
kalikianazsombi: hmmmm I don't quite follow. how do you "freeze" style API?13:11
popey:D13:11
mivoligomzanetti: ok13:11
kalikianazsombi: isn't it all rather specific to the component?13:11
zsombikalikiana: in teh way that you include it in the Style module13:11
mzanettipopey: awesome!13:11
popeysturmflut-work: https://gist.github.com/rschroll/8952015 - but it's broken as JamesTait broke the API :D - hey JamesTait maybe you could fix that as it's quite handy ☻13:11
zsombikalikiana: of course it is, but you can "freeze" it, or make it official, stable for each component, aint it?13:12
kalikianazsombi: so you mean importing "ambiance" would be seen as bad manners?13:12
mzanettiJamesTait: Happy fix-the-api-day!13:12
mzanetti:P13:12
kalikianazsombi: and it comes from ubuntu.components?13:12
zsombikalikiana: never said that... what I'm saying that till the style is not documented for each component, thoise may change freely13:12
popeywell, the API is fixed, I mean, fix anything that uses it ㋛13:12
JamesTaitpopey, ah, I wonder if that's the one client that has issued a broken request since we turned off plain JSON. :-P13:13
kalikianazsombi: well… kinda. we still have apps that will blow up if they change13:13
popeyprobably, i used it a few times and then.. sad face ☹13:13
sturmflut-workpopey: Thanks. Wow. That's an impressively long list.13:13
JamesTaitpopey, I saw it in the logs this morning, and wondered who/what it was.13:13
zsombikalikiana: aham... and that's the same way if they use private properties...13:13
zsombikalikiana: we NEVER said that teh style API is stable13:14
zsombikalikiana: so we should do it now13:14
popeyJamesTait: robert is an app developer, he'd probably appreciate the tips / fixes (he made Beru, the ebook reader)13:14
zsombikalikiana: these people use the text input style on their own risk, and they know it13:14
mivoligomzanetti: about the background design: it took me a lot of time to do a detailed levels, so I decided to do a rather simple „vector look” with gradients here and there but without any textures. I hope it's good enough for the start.13:14
JamesTaitpopey, oh yes, I know (of) Robert.  I e-mailed him personally to let him know the API was changing. :)13:15
zsombikalikiana: and we must publish all style APIs so they can use them from now on officially13:15
kalikianazsombi: true13:15
mzanettimivoligo: it'll do for a start... we can always add more later...13:15
JamesTaitBut I have applist.py cloned locally already, so I can offer a simple patch.13:15
zsombikalikiana: so, that's what I was talking about :)13:15
mzanettimivoligo: most important thing is we have the paths defined so we can start with building proper levels13:15
mivoligomzanetti: I guess I should create and push level.json files13:16
mzanettimivoligo: if you have the time, that would be great13:17
mzanettimivoligo: you can change the existing ones13:17
mivoligomzanetti: I'll try to do that in the evening13:17
mzanettito match your images13:17
mivoligomzanetti: but I'd like to leave enemies waves to you :)13:18
nerochiarokenvandine: what do we do in cases when the user wants to share a photo and a video (i.e. part of a random selection he made in gallery) ?13:18
mzanettimivoligo: sure13:19
kenvandinenerochiaro, ugh... not sure13:19
mzanettimivoligo: just set the fieldsOnPath and forbiddenFields13:19
mivoligomzanetti: great, enjoy your day!13:19
mzanettiyou too!13:19
mzanettio/13:19
mivoligoo/13:19
* popey hugs JamesTait 13:19
nerochiarokenvandine: i think that for now i'll disable the share option if the selection is not homogeneous13:20
kenvandineyeah.. safest13:20
nik90sergiusens: hey were you able to create a unpriviledged lxc container without any issue?13:35
sergiusensnik90: ah, I'm really bad at the terminology; you mean, un as user with all the graphical bling?13:35
* sergiusens does console only13:35
nik90sergiusens: well by default you can call lxc-create only as root13:36
nik90sergiusens: https://www.stgraber.org/2014/01/17/lxc-1-0-unprivileged-containers/13:36
nik90sergiusens: so by assigning uids and gids you can call lxc-create as just your user itself13:36
* nik90 doesn't understand uids and gids..and hence just running commands13:36
sergiusensnik90: I'll take a look13:37
sergiusensnik90: uid and gid is the one matching your user13:37
sergiusensnik90: run 'id' from a terminal13:38
nik90ah it shows 1000 uid and 1000 gid13:38
nik90for my user13:38
sergiusensnik90: after a quick glance, you do't need that info though, just replace stgrabe with what is appropriate for you :-)13:40
nik90sergiusens: I did all that..but when I do lxc-start -n p1, it gives me error that lxc-container could not access /home/krnekhelesh/local. So I gave it access for now using chmod +x13:42
nik90sergiusens: so now I can start the lxc-container as krnekhelesh rather than root13:42
nik90sergiusens: so I think I solved it for now13:42
sergiusensnik90: there is a comment at the very end on how to allow that13:42
sergiusens.local and not local, right?13:42
* sergiusens assumes typo13:43
nik90yes .local13:43
sergiusensnik90: I'll check later today13:44
nik90sergiusens: its ok..it works for now. I am going to now try the gui part in the next blog post :)13:44
ahayzenballoons, ping13:52
balloonsahayzen, pong13:52
ahayzenballoons, i'm trying to write a test which requires the song to be a least 5seconds in...what is the best way of waiting until the player.position > 5000 ?13:53
balloonswait_for not work?13:53
ahayzenballoons, so if i had player.position.wait_for(5000) what happens if gst jumps from 4998 to 5002 or something?13:54
ahayzenballoons, i could try it but not sure how reliable that would be?13:54
balloonswait_for(GreaterThan(5000))13:55
ahayzenballoons, that sounds better, i'll try that thanks :)13:55
balloons:-)13:55
ahayzenballoons, also is this an acceptable way to skip tests when not running as click? http://bazaar.launchpad.net/~andrew-hayzen/music-app/add-url-dispatcher-tests/view/head:/tests/autopilot/music_app/tests/test_music.py#L2413:56
ahayzenballoons, as those url-dispatcher tests only work in a click environment13:56
mihir_balloons: ping13:57
balloonsahayzen, hmmm.. is it only a click env, or is it only on phablet devices?13:57
ahayzenballoons, an environment that has url-dispatcher that works :P13:57
mihir_balloons: this bug https://bugs.launchpad.net/bugs/1367654 was because of the pacakge name change is that correct?13:57
ubot5Ubuntu bug 1367654 in Ubuntu Calendar App "autopilot tests failing mako 235 ImportError: No module named 'address_book_service_testability'" [Undecided,Confirmed]13:58
balloonsahayzen, I think I would be more inclined to only exclude the desktop.. But the click idea I suppose is interesting13:59
ahayzenballoons, i think the reason it doesn't work is that because the click isn't installed the protocol doesn't get registered or something?14:00
balloonsmihir_, we added the depends to the manifest and to the debian control file. However, CI doesn't use autopkgtest to run things, and as such, there's no dependency handling. I added the dependency manually to a config they use14:00
ahayzenballoons, how do i exclude desktop?14:01
balloonsmihir_, the switch was we used to include that code inline in the tests, but then it got packaged14:01
mihir_balloons: okay :) make sense14:01
balloonsso we dropped it from the calendar code and use the upstream version14:01
mihir_popey: ^^14:01
balloonsahayzen, autopilot.model14:01
popeymihir_: yay!14:03
ahayzenballoons, oh god the app just crashed again in jenkins :/ http://91.189.93.70:8080/job/generic-mediumtests-utopic-python3/279/console14:05
ahayzenballoons, for this mp https://code.launchpad.net/~andrew-hayzen/music-app/ap-helper-refactor-004/+merge/233291 ... do you mind rerunning?14:05
balloonsahayzen, is that the latest run?14:12
balloonsand yea.. though it keeps happening?14:12
renatukalikiana, what do you think about this bug? https://bugs.launchpad.net/messaging-app/+bug/136745914:12
ubot5Ubuntu bug 1367459 in Ubuntu UI Toolkit "Messaging app should not rotate upside down" [Undecided,New]14:12
ahayzenballoons, yeah i think so... well it did pass yesterday...but i agree it does seem to keep happening, does it look like us or ap or jenkins?14:13
ahayzenballoons, success \o/ thanks for the rerun :)14:27
balloons:-)14:27
sturmflut-workmzanetti, popey: I built a cheap QML browser for the JSON API to the store that popey mentioned, see https://github.com/Sturmflut/storebrowser14:53
popeyoooh!14:54
* popey hugs sturmflut-work 14:54
Akiva-Thinkpadwhats a qml browser?14:54
Akiva-Thinkpadweb browser?14:54
popeyqml "store browser"14:55
popeynot "qml browser"14:55
Akiva-Thinkpadah14:55
Akiva-Thinkpadvery cool14:55
sturmflut-workpopey: I should probably add some kind of filter, the list is already too long to be properly viewed on a Desktop.14:55
popeysturmflut-work: you going to maintain this? I mean, I have a bunch of issues I could file for you ㋛14:55
mzanettihaha :D14:56
mzanettipopey is like a fly strip14:57
popeyin a good way?14:57
mzanettisure14:57
popey14:58
sturmflut-workpopey: I should probably concentrate a little more on my day job again, I spent a lot of time doing app development instead of "real" work.14:58
popeyhehe15:00
popeyreal work is overrated ㋛15:00
* popey looks around for sabdfl15:00
popeyi never said that15:00
ahayzenlol15:01
mhall119nik90: BTW, my recurring alarm didn't recur15:02
nik90mhall119: not surprised since you told me that the alarm you created did not appear in the indicator-datetime15:03
sturmflut-workpopey: Well, I do work for the government, and they give me a lot of shiny boxes to play with, so better not upset anyone. They might take the shiny boxes away :(15:03
nik90mhall119: if i-dt doesn't see it, then it wont ring since i-dt is the service which does the actual ringing15:03
popeyindeed15:03
nik90mhall119: I am also pretty certain that your old alarm files/clock files are causing this issue since I don't see others having the issue with the only difference being they clean wiped their phone at some point15:04
kalikianarenatu: about rotation… we still don't have control over it afair. either you do or don't. unless unity8 implements that. I'd say ask a unity dev maybe greyback_15:24
kalikianaI second tho that is is a fundamental design violation15:25
kalikianaupside down is no-no15:25
Akiva-Thinkpadpopey, nik90, sturmflut-work , kalikiana - http://imgur.com/a/FIFRy#015:47
Akiva-Thinkpada la this bug https://bugs.launchpad.net/ubuntu-calendar-app/+bug/135726015:48
ubot5Ubuntu bug 1357260 in Ubuntu Calendar App "Font Colour for Time List in "Week" and "Day" view lack contrast." [Low,Confirmed]15:48
Akiva-Thinkpadany thoughts?15:48
nik90Akiva-Thinkpad: I like the 2nd one...first one is too strong imo.15:48
Akiva-Thinkpadnik90, do you think ubuntu dots are possible to use in a design?15:49
nik90I have not seen them in any app yet15:49
Akiva-Thinkpadnik90, you like the second one from the album I just posted? with the faint colours?15:50
nik90Akiva-Thinkpad: if any yes15:50
Akiva-ThinkpadI actually kind of liked the grey one15:50
Akiva-ThinkpadThe ones with the solid colours I am thinking could be made to look nicer if I added a white divider15:51
kalikianaAkiva-Thinkpad: I want to like the use of orange but it's too heavy15:53
Akiva-Thinkpadkalikiana, yah I agree15:53
Akiva-Thinkpadoff the top of your head, do you know which component adds a white line divider?15:54
Akiva-Thinkpadoh, it looks like it is "Divider{}"15:55
nik90actually ThinDivider{}15:56
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
popeyAkiva-Thinkpad: I prefer grey with a subtle highlight, the last two...16:28
Akiva-Thinkpadpopey, hold on, I actually have three more that I think are good, utilizing the divider.16:29
Akiva-Thinkpadpopey, nik90 kalikiana  4 more at the end. I think these ones are pretty swell http://imgur.com/a/FIFRy#716:37
nik90too strong16:37
popeyhttps://imgur.com/a/FIFRy#8  is my fave so far16:38
nik90I would stick with popey's suggestion.16:38
popeymakes the appointments stand out16:38
Akiva-Thinkpadmmm yah16:38
popeyHowever I would point out one details...16:38
popey← Not a designer.16:38
nik908th? The one with the gray background?16:39
Akiva-ThinkpadI'm gonna shoot it up on reddit. The one thing I like about the solid grey, is that it does save battery :P16:39
nik90I like 6th16:39
popeythe one with the dark 7pm16:39
popeyand no orange/purple16:39
popeys/purple/aubergine/16:40
Akiva-Thinkpadyah too strong I think16:40
popeyhttps://i.imgur.com/5xVxwAr.png16:40
nik90how about keep the background white and instead color the event?16:40
* popey shrugs16:40
nik90in combination with 6th16:40
Akiva-Thinkpadpopey, I like what you said about the solid greys making events stand out.16:40
popeydont like the bar under the time16:41
Akiva-Thinkpadif we end up adding colours to the events, it will really make the events stand out.16:41
Akiva-Thinkpadtime should be in the middle?16:41
popeyno, I mean there's a bar in the screenshot I posted16:42
Akiva-Thinkpadah the Divider{}16:42
popeyyes16:43
Akiva-Thinkpadgood to know16:43
Akiva-Thinkpaddo you like the time being moved from x-large to large, and or the fact that it has been moved from the center, to the line which it sits on?16:43
Akiva-Thinkpadnik90?16:44
mihirAkiva-Thinkpad: ping16:45
Akiva-Thinkpadmihir, ah pong16:45
Akiva-Thinkpadwelcome back16:45
Akiva-Thinkpadmihir, http://imgur.com/a/FIFRy#716:45
mihirAkiva-Thinkpad: hahahha finally back home from work :D16:45
* mihir looks at screenshots :D16:45
Akiva-ThinkpadFrom work to work~16:45
nik90Akiva-Thinkpad: well frankly if it were to be my choice, I would move them to the side and decrease their font to normal like google calendar. Having them in the middle only results in the time being hidden by the events.16:46
Akiva-Thinkpadnik90, because you like the white; would you like a version with a white background with the divider?16:46
Akiva-Thinkpadnik90, yah I was thinking of that too.16:46
Akiva-Thinkpadmight play around with that16:46
* popey goes to make food16:48
nik90Akiva-Thinkpad: Here my reasoning as to why I don't prefer strong colors like Orange etc. If one were to use strong colors in the background it reduces the effectiveness of action buttons. That's one of the things I admire in the new clock design. Due to the neutral white background, the action buttons like delete, add, toggle switches become prominent and16:48
nik90let the user know what to do.16:48
nik90Akiva-Thinkpad: which is why I rather let the events take on the color background rather than the entire app itself16:48
Akiva-Thinkpadnik90, yah the new clock design looks great16:48
* Akiva-Thinkpad doesnt disagree16:48
Akiva-Thinkpadnik90, i'll make a few more mockups with your suggestions16:49
nik90Akiva-Thinkpad: Atm the calendar is still a bit cluttered and strong in my opinion. Reducing the font-sizes and moving things to the side would improve it. But then again I am not a designers.16:49
mihirAkiva-Thinkpad: did you make working hours configurable ?16:49
mihirAkiva-Thinkpad: nik90 i liked this one , http://imgur.com/a/FIFRy#516:49
Akiva-Thinkpaderrrr, did he dc?16:50
* Akiva-Thinkpad doesnt have login notifcations16:50
nik90Akiva-Thinkpad: he dced16:50
Akiva-Thinkpad:P16:50
Akiva-Thinkpadokay brb, gonna make another16:50
nik90I don't mind #5 as long colors are made a bit more softer16:51
* nik90 goes back to writing tests16:51
ahayzenelopio, thanks for the review :)16:59
elopiothanks to you ahayzen. You are doing a good job there.16:59
ahayzenelopio, thanks16:59
ahayzenelopio, the encoding of check_call was a weird one17:00
elopioahayzen: does it fail without the enconding?17:00
ahayzenelopio, it was only failing on jenkins17:01
ahayzenelopio, http://91.189.93.70:8080/job/generic-mediumtests-utopic-python3/271/testReport/junit/music_app.tests.test_music/TestMainWindow/test_url_dispatcher_album_play_with_mouse_/17:01
ahayzenelopio, but we are skipping the tests for jenkins now...so...?17:01
elopioumm, you have a unicode character on the test.17:01
ahayzenelopio, yep deliberately ;)17:02
elopioahayzen: ideally, your test files should be really simple. We want lower level tests to check enconding issues like that.17:02
elopioyou are testing two things here, and a test should be simple and focused.17:02
elopiobut if the enconding makes it work, we can leave for later splitting the test in two: one for launching, and one for handling weird characters.17:03
ahayzenelopio, i suppose... so should i have one test with 'normal' characters and one with unicode?17:03
elopioahayzen: you should have one test using the file as simple as possible. There you check that the UI starts playing, that the playlist size is the correct.17:04
elopiothen we need a test to make sure that url-dispatcher can handle unicode characters. That test should live in the url-dispatcher project.17:04
elopioand then we need a test to make sure that the music app displays weird characters correctly.17:05
elopioahayzen: the rule I use comes from the name of the test. I try to name a test as: test_action_must_result_in_x17:05
ahayzenelopio, ok17:06
elopioif I can't make a simple name that explains the test, generally it's because the test is too big.17:06
elopiotest_launch_album_from_url_dispatcher_must_play_it17:06
elopiothat's good.17:06
ahayzenok i'll try and sort it out :)17:06
elopiotest_launch_album_with_unicode_enconding_from_url_dispatcher_must_play_it_and_display_name_correctly, that's bad.17:07
ahayzenelopio, would you want to test that unicode can be launched via url-dispatcher though?17:07
elopioahayzen: I think that won't be necessary. I would ask ted if they have a test for that. And if they don't, just report a bug for him.17:08
ahayzenelopio, ok17:08
ahayzenhmm this test would be much easier to write when 004 refactor lands...17:09
Akiva-Thinkpadnik90, couldnt get the label to be on the side without some extensive coding the label's visible: property, because if I move it to the left, other labels from the other days appear on this one.17:17
Akiva-ThinkpadThat being said17:17
Akiva-Thinkpad4 more17:17
Akiva-Thinkpadhttp://imgur.com/a/FIFRy#1117:17
mihirAkiva-Thinkpad: nik90 sorry DC problem17:19
Akiva-Thinkpadmihir, heh17:19
Akiva-Thinkpadmihir, http://imgur.com/a/FIFRy#11 added a few more that you may like17:19
Akiva-Thinkpadthey are brighter17:19
Akiva-Thinkpadmihir, the other thing I could do, is the ubuntu grey gradient17:19
mihirAkiva-Thinkpad: my only concern is , i would prefer to change label color rather than background17:20
mihirAkiva-Thinkpad: because for me it make more sense, and it looks more elegant..17:20
Akiva-Thinkpadmihir, popey made a good point on the one with the dark solid greys, in that it causes the events to stick out.17:21
nik90Akiva-Thinkpad: Did you just create an event for "Wash popey garden with bleach"? !17:21
Akiva-Thinkpaddid I say wash?17:21
Akiva-ThinkpadI meant to say water17:21
nik90either way how's that good :P17:21
Akiva-Thinkpadgood is a subjective term17:21
* nik90 scrambles to warn popey17:22
Akiva-Thinkpadits too late; hes busy making food17:22
mihirAkiva-Thinkpad: lol, on event names17:22
* Akiva-Thinkpad *mwa ha ha ha*17:22
mihirAkiva-Thinkpad: okay , if he is fine, what are your inputs, nik9017:22
mihiri may have lost the converstation17:22
Akiva-Thinkpadmihir, i'll post it on reddit; see what the community thinks.17:23
mihirAkiva-Thinkpad: excellent :)17:23
mihirAkiva-Thinkpad: btw, did you made that work hours configurable ?17:23
mihirnik90: i am upgrading my VM now..17:23
nik90Akiva-Thinkpad mihir: I like https://i.imgur.com/rs93ZTW.png but with the font-size improvements you just did17:23
mihirnik90: bingo , same choice :)17:23
Akiva-Thinkpadmihir, not yet; once I figure out the design i'll be doing that.17:23
mihirAkiva-Thinkpad: no issues..17:24
JamesTaitpopey, https://gist.github.com/jamestait/539788077e86c89bff7c#file-applist-py17:24
mihirnik90: do you have that branch , or i should propose , and we can improve that together..17:24
nik90mihir: I think you should propose as I can't find a time to get to it..too many stuff to do from all sides17:25
nik90mihir: btw use "bzr mv" to move the files17:25
nik90that will produce a much clean code diff in launchpad17:25
mihirnik90: yeah , i can understand..but i really appriciate your help :) I am lil afraid to make big changes, but with help of you i can think of those changes :)17:26
nik90mihir: ok so first things first.. does all AP tests pass atm?17:27
mihirnik90: i am inbetweent, so let me do that first and then and then i'll pass17:27
mihirpush*17:28
nik90mihir: second, once we start moving stuff, the existing MPs will get code conflicts since the files they are trying to change have been moved17:28
mihirnik90: waiting to run calendar app on my QTc heheh17:28
nik90mihir: are you updating your vm?17:28
=== chihchun is now known as chihchun_afk
mihirnik90: yup it is in progress17:28
nik90mihir: we did this change to make it so much easier to run from QtC17:28
nik90mihir: can you imagine being able to run on desktop, device and emulator with no changes through QtC17:29
nik90that's what we finally have now17:29
mihirnik90: i hope , my vm udpates should fix17:29
nik90so I really hope your issue gets resolved asap17:29
Akiva-Thinkpadbleck; gradient does not work17:29
Akiva-Thinkpadokay; posting it to reddit17:29
ahayzenIs click-buddy installing for anyone on #235 ?17:32
nik90ahayzen: signature issue?17:34
ahayzennik90, yah17:35
nik90ahayzen: try --allow-untrusted ?17:35
ahayzenah17:35
* ahayzen modifies script17:35
nik90ahayzen: might want to check --help first to see if that option is available17:35
ahayzennik90, hmm doesn't appear in help...has it been backported to trusty?17:35
nik90no idea17:36
nik90I haven't updated yet17:36
* nik90 checks which image he is on17:36
ahayzeni was wondering why all of my autopilot tests were failing on device...and then realised it wasn't installing the click lol17:36
nik90lol17:36
* nik90 is#234..updates17:37
ahayzendon't do it! u can't install anything lol17:37
balloonslol17:38
balloonsvictor's bug?17:38
nik90ahayzen: I never use click-buddy17:38
nik90ahayzen: qtcreator should work around this for sure17:38
nik90ahayzen: if not I know who to bug :P17:38
balloonsahayzen, you can avoid using --provision17:38
ahayzenballoons, Fatal error: /tmp/com.ubuntu.music_1.3.10000_all.click failed to install.17:38
ahayzenCannot install /tmp/com.ubuntu.music_1.3.10000_all.click: Signature verification error: debsig: Origin Signature check failed. This deb might not be signed.17:38
nik90ahayzen: also cant let you suffer alone :P17:38
balloonsif you install it manually use --allow-untrusted17:38
balloonsahayzen, ^^ I JUST did this on the new image, it works fine17:38
balloonspkcon install-local --allow-untrusted17:39
ahayzenballoons, /usr/bin/click-buddy: unrecognized option '--allow-untrusted' ... oh pkcon17:39
nik90balloons: I think ahayzen tried click-buddy --dir . --provision17:39
ahayzenyep17:39
* ahayzen hugs click-buddy tightly17:39
ahayzenhow do i use pkcon ?17:40
balloonsahayzen, indeed.. and as nik90 said, feel free to use the sdk to push to devices. If it fails, we can cry wolf :-)17:40
balloonsahayzen, push the click to the device, shell in and run it17:40
nik90mihir: did you know that the events in the indicator-datetime use the same color used by the calendar , cool eh?17:40
balloonspkcon install-local --allow-untrusted my_super_cool_music_click.clik17:40
ahayzenhmm last time i checked qtc couldn't install on device due to a permissions things? it couldn't even enable dev tools17:40
nik90ahayzen: last time? when?17:40
mihirnik90: yup :D17:41
mihirnik90: we intedend that heheh17:41
ahayzennik90, ..now... Build->Ubuntu->Install application on device is greyed17:41
nik90ahayzen: let me try, 1 min..phone still updating17:41
ahayzennik90, if i try and tick 'has device developer tools' i get.. E: Could not open lock file /var/lib/dpkg/lock - open (13: Permission denied)17:42
ahayzenE: Unable to lock the administration directory (/var/lib/dpkg/), are you root?17:42
ahayzenE: Could not open lock file /var/lib/apt/lists/lock - open (13: Permission denied)17:42
ahayzenE: Unable to lock directory /var/lib/apt/lists/17:42
ahayzenE: Could not open lock file /var/lib/dpkg/lock - open (13: Permission denied)17:42
ahayzenE: Unable to lock the administration directory (/var/lib/dpkg/), are you root?17:42
ahayzenE: Could not open lock file /var/lib/dpkg/lock - open (13: Permission denied)17:42
ahayzenE: Unable to lock the administration directory (/var/lib/dpkg/), are you root?17:42
ahayzenE: Could not open lock file /var/lib/dpkg/lock - open (13: Permission denied)17:42
ahayzenE: Unable to lock the administration directory (/var/lib/dpkg/), are you root?17:42
nik90erm we got it :P17:42
ahayzenbah sorry17:42
Akiva-Thinkpadbbl17:44
Akiva-Thinkpadnight all17:44
ahayzenballoons, pkcon install-local --allow-untrusted is also a unknown option guess that hasn't been backported to trusty?17:44
balloonsahayzen, lol.. on the device man!17:44
nik90ahayzen: why do you need that in trusty?17:44
ahayzenoh..17:44
nik90what balloons said ^^17:44
nik90oh ahayzen :P17:45
mihirAkiva-Thinkpad: night :)17:45
ahayzen... i like $ click-buddy --dir . --provision :D it just works...or should i say *worked*17:45
nik90lol17:46
ahayzenyey i think i've manage to install a click package \o/17:47
ahayzenthanks balloons nik9017:47
nik90yw17:50
nik90elopio: hey17:52
nik90elopio: your lab rat brings you https://code.launchpad.net/~nik90/ubuntu-clock-app/checkbox-manual-tests/+merge/234164 :-)17:53
balloonsmihir, still about?17:54
mihirballoons: ?17:55
elopionik90: lab rat sounds terrible. Lets say bleeding edge rat.17:56
balloonsmihir, https://bugs.launchpad.net/ubuntu-calendar-app/+bug/129122517:56
ubot5Ubuntu bug 1291225 in Ubuntu Calendar App "autopilot tests fail when run in UTC+1 timezone" [High,Confirmed]17:56
nik90elopio: +17:56
balloonsmihir, I discovered this bug came back again :-(17:56
mihirballoons: :(17:57
elopionik90: could you add a README about how to run them?17:58
mihirballoons: but now we are getting current locale in AP right ?17:58
elopionik90: and is the namespace of your new version of the clock 2014.com.ubuntu.clock ?17:58
elopioI find it weird to use numbers on the namespace.17:58
nik90elopio: no the namespace is com.ubuntu.clock17:58
nik90elopio: however I was told to you it by zyga17:58
nik90elopio: http://plainbox.readthedocs.org/en/latest/author/provider-namespaces.html#theoretical-considerations17:59
balloonsmihir, I unfortunately didn't spend any time playing with why it was broken, so I'm not sure. But yes we attempt to set to 'C' locale17:59
mihirballoons: if we see mergs, we are using locale instead of UTC17:59
elopionik90: I see. Well, lets follow what they propose.17:59
mihirballoons: it fails on device or desktop ?18:00
balloonsmihir, everywhere.. I found it on the desktop18:00
mihirahhh , i never noticed :|18:00
mihirballoons: it should be UTC or system locale ?18:00
balloonsmihir, I can't remember if calendar or autopilot asserted the weird time18:01
nik90elopio: so we were discussing this on checkbox earlier, but here is the summary of the plan.18:01
mihirballoons: i'll try to reprdocue this , and try to debug this18:01
nik90elopio: In my next MP, I will add debian packaging into the manual tests folder which will depend on the correct version of checkbox.18:01
mihirballoons: as far as i see attached Mergs to the bug, it resolve issues on both way QML and Ap18:02
nik90elopio: so the qa team like yourself, should be able to run the launcher which should open the tests in a GUI (if not console) in a simple fashion where you can run the tests and then enter if they passed or failed.18:02
nik90elopio: that should get outputted to a json file or whichever format you prefer18:02
mihirballoons: there is a confusion , your MR says it has to be utcnow and kunal's MR says it has to be locale18:02
balloonsmihir, right I agree.. So I can't say much other than try setting your device to a local timezone (not utc) and running the tests. They should fail18:02
balloonsmihir, I agree.. the mp's don't see to agree with each other18:03
nik90elopio: I will still need to talk to dholbach about how one goes about installing the clock-manual-tests debian package since it won't be built by jenkins18:03
elopionik90: interesting.18:03
mihirballoons: okay18:03
elopionik90: I don't care about having it properly packaged if it's easy to do from the branch.18:04
nik90elopio: oh ok18:04
elopiobut if you find a way, +1.18:04
elopiothe archive managers will probably don't want a debian package in the archive for a binary that will only be on the store, though.18:05
elopio*probably won't want18:05
elopiobut discussing with daniel might bring good ideas of what to do.18:05
nik90elopio: true but the clock app itself is not in the archive..we test (distribute) it on the desktop using the core-apps ppa18:05
elopionik90: ah, so we could put the package only on the PPA.18:06
balloonsI should talk to zyga about the format for plainbox tests.. I don't like it18:06
elopionik90: as I said, that's not a concern for me, so feel free to do as you like. It's your PPA also :)18:06
nik90balloons: what why...it is so simple though18:06
nik90elopio: http://paste.ubuntu.com/8311480/18:06
sergiusensballoons: it's like the only think I like about them, very standarized format ;-)18:06
nik90elopio: well I am doing all this for the qa team to run manual test easily18:07
nik90elopio: so I rather you make the decision :)18:07
elopioballoons: we discussed with zyga about also supporting the format the qa tracker uses. But that's also not my concern, any format works for me, I just want a file with results.18:07
elopionik90: if you leave it to me, then do not invest any time in packaging the manual tests. Just put a README for now with the instructions to run them.18:08
nik90elopio: ack18:08
elopionik90: remember this is the first trial. If it works good for you and for me, we still need to convince the rest of the team.18:08
nik90true18:09
mihirnik90: after upgrading VM it works :D18:09
* nik90 sighs a relief18:09
balloonssergiusens, nik90 elopio my issue is it doesn't follow the "do this, expect this" format we've adopted: https://wiki.ubuntu.com/QATeam/TestCase18:10
nik90elopio: cool in that case, I will try to make sure the manual test comes up nicely with a launcher and gui to make it super simple.18:10
elopioballoons: oh, I agree there.18:10
elopiobut I guess it would be easy to do to add more verifications between the steps.18:11
nik90elopio: I can check with zyga if that's possible18:11
elopiothat's also the format moztrap uses and I like it a lot.18:11
nik90elopio: But shouldn't that just mean that the test should be split up more?18:11
elopionik90: not necessary in this case.18:11
nik90elopio: for instance If I am testing to check if alarm vibrations works, I need to first have a check to see if alarm even ring in the first place.18:12
elopiomanual testing is too different from automated.18:12
nik90ok18:12
elopiowhat you want in the end is the same: test one single goal, or experience path, or funcionality or how you like to call it.18:12
nik90elopio: actually you should check with zyga since you will be able to explain this better18:12
elopiobut it's really useful to remind the tester what to expect at each step.18:13
brendandballoons, that's just a question of semantics18:13
=== _salem is now known as salem_
elopiolike: open the app from the launcher -> the app must be opened showing the main clock page.18:13
balloonsso, checkbox back in the day used to use a format much closer to what I would like to see, but they migrated way to what you see now18:13
elopiothen if the app is opened on a different page, the tester will know something is wrong and he won't be likely to continue with the test.18:13
balloonsnik90, I am curious though, did you talk with zyga about his plans for the phone and plainbox?18:14
balloonsI never heard back last month18:14
nik90balloons: zyga is already building checkbox-touch which is supposed to run on the phone18:14
nik90I ran the gui and it looks nice18:14
nik90balloons: he linked me to a google document which showed the future plans for ubuntu touch18:15
nik90but I am unable to find it atm, will need to ask him for the link18:15
elopioballoons: and anyway, the results file is so valuable for me, I wouldn't mind to work around that limitation. like writing:18:15
elopiostep 1: do this (this should happen)18:15
elopiostep 2: do that (that should happen)18:15
elopiostep last: finish the test18:15
elopioverification: the test finished successfully.18:15
brendandballoons, i don't think the test case format in checkbox has changed significantly18:16
balloonsbrendand, it changed slightly when they moved from gtk to qt18:16
balloonsbut I might be wrong :-)18:16
brendandballoons, yes - i know :)18:16
balloonsI went through it18:16
brendandballoons, but before that it was worse18:16
elopioballoons: and about running on the phone, that's not really useful for us at the moment.18:17
elopiowe want this only for manual tests. We don't want a combination of automated and manual tests, so we don't really need to execute commands on the phone.18:17
brendandballoons, there was no format18:17
balloonslol.. well, I would really like to see it solidified up18:17
mihirnik90:18:17
balloonsthe bigger thing for me is automated reporting and collation of results18:17
brendandelopio, we may want semi-automated tests though18:17
mihirit is giving errror line below imports and it says use QML_IMPORT_PATH18:17
elopiobrendand: why? when?18:17
balloonsif we don't have that, and buy-in, I'm not interested18:17
mihirnik90: is that okay ? but the project runs fine18:18
brendandelopio, it can be really powerful to part-automate a test18:18
brendandelopio, don't tell me you can't think of a case where that would be useful :)18:18
nik90mihir: ah yes that's fine. Its one of the error in cmake project not being to find ubuntu components. But this should be fixed in a update very soon18:18
balloonsI can agree with brendand on that.. basically it let's you 'automate' things that are hard to get good asserts for18:18
balloonscolors, sounds, etc18:18
nik90mihir: the update will also bring code autocompletion as well18:19
brendandballoons, video playback etc18:19
elopionik90: why do we need summary and purpose? they seem almost the same to me.18:19
brendandelopio, summary is a short description for the ui18:19
nik90elopio: summary can't exceed 80 char18:19
elopiobrendand: got it.18:19
elopionik90: got it.18:19
elopiobut then purpose is needed only if 80 chars are not enough?18:19
nik90I suppose yes..I wanted to maintain uniformity18:20
nik90but I could check with zyga on that18:20
nik90http://plainbox.readthedocs.org/en/latest/author/jobs.html18:20
brendandnik90, where's the branch for this?18:20
nik90brendand: https://code.launchpad.net/~nik90/ubuntu-clock-app/checkbox-manual-tests/+merge/23416418:21
mihirnik90: okay18:21
brendandelopio, purpose is not really part of the job format as such18:22
brendandelopio, here's an example:18:23
brendand_summary: Resolution change tests18:23
brendandPURPOSE:18:23
brendand      This test will verify that the GUI is usable after manually changing resolution on the $product graphics card.18:23
nik90brendand: so summary is more like a test title?18:23
brendandelopio, summary appears in the list of test cases, where purpose only appears in the test case itself18:24
nik90since "Resolution change tests" doesnt explain much18:24
brendandelopio, moreover, all test must have a summary, but only manual tests need a PURPOSE18:24
brendandnik90, it explains more than 'graphics/`echo ${index}`_resolution-change_`echo "${product}" | sed 's/ /_/g;s/[^_a-zA-Z0-9-]//g'`'18:25
brendand:)18:25
nik90lol18:25
brendandnik90, before checkbox could only display the ID, which had to be unique and couldn't have spaces18:27
brendandnik90, not very user friendly18:27
nik90ah ok18:28
nik90sergiusens: hey quick question, how do you access files in the host inside a lxc-container?18:28
nik90sergiusens: I have now a unprivileged utopic amd64 lxc container. I am installing the ubuntu-sdk and other necessary stuff and run them from the commandline18:29
nik90so no gui yet18:29
brendandnik90, do you need the tests to run in a specific order?18:32
nik90brendand: the trigger-alarm test needs to be run first. So I made the other tests depend on it18:32
nik90brendand: if trigger-alarm fails, all the others will also fail18:33
nik90but that's about it18:33
brendandnik90, ok - if you want you can use the alarm/* pattern in your whitelist18:33
brendandnik90, you then won't need to remember to add new tests to the whitelist18:33
nik90ah yes..thnx18:34
brendandnik90, although it's not wrong to be explicit18:34
brendandnik90, in trigger-alarm you have too much space for the VERIFICATION step18:35
brendandnik90, and for me estimated_duration being at the bottom looks weird18:35
nik90yeah I am still struggling with the spacing a bit18:35
brendand_description should always be last18:35
nik90I did the tabbing in gedit, and when I ran .manifest validate -N it gave me errors18:35
nik90so going back to nano messed some things up18:35
balloonsnik90, geany > gedit18:36
nik90true18:36
brendandnik90, gedit needs reconfiguring a bit18:37
balloonsfrom there you can move into more esoteric editors.. it's the gateway18:37
brendandballoons, you say 'esoteric' i say 'better'18:38
brendand:)18:38
sergiusensnik90: I bind mount19:03
sergiusensnik90: there's an automated way to do it; but I do it manually19:04
nik90sergiusens: ah something like lxc.mount.entry = /dev/dri dev/dri none bind,optional,create=dir19:04
killerHey,It's been a long time since I submitted my apps to store but still "pending review"(2 months).19:07
sergiusensnik90: something like that, you can do /home /home too19:10
sergiusensor /home/user /home/user19:10
nik90yup19:10
sergiusensjust make sure user has the same uid and gid or things will break :-)19:11
nik90:-)19:11
nik90balloons: https://docs.google.com/a/canonical.com/document/d/12gpgFGtNBoPet8215bUdeJ-QXLL_peQOsGCzipd4gh0/edit#19:22
balloonskiller, apps for the desktop?19:23
killerballoons: yes19:46
beunokiller, we are solving our lack of man power soon19:49
balloonsnik90, I assume you may have seen the conversation in #ubuntu-touch. Thanks for the document link. I'll peruse it and leave some thoughts on your mp19:58
nik90balloons: not yet, just came from dinner, but will take a look19:59
nik90balloons: sure go ahead19:59
nik90sergiusens: hey I got qtcreator working in my utopic lxc-container!20:22
nik90sergiusens: I can now open the clock app on trusty host with utopic lxc with no overhead20:23
sergiusensnik90: nice!20:23
Letozaf_balloons, hi20:23
balloonsLetozaf_, hello20:23
balloonsnik90, ohh that's awesome20:23
Letozaf_balloons, can you help me solve this : "QQuickListView.count failed: True != dbus.Int32(20, variant_level=1)"20:24
nik90balloons, sergiusens: There are some small kinks that I need to work out like moving my ssh keys, common development folder sharing, etc but woohoo20:24
balloonsnik90, you should also be able to run tests 'locally' from the container20:26
nik90balloons: yup that's my big plan20:27
Letozaf_balloons, I need to check the count property value but it's a dbus.Int3220:27
balloonsLetozaf_, is that your branch?20:27
nik90balloons: no more heavy virtual vm20:27
balloonsthat is the idea behind lxc20:27
Letozaf_balloons, yes20:27
Letozaf_balloons, but I have to push some changes I made20:27
balloonsLetozaf_, sure push them and tell me which test fails. That's easiest ;-)20:27
Letozaf_balloons, ok just a minute20:28
Letozaf_balloons, https://code.launchpad.net/~carla-sella/ubuntu-clock-app/new-add_world_city_test/+merge/231131 the test that fails is ubuntu_clock_app.tests.test_clock.TestClock.test_add_city_by_searching_must_add_world_city20:30
Letozaf_balloons, could this error be related to the count property bug we were taking about with nik90 ?20:40
* balloons looks20:48
* balloons forgets he needs to build first :-)20:54
Letozaf_balloons, :)20:54
balloonsweird, it's still launching the installed version20:56
balloonsohh right.. I know20:57
balloonsLetozaf_, ohh.. when you merged trunk, where did you merge from?20:57
balloonsahh I see lp:ubuntu-clock-app..20:58
Letozaf_balloons, yes20:58
balloonsit's weird clock has been migrated to change launching20:59
Letozaf_balloons, I launch it so: autopilot3 launch -i Qt qmlscene ../../app/ubuntu-clock-app.qml  -I ../../builddir/backend/21:00
Letozaf_balloons, when launching with autopilot21:00
nik90balloons: changed how?21:00
nik90balloons: its the same as before as Letozaf_ launches it21:00
balloonsnik90, I seem to remember we moved things around.. ohh, it was the old tests21:00
balloonswe can fix these up the same way, but that's another issue21:01
nik90yes21:01
nik90balloons: btw just finished reading your long conversation with zyga. Would you like me to convert the test description format to what the manual tests currently use like https://wiki.ubuntu.com/Process/Merges/TestPlan/indicator-datetime21:02
balloonsnik90, yes, ala: https://wiki.ubuntu.com/QATeam/TestCase21:03
balloonsnik90, https://wiki.ubuntu.com/Testing/TestCaseFormat shows the same, with some things we desired for the qatracker21:03
balloonswe might want to drop the verbiage at the bottom21:03
balloonsand we can decide if we want to bold and italics the same way21:03
nik90ok21:04
balloonsnik90, see how it's rendered? http://iso.qa.ubuntu.com/qatracker/testcases/1301/info21:04
balloons1. bold text as action21:04
balloonsitalics as verification21:04
nik90ah yes21:04
nik90well checkbox doesn't support bold and italics yet21:05
balloonsright, and it's not something that is in the test.. it's rendered by css that way :-)21:05
balloonsso plain text is fine21:05
nik90ah :)21:05
balloons 1. Perform X action21:05
balloons21:05
balloonsExpect Y result21:05
balloons2. Perform A action21:05
balloons21:05
balloonsExpect B result21:05
balloons21:05
balloonsExpect C result21:05
balloonsmeh, I don't think that pasted well.. anyways21:05
popeynik90: you totally need to blog about your lxc fun!21:06
nik90I get the idea21:06
nik90popey: :D21:06
balloonsLetozaf_, ok, I got the same issue you did, hehe21:06
balloonsso I'll look at the test21:06
balloonssorry for being slow21:07
Letozaf_balloons, no worries :) thanks21:07
balloonsLetozaf_, ok, so you grab the citylist and that seems to go fine, but you aren't seeing the count increase.21:08
balloonsso we should check things out.. my first thought is to print_tree cityList and to print_tree it's parent21:09
Letozaf_balloons, the count property has a dbus.int32 value21:09
Letozaf_balloons, so when I check >20 it gives an error message21:09
balloonsLetozaf_, ohh21:11
balloonslol21:11
balloonsIt's a syntax issue21:11
balloonsI misunderstoof21:11
Letozaf_balloons, no problem I wasn't sure about the problem :)21:11
balloonsLetozaf_, you want cityList.count.wait_for(GreaterThan(0))21:11
balloonsyou'll need to import GreaterThan21:12
Letozaf_balloons, ah ok ... so easy :P21:12
balloonsfrom testtools.matchers import GreaterThan21:12
Letozaf_balloons, yes thought the problem was something weired :P21:12
balloonsLetozaf_, yep totally. So by default wait_for uses Equals, but you can use any matcher you like21:12
balloonsmake sense?21:13
Letozaf_balloons, yes thanks21:13
Letozaf_balloons, yahoo!! it worked21:15
Letozaf_balloons, going to bed now it's late, thank you21:15
Letozaf_buona notte a tutti :)21:16
balloonsnik90, so did you try running the tests on the device itself? is that possible atm?21:21
nik90balloons: AP tests or qml?21:21
balloonsnik90, sorry I'm still looking at your manual stuff21:21
balloonsalso, I'm not sure you need to use that ppa on utopic or not. Running your instructions for me results in nothing.. just some prints21:22
nik90balloons: oh, I haven't tried it on the device. tbh I haven't given it any thought since it requires more work21:22
balloonsI tried with the archive version and the ppa version of plainbox21:22
nik90balloons: zyga pointed out that I missed a step21:22
nik90balloons: once you are in 2014... folder, type "./manage.py install"21:23
balloonsnik90, I needed sudo to do that.. but no change21:24
nik90balloons: I suggested adding the dev ppa because I was told it was updated almost every few days with all the new stuff. and since it was the qa team who were going to review the MP, I figured that shouldn't matter21:25
nik90balloons: what happens once you are inside the whitelist folder and run "plainbox run -w alarm.whitelist" ?21:25
balloonsnik90, http://paste.ubuntu.com/8313002/21:25
nik90balloons: hmm..okay can you run "../manage.py develop" and then run  "plainbox run -w alarm.whitelist"21:27
balloonsthat made some magic21:27
balloonsncurses ;-)21:28
balloonsnik90, kk21:28
=== salem_ is now known as _salem
nik90balloons: yup this is the console ui, but there is also a proper gui for running tests. but for that I need to figure out the launcher which should happen in my next mp21:31
balloonsnik90, I left questions on the mp; don't assume they are all for you :-) zygmut should answer some21:38
nik90ok ;)21:39
nik90balloons: regarding your file output question, http://plainbox.readthedocs.org/en/latest/usage.html# should come in handy21:40
balloonsdinner time.. ttyl nik90, pleasant evening to you121:41
nik90balloons: cya21:41
=== _salem is now known as salem_
Britohi22:20
rpadovaniSomeone should say to Oliver Ries that is bad making lines with more than 80 chars in mails...22:29
=== salem_ is now known as _salem
popeyrpadovani: get a wider screen ☻23:00
popeyooh, midnight, time for bed!23:00

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