[06:56] <dholbach> good morning
[07:21] <dpm> morning
[08:58] <dholbach> ajmitch, highvoltage: we might have to decide how we move forward with app showdown apps
[08:58] <dholbach> as 'quickly package' will overwrite changes we ask for
[09:13] <PaoloRotolo> Good morning
[09:15] <dholbach> ciao PaoloRotolo
[09:15] <PaoloRotolo> Hi dholbach :)
[09:19] <PaoloRotolo> dholbach, please, can you post a feedback on this app: https://myapps.developer.ubuntu.com/dev/apps/1151/  ? I've uploaded some fixes here: https://code.launchpad.net/~paolorotolo/ubuntu-app-reviews/variety :)
[09:21] <dholbach> PaoloRotolo, it seems like the app uses quickly too, so those changes would get overwritten too the next time it gets updated in the ppa :-/
[09:21] <dholbach> dpm: ^ meet PaoloRotolo, a tireless new helper in the app world :)
[09:22] <dpm> heya PaoloRotolo, nice to meet you and welcome! :-)
[09:22] <dpm> I saw some of your merge proposals already, good work!
[09:22] <didrocks> hey hey
[09:22] <dholbach> salut didrocks
[09:22] <PaoloRotolo> dpm, hi, thanks!
[09:22] <PaoloRotolo> hi didrocks
[09:22] <didrocks> hey dholbach, PaoloRotolo!
[09:22] <dholbach> didrocks, which of the changes in https://code.launchpad.net/~paolorotolo/ubuntu-app-reviews/variety/+merge/113424 should we be able to make in p-d-e itself? :)
[09:23] <dholbach> bug 1002379 is already fixed or going to be fixed soon
[09:23] <didrocks> dholbach: the DEP-5 one
[09:23] <dholbach> bug 1020502 is already filed
[09:23] <didrocks> ah :)
[09:23] <dholbach> I'm not quite sure how to proceed in the .rtupdate case
[09:24] <dholbach> and do you think we should file a bug about extending debian/copyright somehow?
[09:24] <didrocks> why do you override dh_python2?
[09:24] <dholbach> at least to document the problem somewhere
[09:24] <dholbach> didrocks, it creates  usr/share/python/runtime.d/*.rtupdate files
[09:24] <didrocks> dholbach: I would prefer that we discuss that with all the other problems we saw recently on Monday
[09:24] <didrocks> and see what could  be done
[09:24] <didrocks> yeah, but only on quantal?
[09:24] <didrocks> dh_python2 is useful :)
[09:24] <dholbach> I think I saw it in precise too
[09:24] <didrocks> for having compiled .pyc I guess
[09:25] <didrocks> the crashdb.conf is fixing in pde as well
[09:25] <dholbach> I think that's all the problems I currently know of
[09:25] <dholbach> so it's not too bad
[09:25] <dholbach> let me do a test-build on precise and see what I get
[09:25] <didrocks> yeah
[09:26] <PaoloRotolo> didrocks, I override dh_python2 to fix this error:
[09:26] <PaoloRotolo> E: Unallowed file usr/share/python/runtime.d/variety.rtupdate was found in
[09:26] <PaoloRotolo>    the package, which is created by dh_python2. To find out how to override
[09:26] <PaoloRotolo>    dh_python2, please have a look at http://bazaar.launchpad.net/~dholbach
[09:26] <PaoloRotolo>    /ubuntu-app-reviews/harvestwidget/revision/59
[09:26] <dholbach> it was introduced in python-defaults 2.7.2-5
[09:26] <didrocks> PaoloRotolo: right, but you mean that dh_python2 is useless then?
[09:26] <dholbach> PaoloRotolo, yes, I know - that was added to arb-lint before we knew about quickly rewriting things
[09:29] <dholbach> didrocks, the same problem should be in precise as well (.rtupdate)
[09:29] <didrocks> dholbach: hum ok
[09:30] <didrocks> dholbach: I need to look at what this file is for exactly
[09:30] <didrocks> dholbach: and see if we can make use of it
[09:30] <didrocks> I think we should really spend time to ensure that everything works well frmo /opt
[09:30] <didrocks> so that we have a requirement which really works
[09:30] <dholbach> maybe we need a dh_extrasapp :-D
[09:31] <didrocks> dholbach: yeah, maybe, I would be happy that we clearly look at that :)
[09:31] <didrocks> dholbach: can you setup a hangout for Monday?
[09:32] <dholbach> didrocks, so as far as I can see there are two issues which we don't have bugs for: 1) extending debian/copyright and 2) .rtupdate
[09:32] <didrocks> with some people aware and deep into that? :)
[09:32] <dholbach> didrocks, who would you want to have in the hangout
[09:32] <didrocks> dholbach: it will be in French, of course :p
[09:32] <didrocks> dholbach: don't really know, I don't know the arb members ;) probably you, david, already
[09:32] <dholbach> bien sûr :)
[09:34] <didrocks> people who knows about the requirements and exception, as well as different issues we have with it :)
[09:34] <PaoloRotolo> didrocks, are you French :)?
[09:35] <didrocks> PaoloRotolo: you never hear me talking I guess and my english accent, I'm clearly French :p
[09:36] <dholbach> https://bugs.launchpad.net/ubuntu/+source/python-distutils-extra/+bug/1021207
[09:37] <dholbach> PaoloRotolo, I'll update arb-lint to explain that this is something we should ignore for now
[09:37] <PaoloRotolo> nice :)
[09:38] <PaoloRotolo> dholbach, only in the quickly app, right?
[09:39] <dholbach> PaoloRotolo, no, I fear it's a more general issue
[09:41] <dholbach> didrocks, in which cases will p-d-e write the debian/copyright file?
[09:42] <didrocks> for all apps where python-mkdebian --force-copyright is called
[09:42] <didrocks> I added force-control, force-rules and force-copyright some years ago
[09:42] <didrocks> and then, make them called by Quickly
[09:42] <didrocks> I'm not sure anyone else but Quickly is using it
[09:42] <dholbach> and in which cases are they called by quickly? always? or just for extras?
[09:42] <didrocks> always
[09:42] <dholbach> ok
[09:43] <didrocks> as soon as you have to update the packaging
[09:43] <didrocks> meaning release, share, package
[09:43] <dholbach> gotcha
[09:43] <didrocks> so
[09:43] <didrocks> this rtupdate file
[09:43] <didrocks> in fact, it's really interested
[09:43] <didrocks> and definitively relocatable
[09:43] <didrocks> we should have the dh_extraapp that dholbach is talking about :)
[09:44] <didrocks> and moving the additional things we move that way
[09:44] <didrocks> would be way more clean :)
[09:44] <didrocks> this would support changing the prefix, moving files around and such
[09:45] <didrocks> than, just adding dh --with extraapp to debian/rules
[09:45] <didrocks> and the magic happens :)
[09:45] <dholbach> I filed https://bugs.launchpad.net/ubuntu/+source/python-distutils-extra/+bug/1021216
[09:45] <dholbach> dpm, the list of https://bugs.launchpad.net/ubuntu/+bugs?field.tag=arb is growing
[09:46] <didrocks> dholbach: so no special check for rtupdate, the only check is that is should be installed in /opt
[09:46] <dholbach> didrocks, but will it be run if it's located in /opt?
[09:46] <dpm> dholbach, yeah, I saw that. https://bugs.launchpad.net/ubuntu/+source/quickly/+bug/1018038 seems to be a critical one to me. Do you know if many people have hit that bug?
[09:47] <dholbach> yes, a few
[09:47] <dholbach> some fixed it after they saw the answer on AskUbuntu
[09:47] <didrocks> dholbach: yeah
[09:47] <didrocks> dholbach: it's just compiling .py to .pyc
[09:47] <dholbach> didrocks, really? what's running it?
[09:47] <didrocks> nothing
[09:47] <didrocks> should be in the postinst
[09:47] <dholbach> oh yeah?
[09:48] <dholbach> will have a look
[09:48] <dholbach> (if somebody could verify all the test-cases from the bugs in https://bugs.launchpad.net/ubuntu/+source/python-distutils-extra/+bug/1020017 and leave a comment on the individual bugs, that'd be nice)
[09:49] <dpm> dholbach, I'll ask mterry if he can have a look when he's around later on. It's a pretty bad bug if we ask people to use submitubuntu and then the package fails to build
[09:49] <dpm> Btw, does anyone know the answer to http://askubuntu.com/questions/159673/how-to-use-gsettings-from-opt - does Quickly take care of it?
[09:50] <dholbach> didrocks, I see nothing in the postinst calling the .rtupdate file
[09:50] <didrocks> dpm: it's easy to fix with current infra
[09:50] <didrocks> dholbach: yeah, it needs to be set manually in postinst
[09:51] <didrocks> dholbach: http://www.mail-archive.com/debian-python@lists.debian.org/msg04587.html
[09:52] <didrocks> oh
[09:52] <didrocks> it's even worse
[09:52] <didrocks> wow, people are going crazy :)
[09:52] <didrocks> so we have this file
[09:52] <didrocks> which should be executed in a postinst
[09:52] <didrocks> BUT!
[09:52] <didrocks> dh_python3
[09:52] <didrocks> is adding: f which py3compile >/dev/null 2>&1; then
[09:52] <didrocks>         py3compile -p unity-lens-radios /usr/share/unity-lens-radios
[09:52] <didrocks> fi
[09:53] <didrocks> so doing the same :)
[09:54] <dholbach> I don't think I understand what's going on :)
[09:54] <didrocks> dholbach: "it's a mess" is a good summary? :)
[09:55] <dholbach> I think we've had the .rtupdate files much much much longer already
[09:56] <didrocks> dholbach: yeah, but it was removed before, that was what barry told me last week
[09:56] <didrocks> but this file is useless since dh_python it seems
[09:56] <didrocks> basically, dh_python adds the same thing in postinst
[09:57] <didrocks> (compiling the .py in .pyc if you project is not in a standard python directory)
[09:58] <dholbach> aha!
[09:58] <dholbach> the python and python3 maintainer scripts run it
[09:59] <didrocks> but it's useless
[09:59] <didrocks> as the postinst stenza is already doing the same
[10:00] <didrocks> dholbach: I do not see any trigger?
[10:00] <didrocks> ah, you mean in each libs
[10:01] <dholbach> http://paste.ubuntu.com/1076151/
[10:01] <dholbach> for python.p{re,ost}inst
[10:01] <didrocks> dholbach: right, but this is only when you install python
[10:01] <didrocks> see the thread I pointed to you, it confirms that
[10:01] <didrocks> but then, there is no trigger
[10:02] <didrocks> so if you install something afterward installing python
[10:02] <dholbach> I would suggest we override dh_install with p-d-e for now and remove the file and be done with it :)
[10:02] <didrocks> this files isn't used
[10:02] <dholbach> and backport the change to precise as well :)
[10:02] <didrocks> dholbach: yeah, it's not needed
[10:02] <didrocks> dholbach: making this down as something to discuss on Monday? :)
[10:02] <dholbach> or rather delete the whole directory :)
[10:02] <didrocks> I'll try to grab more info to confirm that it's not needed anymore
[10:02] <didrocks> (in our case)
[10:04]  * didrocks is out for some exercice, see you soon :)
[10:04] <dholbach> didrocks, ah, maybe it's needed in case you upgrade to a new python
[10:04] <didrocks> dholbach: hum, interesting… ;)
[10:04] <dholbach> but then we just might have to update the python and python3 maintainer scripts to look into /opt as well
[10:04] <didrocks> yeah
[10:04] <dholbach> bugger bugger bugger
[10:05] <didrocks> I need to try to compile with both :)
[10:05] <dholbach> enjoy the exercise
[10:05] <didrocks> and see how it behaves
[10:05] <didrocks> thanks!
[10:05] <dholbach> .rtupdate is a bad hack really :)
[10:05] <didrocks> enjoy is not the correct them :p
[10:05] <didrocks> well, /opt is as well
[10:05] <didrocks> so 1-1 :)
[10:05] <dholbach> haha
[14:20] <dholbach> lp:~ubuntu-app-review-contributors/ubuntu-app-reviews/webbar ← got PPA info from myApps submission, downloaded last package, imported into branch and uploaded :-D
=== achuni_ is now known as achuni