| === chihchun_afk is now known as chihchun | ||
| mhall119 | zyga: you'd have to talk to beuno for changes in the store, is there a reason you want to use that different namespace rather than com.ubuntu.? | 00:53 |
|---|---|---|
| === 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 | ||
| === _salem is now known as salem_ | ||
| === salem_ is now known as _salem | ||
| mihir | Good Morning all. | 05:57 |
| mihir | balloons: are these pep8 formatting errors on AP for calendar, http://91.189.93.70:8080/job/ubuntu-calendar-app-utopic-amd64-ci/235/console ? | 05:57 |
| dev__ | I'm trying to download the SWORD library on qt but I'm not having any luck. | 06:26 |
| dev__ | I just can't seem to install it | 06:26 |
| dev__ | Has anybody had a hard time installing other API's or engines in Qt or even attempt to? | 06:58 |
| dholbach | good morning | 07:42 |
| justCarakas | good morning dholbach | 07:42 |
| dholbach | hi justCarakas | 07:43 |
| nik90 | Good morning :-) | 07:51 |
| dholbach | nik90, so I merge mzanetti's branch and pushed it | 07:53 |
| dholbach | but it makes the build fail as well | 07:53 |
| nik90 | dholbach: I think you pushed to the wrong branch | 07:54 |
| nik90 | dholbach: balloons yesterday created a common branch that we all can push stuff into -> https://code.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/reboot-packaging/+merge/230530 | 07:54 |
| nik90 | dholbach: this has jenkins passing :) | 07:54 |
| zyga | hey | 07:55 |
| zyga | mhall119: thanks for the answer | 07:55 |
| zyga | mhall119: on com.ubuntu vs com.canonical, not sure, it's not a community app really | 07:55 |
| zyga | beuno: hey | 07:55 |
| zyga | beuno: mhall119 told me you can fix my account's namespace setting for publishing click apps | 07:55 |
| dholbach | nik90, oh? | 07:56 |
| nik90 | dholbach: balloons did that since jenkins failed on your branch due to the find_local_path() function and other small issues in the __init__.py file. So he reverted that file alone to the trunk version and we added more fixes on top of that. | 07:57 |
| nik90 | dholbach: so as of yesterday night, the debian and click packaging works correctly | 07:58 |
| dholbach | nik90, I don't think the build works | 07:59 |
| nik90 | dholbach: What do you mean? The last comment is by jenkins which successfully ran the tests after installing the debian package | 08:00 |
| dholbach | let me retry with a fresh branch | 08:00 |
| dholbach | ah no | 08:02 |
| dholbach | it passes \o/ | 08:02 |
| nik90 | woohoo :D | 08:02 |
| nik90 | you scared me for a moment there (insert angry face here) | 08:02 |
| dholbach | let me do a couple more tests | 08:03 |
| nik90 | ok | 08:03 |
| mzanetti | dholbach: nik90: hey | 08:10 |
| mzanetti | I'm not sure if my branch is really the right thing to do | 08:10 |
| mzanetti | but this makes it compile and run on the phone again for me | 08:10 |
| dholbach | cool | 08:10 |
| mzanetti | however, I've no idea if it breaks other stuff | 08:10 |
| dholbach | I'm just checking a couple of things and let you know how it works :) | 08:10 |
| nik90 | mzanetti: After doing http://bazaar.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/reboot-packaging/revision/80 it seems to run on phone for me. | 08:11 |
| nik90 | it seems on the phone it was unable to find the c++ plugins that we wrote. The above fixes that | 08:11 |
| mzanetti | ah ok... you're disabling the whole tests/autopilot dir for click mode | 08:11 |
| mzanetti | which should make my patch obsolete I guess | 08:12 |
| mzanetti | hmm... for me it was only an issue with finding the python include dir. the rest was fine | 08:12 |
| nik90 | sergiusens came to the conclusion that since we do not run the tests in the click mode might as well not include it in the click package. | 08:13 |
| mzanetti | +1 | 08:13 |
| mzanetti | I thought you guys wanted to have that | 08:13 |
| mzanetti | I for one would not package them up either, no | 08:13 |
| mzanetti | if anything, create a separate package just for the tests | 08:14 |
| nik90 | yes, we have a separate autopilot debian package | 08:14 |
| dholbach | so I don't know how the autopilot tests are supposed to be run | 08:16 |
| dholbach | but run as part of autopkgtest, they still fail: http://paste.ubuntu.com/8034122/ | 08:16 |
| dholbach | maybe autopkgtest is not the way to go, I don't know | 08:16 |
| dholbach | I'll leave this for the gods of testing to decide | 08:16 |
| dholbach | or maybe "adt-run -U -o test-output --source /tmp/ubuntu-clock-app_1.2.dsc --- qemu adt-utopic-amd64-cloud.img" is the wrong way to go about it | 08:17 |
| dholbach | but yeah, making jenkins happy is fantastic | 08:17 |
| dholbach | I guess my branch can be dropped then | 08:17 |
| nik90 | yes | 08:18 |
| nik90 | dholbach: can I add more past release entries into the debian changelog? | 08:18 |
| nik90 | dholbach: I want the release landing on the phone to be 3.0 | 08:19 |
| dholbach | nik90, sure, feel free to | 08:19 |
| dholbach | with "dch -i | 08:19 |
| dholbach | with "dch -i" you can add a template new entry | 08:19 |
| dholbach | or just modify the current one | 08:19 |
| nik90 | I will just modify the current one | 08:19 |
| nik90 | so I will add past release below the current 1.2 version for instance | 08:19 |
| dholbach | sure sure | 08:21 |
| dholbach | good work everyone | 08:21 |
| nik90 | thnx daniel | 08:21 |
| dpm | morning nik90, I've added i18n support to build the .pot file and take into account the .desktop file too to clock reboot, if you've got time to look at the MPs, then I can set up translations in LP once approved. Let me know if you have any questions. Thanks! | 08:57 |
| JamesTait | Good morning all; happy International Lefthanders Day! :-D | 08:59 |
| DanChapman | Good Morning all o/ | 09:03 |
| dpm | hi nerochiaro, around? | 09:05 |
| nerochiaro | dpm: yes | 09:05 |
| nerochiaro | hi | 09:05 |
| dpm | hey :) | 09:05 |
| dpm | nerochiaro, I just wanted to check, as folks start going on holiday on these dates :) - I sent you an e-mail about the translations for the .desktop files for camera, dialer and messaging not being loaded in the dash. Do you think you could have a look at it when you've got a minute? | 09:06 |
| justCarakas | dpm do you know if there is already a system to use translations in HTML5 apps | 09:07 |
| nerochiaro | dpm: when i did that work back then i basically copy-pasted what had been done in other apps for translating these .desktop filtes. i'm not actually sure what the dash does to read them | 09:08 |
| dpm | justCarakas, daker was working on one, but I'm not sure about its status | 09:08 |
| dpm | nerochiaro, do the translations get added correctly to the final .desktop files? That'd be the first thing to look at | 09:09 |
| nerochiaro | dpm: i checked that they did, but i'll double check that. i'm in the middle of something right now but i added it to the todo list | 09:09 |
| dpm | thanks nerochiaro | 09:10 |
| nik90 | dpm: hey, yeah I got the notifications about the Mps by email when I woke up. | 09:37 |
| nik90 | dpm: we are in the process of landing the debian and click packaging fixes in the trunk. Once that happens jenkins can autoland stuff from here on. After that branch lands, I will review your branch since there might be some code conflicts. | 09:37 |
| dpm | ok, cool | 09:38 |
| zyga | dpm: hey | 09:51 |
| zyga | dpm: what's the recommended way for a canonical team to create and upload click packages to the store? | 09:51 |
| zyga | dpm: can we all do it or does it have to be owned by one specific person? | 09:52 |
| dpm | zyga, there is no support for teams in the store, so the pragmatic way is to have a shared account where everyone who needs upload permissions has the password | 09:52 |
| zyga | dpm: thanks | 09:55 |
| zyga | dpm: is that how other teams do it? | 09:55 |
| zyga | dpm: any SOP pages that document that (I'm ok to create them if they are missing) | 09:55 |
| zyga | dpm: last question, is there any magic regarding namespaces, can anyone (e.g. me) just create apps in the com.canonical.* namespace/ | 09:57 |
| dpm | zyga, that's how other teams do it, yes. I'm not sure what SOP is | 09:57 |
| zyga | dpm: thanks :-) | 09:58 |
| zyga | dpm: Statement of Practice or something like that (on the wiki, lots of SOP pages document how we do things) | 09:58 |
| dpm | zyga, I don't think there is anything special about the com.canonical.* namespaces, but beuno would know more (he's away today, though, I think) | 09:58 |
| zyga | ok, I'll double check with beuno | 09:59 |
| popey | zyga: we have a few teams who are uploading multiple apps, for example the core apps are one 'team' (shared account) and webapps are another, perhaps speak to dbarth to see what he did | 10:03 |
| zyga | popey: thanks! good idea | 10:04 |
| daker | justCarakas: yes we have two https://code.launchpad.net/~daker/ubuntu-html5-theme/i18n-pojson | 10:17 |
| daker | https://code.launchpad.net/~daker/ubuntu-html5-theme/i18n-po | 10:17 |
| daker | the json is still WIP and po one is a working prototype that's still wait for a review | 10:18 |
| justCarakas | ok, I'm gonne update BE Mobile soon and atm I'm roling my own for it but it would be nice to integrate | 10:18 |
| zyga | popey: do you have access to that shared account? | 10:26 |
| zyga | popey: we'd rather not create another account and just share this one | 10:26 |
| popey | zyga: what's to be uploaded? | 10:33 |
| zyga | popey: we want to upload under-development versions of checkbox | 10:35 |
| zyga | popey: it will be able to run simple device tests soon and the store is how we want to distribute it | 10:35 |
| popey | ok, so what we do for most apps is we have one shared account which multiple people can upload with, including Mirv, xnox, sergiusens and balloons | 10:35 |
| popey | probably easiest to just provide a click to one of them, and ask them to upload to the store | 10:36 |
| popey | then the semi-automated review and publishing will take care of the rest | 10:36 |
| zyga | popey: can we upload it ourselves or do we need to go through a middle-man each time? | 10:36 |
| popey | you could, but then it's tied to your account | 10:36 |
| zyga | I mean, could we just use the same shared account directly/ | 10:36 |
| popey | we block on you if there's an emergency upload required | 10:36 |
| popey | I don't know what the "rules" are on sharing that account. I don't manage it. | 10:37 |
| popey | I don't have access to upload (on purpose) | 10:37 |
| zyga | ok, do you know who I should talk to (apart from dbarth)? xnox, etc? | 10:37 |
| popey | to what? upload yourself? | 10:38 |
| zyga | to know "rules" and maybe get the access | 10:38 |
| zyga | yes | 10:38 |
| popey | how many apps is this for? are they expected to be baked into the device? | 10:40 |
| popey | or are we just talking about effectively 3rd party apps | 10:40 |
| zyga | popey: just the one | 10:40 |
| zyga | popey: not sure, maybe, maybe not, that's up in the air (baked or not) | 10:41 |
| zyga | popey: this is a system-level kind of app | 10:41 |
| zyga | popey: right now it's just a public prototype | 10:41 |
| zyga | popey: though there are some non-public derivatives that might be baked in later | 10:41 |
| zyga | popey: I don't manage that part as it's not really a tangible thing yet | 10:41 |
| dpm | popey, there are 2 clock branches blocked on data missing from geoname-lookup.u.c - have you followed with ev or someone else on how to add additional data to the service? http://people.canonical.com/~dpm/core-apps-sponsoring/ | 10:44 |
| popey | dpm: am looking at it myself | 10:51 |
| popey | zyga: as dpm says I think beuno is the best person to speak to, but in the short term, the quickest way to get your click in the store is to ask dpm / balloons or mirv to upload | 10:53 |
| Laney | hey Kaleo, do you know if camera-app can move to content1 instead of content0.1? | 10:54 |
| dpm | thanks popey, if you could update the clock MP with the results, that'd be great | 10:55 |
| Laney | actually I don't know what this build dependency is for, maybe tests? | 10:55 |
| popey | will do when I figure it out | 10:55 |
| zyga | popey: thanks, I'll follow up with dpm and beuno when he's around :) | 10:56 |
| nik90 | dpm, popey: Is there a person with debian packaging know how to do a quick review of https://code.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/reboot-packaging/+merge/230530. It was done by Daniel, so he can't review it :) | 10:57 |
| nik90 | dpm, popey: I reviewed the cmake bits and think it is okay | 10:57 |
| dpm | Mirv, do you think you could help us with a quick packaging review? It's about adding .deb packaging support for the new clock app -> https://code.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/reboot-packaging/+merge/230530 | 10:58 |
| Kaleo | Laney, you mean the import, or deb deps? or what? | 11:00 |
| Laney | Kaleo: ah yes, I see the imports now, I mean both I guess | 11:02 |
| Laney | content-hub doesn't even build the old one any more | 11:03 |
| Mirv | dpm: I'm looking at it, is it meant for archive upload? (like, to be used on desktop) | 11:23 |
| dpm | Mirv, it's only for Jenkins purposes, to be uploaded in the core apps PPA, so that Jenkins can run desktop tests | 11:23 |
| dpm | i.e. not for the archive | 11:24 |
| nik90 | Mirv: any idea why the deb packages created (as shown in jenkins) at http://91.189.93.70:8080/job/generic-mediumtests-utopic-python3/12/? is named ubuntu-clock-app-autopilot_3.0+autopilot0_all.deb instead of ubuntu_clock_app_3.0_all.deb? | 11:32 |
| nik90 | Mirv: not sure why it appends autopilot to the name when we have a separate package for that. | 11:32 |
| popey | nik90: just fyi, I'm fiddling with the geonames stuff, and will have it up soon on a server, can we test your clock, with it pointing at my geonames server once done? | 11:33 |
| nik90 | popey: yup, I need to make a tiny change to the branch when the server is ready | 11:33 |
| popey | k | 11:33 |
| * popey goes back to battling postgres | 11:33 | |
| * popey gets coffee | 11:34 | |
| nik90 | dpm: ping | 11:34 |
| nerochiaro | Elleo: did you try building the gallery in a chroot last week ? | 11:35 |
| Elleo | nerochiaro: oh no sorry, only natively; although I did hit the QtDbus issue there too. artmello also ran into it and was going to submit a branch I think (adding a missing build-dep instead of removing the test) | 11:38 |
| nerochiaro | Elleo: the test has been removed in trunk already | 11:39 |
| Elleo | ah, okay | 11:39 |
| nerochiaro | Elleo: can you please try building in the chroot ? I'm having other issues, like cmake not being able to find Qt5 | 11:39 |
| Elleo | nerochiaro: okay, I'll setup a chroot now | 11:40 |
| nerochiaro | Elleo: thanks | 11:40 |
| dpm | hey nik90 | 11:42 |
| nik90 | dpm: Hi, at the moment the debian/control files are point to https://code.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/trunk which is the old branch. Should we change this for the clock reboot? | 11:43 |
| Mirv | nik90: not sure, possibly related to how they mangle the versions in jenkins and the fact that the package has native version (instead of what's 'normally' used ie distro version appended) | 11:43 |
| nerochiaro | zbenjamin: do you know what might be causing this when building gallery-app in a chroot from qtcreator ? http://pastebin.ubuntu.com/8035417/ i updated the chroot from qtcreator and also used "maintain" to do "apt-get build-dep gallery-app" but it's not getting better | 11:43 |
| nik90 | dpm: or are we planning to keep the reboot branch as it is? or copy it all over to the other main branch? | 11:43 |
| Mirv | nik90: I just pushed some more debian/ changes in, please pull | 11:43 |
| nik90 | Mirv: ah ok. sure | 11:43 |
| nik90 | Mirv: can you change the description in debian/control to Ubuntu Core Clock application which provides Alarm, Clock and World Clock functionalities. | 11:45 |
| nik90 | we no longer have timer and stopwatch for RTM | 11:45 |
| nik90 | Mirv: and lastly, why do we have ubuntu-clock-app (3.0-0ubuntu1) UNRELEASED; urgency=medium as UNRELEASED rather than say utopic? | 11:45 |
| dpm | nik90, I'd say we update debian/control to point to the reboot branch, so it's got the correct branch, and then we have a separate discussion on how we'll swap or branch off. Does that make sense? | 11:46 |
| nik90 | dpm: +1 | 11:46 |
| dpm | ok, cool | 11:46 |
| nik90 | dpm: I guess the same applies then to the manifest.json files as well | 11:46 |
| dpm | right | 11:47 |
| nerochiaro | dpm: in the email you sent me you say that for camera-app I need to remove from the .desktop file template the X-Ubuntu-Gettext-Domain=camera-app' line. However I did that, as you can see here: https://code.launchpad.net/~phablet-team/camera-app/camera-app-desktop-translation/+merge/223229 | 11:47 |
| nik90 | ack | 11:47 |
| Mirv | nik90: ok, changing. UNRELEASED it would be as long as it's not in ubuntu archives (I just wrote a comment to the MP), but I can revert that back since not all distro policies are really needed if you're not aiming there but the core-apps PPA | 11:47 |
| dpm | nerochiaro, I'm perhaps looking at the wrong branch, but I can still see it here: http://bazaar.launchpad.net/~phablet-team/camera-app/trunk/view/head:/camera-app.desktop.in | 11:48 |
| nerochiaro | dpm: you're looking at trunk, the branch i submitted for review is lp:~phablet-team/camera-app/camera-app-desktop-translation and it hasn't been reviewed yet I think | 11:48 |
| nerochiaro | dpm: jenkins was rejecting it so i merged trunk and pushed it again | 11:49 |
| Mirv | nik90: pushed another commit | 11:49 |
| nik90 | Mirv: ok. I will make a few other changes now (vcs-bzr needs to point at the correct bzr branch for instance). But nothing that should break the debian build. | 11:50 |
| nik90 | Mirv: thnx | 11:50 |
| nik90 | Mirv: can you give me 5 mins before approving the branch for debian code review? | 11:50 |
| Mirv | it surely looks good enough for the PPA | 11:50 |
| Mirv | nik90: yeah, no problem, and maybe dpm or someone can do the top-approve. I'm fine enough with the packaging. | 11:51 |
| nik90 | ack | 11:51 |
| === MacSlow is now known as MacSlow|lunch | ||
| dpm | nerochiaro, thanks, I've just added a fet comments. Do you think someone else could review it to get it landed? | 12:01 |
| nerochiaro | dpm: i'll fix your comments, then i think you can review it and we can check with oSoMoN to get it landed | 12:02 |
| dpm | cool | 12:03 |
| nik90 | dholbach: hey, ready to merge https://code.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/reboot-packaging/+merge/230530 ? | 12:05 |
| nik90 | dholbach: can you top approve? | 12:05 |
| dholbach | nik90, can we change 3.0ubuntu1 → 3.0? | 12:07 |
| dholbach | nik90, sorry | 12:07 |
| dholbach | nik90, can we change 3.0-0ubuntu1 → 3.0? | 12:07 |
| nik90 | dholbach: it was done by Mirv just now | 12:07 |
| dholbach | oh ok | 12:07 |
| nik90 | Mirv: is that okay ^^ | 12:07 |
| dholbach | Mirv, ^ is this something with daily landing I don't understand? :-) | 12:07 |
| Mirv | dholbach: well we don't have non-distronative packages as native packages normally? | 12:09 |
| Mirv | dholbach: it's irrelevant for PPA usage though and can be reverted | 12:09 |
| nerochiaro | dpm: i don't understand why the file should be named .in.in.h, in your filemanager branch it is named .in.in, not .in.in.h | 12:09 |
| dholbach | Mirv, I don't have a hard opinion on it - it just surprised me somewhat | 12:10 |
| nik90 | dholbach, Mirv: May be we do what the camera app does? https://bazaar.launchpad.net/~phablet-team/camera-app/trunk/view/head:/debian/changelog | 12:11 |
| nik90 | as far as I understand it is also pushed as a click to the store | 12:11 |
| dpm | nerochiaro, it's the name of the file that intltool-extract generates. It's .in.in.h in L20 of https://code.launchpad.net/~dpm/ubuntu-filemanager-app/desktop-i18n/+merge/230591 | 12:12 |
| dholbach | top-approved | 12:12 |
| nik90 | dholbach: jenkins should automerge right? | 12:13 |
| Mirv | nik90: it'd be now the same for clock app if it would be released through CI Train (which adds the date) | 12:13 |
| dholbach | I don't know | 12:13 |
| nik90 | dholbach: alrite I will wait and see | 12:13 |
| nik90 | Mirv: ah ok | 12:14 |
| dholbach | excellent | 12:14 |
| Mirv | so the "3.0.0-0ubuntu1) UNRELEASED" is what CI Train would expect, except that we're not doing this for CI Train now, or archives | 12:14 |
| nik90 | ack | 12:14 |
| nerochiaro | dpm: but if i change CMakeFile.txt to configure_file(${DESKTOP_FILE}.in.in.h ${DESKTOP_FILE}.in) and then run cmake it tells me that the .in.in.h file doens't exist | 12:19 |
| nerochiaro | Elleo: news with the chroot ? | 12:20 |
| Elleo | nerochiaro: still downloading packages | 12:20 |
| Elleo | think it's almost done as its on to the qt packages now | 12:20 |
| nerochiaro | dpm: i also change the po/CMakeLists.txt to create a .in.in.h but i get the same problem | 12:22 |
| nik90 | dholbach: the autolanding failed :/ | 12:24 |
| nik90 | hmm I see the error could not import package ubuntu_clock_app: No module named ubuntu_clock_app | 12:25 |
| popey | nik90: TimeZoneID is the only extra field you need, right? | 12:26 |
| nerochiaro | zbenjamin: ping | 12:27 |
| nik90 | popey: one sec, let me check | 12:27 |
| nik90 | popey: https://bugs.launchpad.net/ubuntu-weather-app/+bug/1326918/comments/5 | 12:28 |
| ubot5 | Ubuntu bug 1326918 in Ubuntu Clock App reboot "API key for geonames.org" [High,In progress] | 12:28 |
| nik90 | popey: essentially admin2, admin3 and timezoneID which are also required by weather app | 12:28 |
| popey | hm, ok | 12:28 |
| * popey fiddles with postgres queries | 12:28 | |
| nik90 | popey: admin2, admin3 are basically used to distuinguish cities with the same names. | 12:29 |
| nik90 | so it is not essential right away | 12:29 |
| popey | ok | 12:29 |
| === _salem is now known as salem_ | ||
| dpm | nerochiaro, you don't have to configure .in.in.h to .in. the .in.in.h file is a product of running intltool-extract | 12:32 |
| nerochiaro | dpm: oh, i see | 12:32 |
| dpm | so intltool-extract generates the .in.in.h file, which is added to the list of files to pass to xgettext | 12:34 |
| nik90 | balloons: ping | 12:34 |
| Elleo | nerochiaro: chroot creation failed, will try again | 12:35 |
| nerochiaro | Elleo: failed how ? | 12:36 |
| Elleo | nerochiaro: it just gave a generic error saying that the chroot creation had failed and then cleaned up (using the Qt chroot creation stuff) | 12:37 |
| nerochiaro | Elleo: bleh, we really need some help from zbenjamin here | 12:38 |
| Elleo | nerochiaro: restarted the sdk and letting it have another try; if that fails I'll try doing it using the click tools at the command line and see if that spits out any more info | 12:39 |
| renatu | dpm, thanks for review my MR. I have fixed your comments could you check it again? https://code.launchpad.net/~renatofilho/address-book-app/update-translation/+merge/230556 | 12:41 |
| nerochiaro | Elleo: thanks | 12:41 |
| dpm | renatu, done, thanks! | 12:43 |
| dpm | popey, does the accounts button in system settings work for you? | 12:44 |
| popey | i saw davmor2 mention aproblem with it earlier... | 12:44 |
| dpm | I've noticed since a couple of images ago that it does nothing for me | 12:44 |
| dpm | davmor2, are you having the same problem? ^ | 12:44 |
| popey | works here | 12:44 |
| popey | back button doesn't though, black screen | 12:45 |
| renatu | dpm, thank you | 12:45 |
| dpm | yeah, I had that too, I'd be happy to see the accounts at all, nevermind going back one screen and crashing :) | 12:45 |
| popey | you want the moon on a stick! | 12:45 |
| dpm | :) | 12:46 |
| davmor2 | dpm: no it's not crashing it's just not opened the normal way so the close isn't handled quite correctly. | 13:02 |
| nik90 | fginther: quick question, how does something that passes jenkins continous-integration fail autolanding? | 13:03 |
| davmor2 | dpm: if you got bored of waiting and swiped the accounts page closed that used to stop it from opening only way to get around that was to reboot | 13:03 |
| dpm | davmor2, no, in my case it's not opening at all | 13:03 |
| justCarakas | is it allowed to provide things like this https://developer.cdn.mozilla.net/media/uploads/demos/v/i/viliusl/66924b4d3bff8b1c0abecfd52e72407f/minipaint_1375196685_demo_package/index.html as a web app ? | 13:05 |
| dpm | hm, and I can't find a log file for the the system settings app | 13:05 |
| nerochiaro | dpm: i'm still quite confused by the other MRs you point out. the filemanager does explicitly configure .in.in.h for example, if you look at http://bazaar.launchpad.net/~dpm/ubuntu-filemanager-app/desktop-i18n/view/head:/CMakeLists.txt | 13:07 |
| dpm | nerochiaro, I can only see "configure_file(${DESKTOP_FILE}.in.in ${DESKTOP_FILE}.in)" | 13:07 |
| dpm | no .h | 13:07 |
| dholbach | nik90, hum, not sure | 13:09 |
| Elleo | nerochiaro: qtcreator crashed partway through :/ attempting from the command line now | 13:10 |
| nik90 | dholbach: I kicked it again since after the failed autolanding, it said continuous integration said it passed. Not sure what changed in that mean time. | 13:11 |
| * nik90 waits for few mins | 13:11 | |
| nerochiaro | Elleo: thanks. we really need to report all these bugs. I don't know how you manage, but I'm wasting a ton of time trying to build stuff for the device these days | 13:11 |
| davmor2 | dpm: try online-accounts it's a separate application | 13:11 |
| dholbach | nik90, it looks like it still hasn't landed in lp:ubuntu-clock-app/reboot | 13:11 |
| Elleo | nerochiaro: I tend to mostly build stuff directly on device, as its a lot less hassle for me that way | 13:12 |
| nerochiaro | Elleo: i wish i could, it doesn't work for me. maybe you can give me an hand setting that up, if it works for you ? | 13:13 |
| nerochiaro | Elleo: or tell me your process | 13:13 |
| nik90 | dholbach: I know since the autolanding failed | 13:13 |
| Elleo | nerochiaro: I basically just ssh in and treat it like any other linux system | 13:14 |
| nerochiaro | Elleo: so for gallery you build a deb with debuild and then install it with dpkg ? | 13:14 |
| Elleo | nerochiaro: no I use the click build script | 13:14 |
| bitgandtter | good morning everyone | 13:15 |
| nerochiaro | Elleo: what's the command you use ? | 13:15 |
| bitgandtter | Im writing a ubuntu touch app with u1db as db, but im getting troubles with query and index. Im saving a doc with sub-fields, but at retrieve point with index and query i wont get the expected resutls. I need the list to show two rows one for the account "Ahorro" another one for the account "Corriente" but only one row show, the first one. | 13:15 |
| Elleo | nerochiaro: there's a script called click-build.sh that's part of the gallery repo, that has everything in it already | 13:15 |
| Elleo | nerochiaro: you'll need to install all the build-deps first though | 13:15 |
| bitgandtter | here is the code http://pastie.org/9470062 | 13:15 |
| bitgandtter | can anyone helpme? | 13:16 |
| Elleo | nerochiaro: I haven't built gallery in a couple of months though, so can't guarantee it still works okay | 13:16 |
| zbenjamin | nerochiaro: pong, sorry for the delay i was in my lunch break | 13:21 |
| nerochiaro | zbenjamin: both me and Elleo are having problems building things in chroots (or in his case building the chroot itself) | 13:22 |
| zbenjamin | Elleo: try to use click chroot -a <arch> -f ubuntu-sdk-<version> create on the commandline and look if you can find any errors there | 13:23 |
| Elleo | zbenjamin: yep, that's what I'm doing now :) | 13:23 |
| zbenjamin | nerochiaro: whats your problem? | 13:24 |
| nik90 | dholbach: nope still fails. I guess I will have to wait for balloons or fginther to come online | 13:24 |
| dholbach | ok, or somebody in #ubuntu-ci-eng? | 13:24 |
| fginther | nik90, hello, what's the MP? | 13:24 |
| nik90 | fginther: https://code.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/reboot-packaging/+merge/230530 | 13:24 |
| bitgandtter | also im trying to desing an app that have relation between documents, there is a best practice for do that on on u1db? | 13:25 |
| nerochiaro | zbenjamin: when building gallery in the chroot i get this: http://pastebin.ubuntu.com/8035417/ i tried updating the chroot and installing build-dep of gallery-app in it, but no luck | 13:25 |
| nik90 | fginther: I see, | 13:26 |
| nik90 | could not import package ubuntu_clock_app: No module named ubuntu_clock_app | 13:26 |
| nik90 | Loading tests from: | 13:26 |
| nik90 | Did not find any tests | 13:26 |
| fginther | nik90, I found the problem. | 13:26 |
| nik90 | looks like an autopilot error | 13:26 |
| nik90 | fginther: ? | 13:27 |
| fginther | nik90, I didn't update the testing for the autolanding job to use python3. I have that fixed and will re-approve that MP | 13:27 |
| nik90 | fginther: ah :) | 13:27 |
| nik90 | fginther: hope that fixes the issue | 13:27 |
| zbenjamin | nerochiaro: ugh that does not look good, is this a new chroot? | 13:29 |
| nerochiaro | zbenjamin: i created it some weeks ago | 13:29 |
| nerochiaro | zbenjamin: should i try a new one ? do they bitrot that quick ? | 13:30 |
| zbenjamin | wait | 13:32 |
| popey | dholbach: you and I both reviewed dinko's app at the same time, you beat me by 1s and I got a server error ⍨ | 13:33 |
| dholbach | \0/ | 13:34 |
| zbenjamin | nerochiaro: its weird, can you try to delete all build files and try again? | 13:34 |
| nerochiaro | zbenjamin: already tried | 13:35 |
| zbenjamin | nerochiaro: the weird thing is apt-file does not find those cmake files either | 13:36 |
| zbenjamin | nerochiaro: the weird thing is apt-file does not find those cmake files either | 13:37 |
| nerochiaro | zbenjamin: i think they are called something different, that's the name CMake use | 13:38 |
| zbenjamin | nerochiaro: yeah the problem is that cmake is still looking for those files | 13:39 |
| zbenjamin | nerochiaro: or something in the click chroots changed that broke cmake, which is also possible | 13:39 |
| nerochiaro | zbenjamin: anything i can do to help debug the problem ? | 13:39 |
| * zbenjamin updates his chroot | 13:40 | |
| zbenjamin | nerochiaro: i have not enough knowledge about how cmake searches for qt now | 13:40 |
| zbenjamin | nerochiaro: let me try if i can reproduce | 13:40 |
| nerochiaro | zbenjamin: i'm on 14.10 if it changes anything, but it shouldn't | 13:41 |
| nik90 | dholbach, fginther: Autolanding PASSED! | 13:41 |
| nik90 | fginther: thnx for the hotfix | 13:41 |
| fginther | nik90, you're welcome, sorry it was borked in the first place :-) | 13:42 |
| nik90 | np | 13:42 |
| dholbach | yeehaw! | 13:42 |
| nik90 | dholbach: got a ton of branches to land.. | 13:42 |
| * nik90 gets busy busy | 13:42 | |
| dholbach | :-D | 13:43 |
| mara___ | please help. I have a Java Swing project in Netbeans. I need him to create the Debian Source package. Can someone help me. thanks | 13:43 |
| zbenjamin | nerochiaro: hm, it works for me even with a up2date chroot. | 13:44 |
| zbenjamin | nerochiaro: you _could_ try to recreate it | 13:45 |
| zbenjamin | nerochiaro: and pray that the problem is gone | 13:45 |
| nerochiaro | zbenjamin: well, i'm trying with a fresh chroot, but i'm not a religious person | 13:46 |
| nerochiaro | so no praying to technology | 13:46 |
| nerochiaro | zbenjamin: i'm going t try this with the new chroot. if it doesn't work if you don't mind it might be a good idea to get on an hangout and talk about it by voice, will be quickler | 13:50 |
| nerochiaro | quicker | 13:50 |
| zbenjamin | nerochiaro: you can also pray to the FSM , makes no difference ;) | 13:50 |
| zbenjamin | nerochiaro: we can try this, but when its still not working we probably should have a click developer there | 13:51 |
| nerochiaro | zbenjamin: who would that be ? | 13:52 |
| zbenjamin | nerochiaro: good question | 13:53 |
| nik90 | dholbach: need your opinion on the debian versioning. So right now the latest version is 3.0-0ubuntu1. If I fix a bug and update the changelog, it seems a bit strange to release a new version just for that 1 bug fix. So do I keep appending to this existing version? And secondly, for new versions, does 3.0.1-0ubuntu1 look good? | 13:53 |
| zbenjamin | nerochiaro: seems mvo is the one to ask | 13:54 |
| dholbach | nik90, I personally would just have gone with 3.0 and then 3.1, etc - adding something like "-0ubuntu1" usually indicates that you do "upstream releases" (like tarball releases) or your piece of software | 13:54 |
| nik90 | dholbach: ack | 13:55 |
| dholbach | and then "-0ubuntu1" means: not been in Debian yet, one revision of this particular upstream version has been uploaded to Ubuntu | 13:55 |
| nerochiaro | zbenjamin: i get this when trying to create the new chroot: http://pastebin.ubuntu.com/8036366/ | 13:55 |
| dholbach | for a small fix like a dependency change you'd then go with 0ubuntu2 | 13:55 |
| nerochiaro | Elleo: is it the same problem you were having ? http://pastebin.ubuntu.com/8036366/ | 13:55 |
| dholbach | but maybe there's a good reason in the autolanding scheme of things to use -0ubuntu1 | 13:55 |
| zbenjamin | nerochiaro: check with mount if you old chroot is still mounted | 13:56 |
| nik90 | dholbach: ok. I am going to add 3.1 and then keep appending bug fixes to it before I consider 3.2 worthy. | 13:56 |
| dpm | mzanetti, would you mind if we move today's Reminders meeting to tomorrow same time? | 13:57 |
| zbenjamin | nerochiaro: if it is try schroot --all-sessions -e | 13:57 |
| dholbach | nik90, maybe have a chat with Mirv as well to confirm | 13:57 |
| Elleo | nerochiaro: possibly, I didn't get as much output as that only the "There was an error creating the click target, cleaning up" line onwards in qtcreator | 13:58 |
| nik90 | dholbach: ok | 13:58 |
| nerochiaro | zbenjamin: i asked qtcreator to delete it. that should've unmounted it no ? | 13:58 |
| nerochiaro | zbenjamin: to delete the old one | 13:58 |
| nik90 | Mirv: hey, when you are free, can you ping me. Just a really low priority question. So no hurry :) | 13:58 |
| mzanetti | dpm: works for me | 13:59 |
| mara___ | please help. I have a Java Swing project in Netbeans. I need him to create the Debian Source package. Can someone help me. thanks | 13:59 |
| dpm | mzanetti, thanks. Actually, it'd be half an hour earlier, if that still works | 14:00 |
| mzanetti | dpm: sure | 14:00 |
| dpm | excellent | 14:00 |
| zbenjamin | nerochiaro: its chroot, sometimes it fails | 14:00 |
| nik90 | dpm: can you merge trunk to your branches. we just landed a mega branch into trunk :) | 14:01 |
| nerochiaro | zbenjamin: interestingly enough when i try to build on the device (manually, not via qtcreator) i have the same problem | 14:01 |
| nerochiaro | zbenjamin: of qt5 libs not being found | 14:01 |
| zbenjamin | nerochiaro: the dev packages are not installed on the device | 14:02 |
| nerochiaro | zbenjamin: i installed build-dep of gallery-app, that should've brought them in | 14:03 |
| zbenjamin | nerochiaro: true | 14:04 |
| nerochiaro | zbenjamin: wait, trying again now installs some more packages | 14:04 |
| zbenjamin | nerochiaro: wtf | 14:04 |
| nerochiaro | zbenjamin: yeah, wtf... but it's building now, on device, let's see what happens. in the meantime, i'm trying these chroots | 14:05 |
| nerochiaro | zbenjamin: this is the result of the commmand you suggested before: http://pastebin.ubuntu.com/8036416/ | 14:06 |
| dpm | nik90, sure. In about 2 hours I should be able to | 14:08 |
| nik90 | dpm: ok | 14:08 |
| zbenjamin | nerochiaro: as i thought, schroot did not unmount correctly | 14:09 |
| zbenjamin | nerochiaro: check if the /var/lib/schroot/chroots/click-ubuntu-sdk-14.10-armhf is deleted | 14:10 |
| zbenjamin | nerochiaro: if its gone, restart your command | 14:10 |
| nerochiaro | zbenjamin: the dir is not there but if i try to create a new chroot from qtcreator it gives me the error i pastebin'd you before | 14:13 |
| zbenjamin | nerochiaro: click chroot -a armhf -f ubuntu-sdk-14.10 destroy | 14:14 |
| popey | nik90: http://146.185.188.87/?query=Farnborough now has timezone.. any good? | 14:14 |
| nik90 | popey: oh yeah that's awesome | 14:14 |
| nik90 | popey: that should unblock my branches | 14:14 |
| nerochiaro | zbenjamin: "A chroot for that name and architecture does not exist." | 14:15 |
| nerochiaro | zbenjamin: if i try "create" instead of destroy i get that error | 14:15 |
| nik90 | popey: so should I point the clock app to your server link or are you merging this into geonames lookup | 14:15 |
| nerochiaro | zbenjamin: "that error" i mean the one i put on pastebin | 14:16 |
| nerochiaro | zbenjamin: so it seems like a problem with debootstrap, who's responsible for that ? | 14:17 |
| popey | nik90: I'll prepare a patch, no guarantees when it will land. | 14:18 |
| zbenjamin | nerochiaro: Cannot install into target '/var/lib/schroot/chroots/click-ubuntu-sdk-14.10-armhf' mounted with noexec or nodev thats the error and it tells you that its still mounted | 14:18 |
| nik90 | popey: ok | 14:18 |
| popey | nik90: the box that is running on is a vps I'm personally pay for, not canonical infra.. | 14:18 |
| zbenjamin | nerochiaro: you can try to reboot or force unmount them | 14:18 |
| popey | nik90: I'll speak to the guys in our community call and see what we can do to get it landed, but feel free to test on that IP for now | 14:19 |
| nerochiaro | zbenjamin: well the real cause seems "/usr/sbin/debootstrap: 1290: /usr/sbin/debootstrap: cannot create /var/lib/schroot/chroots/click-ubuntu-sdk-14.10-armhf/test-dev-null: Permission denied | 14:19 |
| nerochiaro | " | 14:19 |
| nik90 | popey: ok. I can test my branch with your server but cannot land it before your patch lands. | 14:19 |
| nerochiaro | zbenjamin: shit, /var/lib/schroot does not actually exist | 14:20 |
| zbenjamin | nerochiaro: i would try a reboot | 14:20 |
| nerochiaro | zbenjamin: i created that dir instead | 14:20 |
| zbenjamin | nerochiaro: wut | 14:20 |
| popey | nik90: ok | 14:25 |
| nerochiaro | zbenjamin: ok, seems to be finally building the chroot | 14:26 |
| zbenjamin | nerochiaro: how did you fix it? | 14:26 |
| nerochiaro | zbenjamin: created the missing dirs | 14:30 |
| nerochiaro | zbenjamin: not sure why they were not there | 14:31 |
| popey | dholbach: myapps completely gone for you? | 14:32 |
| popey | The requested URL / was not found on this server. | 14:32 |
| popey | woah, its back | 14:32 |
| dholbach | yep, wfm | 14:32 |
| zbenjamin | nerochiaro: good that its working now | 14:32 |
| dholbach | maybe a deployment in progress? | 14:32 |
| nerochiaro | zbenjamin: well, it's creating the chroot | 14:32 |
| nerochiaro | zbenjamin: we'll see if it can build in there | 14:32 |
| zbenjamin | nerochiaro: true ;) | 14:33 |
| nerochiaro | zbenjamin: also i built the click on the device. i get this when i try to install it: | 14:33 |
| nerochiaro | ** (process:24247): CRITICAL **: ubuntu_app_launch_get_primary_pid: assertion 'con != NULL' failed | 14:33 |
| nerochiaro | ** Message: database.vala:342: Removing /opt/click.ubuntu.com/com.ubuntu.gallery/2.9.1.1034 | 14:33 |
| nerochiaro | zbenjamin: any idea who can i ask to ? | 14:33 |
| popey | dholbach: i always seem to catch the deployments! | 14:34 |
| zbenjamin | nerochiaro: you can try to verify the packet you created on the device , with the click review tools | 14:35 |
| zbenjamin | they often tell you when something is wrong | 14:35 |
| nik90 | popey: my branch works perfectly with your server :) cities are returned fast with the correct times | 14:35 |
| popey | awesome! | 14:36 |
| popey | nik90: should the field name be timezone or timezoneid? | 14:39 |
| Elleo | nerochiaro: with the content-hub integration one little extra thing that might not be obvious, when the content type is set to unknown you'll need to allow picking both videos and pictures (this is the case when the user hasn't requested a specific content-type, so all are valid) | 14:40 |
| popey | the column in the db is timezoneid, so probably should be that? | 14:40 |
| Elleo | nerochiaro: haven't seen your branch yet so you might already be doing that, but thought I should mention it since it's not an immediately obvious usecase | 14:40 |
| nik90 | popey: I am fine with either since it is just a simple change on my end. I think that timezoneID fits better since it is indeed the timezoneID of that location. | 14:40 |
| popey | (just preparing the merge but don't want to have to re-do it) | 14:40 |
| popey | its all lower case, so will be timezoneid | 14:40 |
| popey | thanks | 14:40 |
| nik90 | ok | 14:40 |
| nik90 | np | 14:41 |
| nerochiaro | Elleo: i accounted for that, the only thing i haven't tested yet is what is being passed to me when the content typei is not specified. an empty string ? | 14:41 |
| nik90 | popey: ping me the MP links when you are done so that I can include that in MP for easy tracking | 14:41 |
| Elleo | nerochiaro: ah, you're using the C++ API in gallery, so yeah, it might be an empty string; might want to check with kenvandine | 14:42 |
| Elleo | nerochiaro: for QML I think it gets an explicit ContentType.Unknown value set, but I'm not sure about the C++ API | 14:42 |
| kenvandine | ugh | 14:43 |
| kenvandine | right | 14:43 |
| kenvandine | so that only gets set to a valid value when the transfer is created in the qml bindings | 14:43 |
| popey | nik90: https://code.launchpad.net/~popey/ubuntu-geonames/fix-1326918/+merge/230650 | 14:43 |
| nerochiaro | kenvandine: should i just go ahead and spend some extra time making it all work in pure qml ? | 14:44 |
| nerochiaro | kenvandine: it seems such a pain in the neck to do it in c++ | 14:44 |
| kenvandine | nerochiaro, that would be ideal... | 14:44 |
| nerochiaro | kenvandine: but what do i get now in c++ when no type is specified ? empty string ? that will work for me | 14:44 |
| kenvandine | we really don't want to support the c++ api... that's not part of the platform definition | 14:44 |
| kenvandine | you should get "unknown" | 14:45 |
| kenvandine | i think | 14:45 |
| nerochiaro | ok, i'll debug it i guess | 14:45 |
| nerochiaro | kenvandine: this is unrelated to the above, but i'm trying to install build-dep of gallery-app in an armhf chroot and i get "libcontent-hub-dev : Depends: libcontent-hub0 (= 0.0+14.10.20140806.2-0ubuntu1) but it is not going to be installed" | 14:47 |
| nerochiaro | kenvandine: any idea what's amiss there ? | 14:47 |
| kenvandine | not off hand... what if you try to install both? | 14:48 |
| kenvandine | does it resolve the dep right? | 14:49 |
| * kenvandine wonders if something libcontent-hub0 deps on is broken | 14:49 | |
| kenvandine | ah, the bindings i bet | 14:49 |
| kenvandine | if you do a dist-upgrade it'll fix it | 14:49 |
| kenvandine | or include qtdeclarative5-ubuntu-content1 in your install | 14:50 |
| zbenjamin | nerochiaro: you sure it installs the armhf version? | 14:52 |
| zbenjamin | nerochiaro: try apt-get install libcontent-hub-dev:armhf | 14:53 |
| renatu | nik90, did you talk with timp about the PageHeader state "" | 14:54 |
| nerochiaro | zbenjamin: i'm just doing "apt-get build-dep gallery-app" in the chroot | 14:57 |
| zbenjamin | nerochiaro: if that does not pull the right arch | 14:57 |
| zbenjamin | you are doomed ;) | 14:57 |
| nerochiaro | zbenjamin: well, looks like the case, or some dependency mix up. but i need to get to the bottom of it | 14:58 |
| nerochiaro | kenvandine: ^ | 14:58 |
| nik90 | renatu: oh no not yet..I got busy with some other stuff. I will ask him now. | 14:58 |
| kenvandine | nerochiaro, i suspect it's because of the package rename for qtdeclarative5-ubuntu-content1 | 14:59 |
| kenvandine | not sure if build-dep will do the right thing there | 14:59 |
| nik90 | t1mp: Hey, while using PageHeadState in the clock app, it was not possible to set the PageHeadState name to "" instead of "default". Is there any particular why this is the case? | 14:59 |
| kenvandine | does your chroot had qtdeclarative5-content1 or qtdeclarative5-ubuntu-content0.1 ? | 14:59 |
| kenvandine | nerochiaro, which depends on libcontent-hub0 | 15:00 |
| nerochiaro | kenvandine: it has -ubuntu-content0.1 | 15:00 |
| kenvandine | ok | 15:00 |
| kenvandine | install qtdeclarative5-ubuntu-content1 | 15:01 |
| kenvandine | that'll remove the old one | 15:01 |
| kenvandine | it's the quickest solution | 15:01 |
| kenvandine | a dist-upgrade would work too | 15:01 |
| nik90 | t1mp: if I set the PageHeadState name to "", and the assign the default page to "", it does not load the header action buttons that were defined in the "" pageheadstate. | 15:01 |
| nerochiaro | kenvandine: dist upgrade doesn't do anything. the other solution you just suggested is now removeing a lot of packages and installing others | 15:02 |
| nerochiaro | kenvandine: let's see if it works | 15:02 |
| kenvandine | ugh | 15:02 |
| nik90 | renatu: oh btw I overcame that limitation (or bug) by adding when: !alarmListView.isInSelectionMode in the default pageheadstate :) | 15:02 |
| kenvandine | dist-upgrade should... | 15:02 |
| nik90 | renatu: this way they switch back to the correct state based on if the selectionMode is active or not | 15:03 |
| nerochiaro | kenvandine: it just says nothing to do | 15:03 |
| nerochiaro | kenvandine: it's a fresh chroot so it's already as up to date as possbile | 15:03 |
| nerochiaro | kenvandine: ok, the other solution worked and i can instal build-dep of gallery-app now | 15:03 |
| renatu | nik90, could you report a bug about that? I am reporting another bug that I facing with PageHeaderState | 15:03 |
| kenvandine | cool | 15:03 |
| renatu | nik90, with a example | 15:03 |
| nik90 | renatu: sure, creating one now | 15:04 |
| nerochiaro | but zbenjamin now i have the fresh chroot, i installed the build-dep gallery-app successfully. removed all kits from the project except the armhf one. trying to configure the project still fails as it can't find the qt5 libraries it seems | 15:05 |
| zbenjamin | bzoltan: we probably broken chroots ^^^ | 15:06 |
| zbenjamin | +have | 15:06 |
| renatu | nik90, this is another bug that I found: https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1356420 | 15:13 |
| ubot5 | Ubuntu bug 1356420 in Ubuntu UI Toolkit "[PageHeaderState] Does not work in a state with other PropertyChanges" [Undecided,New] | 15:13 |
| nik90 | renatu: yeah I hit that bug as well, So I just did contents: Label {} where I set the label | 15:14 |
| renatu | kkk | 15:14 |
| kenvandine | nerochiaro, do you have any idea how much work it would be to refactor gallery-app to use the qml bindings? | 15:15 |
| nik90 | renatu: btw in the MultiSelectionView.qml file, the functions startSelection() and other, what state do they change? The PageHeadState or the listview state? | 15:15 |
| renatu | nik90, listview | 15:15 |
| nik90 | renatu: I don't see any reference to which state they are changing except the assignments like state = "selected" | 15:15 |
| nik90 | hmm | 15:16 |
| nik90 | renatu: ok, I better understand the bug. | 15:17 |
| nerochiaro | kenvandine: probably not more than a day | 15:18 |
| elopio | boiko: for when you have some time, this is ready for a review: https://code.launchpad.net/~canonical-platform-qa/dialer-app/fake_url-dispatcher/+merge/230403 | 15:19 |
| kenvandine | we've been bit a couple times by gallery already, but i guess it's up to bfiller :) | 15:20 |
| bzoltan | nerochiaro: could you give us a branch what we can try out with zbenjamin? | 15:20 |
| nerochiaro | bzoltan: lp:gallery-app | 15:22 |
| === chihchun is now known as chihchun_afk | ||
| nik90 | popey: do you mind top approving https://code.launchpad.net/~nik90/ubuntu-clock-app/fix-alarm-label-behavior/+merge/230387. it is ready to go | 15:42 |
| popey | nik90: will do in 15, on a hangout right now | 15:43 |
| nik90 | popey: np | 15:43 |
| justCarakas | popey: what is the oposit of a hangout :p | 15:43 |
| nik90 | renatu: https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1356434 | 15:48 |
| ubot5 | Ubuntu bug 1356434 in Ubuntu UI Toolkit "[PageHeadState] Cannot create pageheadstate with name """ [Undecided,New] | 15:48 |
| zbenjamin | nerochiaro: for me it finds the qt stuff but fails at the end when linking to expat | 15:50 |
| zbenjamin | nerochiaro: i sadly have to run now, its my dads birthday | 15:51 |
| bzoltan1 | nerochiaro: I have installed the deps to my chroot and cmake went fine... now compiling | 16:16 |
| nerochiaro | bzoltan1: how did you install the deps ? | 16:18 |
| rpadovani | dpm, hey, I see the email for tomorrow meeting... Thanks, it definitely works for me :-) | 16:26 |
| bzoltan1 | nerochiaro: apt-get install | 16:28 |
| nerochiaro | bzoltan1: install whta ? | 16:30 |
| bzoltan1 | nerochiaro: libmediainfo-dev libkexiv2-dev libcontent-hub-dev | 16:32 |
| bzoltan1 | nerochiaro: I made it on an i386 target ... let me do for armhf too | 16:32 |
| t1mp | nik90: yeah I noticed a problem with PageHeadState.. it seems like all properties of the standard State get broken :( | 16:35 |
| t1mp | I'll need to find a solution for that, but there is a lot of other stuff to do first :( | 16:35 |
| t1mp | nik90: I gotta go now, let's discuss it tomorrow | 16:35 |
| bzoltan | nerochiaro: the libmediainfo-dev:armhf was trivial ... other other too are not | 16:39 |
| bzoltan | nerochiaro: I am afraid one of the reason for your broken chroot was that you have installed libcontent-hub-dev:armhf and that has removed some essential libs | 16:40 |
| bzoltan | nerochiaro: zbenjamin: for questions like this :) http://pastebin.ubuntu.com/8037566/ my answer is _NO_ | 16:41 |
| dpm | rpadovani, awesome, thanks! | 16:43 |
| bzoltan | nerochiaro: so in short, the gallery app is not supported by click chroots ... good point to file a big against the click package | 16:45 |
| nik90 | bzoltan: hi, had any luck with the failing builds in the silo? | 16:51 |
| nik90 | popey, balloons: If either of you are free, can you top approve https://code.launchpad.net/~nik90/ubuntu-clock-app/update-innerclock-texture/+merge/230669 and https://code.launchpad.net/~nik90/ubuntu-clock-app/sync-upstream-fastscroll/+merge/230666. Both these MPs require no code review since one replaces an image and the other syncs up with upstream code | 16:53 |
| nik90 | which is already tested. | 16:53 |
| balloons | I win! | 16:54 |
| mihir | balloons: hi , Good morning | 16:54 |
| nik90 | lol | 16:54 |
| balloons | :-) | 16:54 |
| balloons | morning mihir | 16:54 |
| mihir | balloons: i did write an AP for editing an event and jenkins failed it | 16:55 |
| mihir | https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/1188819/+merge/230554 | 16:55 |
| mihir | balloons: are these errors regarding pep8 formatting issues? | 16:55 |
| bzoltan | nerochiaro: zbenjamin: https://bugs.launchpad.net/ubuntu/+source/click/+bug/1356481 | 16:56 |
| ubot5 | Ubuntu bug 1356481 in click (Ubuntu) "Can not install build deps for gallery-app in an armhf click chroot" [Undecided,New] | 16:56 |
| balloons | mihir, yep.. you can use autopep8 to fix it if you wish, or do the edits yourself | 16:57 |
| nerochiaro | bzoltan: i actually recreated the chroot from scratch then installed the armhf deps explicitly as Kaleo suggested and it worked | 16:57 |
| balloons | python-pep8 | 16:57 |
| mihir | balloons: sure i'll do that | 16:57 |
| balloons | then run pep8 name_of_fille.py | 16:57 |
| bzoltan | nerochiaro: where can I see Kaleo's suggestion? | 16:57 |
| mihir | balloons: if you have got my previous messages, i gave one link to paste.ubuntu , which contains errors while adding guest. | 16:58 |
| nerochiaro | bzoltan: apt-get install libexpat1-dev:armhf libcontent-hub-dev:armhf libexiv2-dev:armhf libmediainfo-dev:armhf | 16:58 |
| mihir | balloons: actually i have lost the link :| and on web IRC client. | 16:59 |
| bzoltan | nerochiaro: that command removes half of the APIs from the chroot | 16:59 |
| balloons | mihir, not sure I have anything else from you.. | 16:59 |
| nerochiaro | bzoltan: didn't do that in mine | 17:00 |
| nerochiaro | bzoltan: Kaleo ? ^ | 17:00 |
| bzoltan | nerochiaro: OK.. let me create a fresh chroot | 17:00 |
| bzoltan | nerochiaro: anyhow.. if the gallery app is meant to be built with our SDK and these libs are part of our public API set then these packages should be present on the click chroot by default | 17:02 |
| nerochiaro | bzoltan: i've been told the dependencies of an app need to be installed manually isn't what we're doing here ? | 17:03 |
| bzoltan | nerochiaro: who told you that? | 17:03 |
| nerochiaro | bzoltan: benjamin | 17:04 |
| bzoltan | nerochiaro: I mean clearly, you do not have other choice | 17:04 |
| nerochiaro | bzoltan: i think | 17:04 |
| bzoltan | nerochiaro: but optimally the builder sysroot should be static. Because if the developers start to add there APIs then the risk will grow that they build and package apps what will not work on the device. | 17:05 |
| mihir | balloons: balloons: for somehow it is not letting me add guest while creating new event , giving following log http://paste.ubuntu.com/8030498/ | 17:06 |
| mihir | balloons: ^^^ | 17:06 |
| nerochiaro | bzoltan: well, so if apps want to use a certian library that's not in the standard image they have to statically link to it ? | 17:07 |
| bzoltan | nerochiaro: what else can they do? | 17:12 |
| bzoltan | nerochiaro: or package the lib side by to the app | 17:12 |
| nerochiaro | bzoltan: well i guess it's a limitation/design choice of click | 17:12 |
| === gatox is now known as gatox_lunch | ||
| bzoltan | nerochiaro: it is the same on every mobile platform I know... Sailfish, Maemo, Meego ... | 17:13 |
| nerochiaro | bzoltan: fair enough. so i guess i should change gallery app at some point to statically link all that stuff | 17:14 |
| bzoltan | nerochiaro: if something is not available build time than that library will not be available at runtime either... | 17:14 |
| bzoltan | nerochiaro: or we should include those libs to the image and th sdk | 17:14 |
| nerochiaro | bzoltan: whatever works best. libcontent surely should be in the SDK no ? | 17:14 |
| bzoltan | nerochiaro: I would think so | 17:15 |
| nerochiaro | bzoltan: who would be responsible for making it so ? | 17:15 |
| nerochiaro | kenvandine: is there any quick way to have an app request specifically video content from the content hub ? | 17:16 |
| nerochiaro | kenvandine: to test that gallery honors the content type request | 17:16 |
| bzoltan | nerochiaro: I am afraid it is me :) at least from the SDK side.. if those libs are expected to be used by other apps then we should consider to include them to our API offering. | 17:17 |
| nerochiaro | bzoltan: i think only the content hub but i guess it's not up to me to decide | 17:18 |
| nerochiaro | bzoltan: as i don't know if other apps use exiv etc | 17:18 |
| nik90 | mzanetti: hey you got a minute? Got a question about QTest. | 17:38 |
| mzanetti | nik90: shoot | 17:39 |
| nik90 | mzanetti: say I want to test if the back button is disabled when the alarm label is empty. Do I do the usual mouseClick(), type string and check back button status? Or do I just say alarmLabel.text = "value", now check button status? | 17:40 |
| nik90 | mzanetti: oh I just wrote my first QTest, it is nice to write in qml :D | 17:40 |
| dpm | nik90, ok, https://code.launchpad.net/~dpm/ubuntu-clock-app/reboot-i18n/+merge/230578 merged with trunk | 17:41 |
| mzanetti | nik90: yeah... I'd do something like this: | 17:41 |
| mzanetti | compare(backButton.enabled, false) | 17:42 |
| nik90 | dpm: thnx :) will check it out | 17:42 |
| mzanetti | mouseClick, type etc | 17:42 |
| mzanetti | compare(backButton.enabled, true) | 17:42 |
| balloons | mihir, the guest must be created by the test | 17:42 |
| balloons | There's code in trunk that should be taking care of it | 17:42 |
| nik90 | mzanetti: but why do the mouseClick and type when you can just set the text value? | 17:42 |
| kenvandine | nerochiaro, specifying the type is only used to find the peers that support it | 17:43 |
| mihir | balloons: hmmm , i just ran existing test, and it failed while entering guest name UX user. | 17:43 |
| mzanetti | nik90: well, yes, you can also do that... depends a bit on what exactly you want to test | 17:43 |
| ahayzen | balloons, we've landed a fix to the ap tests so could u rerun jenkins for https://code.launchpad.net/~vrruiz/music-app/i18n-strings/+merge/230512 | 17:43 |
| ahayzen | balloons, and when u have a free period of time could u ping me so we can discuss the ap refactor \o/ | 17:43 |
| nik90 | mzanetti: ah ok. If it is purely a unit test, then I suppose i could just set text. If I am looking to test user interaction then I do the mouseClick and type stuff. | 17:43 |
| balloons | ahayzen, victor top approved that so it should be good | 17:43 |
| ahayzen | balloons, damn he is fast | 17:43 |
| balloons | mihir, trunk? let me try | 17:43 |
| mihir | balloons: i just got latest trunk, am I missing anything on utopic desktop for contacts ? | 17:44 |
| nerochiaro | kenvandine: sorry, i thought i was supposed to receive the content type in the transfer request as well once my app was selected | 17:44 |
| nerochiaro | kenvandine: in case my app supports more than one content type (e.g. gallery) | 17:44 |
| mihir | balloons: i just wrote AP , and formated with pep8 could you just give me your inputs , https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/1188819/+merge/230554 | 17:44 |
| mihir | balloons: finally i wrote an AP :D | 17:44 |
| balloons | mihir, adding a guest works fine here from trunk. tests run properly | 17:45 |
| * balloons branches lp:~mihirsoni/ubuntu-calendar-app/1188819 | 17:47 | |
| kenvandine | nerochiaro, the content type is in the transfer object | 17:48 |
| kenvandine | now... just added that | 17:48 |
| kenvandine | but it only includes it for transfers created by an app using the QML bindings | 17:48 |
| nerochiaro | kenvandine: i receive it and it is currently "unknown" | 17:49 |
| nerochiaro | kenvandine: is that what it is always supposed to be in c++ for now ? | 17:49 |
| kenvandine | yeah | 17:49 |
| kenvandine | i didn't want to clutter up the c++ API right now, since we aren't really supporting that | 17:50 |
| balloons | mihir, tried with your branch as well.. tests run fine, adds user fine | 17:52 |
| mihir | balloons: okay , do i need to create that Guest manually on desktop ? | 17:53 |
| mihir | balloons: btw, the new test case runs fine ? | 17:53 |
| kenvandine | nerochiaro, actually... which app is creating the transfer? | 17:53 |
| nerochiaro | kenvandine: so why did you even add the property to the c++ API if it's always going to be "unknown" ? | 17:53 |
| kenvandine | it should have a valid content type there if it was created by anything other than gallery | 17:53 |
| nerochiaro | kenvandine: i am testing by creating the transfer from browser | 17:53 |
| nerochiaro | kenvandine: what other app can i try ? | 17:53 |
| kenvandine | oh... is it using ContentType.All maybe? | 17:54 |
| kenvandine | the property needs to be there to get it sent over dbus, etc | 17:54 |
| nerochiaro | kenvandine: sorry, i'm not following you. can you hop quickly on mumble for a minute ? | 17:54 |
| kenvandine | sure | 17:54 |
| balloons | mihir, nope the tests should just work. post a full log where it doesn't. Also check and ensure you have everything installed you need to. my guess is you are missing depends | 17:55 |
| balloons | mihir, you rneed address-book-service-dummy installed | 17:56 |
| kenvandine | https://code.launchpad.net/~ken-vandine/+junk/hub-importer | 17:56 |
| kenvandine | nerochiaro, ^^ | 17:56 |
| m-b-o | balloons: ping | 18:00 |
| nerochiaro | kenvandine: when i try to run it from qtcreator on the device it just exits immediately, nothing is displayed. is that expected ? | 18:00 |
| balloons | m-b-o, pong | 18:00 |
| m-b-o | balloons: something seems not right with install instructions on https://wiki.ubuntu.com/Touch/Testing/Autopilot | 18:00 |
| kenvandine | nerochiaro, nope... | 18:00 |
| kenvandine | nerochiaro, i just build a click for it | 18:01 |
| m-b-o | balloons: tried ppa:autopilot/1.5 too, but apt-get update gets me a 404 | 18:01 |
| kenvandine | nerochiaro, the content-hub relies on ubuntu-app-launch for switching between the apps, etc | 18:01 |
| kenvandine | and it doesn't behave well with apps that aren't installed as click | 18:01 |
| nerochiaro | kenvandine: i think that's what qtcreator does, installs it as click then launches it | 18:02 |
| nik90 | mzanetti: do you happen to know if it is possible to click on the clear button in a text field? I cannot define its objectName in my qml code since it is a sdk component. So curious how one goes about that. | 18:02 |
| mihir | balloons: awesome , that was the cause :D | 18:02 |
| nerochiaro | kenvandine: Installed com.ubuntu.developer.ken-vandine.hub-importer-0.2.all (installed:click,removable=1,app_name=hub-importer) | 18:02 |
| mihir | balloons: thanks a lot :D | 18:02 |
| kenvandine | ok, try launching it from the shell then | 18:02 |
| mihir | balloons: root cause* | 18:02 |
| balloons | mihir, :-) awesome. first glance at your test looks good | 18:02 |
| mihir | balloons: thanks for your review time and thanks a lot for your help to setting this up :) , i'll try to add pending AP listed in Bugs :) | 18:03 |
| mzanetti | nik90: well... you could use findChild to find the button inside... but I guess in this case its probably easier to click the label at the right | 18:04 |
| kenvandine | nerochiaro, oh.. it might have the wrong policy version specified | 18:04 |
| kenvandine | that branch is pretty old | 18:04 |
| balloons | m-b-o, looking | 18:04 |
| kenvandine | or wrong framework | 18:04 |
| nerochiaro | kenvandine: not sure how to fix the policy | 18:04 |
| kenvandine | nerochiaro, fix the framework in the manifest | 18:04 |
| nik90 | mzanetti: hmm..I am tryingn to clear the text inside the textfield | 18:04 |
| mzanetti | nik90: mouseClick(textField, textField.width - units.gu(2), textfield.height / 2) | 18:04 |
| mzanetti | nik90: assuming the clear button is at the right hand side in the textfield | 18:04 |
| balloons | m-b-o, ppa:autopilot/1.5 is correct | 18:05 |
| kenvandine | ubuntu-sdk-14.10-qml-dev3 | 18:05 |
| kenvandine | nerochiaro, ^^ | 18:05 |
| balloons | it's only published for trusty tho | 18:05 |
| nik90 | mzanetti: hmm yeah that will work | 18:05 |
| m-b-o | balloons: yes, but apt-get update gets me a 404 for the Packages | 18:05 |
| balloons | m-b-o, you are on trusty yes? | 18:06 |
| m-b-o | balloons: nope utopic in vm | 18:06 |
| nerochiaro | kenvandine: still fails. the json file says "policy_version": 1, should i change that ? | 18:06 |
| mihir | popey: you there ? | 18:06 |
| balloons | m-b-o, lol.. then you don't need a ppa. just install python3-autopilot and python3-autopilot-vis | 18:06 |
| nerochiaro | kenvandine: also how would you run it from the console ? | 18:06 |
| popey | mihir: yo | 18:06 |
| balloons | I'll update the instructions to clarify | 18:06 |
| balloons | m-b-o, ^^ | 18:06 |
| mihir | popey: did you get time for getting some idea about provider ID ? | 18:07 |
| m-b-o | balloons: hehe | 18:07 |
| kenvandine | nerochiaro, use ubuntu-app-launch on the console | 18:07 |
| kenvandine | and look at the logs in ~/.cache/upstart | 18:07 |
| nerochiaro | kenvandine: i mean, what id should i pass to ubuntu-app-launch, i always get that wrong | 18:07 |
| kenvandine | com.ubuntu.developer.ken-vandine.hub-importer_hub-importer_0.1 | 18:08 |
| kenvandine | when in doubt, look in the .desktop file in ~/.local/share/applications | 18:08 |
| kenvandine | the APP_ID in put in there | 18:08 |
| nerochiaro | kenvandine: ah, that's interesting, it's not there. i guess qtcreator uninstalls it after finishing to run it. i'll try building the click manually and installing it | 18:09 |
| mihir | renatu: ping !! | 18:09 |
| balloons | mihir, so I'll try to run your test now. it works for you? | 18:11 |
| mihir | balloons: yeah it works for me :) | 18:11 |
| mihir | it create new event , edit that event and assert on details view. | 18:11 |
| === gatox_lunch is now known as gatox | ||
| mihir | balloons: let me know if I have done wrongly | 18:11 |
| popey | mihir: it should work with "google".. | 18:11 |
| popey | mihir: have you tested in the emulator? | 18:12 |
| mihir | popey: okay thanks. | 18:12 |
| mihir | popey: nope , but balloons did tried on device but it didn't work. | 18:12 |
| popey | ☹ | 18:12 |
| popey | any apparmor denials? | 18:13 |
| popey | balloons: ? | 18:13 |
| mihir | popey: on desktop , if I write evernote it does open window to add account , but when i write google , it doesn't do anything :( | 18:13 |
| balloons | mihir, looks awesome. My only thought is that it should probably change all the information. it seems to use the same uuid | 18:13 |
| popey | well thats odd | 18:13 |
| mihir | balloons: while editing an event , ? | 18:13 |
| mihir | balloons: nope i took another UUID for editing event, if you observe :| | 18:13 |
| balloons | mihir, yes when you edit, change the details to something different | 18:13 |
| balloons | mihir, I say that.. but it's weird as the uuid id was the same for me | 18:14 |
| nerochiaro | kenvandine: the id from the desktop is com.ubuntu.developer.ken-vandine.hub-importer_hub-importer_0.2 and when i launch it i get no feedback at all, and there's no log for it in the logs dir | 18:14 |
| balloons | mihir, yes I saw test_event = data.Event.make_unique() | 18:14 |
| mihir | balloons: hmmm , for me all the cases it was diff :| heheh weried | 18:14 |
| mihir | popey: i'll try to digout this more :) | 18:14 |
| balloons | hehe.. ok, so yea. I'll try again.. maybe I was crazy | 18:14 |
| kenvandine | nerochiaro, weird... do you see it in the app scope? | 18:14 |
| balloons | mihir, so +1 overall from me | 18:15 |
| mihir | balloons: awesome , i'll try to finish pending ones :) | 18:15 |
| kenvandine | if anything is wrong, ubuntu-app-launch does blindly not tell you :) | 18:15 |
| mihir | popey: what do you think for this bug, https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1349531 | 18:15 |
| ubot5 | Ubuntu bug 1349531 in Ubuntu Calendar App "Delete future recurring events missing" [Wishlist,Confirmed] | 18:15 |
| nerochiaro | kenvandine: no, doesn't seem to be there | 18:15 |
| nerochiaro | kenvandine: in the scope | 18:15 |
| kenvandine | it must not have properly installed | 18:15 |
| kenvandine | you changed the framework right? | 18:15 |
| kenvandine | nerochiaro, change policy_version to 1.2 in hub-importer.json | 18:16 |
| nerochiaro | kenvandine: yes, then i did "click build .", adb pushed the click, then used pkcon install-local to install it | 18:16 |
| kenvandine | i bet it failed to generate the click profile for it because of that policy_version | 18:16 |
| kenvandine | i need to update these +junk branches :) | 18:17 |
| mihir | balloons: one question , did we got helper for datetime picker ? | 18:17 |
| nerochiaro | kenvandine: ok, launches now | 18:18 |
| mihir | balloons: why did Jenkis failed again :( | 18:19 |
| nerochiaro | kenvandine: ok, so now if i click on "import pictures" the gallery is not part of the apps that get proposed | 18:20 |
| kenvandine | right, you changed to to look for ContentType.Videos right? | 18:21 |
| balloons | mihir, there's video for jenkins tests.. really handy. I'm watching now to see what happened | 18:21 |
| balloons | http://91.189.93.70:8080/job/generic-mediumtests-utopic/1512/artifact/calendar_app.tests.test_new_event.NewEventTestCase.test_add_new_event_with_default_values%20%28with%20mouse%29.ogv | 18:21 |
| nerochiaro | kenvandine: no, not yet. i wanted to test with pictures first | 18:21 |
| kenvandine | oh | 18:21 |
| kenvandine | what is listed? | 18:21 |
| balloons | mihir, ohh one thing.. you should add @autopilot.logging.log_action(logger.debug) above def edit(self): in emulators.py | 18:22 |
| nerochiaro | kenvandine: camera and file manager | 18:22 |
| kenvandine | nerochiaro, there's something wrong with your gallery click install | 18:22 |
| nerochiaro | kenvandine: ok, what should i be looking for ? | 18:23 |
| kenvandine | do you have your own build of gallery installed? | 18:23 |
| nerochiaro | kenvandine: i know what the problem was, i didn't have it properly installed | 18:25 |
| nerochiaro | kenvandine: galler | 18:25 |
| nerochiaro | kenvandine: now i fixed that and it does appear | 18:25 |
| kenvandine | great | 18:25 |
| balloons | mihir, ohh you might need to use self._workaround_bug_1350605() as well | 18:26 |
| balloons | mm.. it's called in _add_event | 18:26 |
| mihir | balloons: it has to be called in _edit_event ? | 18:27 |
| mihir | balloons: i did removed it because after we edit we go back to details page and asserting the values right ? | 18:27 |
| balloons | mihir, I may have spoke too soon | 18:27 |
| balloons | mihir, mm.. | 18:28 |
| balloons | mihir, so I see one small bit of logic that might be the cause. I'll comment in the mp | 18:28 |
| mihir | balloons: hmm okay. | 18:29 |
| mihir | balloons: one more que, do we have AP helper for datetime picker? | 18:35 |
| balloons | mihir, yes we do. it's in trunk for calendar | 18:35 |
| mihir | balloons: okay :) | 18:35 |
| balloons | mihir, http://pad.lv/1328600 is the issue with using it | 18:36 |
| ubot5 | Launchpad bug 1328600 in Autopilot "Autopilot lacks support for large timestamps" [High,In progress] | 18:36 |
| mihir | balloons: thanks for help :) | 18:36 |
| mihir | balloons: sorry i lost , this is the cause of failing Jenkins on my NEW AP ? | 18:37 |
| balloons | mihir, nope, it's not an issue for your mp | 18:38 |
| mihir | balloons: okay got it :) | 18:39 |
| mihir | balloons: i'll be off to bed, i'll try to look at that MR tomorrow. let me know your inputs on MR. | 18:41 |
| balloons | mihir, yep.. almost ready to push. it'll be a branch for you to merge | 18:42 |
| balloons | just some tweaks | 18:42 |
| mihir | balloons: Hmm okay. | 18:43 |
| mihir | balloons: okay waiting , so that we can land this today itself. | 18:43 |
| * balloons runs changes | 18:44 | |
| balloons | k, I think it might be good | 18:50 |
| mihir | balloons: you pushing in saperate MR ? | 18:51 |
| popey | m-b-o: nik90 do you have examples of cities which have an admin2/admin3? | 18:52 |
| balloons | mihir, I reproduced the issue. The trouble is timing | 18:52 |
| balloons | mihir, no just a branch for you to merge | 18:53 |
| mihir | hmm okay. | 18:53 |
| balloons | mihir, anyways, you can see what I did here: lp:~nskaggs/ubuntu-calendar-app/mihir-edit-tweaks | 18:54 |
| balloons | it's not exactly ready, but the original issue I found so I thought was in _edit_event | 18:54 |
| balloons | mihir, it can merge tomorrow :-) | 18:55 |
| nik90 | popey: let me check | 18:55 |
| mihir | balloons: Hmm i see, but i couldn't get why did you add cleanup ? | 18:56 |
| popey | nik90: geonames=# SELECT * FROM geoname WHERE admin2 != "" | 18:57 |
| popey | geonames-# SELECT * FROM geoname WHERE admin3 != "" | 18:57 |
| popey | geonames-# | 18:57 |
| balloons | mihir, the cleanup step will delete the test after we are done | 18:58 |
| balloons | mihir, did I duplicate it? | 18:58 |
| mihir | balloons: i see my mistake, i called add_event , instead of edit in edit_event , is that correct? | 18:59 |
| nik90 | popey: http://api.geonames.org/search?q=london&maxRows=2&username=krnekhelesh&style=full&featureClass=P&style=full&featureClass=P | 18:59 |
| balloons | mihir, something about that method is still a bit off I think yes | 19:00 |
| mihir | balloons: i am trying understand my mistake and why did it scuccessfully ran on machine | 19:01 |
| balloons | mihir, yes I broke it, but I'm hoping to make a bit more sense | 19:02 |
| balloons | ;-) | 19:02 |
| mihir | balloons: Thanks for breaking , it made me to become perfect :D | 19:02 |
| nik90 | popey: for clock we use only adminName and adminName2 | 19:02 |
| nik90 | popey: never used adminName3, not sure why weather app needs it | 19:02 |
| m-b-o | popey nik90: http://bazaar.launchpad.net/~ubuntu-weather-dev/ubuntu-weather-app/trunk/view/head:/components/CitiesList.js | 19:03 |
| m-b-o | that's the data used for the defined list of cities | 19:03 |
| popey | I'm trying to figure out with mhall119 why my crappy mod isn't showing admin2 | 19:04 |
| nik90 | m-b-o: ok, but isn't adminName and adminName2 sufficient enough to distinguish cities with same name? | 19:05 |
| nik90 | m-b-o: In the list you provides, I am not seeing a benefit of adminName3 | 19:05 |
| mihir | balloons: pushed :) | 19:05 |
| boiko | elopio: late reply, sure, I will take it a look later today | 19:06 |
| m-b-o | nik90: yes, the list is a bad example | 19:06 |
| m-b-o | nik90: I can't actually rememebr why we used admin3 over admin2 | 19:06 |
| nik90 | m-b-o: ah ok | 19:07 |
| elopio | boiko: thanks. And I have a branch that introduces a qml suite. It works on my machine, but segfaults on Jenkins. | 19:07 |
| elopio | whenever you have some time please ping me to see if you can make sense of the error log. | 19:07 |
| mihir | balloons: have good night :) | 19:07 |
| boiko | elopio: ok, mind if we check that tomorrow? | 19:08 |
| m-b-o | nik90: I remember, admin3 was county, admin2 the next bigger area. It makes at least in Germany more sense | 19:08 |
| balloons | mihir, night.. I pushed some more changes, but it still fails locally, heh | 19:08 |
| elopio | boiko: yes, no problem with that one. It's not urgent. Whenever you can make some space on your agenda. | 19:08 |
| balloons | it's the same as jenkins | 19:08 |
| mihir | balloons: Hmmm | 19:08 |
| mihir | balloons: ahh, you just ran normally and it broke , or some tweaks to break like jenkins? | 19:09 |
| balloons | mihir, I restructed it to try and fix things. | 19:10 |
| balloons | lol. I think _edit_event makes more sense now | 19:10 |
| balloons | well.. actually no | 19:11 |
| balloons | heh.. but mihir you should get some sleep | 19:11 |
| mihir | balloons: hmmm let me re-run on my VM | 19:11 |
| balloons | more code cutting done, rev 39 | 19:14 |
| balloons | 397 | 19:14 |
| nik90 | mzanetti: hey I just finished my test at https://code.launchpad.net/~nik90/ubuntu-clock-app/add-first-qmltest/+merge/230702. Can you check if I did it right? It passes for me locally. | 19:18 |
| mihir | balloons: okay trying to run on my machine | 19:18 |
| nik90 | mzanetti: make test doesn't work. I think the location is incorrect | 19:18 |
| israel_ | Hi, I have been searching for a library for X11 to Hexidecimal (or even RGB) colors in C++ anyone have any suggestions? | 19:22 |
| mihir | balloons: i see the error, it is not going into edit mode now : | 19:23 |
| balloons | mihir, yes ofc.. it's not finding the event.. yet it can find it a second later to delete | 19:24 |
| mihir | but balloons it won't , the action should be after event Details , it should go to Edit page , right? | 19:24 |
| mhall119 | nik90: I think you're gonna owe popey a beer or two, after looking at this geonames code | 19:24 |
| popey | haha | 19:25 |
| nik90 | mhall119: :D | 19:25 |
| nik90 | popey: I will mail you the beer | 19:25 |
| mihir | balloons: on event Details page, instead of going to Delete action , it should go and click edit this event right? | 19:26 |
| balloons | mihir, pushed rev 398 now. | 19:26 |
| balloons | mihir, it's running the cleanup step | 19:26 |
| balloons | which uses the exact same method to discover the event | 19:26 |
| balloons | however, when it goes to click the event for edit_event, it fails to find the event name, as the error says | 19:27 |
| balloons | mihir, def get_event(self, event_name, visible=True): is where the error comes from | 19:28 |
| israel_ | Hi, I have been searching for a library *TO CONVERT* X11 to Hexidecimal (or even RGB) colors in C++ anyone have any suggestions? | 19:29 |
| === mardy_ is now known as mardy | ||
| mihir | balloons: hmm but again it is failing ,did it pass on your machine? | 19:31 |
| nik90 | mhall119, popey: Hope this is not late :/ but once we get AGPS working, it will return the user's lat and long. I will need to query the same server to query the cityname using its lat and long. Do you know if http://geoname-lookup.ubuntu.com/ supports that? | 19:31 |
| nik90 | with geoname I do http://api.geonames.org/timezoneJSON?lat=11.58&lng=43.14&username=krnekhelesh | 19:32 |
| mihir | balloons: why are we cleaning before editing ? | 19:32 |
| israel_ | It seems that this should exist, as there is /etc/X11/rgb.txt installed... just not sure where to begin looking | 19:32 |
| balloons | mihir, I think I see the issue | 19:32 |
| m-b-o | popey: fyi http://imgur.com/i8pLiiB | 19:32 |
| balloons | mihir, the cleanup step runs last. it matters not when you add it | 19:32 |
| nik90 | mhall119, popey: actually http://api.geonames.org/findNearbyJSON?lat=47.3&lng=9&username=krnekhelesh&style=full | 19:33 |
| mihir | balloons: cleanup delete the created event , have I understood correctly? | 19:33 |
| popey | nik90: it doesnt | 19:33 |
| popey | m-b-o: ooh! | 19:34 |
| balloons | mihir, we aren't passing the original test_event name to edit_event | 19:34 |
| balloons | that's the issue | 19:34 |
| * balloons testing | 19:36 | |
| balloons | works :-) | 19:37 |
| mihir | you made change in _edit_event right | 19:38 |
| mihir | ? | 19:38 |
| balloons | I made lots of changes, heh | 19:39 |
| mihir | balloons: ahahah :| | 19:39 |
| balloons | but no, the final was in test_edit_event_must_change_it_from_day_view | 19:39 |
| mihir | did you push ? | 19:39 |
| balloons | not yet | 19:40 |
| balloons | mihir, pushed 399, but the asserts need fixed now | 19:42 |
| mihir | balloons: asserts should be either on Details page or Day view | 19:43 |
| mihir | balloons: i'll try to play with it tomorrow. | 19:44 |
| balloons | mihir, rev 400 closer.. hehe | 19:45 |
| mihir | balloons: thanks a lot for your help :) | 19:45 |
| israel_ | no one knows? | 19:46 |
| balloons | israel_, #ubuntu-devel might be a better place for your question | 19:47 |
| mzanetti | nik90: do you really need the MainView there? | 19:48 |
| nik90 | mzanetti: if I dont include mainView, then the header buttons don't appear | 19:49 |
| nik90 | the whole header is missing | 19:49 |
| mzanetti | oh, I see | 19:51 |
| israel_ | ballons: thanks! The title of that one said "not for app development' But I will try there! | 19:51 |
| nik90 | mzanetti: I looked at the SDK unit tests, and noticed that they also include MainView to test the header sections. | 19:51 |
| nik90 | mzanetti: but yeah it complicates things :) | 19:51 |
| mzanetti | nik90: well, in that case you might want to just use MainView, instead of the wrapping Item {} | 19:51 |
| mzanetti | just an idea, didn't try it | 19:51 |
| mzanetti | but both is fine I guess | 19:51 |
| mzanetti | the test is ok | 19:51 |
| nik90 | awesome. I will try without the item | 19:52 |
| nik90 | mzanetti: any thoughts why make test doesn't work? | 19:53 |
| mzanetti | not really atm | 19:56 |
| === salem_ is now known as _salem | ||
| mzanetti | nik90: http://paste.ubuntu.com/8038996 | 19:58 |
| nik90 | mzanetti: yup I am not sure why it doesn't the test | 19:59 |
| nik90 | mzanetti: it has the same working folder as the autopilot test | 19:59 |
| mzanetti | but its working, no? | 19:59 |
| mzanetti | bbiab | 19:59 |
| nik90 | mzanetti: ah..I expected some verbose output about what passed. But yeah it works | 20:01 |
| popey | nik90: m-b-o mhall119 http://146.185.188.87/test/?query=Bromley | 20:05 |
| popey | pls test | 20:05 |
| popey | has admin1 and admin2 | 20:06 |
| m-b-o | popey: admin3 is used in weather | 20:06 |
| popey | will figure out how to add admin2, it's not in the database dump | 20:07 |
| mzanetti | nik90: ctest -V | 20:08 |
| m-b-o | popey: admin2 or admin3? | 20:08 |
| popey | 3 | 20:08 |
| m-b-o | popey: ok. it's very usefull, at least in germany | 20:08 |
| popey | I'll figure it out ☻ | 20:09 |
| nik90 | m-b-o: example city pls? I want to check | 20:09 |
| davmor2 | nik90: Wolverhampton | 20:09 |
| nik90 | davmor2: cities with duplicate names :) Yours returns only 1 result :D | 20:10 |
| nik90 | no other Wolverhampton in the world | 20:10 |
| davmor2 | nik90: Birmingham there is one in the uk and one in the states | 20:10 |
| davmor2 | nik90: Washington | 20:10 |
| nik90 | davmor2: awesome thnx | 20:11 |
| davmor2 | nik90: leek, leeds, new castle | 20:11 |
| nik90 | davmor2: good lord there are so many Birminghams in uk and us | 20:11 |
| nik90 | perfect example to test | 20:11 |
| === _salem is now known as salem_ | ||
| davmor2 | nik90: no worries chap :) | 20:12 |
| davmor2 | nik90: shrewsbury, stratford will be another good example too I think | 20:14 |
| nik90 | mzanetti: where do I add the cTest -V ? | 20:14 |
| mzanetti | nik90: run that instead of make test if you want more verbose | 20:14 |
| mzanetti | lowercase t btw, ctest -V | 20:14 |
| nik90 | mzanetti: I got http://paste.ubuntu.com/8039105/. Is there a way to replace basic with AlarmLabelPage (test suite name for instance) | 20:16 |
| popey | m-b-o: do you have an example German city? | 20:17 |
| m-b-o | Ludwigshafen | 20:18 |
| m-b-o | nik90 popey: ^ | 20:18 |
| mzanetti | nik90: strange... this is what I get: http://paste.ubuntu.com/8039113 | 20:18 |
| popey | ta | 20:18 |
| popey | it's odd, in all the dumps that geonames provide, the lookup for admin3 isn't there... | 20:20 |
| popey | http://download.geonames.org/export/dump/ | 20:20 |
| popey | so while I can find out that Ludwigshafen has admin code 3 of 07338, I can't look that up | 20:20 |
| m-b-o | nik90 popey: most time you can distinguish enough with admin1 and admin2, but the name used in admin2 is very common. admin3 is on the car plates | 20:20 |
| m-b-o | sorry: admin2 not verycommon | 20:20 |
| popey | I don't doubt admin3 is useful, just trying to figure out how to look it up ☻ | 20:21 |
| nik90 | mzanetti: strange after running cmake again, it shows what you pasted :) | 20:23 |
| popey | found it! | 20:28 |
| popey | m-b-o: so would you expect in that json response to say admin3: 07338, or admin3: Rhein-Pfalz-Kreis ? | 20:36 |
| m-b-o | popey: the last one, the name | 20:37 |
| popey | hmmm | 20:39 |
| m-b-o | popey: there's a premium dump too. so my guess is, admin3 is there.... | 20:40 |
| popey | oh? | 20:41 |
| popey | premium. as in not cc licensed? | 20:41 |
| popey | I can't see any easy way to pick the admin3name out. hmm | 20:41 |
| m-b-o | popey: yes, in the postal codes dump is a field for it, but no data | 20:45 |
| m-b-o | balloons: can you have a look at http://91.189.93.70:8080/job/generic-mediumtests-utopic-weather-app/46/? please? | 20:53 |
| m-b-o | balloons: seems there's an issue with the header emulator and small icons | 20:53 |
| mhall119 | popey: m-b-o: nik90: what data is it that you actually *need* from this? | 20:55 |
| popey | mhall119: https://bugs.launchpad.net/ubuntu-weather-app/+bug/1326918/comments/5 | 20:55 |
| ubot5 | Ubuntu bug 1326918 in Ubuntu Clock App reboot "API key for geonames.org" [High,In progress] | 20:55 |
| mhall119 | popey: but what is the data in admin3 that we need? | 20:56 |
| m-b-o | mhall119 popey: https://bugs.launchpad.net/ubuntu-weather-app/+bug/1326918/comments/4 | 20:56 |
| popey | mhall119: it's a regional name, like in the UK example we had GB (country), ENG (admin1), GLA (admin2), the Admin3 might be "Bromley" | 20:57 |
| m-b-o | mhall119 popey: if admin3 it's not possible, it's not that big problem. | 20:59 |
| mhall119 | popey: do you have an example of what's in the admin3 field of geonames for Bromley? | 21:01 |
| popey | well, bromley was an example | 21:01 |
| m-b-o | mhall119 popey: admin3 gets only displayed, when the location name and admin1 is not unique | 21:04 |
| popey | in general admin3 is a numeric value | 21:04 |
| m-b-o | adminName3, sorry | 21:04 |
| m-b-o | adminName1 an adminName3 gives you a distinction around up to 50km, adminName1 and adminName2 up to 200km. At least here | 21:06 |
| m-b-o | but adminName3 is not very often used | 21:07 |
| nik90 | mhall119, popey: In http://146.185.188.87/test/?query=Birmingham do I really need to show adminName2 ? | 21:08 |
| balloons | m-b-o, what's the issue you think? | 21:08 |
| nik90 | Shouldn't Name + adminName1 + Country suffice? | 21:08 |
| m-b-o | balloons: the click for opening the Sheet happened not on the icon | 21:09 |
| m-b-o | balloons: but that's just a guess | 21:09 |
| mhall119 | nik90: it appears that would give enough data to pick the correct one | 21:16 |
| mhall119 | popey: I still can't figure out what sphinx is being used for | 21:16 |
| popey | heh | 21:17 |
| nik90 | mhall119: yup that's what I think. I am in a bit of dilemma as to how to present this data properly. | 21:17 |
| popey | its the initial lookup of city name to city id i think | 21:17 |
| mhall119 | popey: yeah....but that's all in the database, so I don't know *why* sphinx would be needed | 21:18 |
| nik90 | ideally I think name \n AdminName, Country should do | 21:18 |
| mhall119 | nik90: that should work for > 90% of the cases at least | 21:18 |
| popey | you need to try some names which have lots of dupes | 21:18 |
| popey | like Newport | 21:18 |
| popey | and make sure you can differentiate between them | 21:19 |
| mhall119 | I tried Springfield which we have a lot of in the USA, and nik90's proposed display would be fine | 21:19 |
| mhall119 | ah, nope, Jefferson breaks it, because there's 2 on Georgia | 21:21 |
| m-b-o | adminName1 is "England" | 21:21 |
| nik90 | mhall119, popey: Do we have so much accuracy though? I mean we need this for the world clocks to choose timezones essentially. What does it matter if there are 2 jeffersons in Georgia? They are bound to have the same timezone right? | 21:22 |
| nik90 | mhall119, popey: Choosing either one without knowing which one is still alright since the timezone will be correct | 21:22 |
| popey | i think there are some edge cases, but it's "good enough" | 21:23 |
| popey | we have other bigger issues to solve ☻ | 21:23 |
| nik90 | :) | 21:23 |
| mhall119 | yeah, I'm happy with "good enough" | 21:23 |
| nik90 | I say for the clock app, I should be good with Name, AdminName1 and Country. | 21:23 |
| nik90 | adding too much will clutter our beautiful clock :D | 21:23 |
| mhall119 | nobody likes Georgia anyway | 21:24 |
| nik90 | rofl | 21:24 |
| popey | http://en.wikipedia.org/wiki/List_of_popular_place_names | 21:24 |
| popey | there's your test cases | 21:24 |
| popey | in order | 21:24 |
| nik90 | good lord | 21:25 |
| * mhall119 loves the List_of_* articles in Wikipedia | 21:25 | |
| mhall119 | http://en.wikipedia.org/wiki/List_of_Wikipedia_Lists goodbye everybody, I'll be on wikipedia until sometime in 2021 | 21:26 |
| popey | http://uncyclopedia.wikia.com/wiki/List_of_weapons_that_don't_exist,_but_should is better :) | 21:28 |
| nik90 | lol | 21:32 |
| ybon | Is there a way from an app to know the wifi around and/or the cell tower? I'm thinking about implementing https://mozilla-ichnaea.readthedocs.org/en/latest/api/index.html#service-api | 21:35 |
| m-b-o | mhall119 popey nik90: adminName2 would be a nicetohave in weather | 21:41 |
| popey | ok, I'll push my changes which are at http://146.185.188.87/test/?query=San%20Jose | 21:42 |
| popey | I'll do that in the morning when I am not half asleep ☻ | 21:43 |
| popey | this does mean I'll be the most recent person to change that code which probably means it's now mine forever | 21:43 |
| m-b-o | popey: thank you | 21:45 |
| popey | np | 21:45 |
| nik90 | popey: thnx. I will test again with your server tomorrow morning as well | 21:47 |
| popey | kk | 21:48 |
| === salem_ is now known as _salem | ||
| nik90 | balloons: ping | 22:42 |
| karni | Can my click package not contain a '+' symbol? If it does, I can't launch it from the Dash. http://paste.ubuntu.com/8040118/ | 22:43 |
| karni | GDBus.Error:com.canonical.URLDispatcher.BadURL | 22:43 |
| popey | karni: i have no clicks on my device that have a + in the name | 22:46 |
| popey | karni: but it probably isn't good, no | 22:46 |
| karni | popey: ^([a-z0-9][a-z0-9+.-]+)_([a-zA-Z0-9+.-]+)(?:_([0-9][a-zA-Z0-9.+:~-]*))?$ | 22:46 |
| karni | popey: just got this from Chipaca | 22:46 |
| popey | cat fall asleep on the keyboard? | 22:46 |
| karni | popey: trying to append +build to the version (semantic versioning), and it's a no go | 22:47 |
| karni | popey: treasure found, https://wiki.ubuntu.com/AppStore/Interfaces/ApplicationId | 22:48 |
| popey | gotcha | 22:49 |
| karni | kalikiana: hi! when's the chance we'll get the config path (with application domain) fix in the image? or is it already there, and I should fix my app somehow? | 22:53 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!