[06:59] <dholbach> good morning
[07:02] <PaoloRotolo> Good morning
[07:03] <dholbach> ciao PaoloRotolo
[07:03] <dholbach> come stai?
[07:03] <PaoloRotolo> dholbach, fine, thanks :D
[07:03] <dholbach> :-)
[07:04] <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:05] <PaoloRotolo> dholbach, nice, where?
[07:06] <dholbach> we're invited to a wedding in Massafra, but we're going to see other parts of the country as well :)
[07:08] <PaoloRotolo> dholbach, it's near my town :) I live in Puglia
[07:08] <dholbach> ah nice :)
[07:08] <didrocks> hey dholbach ;)
[07:09] <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:10] <didrocks> PaoloRotolo: no worry, happens to me every single day! :)
[07:10] <PaoloRotolo> didrocks, me too :)
[07:23] <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:42] <dholbach> ajmitch, are you feeling any better now?
[07:42] <dholbach> ajmitch, what do we do with the wikipedia lens?
[07:50] <ajmitch> 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] <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
[08:15] <BerndSch> ajmitch: are there any news for my submitted app (https://myapps.developer.ubuntu.com/dev/apps/548/) for the precise software center?
[08:20] <PaoloRotolo> ok, can I review the last app: variety?
[08:21] <dholbach> PaoloRotolo, sure
[08:21] <PaoloRotolo> good
[08:21] <dholbach> BerndSch, I'll take a look
[08:21] <PaoloRotolo> Let's start :)
[08:32] <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:33] <dholbach> I never dealt with any of this, so I'm not 100% sure
[08:35] <dholbach> BerndSch, I pushed a few changes to lp:~dholbach/ubuntu-app-reviews/unity-lens-sshsearch which you might be interested in
[08:36] <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:48] <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:57] <dholbach> oh
[08:58] <dholbach> you mean the dh_gencontrol stuff?
[09:00] <dholbach> 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] <dholbach> I'll have a look at utrails next
[10:48] <dholbach> this is actually fun :)
[10:55] <dholbach> utrails might actually be ready soon
[11:03]  * dholbach has a look at protoborsa
[11:10] <achuni> good morning!
[11:11] <dholbach> hey achuni
[11:11] <achuni> hi dholbach :)
[12:30] <didrocks> hey dholbach :)
[12:31] <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:32] <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:33] <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:34] <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:35] <didrocks> TommyBrunn: so normally, that would ask for a separate source package
[12:40] <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:42] <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:43] <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:44] <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:45] <dholbach> but I guess it's more of a p-d-e problem anyway, right?
[12:49]  * 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:50] <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:53] <dholbach> didrocks, I'm not sure what we do about 'quickly package' overwriting things :-/
[12:57] <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:58] <coolbhavi> highvoltage, ajmitch dholbach https://myapps.developer.ubuntu.com/dev/apps/1111/ < -- can I reject this request?
[12:59] <coolbhavi> its available freely and readily for download I see no point in pushing this through as an app
[13:00] <coolbhavi> available as is as a pdf
[13:08] <coolbhavi> any comments?
[13:10] <dholbach> didrocks, the requirements stayed the same - it's the packaging infrastructure which changed :)
[13:22] <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:25] <coolbhavi> highvoltage, thanks for that
[13:25] <coolbhavi> dholbach, whom to contact for the commercial queue?
[13:26] <coolbhavi> for moving the app?
[13:29] <coolbhavi> this is the first time I am encountering such situations
[13:39] <coolbhavi> achuni, hi
[13:39] <achuni> hey coolbhavi
[13:40] <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:41] <coolbhavi> achuni, ^^^
[13:42] <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:43] <achuni> coolbhavi: ah, I see your comments above (I might have missed a few, I dropped out a bit I think)
[13:44] <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:45] <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:46] <achuni> coolbhavi: to move over, during review you should have a button at the bottom of the page that says "Move to MyApps"
[13:47] <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:49] <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:51] <achuni> coolbhavi: np :)
[14:09] <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:10] <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:11] <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:12] <didrocks> right now, I can only see this as copying it over
[14:13] <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:14] <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:15] <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:16] <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:17] <dholbach> right
[14:17] <dholbach> debuild -i.bzr then ;-)
[14:18] <didrocks> same issue if you add new deps or update AUTHORS, but yeah, at least, new files are handled :)
[14:21] <dholbach> didrocks, or we would tell app authors to just give us a link to their bzr branch
[14:22] <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:23] <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:24] <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:25]  * dholbach goes back to teaching the apps-brancher to pull from PPAs :)
[14:40] <PaoloRotolo> Hi all!
[14:42] <highvoltage> hi PaoloRotolo
[14:42] <PaoloRotolo> hi highvoltage
[14:42] <dholbach> ciao PaoloRotolo
[14:43] <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:44] <dholbach> hey highvoltage
[14:45] <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:46] <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:48] <PaoloRotolo> dholbach, and... what happens if someone doesn't use Quickly?
[14:48] <PaoloRotolo> is not obligatory
[14:52] <dholbach> in that case the change would make sense :)
[14:57] <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:58] <dholbach> didrocks, is the .quickly file always shipped?
[14:59] <didrocks> dholbach: in the source package? right, but it shouldn't be installed
[14:59] <dholbach> sure, sorry - that's what I meant
[15:00] <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:02] <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:03] <dholbach> didrocks, can we get a 'quickly package --dont-do-it' option? :-P
[15:04] <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:05] <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:06] <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:09] <PaoloRotolo> dholbach, oh, I finally got it :D
[15:11] <PaoloRotolo> dholbach, So this affects only the apps in development. (for the contest)
[15:12] <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:14] <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:48] <PaoloRotolo> ok, fixed 4 issues with "variety"
[15:48] <PaoloRotolo> I'm uploading it into a branch
[16:13] <PaoloRotolo> Here it is: https://code.launchpad.net/~ubuntu-app-review-contributors/ubuntu-app-reviews/variety
[17:48] <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/