Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 76204 - games-kids/pytraffic revised patch
Summary: games-kids/pytraffic revised patch
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Games (show other bugs)
Hardware: All All
: High minor (vote)
Assignee: Gentoo Games
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-12-30 21:14 UTC by Yaakov S
Modified: 2005-01-01 23:10 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
revised patch (1.2-gentoo-dirs.patch,1.69 KB, patch)
2004-12-30 21:15 UTC, Yaakov S
Details | Diff
revised ebuild (pytraffic-1.2.1.ebuild,1.04 KB, text/plain)
2004-12-30 21:40 UTC, Yaakov S
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Yaakov S 2004-12-30 21:14:22 UTC
The patch to pytraffic so that it should install in the games hierarchy missed something in the .desktop file.  I'm attaching a revised patch to fix this.
Comment 1 Yaakov S 2004-12-30 21:15:22 UTC
Created attachment 47261 [details, diff]
revised patch
Comment 2 Yaakov S 2004-12-30 21:40:44 UTC
Created attachment 47262 [details]
revised ebuild 

Now that I look at the ebuild, there's a few revisions to make.  One, bytecode
isn't compiled (since the python modules aren't installed into
/usr/lib/python*.  Also, there's a typo DDOCS instead of DOCS.

I tried out the attached ebuild, and now it properly postinstalls both for
games and python bytecode compiling.
Comment 3 SpanKY gentoo-dev 2004-12-30 22:13:05 UTC
merged with cvs, thanks
Comment 4 Yaakov S 2005-01-01 19:37:37 UTC
Thank you.  I noticed however that pkg_postrm from the new ebuild wasn't merged.  IFAIK without it the bytecode created by pkg_postinst will be left behind if the packages is unmerged.  If it doesn't belong, could you please explain why so that I know for the future?
Comment 5 SpanKY gentoo-dev 2005-01-01 23:10:42 UTC
added this to the ebuild:
pkg_postrm() {
    distutils_pkg_postrm
}