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