194343 2007-10-01 06:40 0000 dev-python/pygtk incorrectly stores .pyc files in /usr/share/ 2008-07-23 22:11:24 0000 1 1 1 Unclassified Gentoo Linux Applications unspecified All Linux RESOLVED FIXED P2 normal --- 1 vapier@gentoo.org gnome@gentoo.org Michael_Dehler@web.de mikachu@gmail.com python@gentoo.org radek@podgorny.cz shawvrana@gmail.com wpan03@gmail.com vapier@gentoo.org 2007-10-01 06:40:25 0000 it is a FHS violation for non-portable binary files to be stored in /usr/share/ dev-python/pygtk sticks .pyc/.pyo files in there which causes problems obviously when upgrading python versions $ /usr/share/pygtk/2.0/codegen/ -v dev-python/pygtk-2.12.0 (/usr/share/pygtk/2.0/codegen) $ ls /usr/share/pygtk/2.0/codegen/*.pyc /usr/share/pygtk/2.0/codegen/argtypes.pyc /usr/share/pygtk/2.0/codegen/docextract.pyc /usr/share/pygtk/2.0/codegen/mkskel.pyc /usr/share/pygtk/2.0/codegen/codegen.pyc /usr/share/pygtk/2.0/codegen/docgen.pyc /usr/share/pygtk/2.0/codegen/override.pyc /usr/share/pygtk/2.0/codegen/createdefs.pyc /usr/share/pygtk/2.0/codegen/h2def.pyc /usr/share/pygtk/2.0/codegen/reversewrapper.pyc /usr/share/pygtk/2.0/codegen/definitions.pyc /usr/share/pygtk/2.0/codegen/__init__.pyc /usr/share/pygtk/2.0/codegen/scmexpr.pyc /usr/share/pygtk/2.0/codegen/defsparser.pyc /usr/share/pygtk/2.0/codegen/mergedefs.pyc eva@gentoo.org 2008-04-14 10:10:47 0000 *** Bug 217591 has been marked as a duplicate of this bug. *** ford_prefect@gentoo.org 2008-04-14 16:14:21 0000 Created an attachment (id=149701) patch to install codegen into /usr/lib/python-2.x/site-packages remi@gentoo.org 2008-04-14 17:34:58 0000 Patch looks good, but I don't really see the point of : pkgpythondir = $(pyexecdir)/gtk-2.0 codegendir = $(pkgpythondir)/codegen Why not do : codegendir = $(pyexecdir)/gtk-2.0/codegen ...unless "pkgpythondir" is reused someplace... Am I missing something? Patch should be sent upstream too imho :) Great stuff remi@gentoo.org 2008-04-14 17:36:24 0000 (In reply to comment #3) > Patch should be sent upstream too imho :) Great stuff /me should have checked his inbox thoroughly before writing dumb comments :p ford_prefect@gentoo.org 2008-04-14 19:48:46 0000 (In reply to comment #3) > Patch looks good, but I don't really see the point of : > > pkgpythondir = $(pyexecdir)/gtk-2.0 > codegendir = $(pkgpythondir)/codegen > > Why not do : > > codegendir = $(pyexecdir)/gtk-2.0/codegen I just copied the way upstream did it for other stuff, though I do take your point. And the set of functions exported by the python m4 macros just added to the confusion, till I found the documentation. ford_prefect@gentoo.org 2008-04-14 19:53:38 0000 (In reply to comment #5) > I just copied the way upstream did it for other stuff, though I do take your > point. And the set of functions exported by the python m4 macros just added to > the confusion, till I found the documentation. To reiterate aforementioned confusion -- I reviewed the patch again, and I believe pkgpythondir is used by the codegen_PYTHON line below in the same file to determine where all those .py files are sent. remi@gentoo.org 2008-05-28 10:12:32 0000 Created an attachment (id=154565) patch to install codegen into /usr/lib/python-2.x/site-packages Simpler patch, should be safer imho. remi@gentoo.org 2008-05-28 14:04:26 0000 I've just committed my patch to portage on pygtk-2.12.1-r1. Closing mikachu@gmail.com 2008-05-28 20:11:35 0000 After i installed the new revision i got this error while building the gimp: /usr/bin/python: can't open file '/lib/python2.5/site-packages/gtk-2.0/codegen/codegen.py': [Errno 2] No such file or directory Which is probably due to the fact that /usr/bin/pygtk-codegen-2.0 looks like this: #!/bin/sh prefix=/usr datarootdir=${prefix}/share datadir=/usr/share codegendir=${exec_prefix}/lib/python2.5/site-packages/gtk-2.0/codegen PYTHONPATH=$codegendir export PYTHONPATH exec /usr/bin/python $codegendir/codegen.py "$@" Note that exec_prefix isn't set anywhere... dang@gentoo.org 2008-05-29 02:28:46 0000 Thanks for pointing that out. I've fixed it. wpan03@gmail.com 2008-05-29 10:54:40 0000 (In reply to comment #10) > Thanks for pointing that out. I've fixed it. > Just out of curiosity, if the old version of pygtk-2.12.1-r1 needed the exec_prefix correction in the patch to fix this issue, why wasn't it bumped to pygtk-2.12.1-r2? I ask because the exec_prefix bug caused gtk-vnc not to build (see bug #224039), and if the pygtk version number isn't bumped, doing another emerge--sync and emerge -u world won't pull the fix in. remi@gentoo.org 2008-05-29 12:27:36 0000 Rebuilding pygtk should have been enough. I've forced a revbump just to be sure, ebuilds are identical. Thanks eva@gentoo.org 2008-07-23 22:11:24 0000 *** Bug 232279 has been marked as a duplicate of this bug. *** 149701 2008-04-14 16:14 0000 patch to install codegen into /usr/lib/python-2.x/site-packages pygtk-2.12.1-install-codegen-to-usr-lib.patch text/plain ZGlmZiAtdXJOcCBweWd0ay0yLjEyLjEtb3JpZy9jb2RlZ2VuL01ha2VmaWxlLmFtIHB5Z3RrLTIu MTIuMS1uZXcvY29kZWdlbi9NYWtlZmlsZS5hbQotLS0gcHlndGstMi4xMi4xLW9yaWcvY29kZWdl bi9NYWtlZmlsZS5hbQkyMDA4LTA0LTE0IDE2OjE1OjA4LjAwMDAwMDAwMCArMDUzMAorKysgcHln dGstMi4xMi4xLW5ldy9jb2RlZ2VuL01ha2VmaWxlLmFtCTIwMDgtMDQtMTQgMTY6NDg6NTAuMDAw MDAwMDAwICswNTMwCkBAIC0yLDcgKzIsMTAgQEAgUExBVEZPUk1fVkVSU0lPTiA9IDIuMAogCiBi aW5fU0NSSVBUUyA9IHB5Z3RrLWNvZGVnZW4tJChQTEFURk9STV9WRVJTSU9OKQogCi1jb2RlZ2Vu ZGlyID0gJChwa2dkYXRhZGlyKS8kKFBMQVRGT1JNX1ZFUlNJT04pL2NvZGVnZW4KKyMgcHlndGsg c2NyaXB0cworcGtncHl0aG9uZGlyID0gJChweWV4ZWNkaXIpL2d0ay0yLjAKKworY29kZWdlbmRp ciA9ICQocGtncHl0aG9uZGlyKS9jb2RlZ2VuCiAKIGNvZGVnZW5fUFlUSE9OID0gXAogCV9faW5p dF9fLnB5IFwKZGlmZiAtdXJOcCBweWd0ay0yLjEyLjEtb3JpZy9jb2RlZ2VuL3B5Z3RrLWNvZGVn ZW4tMi4wLmluIHB5Z3RrLTIuMTIuMS1uZXcvY29kZWdlbi9weWd0ay1jb2RlZ2VuLTIuMC5pbgot LS0gcHlndGstMi4xMi4xLW9yaWcvY29kZWdlbi9weWd0ay1jb2RlZ2VuLTIuMC5pbgkyMDA4LTA0 LTE0IDE2OjE1OjA4LjAwMDAwMDAwMCArMDUzMAorKysgcHlndGstMi4xMi4xLW5ldy9jb2RlZ2Vu L3B5Z3RrLWNvZGVnZW4tMi4wLmluCTIwMDgtMDQtMTQgMTY6NDg6NDcuMDAwMDAwMDAwICswNTMw CkBAIC0xLDkgKzEsOSBAQAogIyEvYmluL3NoCiAKIHByZWZpeD1AcHJlZml4QAotZGF0YXJvb3Rk aXI9QGRhdGFyb290ZGlyQAotZGF0YWRpcj1AZGF0YWRpckAKLWNvZGVnZW5kaXI9JHtkYXRhZGly fS9weWd0ay8yLjAvY29kZWdlbgorZXhlY19wcmVmaXg9QGV4ZWNfcHJlZml4QAorcHlleGVjZGly PUBweWV4ZWNkaXJACitjb2RlZ2VuZGlyPSR7cHlleGVjZGlyfS9ndGstMi4wL2NvZGVnZW4KIAog UFlUSE9OUEFUSD0kY29kZWdlbmRpcgogZXhwb3J0IFBZVEhPTlBBVEgKZGlmZiAtdXJOcCBweWd0 ay0yLjEyLjEtb3JpZy9weWd0ay0yLjAucGMuaW4gcHlndGstMi4xMi4xLW5ldy9weWd0ay0yLjAu cGMuaW4KLS0tIHB5Z3RrLTIuMTIuMS1vcmlnL3B5Z3RrLTIuMC5wYy5pbgkyMDA4LTA0LTE0IDE2 OjE1OjA4LjAwMDAwMDAwMCArMDUzMAorKysgcHlndGstMi4xMi4xLW5ldy9weWd0ay0yLjAucGMu aW4JMjAwOC0wNC0xNCAxOTo0MDoxOC4wMDAwMDAwMDAgKzA1MzAKQEAgLTEyLDcgKzEyLDcgQEAg cHlndGtpbmNsdWRlZGlyPSR7aW5jbHVkZWRpcn0vcHlndGstMi4wCiAKICMgc2FtZSBoZXJlLiAg VXNlZnVsIHdoZW4gY2FsbGluZyB0aGUgY29kZSBnZW5lcmF0b3IgaW4gYWRkb24gcGFja2FnZXMu CiBkZWZzZGlyPSR7ZGF0YWRpcn0vcHlndGsvMi4wL2RlZnMKLWNvZGVnZW5kaXI9JHtkYXRhZGly fS9weWd0ay8yLjAvY29kZWdlbgorY29kZWdlbmRpcj0ke3B5ZXhlY2Rpcn0vZ3RrLTIuMC9jb2Rl Z2VuCiAKIE5hbWU6IFB5R1RLCiBEZXNjcmlwdGlvbjogUHl0aG9uIGJpbmRpbmdzIGZvciBHVEsr IGFuZCByZWxhdGVkIGxpYnJhcmllcwo= 154565 2008-05-28 10:12 0000 patch to install codegen into /usr/lib/python-2.x/site-packages patch text/plain SW5kZXg6IHB5Z3RrLTIuMC5wYy5pbgo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBweWd0ay0yLjAucGMuaW4JKHJl dmlzaW9uIDI5ODMpCisrKyBweWd0ay0yLjAucGMuaW4JKHdvcmtpbmcgY29weSkKQEAgLTEyLDcg KzEyLDcgQEAKIAogIyBzYW1lIGhlcmUuICBVc2VmdWwgd2hlbiBjYWxsaW5nIHRoZSBjb2RlIGdl bmVyYXRvciBpbiBhZGRvbiBwYWNrYWdlcy4KIGRlZnNkaXI9JHtkYXRhZGlyfS9weWd0ay8yLjAv ZGVmcwotY29kZWdlbmRpcj0ke2RhdGFkaXJ9L3B5Z3RrLzIuMC9jb2RlZ2VuCitjb2RlZ2VuZGly PSR7cHlleGVjZGlyfS9ndGstMi4wL2NvZGVnZW4KIAogTmFtZTogUHlHVEsKIERlc2NyaXB0aW9u OiBQeXRob24gYmluZGluZ3MgZm9yIEdUSysgYW5kIHJlbGF0ZWQgbGlicmFyaWVzCkluZGV4OiBj b2RlZ2VuL3B5Z3RrLWNvZGVnZW4tMi4wLmluCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIGNvZGVnZW4vcHlndGst Y29kZWdlbi0yLjAuaW4JKHJldmlzaW9uIDI5ODMpCisrKyBjb2RlZ2VuL3B5Z3RrLWNvZGVnZW4t Mi4wLmluCSh3b3JraW5nIGNvcHkpCkBAIC0zLDcgKzMsNyBAQAogcHJlZml4PUBwcmVmaXhACiBk YXRhcm9vdGRpcj1AZGF0YXJvb3RkaXJACiBkYXRhZGlyPUBkYXRhZGlyQAotY29kZWdlbmRpcj0k e2RhdGFkaXJ9L3B5Z3RrLzIuMC9jb2RlZ2VuCitjb2RlZ2VuZGlyPUBweWV4ZWNkaXJAL2d0ay0y LjAvY29kZWdlbgogCiBQWVRIT05QQVRIPSRjb2RlZ2VuZGlyCiBleHBvcnQgUFlUSE9OUEFUSApJ bmRleDogY29kZWdlbi9NYWtlZmlsZS5hbQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBjb2RlZ2VuL01ha2VmaWxl LmFtCShyZXZpc2lvbiAyOTgzKQorKysgY29kZWdlbi9NYWtlZmlsZS5hbQkod29ya2luZyBjb3B5 KQpAQCAtMiw3ICsyLDcgQEAKIAogYmluX1NDUklQVFMgPSBweWd0ay1jb2RlZ2VuLSQoUExBVEZP Uk1fVkVSU0lPTikKIAotY29kZWdlbmRpciA9ICQocGtnZGF0YWRpcikvJChQTEFURk9STV9WRVJT SU9OKS9jb2RlZ2VuCitjb2RlZ2VuZGlyID0gJChweWV4ZWNkaXIpL2d0ay0yLjAvY29kZWdlbgog CiBjb2RlZ2VuX1BZVEhPTiA9IFwKIAlfX2luaXRfXy5weSBcCg==