Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 381587

Summary: gnome-extra/gnome-dvb-daemon-0.2.2 uses package build system to byte-compile Python
Product: Gentoo Linux Reporter: Chris Mayo <aklhfex>
Component: New packagesAssignee: Matt Turner <mattst88>
Status: RESOLVED FIXED    
Severity: normal CC: gnome
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: ebuild patch

Description Chris Mayo 2011-09-02 16:59:11 UTC
Created attachment 285367 [details]
ebuild patch

Attached patch disables Python byte-compilation supplied with gnome-dvb-daemon and gets portage to do it.
Comment 1 Matt Turner gentoo-dev 2011-09-03 01:59:27 UTC
Thanks for the patch.

The python stuff is foreign to me. Maybe you could explain?
Comment 2 Chris Mayo 2011-09-03 12:51:46 UTC
It's considered a good idea not to let the package compile its own Python:
http://www.gentoo.org/proj/en/Python/developersguide.xml#doc_chap2_sect8

to ensure a platform consistent way of generating the byte code. One benefit I like is that if you do a:
 qlist gnome-extra/gnome-dvb-daemon
you only see the .py files and not all the compiled versions too - even though they are still managed by portage.
Comment 3 Matt Turner gentoo-dev 2011-09-06 18:40:31 UTC
Arfrever said the patch looks good.

Thanks a lot! :)
Comment 4 Chris Mayo 2011-09-07 21:52:49 UTC
Changes actually applied in the tree don't work for me.
The:
   ln -s $(type -P true) py-compile

fails because py-compile already exists. Other ebuilds in the tree that do this put this line before:
   mv py-compile py-compile.orig.

Others have ln -sf which seems to work OK for me.
Comment 5 Matt Turner gentoo-dev 2011-09-07 22:00:06 UTC
Ah! Crap!

Thanks for pointing that out, that was totally my fault.

Fixed.