| dholbach | good morning | 06:59 |
|---|---|---|
| PaoloRotolo | Good morning | 07:02 |
| dholbach | ciao PaoloRotolo | 07:03 |
| dholbach | come stai? | 07:03 |
| PaoloRotolo | dholbach, fine, thanks :D | 07:03 |
| dholbach | :-) | 07:03 |
| PaoloRotolo | dholbach, your italian is good :P | 07:04 |
| dholbach | no, it's not ;-) | 07:04 |
| dholbach | but I picked up a few words and hope to pick up a few more - because I'll be on holiday in Italy in ~2 weeks :) | 07:04 |
| PaoloRotolo | dholbach, nice, where? | 07:05 |
| dholbach | we're invited to a wedding in Massafra, but we're going to see other parts of the country as well :) | 07:06 |
| PaoloRotolo | dholbach, it's near my town :) I live in Puglia | 07:08 |
| dholbach | ah nice :) | 07:08 |
| didrocks | hey dholbach ;) | 07:08 |
| dholbach | salut didrocks | 07:09 |
| didrocks | hi PaoloRotolo | 07:09 |
| dholbach | PaoloRotolo, I'm looking forward to it | 07:09 |
| PaoloRotolo | dholbach, hi! | 07:09 |
| didrocks | wrong d<tab> ;) | 07:09 |
| PaoloRotolo | didrocks, yes, sorry | 07:09 |
| PaoloRotolo | didrocks, hi | 07:09 |
| didrocks | PaoloRotolo: no worry, happens to me every single day! :) | 07:10 |
| PaoloRotolo | didrocks, me too :) | 07:10 |
| dholbach | if any of you could help test the bugs mentioned in https://bugs.launchpad.net/ubuntu/+source/python-distutils-extra/+bug/1020017 (test-cases are in there), it'd be much appreciated | 07:23 |
| dholbach | ajmitch, are you feeling any better now? | 07:42 |
| dholbach | ajmitch, what do we do with the wikipedia lens? | 07:42 |
| ajmitch | dholbach: usually the person who puts it up for vote has been the uploader, but that obviously won't work | 07:50 |
| * ajmitch has just got back in from pycon meeting, needs to sort dinner | 07:51 | |
| dholbach | ajmitch, wow, you're in meeting throughout the week :) | 07:51 |
| ajmitch | well, last night was codecraft, a monthly meeting | 07:51 |
| ajmitch | so we moved the pycon one to tonight | 07:51 |
| BerndSch | ajmitch: are there any news for my submitted app (https://myapps.developer.ubuntu.com/dev/apps/548/) for the precise software center? | 08:15 |
| PaoloRotolo | ok, can I review the last app: variety? | 08:20 |
| dholbach | PaoloRotolo, sure | 08:21 |
| PaoloRotolo | good | 08:21 |
| dholbach | BerndSch, I'll take a look | 08:21 |
| PaoloRotolo | Let's start :) | 08:21 |
| dholbach | ajmitch, does etc/apparmor.d/opt.extras.ubuntu.com.unity-lens-sshsearch.unity-lens-sshsearch.py look like a correct path to you? | 08:32 |
| dholbach | I never dealt with any of this, so I'm not 100% sure | 08:33 |
| dholbach | BerndSch, I pushed a few changes to lp:~dholbach/ubuntu-app-reviews/unity-lens-sshsearch which you might be interested in | 08:35 |
| dholbach | BerndSch, I'm a bit unsure about the location of the apparmor profile, but as I'm not an expert, I'll leave it as it is :) | 08:36 |
| ajmitch | dholbach: I left that & the debian/{control,rules} stuff in - what I wanted was for other people to vote on it | 08:48 |
| ajmitch | there were some problems with qreator & the metadata getting through to software-center from what I recall | 08:48 |
| dholbach | oh | 08:57 |
| dholbach | you mean the dh_gencontrol stuff? | 08:58 |
| dholbach | ok, I reapplied it to lp:~dholbach/ubuntu-app-reviews/unity-lens-sshsearch | 09:00 |
| * dholbach just had a look at pyMi | 10:34 | |
| * dholbach has a look at indicator pingdom | 10:43 | |
| dholbach | I'll have a look at utrails next | 10:48 |
| dholbach | this is actually fun :) | 10:48 |
| dholbach | utrails might actually be ready soon | 10:55 |
| * dholbach has a look at protoborsa | 11:03 | |
| achuni | good morning! | 11:10 |
| dholbach | hey achuni | 11:11 |
| achuni | hi dholbach :) | 11:11 |
| didrocks | hey dholbach :) | 12:30 |
| didrocks | dholbach: so the new requirements on http://bazaar.launchpad.net/~dholbach/ubuntu-app-reviews/indicator-pingdom/changes/4?start_revid=4 (like apport and the .rtupdate) makes it difficult for people using Quickly for the context | 12:31 |
| didrocks | as it won't be update for this | 12:31 |
| didrocks | and next quickly package will overwrite this | 12:31 |
| didrocks | TommyBrunn: for 2. you can quickly configure dependencies | 12:31 |
| didrocks | then manually add in a line the dep | 12:31 |
| didrocks | then, quickly package will add this dep when recreating debian/control | 12:31 |
| TommyBrunn | didrocks: Ok, that I can certainly do. | 12:32 |
| didrocks | TommyBrunn: for the copyright, right, it's a case we don't support, apart from copying over the debian/copyright, I'm unsure :) | 12:32 |
| didrocks | but it's interesting, yeah, didn't think about apps wrapping multiple libs | 12:32 |
| TommyBrunn | didrocks: Wouldn't it be fairly common to have at least something that's not necessarily your own. Such as using a CC icon or something? | 12:33 |
| dholbach | didrocks, urgh :-( | 12:34 |
| didrocks | TommyBrunn: well, Quickly was done for applications in the distro first, not ARB one | 12:34 |
| dholbach | didrocks, it wasn't clear to me that quickly would overwrite it again | 12:34 |
| didrocks | TommyBrunn: so normally, that would ask for a separate source package | 12:35 |
| TommyBrunn | didrocks: doing quickly configure dependencies will add libglib2.0-bin as a regular dependency, not a build-dependency. Is that a problem? | 12:40 |
| didrocks | TommyBrunn: not really, automagic build-dep doesn't work in dh_python2 anyway | 12:40 |
| didrocks | but yeah, something to note when it will work :) | 12:40 |
| dholbach | didrocks, do you know what might cause something like lines 8-10 in http://bazaar.launchpad.net/~dholbach/ubuntu-app-reviews/protoborsa/revision/24? | 12:42 |
| didrocks | dholbach: what is wrong? :) | 12:43 |
| dholbach | didrocks, the removed lines | 12:43 |
| dholbach | didrocks, I think it's auto-generated stuff | 12:43 |
| didrocks | it's a quickly project? | 12:43 |
| dholbach | and I've seen it many times reviewing apps | 12:43 |
| dholbach | yes | 12:43 |
| didrocks | this comes from the AUTHORS file | 12:43 |
| dholbach | almost all of them are | 12:43 |
| didrocks | so if the AUTHORS file changed, it's put there | 12:43 |
| dholbach | "</property>" does not look like a standard AUTHORS bit | 12:43 |
| didrocks | the first stenza changed is the new format | 12:44 |
| didrocks | http://bazaar.launchpad.net/~dholbach/ubuntu-app-reviews/protoborsa/view/head:/AUTHORS | 12:44 |
| dholbach | what about the "</property>" bit? | 12:44 |
| dholbach | but I guess it's more of a p-d-e problem anyway, right? | 12:45 |
| * didrocks doesn't see </property> | 12:49 | |
| didrocks | ah | 12:49 |
| didrocks | in the lines removed | 12:49 |
| didrocks | yeah, it's an old bug of p-d-e | 12:49 |
| didrocks | the new update, as you can see, removes it :) | 12:49 |
| dholbach | sweet | 12:50 |
| dholbach | I can't wait for the update to go into -updates | 12:50 |
| dholbach | it's in -proposed already | 12:50 |
| dholbach | didrocks, I'm not sure what we do about 'quickly package' overwriting things :-/ | 12:53 |
| didrocks | dholbach: well, if the requirements are moving, we won't be ever able to stabilize some stuff on quickly | 12:57 |
| didrocks | dholbach: we do have to update because the packaging is changing | 12:57 |
| coolbhavi | highvoltage, ajmitch dholbach https://myapps.developer.ubuntu.com/dev/apps/1111/ < -- can I reject this request? | 12:58 |
| coolbhavi | its available freely and readily for download I see no point in pushing this through as an app | 12:59 |
| coolbhavi | available as is as a pdf | 13:00 |
| coolbhavi | any comments? | 13:08 |
| dholbach | didrocks, the requirements stayed the same - it's the packaging infrastructure which changed :) | 13:10 |
| highvoltage | coolbhavi: hmm, PDFs are usually allowed, I think it usually goes through the commercial process because they have some process for that. I agree that it's kind of pointless but I'm not sure if that's enough basis to reject it on | 13:22 |
| dholbach | you can punt it to the other queue | 13:22 |
| coolbhavi | highvoltage, thanks for that | 13:25 |
| coolbhavi | dholbach, whom to contact for the commercial queue? | 13:25 |
| coolbhavi | for moving the app? | 13:26 |
| coolbhavi | this is the first time I am encountering such situations | 13:29 |
| coolbhavi | achuni, hi | 13:39 |
| achuni | hey coolbhavi | 13:39 |
| coolbhavi | https://myapps.developer.ubuntu.com/dev/apps/1111/ there is a pdf which is readily available for download.. Yet submitted to the ARB for processing | 13:40 |
| coolbhavi | achuni, ^^^ | 13:41 |
| achuni | coolbhavi: right, MyApps bumps everything that is $0 and uses a free license over to the ARB | 13:42 |
| achuni | coolbhavi: even though the ARB doesn't do pdf content, it's hard to tell automatically when an app is a pdf-in-a-box | 13:42 |
| achuni | coolbhavi: ah, I see your comments above (I might have missed a few, I dropped out a bit I think) | 13:43 |
| coolbhavi | achuni, personally I see no point in pushing this as an app because its available as is for download as pdf as I said above | 13:44 |
| achuni | coolbhavi: I'd say bump over to the non-ARB queue | 13:44 |
| coolbhavi | achuni, do you guys have a process for packing a pdf document btw? and how to move it over? | 13:45 |
| achuni | coolbhavi: until there's a separate "book store" for ubuntu, pdfs and apps are being mingled... there's an intent to separate them out at some point soon | 13:45 |
| achuni | coolbhavi: to move over, during review you should have a button at the bottom of the page that says "Move to MyApps" | 13:46 |
| coolbhavi | achuni, ah ok never used that thanks! | 13:47 |
| achuni | coolbhavi: which is a very non-intuitive label, yep. at first we referred to the non-ARB queue as the "MyApps" queue | 13:47 |
| achuni | coolbhavi: I still don't think we have a good name for the non-ARB queue :) | 13:47 |
| coolbhavi | achuni, seemed a bit confusing at first look because we are reviewing in the My apps portal atleast going by the website :) so I just got a bit dumb I guess | 13:49 |
| coolbhavi | sorry for that | 13:49 |
| achuni | coolbhavi: np :) | 13:51 |
| TommyBrunn | So dholbach, didrocks, any ideas on how to solve this problem? I imagine I'm not going to be the only person in the Ubuntu App Showdown to use quickly and have some third-party stuff in there. I'd be fine with a hacky solution for now, as long as there _is_ a solution. | 14:09 |
| dholbach | TommyBrunn, which problem are you referring to? the .rtupdate and apport issue? | 14:10 |
| dholbach | if that's the case, ignore it for now - I'll send a mail to the ARB list about it | 14:10 |
| TommyBrunn | No, I was referring to the issue with the copyright file | 14:11 |
| didrocks | well, you are the only one about the copyright file as of today TBH :/ | 14:11 |
| didrocks | it's something to open a bug for, it's an interesting case quickly wasn't design for | 14:11 |
| didrocks | right now, I can only see this as copying it over | 14:12 |
| TommyBrunn | What do you mean about that last part? | 14:13 |
| dholbach | I think I'm missing part of the discussion right now - is debian/copyright auto-generated too and overwritten or what is the issue? | 14:13 |
| TommyBrunn | That is correct | 14:13 |
| TommyBrunn | When quickly packages the project, it generates a new copyright file and overwrites the old one | 14:14 |
| dholbach | ok | 14:14 |
| didrocks | as you can have added AUTHORS to this file | 14:14 |
| didrocks | bundling other libs in your app was not what quickly was designed for | 14:15 |
| dholbach | didrocks, once the project is in the "finishing touches" stage, would it be possible to use something like "bzr bd -- -S -sa; cd ..; dput ppa package*.changes" instead of 'quickly package'? or would it cause problems? | 14:15 |
| dholbach | so we could tell people to concentrate on the code and once they're happy, "upload it regularly"? | 14:16 |
| didrocks | dholbach: that won't cause any problem | 14:16 |
| didrocks | well | 14:16 |
| didrocks | until you add new deps :) | 14:16 |
| didrocks | or update AUTHORS | 14:16 |
| didrocks | or add new files :p | 14:16 |
| dholbach | right | 14:17 |
| dholbach | debuild -i.bzr then ;-) | 14:17 |
| didrocks | same issue if you add new deps or update AUTHORS, but yeah, at least, new files are handled :) | 14:18 |
| dholbach | didrocks, or we would tell app authors to just give us a link to their bzr branch | 14:21 |
| didrocks | yeah, but that will break at next release | 14:22 |
| didrocks | I think it's something to discuss for next week | 14:22 |
| didrocks | it's a tricky question | 14:22 |
| dholbach | I just sent https://lists.ubuntu.com/archives/app-review-board/2012-July/001056.html | 14:22 |
| didrocks | until now, we told "the debian directory is owned by quickly" | 14:23 |
| dholbach | and CCed you | 14:23 |
| didrocks | thanks :) | 14:23 |
| dholbach | TommyBrunn, for now I'd recommend to ignore the issue | 14:23 |
| dholbach | and just keep on working on the app :) | 14:24 |
| dholbach | as soon as we have a solution we'll advise through myApps comments | 14:24 |
| TommyBrunn | Okay, thanks for your time. :) | 14:24 |
| dholbach | thanks a lot for bringing it up | 14:24 |
| * dholbach goes back to teaching the apps-brancher to pull from PPAs :) | 14:25 | |
| PaoloRotolo | Hi all! | 14:40 |
| highvoltage | hi PaoloRotolo | 14:42 |
| PaoloRotolo | hi highvoltage | 14:42 |
| dholbach | ciao PaoloRotolo | 14:42 |
| dholbach | PaoloRotolo, I'm not sure if you saw https://lists.ubuntu.com/archives/app-review-board/2012-July/001056.html yet | 14:43 |
| PaoloRotolo | dholbach, ciao :) | 14:43 |
| dholbach | (not sure if you are on the list) | 14:43 |
| dholbach | but for now we should probably hold off making .rtupdate, crashdb and DEP5 changes | 14:43 |
| dholbach | as they will be overwritten anway :-/ | 14:43 |
| dholbach | hey highvoltage | 14:44 |
| dholbach | but still it should make sense to branch apps, install them, test them and review them - there might be other issues we can point out or fix :) | 14:45 |
| PaoloRotolo | dholbach, sorry I don't understand... We can't fix the DEP5 in the copyright, for example? | 14:45 |
| dholbach | PaoloRotolo, yes, even if the app author merges the changes, they will be overwritten the next time they run 'quickly package' | 14:46 |
| dholbach | so until we have a good solution to the problem, it doesn't make sense yet | 14:46 |
| PaoloRotolo | dholbach, and... what happens if someone doesn't use Quickly? | 14:48 |
| PaoloRotolo | is not obligatory | 14:48 |
| dholbach | in that case the change would make sense :) | 14:52 |
| PaoloRotolo | dholbach, so, if the app don't use Quickly I can fix the issue, right? How can I know if the app are using it? | 14:57 |
| dholbach | didrocks, is the .quickly file always shipped? | 14:58 |
| didrocks | dholbach: in the source package? right, but it shouldn't be installed | 14:59 |
| dholbach | sure, sorry - that's what I meant | 14:59 |
| didrocks | yeah yeah ;) | 15:00 |
| didrocks | apart if the dev is doing some tricks ;) | 15:00 |
| didrocks | but no garantee then! | 15:00 |
| dholbach | PaoloRotolo, if there's a .quickly file in the source package then it uses Quickly | 15:00 |
| dholbach | the apport crash db we filed a bug report for, the DEP5 thing should be fixed once python-distutils-extra gets into -updates | 15:02 |
| dholbach | but for the .rtupdate thing we still don't have a good answer :/ | 15:02 |
| dholbach | didrocks, can we get a 'quickly package --dont-do-it' option? :-P | 15:03 |
| PaoloRotolo | dholbach, good. If there isn't a quickly file I can fix the issue. If there is, we can invite the owner of the app to run 'quicly package', ok? | 15:04 |
| didrocks | dholbach: well, we need to find a way, as told in the email, there are a lot of reasons to have those files owned by quickly | 15:04 |
| dholbach | PaoloRotolo, 'quickly package' will overwrite all our changes to debian/rules and debian/copyright | 15:04 |
| didrocks | the issue is when the platform and requirements changes :) | 15:04 |
| didrocks | like this new rtupdate file | 15:05 |
| didrocks | the copyright dep5, it's not mandatory to push latest versions | 15:05 |
| dholbach | and in more complicated cases like a more complicated debian/copyright | 15:05 |
| dholbach | no, it's not - agreed | 15:05 |
| didrocks | the older spec is still valid | 15:05 |
| didrocks | and we have a bunch of packages in main still following it :) | 15:05 |
| didrocks | the "I embeed a separate lib in my app" is more worrying | 15:06 |
| didrocks | not really sure how we can do that | 15:06 |
| didrocks | I'm sure pkgme has the same issue btw | 15:06 |
| dholbach | ask authors to add a EXTRA_COPYING file which gets >> debian/copyright ;-) | 15:06 |
| PaoloRotolo | dholbach, oh, I finally got it :D | 15:09 |
| PaoloRotolo | dholbach, So this affects only the apps in development. (for the contest) | 15:11 |
| dholbach | PaoloRotolo, let's wait which replies come up in the discussion on the mailing list | 15:12 |
| PaoloRotolo | It's not valid for the mature apps submitted to the ARB | 15:12 |
| dholbach | right now, I'd say best ignore it for quickly apps | 15:12 |
| PaoloRotolo | dholbach, ok, thanks! I'm going to subscribe the mailing list | 15:14 |
| dholbach | thanks :) | 15:14 |
| PaoloRotolo | [I'm afk for 5 minutes] | 15:14 |
| PaoloRotolo | ok, fixed 4 issues with "variety" | 15:48 |
| PaoloRotolo | I'm uploading it into a branch | 15:48 |
| PaoloRotolo | Here it is: https://code.launchpad.net/~ubuntu-app-review-contributors/ubuntu-app-reviews/variety | 16:13 |
| BerndSch | dholbach, ajmitch: I updated my branch with dholbach changes and submitted the new tar.gz for my app https://myapps.developer.ubuntu.com/dev/apps/548/changerequest/145/ | 17:48 |
| === achuni_ is now known as achuni | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!