| dholbach | good morning | 06:56 |
|---|---|---|
| dpm | morning | 07:21 |
| 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 | 08:58 |
| PaoloRotolo | Good morning | 09:13 |
| dholbach | ciao PaoloRotolo | 09:15 |
| PaoloRotolo | Hi dholbach :) | 09:15 |
| 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:19 |
| 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:21 |
| 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:22 |
| 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:23 |
| 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:24 |
| 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:25 |
| 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:26 |
| dholbach | didrocks, the same problem should be in precise as well (.rtupdate) | 09:29 |
| didrocks | dholbach: hum ok | 09:29 |
| 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:30 |
| 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:31 |
| 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:32 |
| 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:34 |
| didrocks | PaoloRotolo: you never hear me talking I guess and my english accent, I'm clearly French :p | 09:35 |
| dholbach | https://bugs.launchpad.net/ubuntu/+source/python-distutils-extra/+bug/1021207 | 09:36 |
| dholbach | PaoloRotolo, I'll update arb-lint to explain that this is something we should ignore for now | 09:37 |
| PaoloRotolo | nice :) | 09:37 |
| PaoloRotolo | dholbach, only in the quickly app, right? | 09:38 |
| dholbach | PaoloRotolo, no, I fear it's a more general issue | 09:39 |
| dholbach | didrocks, in which cases will p-d-e write the debian/copyright file? | 09:41 |
| 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:42 |
| 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:43 |
| 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:44 |
| 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:45 |
| 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:46 |
| 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:47 |
| 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:48 |
| 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:49 |
| 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:50 |
| didrocks | dholbach: http://www.mail-archive.com/debian-python@lists.debian.org/msg04587.html | 09:51 |
| 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:52 |
| didrocks | so doing the same :) | 09:53 |
| dholbach | I don't think I understand what's going on :) | 09:54 |
| didrocks | dholbach: "it's a mess" is a good summary? :) | 09:54 |
| dholbach | I think we've had the .rtupdate files much much much longer already | 09:55 |
| 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:56 |
| didrocks | (compiling the .py in .pyc if you project is not in a standard python directory) | 09:57 |
| dholbach | aha! | 09:58 |
| dholbach | the python and python3 maintainer scripts run it | 09:58 |
| didrocks | but it's useless | 09:59 |
| didrocks | as the postinst stenza is already doing the same | 09:59 |
| didrocks | dholbach: I do not see any trigger? | 10:00 |
| didrocks | ah, you mean in each libs | 10:00 |
| 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:01 |
| 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:02 |
| * 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:04 |
| 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 | 10:05 |
| 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 | 14:20 |
| === achuni_ is now known as achuni | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!