/srv/irclogs.ubuntu.com/2012/07/04/#ubuntu-arb.txt

dholbachgood morning06:59
PaoloRotoloGood morning07:02
dholbachciao PaoloRotolo07:03
dholbachcome stai?07:03
PaoloRotolodholbach, fine, thanks :D07:03
dholbach:-)07:03
PaoloRotolodholbach, your italian is good :P07:04
dholbachno, it's not ;-)07:04
dholbachbut 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
PaoloRotolodholbach, nice, where?07:05
dholbachwe're invited to a wedding in Massafra, but we're going to see other parts of the country as well :)07:06
PaoloRotolodholbach, it's near my town :) I live in Puglia07:08
dholbachah nice :)07:08
didrockshey dholbach ;)07:08
dholbachsalut didrocks07:09
didrockshi PaoloRotolo07:09
dholbachPaoloRotolo, I'm looking forward to it07:09
PaoloRotolodholbach, hi!07:09
didrockswrong d<tab> ;)07:09
PaoloRotolodidrocks, yes, sorry07:09
PaoloRotolodidrocks, hi07:09
didrocksPaoloRotolo: no worry, happens to me every single day! :)07:10
PaoloRotolodidrocks, me too :)07:10
dholbachif 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 appreciated07:23
dholbachajmitch, are you feeling any better now?07:42
dholbachajmitch, what do we do with the wikipedia lens?07:42
ajmitchdholbach: usually the person who puts it up for vote has been the uploader, but that obviously won't work07:50
* ajmitch has just got back in from pycon meeting, needs to sort dinner07:51
dholbachajmitch, wow, you're in meeting throughout the week :)07:51
ajmitchwell, last night was codecraft, a monthly meeting07:51
ajmitchso we moved the pycon one to tonight07:51
BerndSchajmitch: are there any news for my submitted app (https://myapps.developer.ubuntu.com/dev/apps/548/) for the precise software center?08:15
PaoloRotolook, can I review the last app: variety?08:20
dholbachPaoloRotolo, sure08:21
PaoloRotologood08:21
dholbachBerndSch, I'll take a look08:21
PaoloRotoloLet's start :)08:21
dholbachajmitch, does etc/apparmor.d/opt.extras.ubuntu.com.unity-lens-sshsearch.unity-lens-sshsearch.py look like a correct path to you?08:32
dholbachI never dealt with any of this, so I'm not 100% sure08:33
dholbachBerndSch, I pushed a few changes to lp:~dholbach/ubuntu-app-reviews/unity-lens-sshsearch which you might be interested in08:35
dholbachBerndSch, 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
ajmitchdholbach: I left that & the debian/{control,rules} stuff in - what I wanted was for other people to vote on it08:48
ajmitchthere were some problems with qreator & the metadata getting through to software-center from what I recall08:48
dholbachoh08:57
dholbachyou mean the dh_gencontrol stuff?08:58
dholbachok, I reapplied it to lp:~dholbach/ubuntu-app-reviews/unity-lens-sshsearch09:00
* dholbach just had a look at pyMi10:34
* dholbach has a look at indicator pingdom10:43
dholbachI'll have a look at utrails next10:48
dholbachthis is actually fun :)10:48
dholbachutrails might actually be ready soon10:55
* dholbach has a look at protoborsa11:03
achunigood morning!11:10
dholbachhey achuni11:11
achunihi dholbach :)11:11
didrockshey dholbach :)12:30
didrocksdholbach: 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 context12:31
didrocksas it won't be update for this12:31
didrocksand next quickly package will overwrite this12:31
didrocksTommyBrunn: for 2. you can quickly configure dependencies12:31
didrocksthen manually add in a line the dep12:31
didrocksthen, quickly package will add this dep when recreating debian/control12:31
TommyBrunndidrocks: Ok, that I can certainly do.12:32
didrocksTommyBrunn: for the copyright, right, it's a case we don't support, apart from copying over the debian/copyright, I'm unsure :)12:32
didrocksbut it's interesting, yeah, didn't think about apps wrapping multiple libs12:32
TommyBrunndidrocks: 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
dholbachdidrocks, urgh :-(12:34
didrocksTommyBrunn: well, Quickly was done for applications in the distro first, not ARB one12:34
dholbachdidrocks, it wasn't clear to me that quickly would overwrite it again12:34
didrocksTommyBrunn: so normally, that would ask for a separate source package12:35
TommyBrunndidrocks: doing quickly configure dependencies will add libglib2.0-bin as a regular dependency, not a build-dependency. Is that a problem?12:40
didrocksTommyBrunn: not really, automagic build-dep doesn't work in dh_python2 anyway12:40
didrocksbut yeah, something to note when it will work :)12:40
dholbachdidrocks, 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
didrocksdholbach: what is wrong? :)12:43
dholbachdidrocks, the removed lines12:43
dholbachdidrocks, I think it's auto-generated stuff12:43
didrocksit's a quickly project?12:43
dholbachand I've seen it many times reviewing apps12:43
dholbachyes12:43
didrocksthis comes from the AUTHORS file12:43
dholbachalmost all of them are12:43
didrocksso if the AUTHORS file changed, it's put there12:43
dholbach"</property>" does not look like a standard AUTHORS bit12:43
didrocksthe first stenza changed is the new format12:44
didrockshttp://bazaar.launchpad.net/~dholbach/ubuntu-app-reviews/protoborsa/view/head:/AUTHORS12:44
dholbachwhat about the "</property>" bit?12:44
dholbachbut I guess it's more of a p-d-e problem anyway, right?12:45
* didrocks doesn't see </property>12:49
didrocksah12:49
didrocksin the lines removed12:49
didrocksyeah, it's an old bug of p-d-e12:49
didrocksthe new update, as you can see, removes it :)12:49
dholbachsweet12:50
dholbachI can't wait for the update to go into -updates12:50
dholbachit's in -proposed already12:50
dholbachdidrocks, I'm not sure what we do about 'quickly package' overwriting things :-/12:53
didrocksdholbach: well, if the requirements are moving, we won't be ever able to stabilize some stuff on quickly12:57
didrocksdholbach: we do have to update because the packaging is changing12:57
coolbhavihighvoltage, ajmitch dholbach https://myapps.developer.ubuntu.com/dev/apps/1111/ < -- can I reject this request?12:58
coolbhaviits available freely and readily for download I see no point in pushing this through as an app12:59
coolbhaviavailable as is as a pdf13:00
coolbhaviany comments?13:08
dholbachdidrocks, the requirements stayed the same - it's the packaging infrastructure which changed :)13:10
highvoltagecoolbhavi: 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 on13:22
dholbachyou can punt it to the other queue13:22
coolbhavihighvoltage, thanks for that13:25
coolbhavidholbach, whom to contact for the commercial queue?13:25
coolbhavifor moving the app?13:26
coolbhavithis is the first time I am encountering such situations13:29
coolbhaviachuni, hi13:39
achunihey coolbhavi13:39
coolbhavihttps://myapps.developer.ubuntu.com/dev/apps/1111/ there is a pdf which is readily available for download.. Yet submitted to the ARB for processing13:40
coolbhaviachuni, ^^^13:41
achunicoolbhavi: right, MyApps bumps everything that is $0 and uses a free license over to the ARB13:42
achunicoolbhavi: even though the ARB doesn't do pdf content, it's hard to tell automatically when an app is a pdf-in-a-box13:42
achunicoolbhavi: ah, I see your comments above (I might have missed a few, I dropped out a bit I think)13:43
coolbhaviachuni, personally I see no point in pushing this as an app because its available as is for download as pdf  as I said above13:44
achunicoolbhavi: I'd say bump over to the non-ARB queue13:44
coolbhaviachuni, do you guys have a process for packing a pdf document btw? and how to move it over?13:45
achunicoolbhavi: 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 soon13:45
achunicoolbhavi: to move over, during review you should have a button at the bottom of the page that says "Move to MyApps"13:46
coolbhaviachuni, ah ok never used that thanks!13:47
achunicoolbhavi: which is a very non-intuitive label, yep.  at first we referred to the non-ARB queue as the "MyApps" queue13:47
achunicoolbhavi: I still don't think we have a good name for the non-ARB queue :)13:47
coolbhaviachuni, 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 guess13:49
coolbhavisorry for that13:49
achunicoolbhavi: np :)13:51
TommyBrunnSo 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
dholbachTommyBrunn, which problem are you referring to? the .rtupdate and apport issue?14:10
dholbachif that's the case, ignore it for now - I'll send a mail to the ARB list about it14:10
TommyBrunnNo, I was referring to the issue with the copyright file14:11
didrockswell, you are the only one about the copyright file as of today TBH :/14:11
didrocksit's something to open a bug for, it's an interesting case quickly wasn't design for14:11
didrocksright now, I can only see this as copying it over14:12
TommyBrunnWhat do you mean about that last part?14:13
dholbachI 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
TommyBrunnThat is correct14:13
TommyBrunnWhen quickly packages the project, it generates a new copyright file and overwrites the old one14:14
dholbachok14:14
didrocksas you can have added AUTHORS to this file14:14
didrocksbundling other libs in your app was not what quickly was designed for14:15
dholbachdidrocks, 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
dholbachso we could tell people to concentrate on the code and once they're happy, "upload it regularly"?14:16
didrocksdholbach: that won't cause any problem14:16
didrockswell14:16
didrocksuntil you add new deps :)14:16
didrocksor update AUTHORS14:16
didrocksor add new files :p14:16
dholbachright14:17
dholbachdebuild -i.bzr then ;-)14:17
didrockssame issue if you add new deps or update AUTHORS, but yeah, at least, new files are handled :)14:18
dholbachdidrocks, or we would tell app authors to just give us a link to their bzr branch14:21
didrocksyeah, but that will break at next release14:22
didrocksI think it's something to discuss for next week14:22
didrocksit's a tricky question14:22
dholbachI just sent https://lists.ubuntu.com/archives/app-review-board/2012-July/001056.html14:22
didrocksuntil now, we told "the debian directory is owned by quickly"14:23
dholbachand CCed you14:23
didrocksthanks :)14:23
dholbachTommyBrunn, for now I'd recommend to ignore the issue14:23
dholbachand just keep on working on the app :)14:24
dholbachas soon as we have a solution we'll advise through myApps comments14:24
TommyBrunnOkay, thanks for your time. :)14:24
dholbachthanks a lot for bringing it up14:24
* dholbach goes back to teaching the apps-brancher to pull from PPAs :)14:25
PaoloRotoloHi all!14:40
highvoltagehi PaoloRotolo14:42
PaoloRotolohi highvoltage14:42
dholbachciao PaoloRotolo14:42
dholbachPaoloRotolo, I'm not sure if you saw https://lists.ubuntu.com/archives/app-review-board/2012-July/001056.html yet14:43
PaoloRotolodholbach, ciao :)14:43
dholbach(not sure if you are on the list)14:43
dholbachbut for now we should probably hold off making .rtupdate, crashdb and DEP5 changes14:43
dholbachas they will be overwritten anway :-/14:43
dholbachhey highvoltage14:44
dholbachbut 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
PaoloRotolodholbach, sorry I don't understand... We can't fix the DEP5 in the copyright, for example?14:45
dholbachPaoloRotolo, yes, even if the app author merges the changes, they will be overwritten the next time they run 'quickly package'14:46
dholbachso until we have a good solution to the problem, it doesn't make sense yet14:46
PaoloRotolodholbach, and... what happens if someone doesn't use Quickly?14:48
PaoloRotolois not obligatory14:48
dholbachin that case the change would make sense :)14:52
PaoloRotolodholbach, 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
dholbachdidrocks, is the .quickly file always shipped?14:58
didrocksdholbach: in the source package? right, but it shouldn't be installed14:59
dholbachsure, sorry - that's what I meant14:59
didrocksyeah yeah ;)15:00
didrocksapart if the dev is doing some tricks ;)15:00
didrocksbut no garantee then!15:00
dholbachPaoloRotolo, if there's a .quickly file in the source package then it uses Quickly15:00
dholbachthe apport crash db we filed a bug report for, the DEP5 thing should be fixed once python-distutils-extra gets into -updates15:02
dholbachbut for the .rtupdate thing we still don't have a good answer :/15:02
dholbachdidrocks, can we get a 'quickly package --dont-do-it' option? :-P15:03
PaoloRotolodholbach, 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
didrocksdholbach: well, we need to find a way, as told in the email, there are a lot of reasons to have those files owned by quickly15:04
dholbachPaoloRotolo, 'quickly package' will overwrite all our changes to debian/rules and debian/copyright15:04
didrocksthe issue is when the platform and requirements changes :)15:04
didrockslike this new rtupdate file15:05
didrocksthe copyright dep5, it's not mandatory to push latest versions15:05
dholbachand in more complicated cases like a more complicated debian/copyright15:05
dholbachno, it's not - agreed15:05
didrocksthe older spec is still valid15:05
didrocksand we have a bunch of packages in main still following it :)15:05
didrocksthe "I embeed a separate lib in my app" is more worrying15:06
didrocksnot really sure how we can do that15:06
didrocksI'm sure pkgme has the same issue btw15:06
dholbachask authors to add a EXTRA_COPYING file which gets >> debian/copyright ;-)15:06
PaoloRotolodholbach, oh, I finally got it :D15:09
PaoloRotolodholbach, So this affects only the apps in development. (for the contest)15:11
dholbachPaoloRotolo, let's wait which replies come up in the discussion on the mailing list15:12
PaoloRotoloIt's not valid for the mature apps submitted to the ARB15:12
dholbachright now, I'd say best ignore it for quickly apps15:12
PaoloRotolodholbach, ok, thanks! I'm going to subscribe the mailing list15:14
dholbachthanks :)15:14
PaoloRotolo[I'm afk for 5 minutes]15:14
PaoloRotolook, fixed 4 issues with "variety"15:48
PaoloRotoloI'm uploading it into a branch15:48
PaoloRotoloHere it is: https://code.launchpad.net/~ubuntu-app-review-contributors/ubuntu-app-reviews/variety16:13
BerndSchdholbach, 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!