Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 229687 - sci-chemistry/pymol version bump to 1.1
Summary: sci-chemistry/pymol version bump to 1.1
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Gentoo Chemistry-Related Packages
URL: http://pymol.sourceforge.net/
Whiteboard:
Keywords: EBUILD
Depends on:
Blocks:
 
Reported: 2008-06-27 08:46 UTC by Justin Lecher (RETIRED)
Modified: 2008-08-13 05:44 UTC (History)
1 user (show)

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


Attachments
diff to latest in maintree (clipboard.txt,5.34 KB, patch)
2008-06-27 08:50 UTC, Justin Lecher (RETIRED)
Details | Diff
pymol-1.1-r1.ebuild (pymol-1.1-r1.ebuild,3.33 KB, text/plain)
2008-06-27 08:50 UTC, Justin Lecher (RETIRED)
Details
apbs-070604.patch (apbs-070604.patch,86.16 KB, text/plain)
2008-06-27 08:52 UTC, Justin Lecher (RETIRED)
Details
nosplash-gentoo.patch (nosplash-gentoo.patch,467 bytes, text/plain)
2008-06-27 08:54 UTC, Justin Lecher (RETIRED)
Details
pymol-1.1-r1-data-path.patch (pymol-1.1-r1-data-path.patch,2.52 KB, text/plain)
2008-06-27 08:54 UTC, Justin Lecher (RETIRED)
Details
pymol-1.1-r1-shaders.patch (pymol-1.1-r1-shaders.patch,524 bytes, text/plain)
2008-06-27 08:55 UTC, Justin Lecher (RETIRED)
Details
pymol-1.1-r1.ebuild (pymol-1.1-r1.ebuild,3.04 KB, text/plain)
2008-06-27 20:12 UTC, Donnie Berkholz (RETIRED)
Details
diff to Donnies Version (clipboard.txt,766 bytes, text/plain)
2008-06-28 09:06 UTC, Justin Lecher (RETIRED)
Details
branch from upstream (pymol-1.1-r1.ebuild.diff,1.59 KB, patch)
2008-07-15 11:55 UTC, Justin Lecher (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Justin Lecher (RETIRED) gentoo-dev 2008-06-27 08:46:59 UTC
As the news show on the pymol homepage, version 1.1 is released. 
With this ebuild a couple of bugs are fixed.

Reproducible: Always

Steps to Reproduce:
Comment 1 Justin Lecher (RETIRED) gentoo-dev 2008-06-27 08:50:29 UTC
Created attachment 158587 [details, diff]
diff to latest in maintree
Comment 2 Justin Lecher (RETIRED) gentoo-dev 2008-06-27 08:50:59 UTC
Created attachment 158589 [details]
pymol-1.1-r1.ebuild
Comment 3 Justin Lecher (RETIRED) gentoo-dev 2008-06-27 08:52:40 UTC
Created attachment 158591 [details]
apbs-070604.patch

Update to the latest version of apbs plugin
Comment 4 Justin Lecher (RETIRED) gentoo-dev 2008-06-27 08:54:11 UTC
Created attachment 158593 [details]
nosplash-gentoo.patch

updated version 
fixes things mentioned in #210910
Comment 5 Justin Lecher (RETIRED) gentoo-dev 2008-06-27 08:54:55 UTC
Created attachment 158595 [details]
pymol-1.1-r1-data-path.patch

only two files left for patching
Comment 6 Justin Lecher (RETIRED) gentoo-dev 2008-06-27 08:55:42 UTC
Created attachment 158597 [details]
pymol-1.1-r1-shaders.patch

fixes #210910
Comment 7 Justin Lecher (RETIRED) gentoo-dev 2008-06-27 08:57:33 UTC
Fixes bugs

#210910
#213616
#214590
#215602


Don't know if this bug depends or block those.
Comment 8 Donnie Berkholz (RETIRED) gentoo-dev 2008-06-27 17:50:41 UTC
Looks pretty good. Thanks for making the indentation normal!

I made a couple changes related to the distutils stuff -- removed the postrm, added a call to distutils_pkg_postinst in your postinst, and added PYTHON_MODNAME="chempy pmg_tk pymol" to the top of the ebuild so distutils would handle that stuff right automatically. It's still building, so I'll let you know how it goes.
Comment 9 Justin Lecher (RETIRED) gentoo-dev 2008-06-27 18:10:03 UTC
 (In reply to comment #8)

> PYTHON_MODNAME="chempy pmg_tk pymol" 
Thats a good feature! Didn't know that! There is a need for better docs of the eclasses.
Comment 10 Donnie Berkholz (RETIRED) gentoo-dev 2008-06-27 20:12:12 UTC
Created attachment 158689 [details]
pymol-1.1-r1.ebuild

Looks like the ppc keyword was removed so that apbs could be pulled in? Instead, the apbs flag should be masked on that architecture. One other detail I noticed: there's a new dosym to /usr/share/data. That sounds like a bug in our data-path patch, so I removed it to aid in diagnosis.

I couldn't actually open numerous PDBs, because my CPU pegged at 100% in /usr/lib64/python2.5/site-packages/pymol/_cmd.so:ObjectMoleculeConnect. I could open some small ones, but not a few larger ones (for example PDB ID 3GRS).

Here's an updated ebuild, with the above changes remaining to be looked into.
Comment 11 Justin Lecher (RETIRED) gentoo-dev 2008-06-28 09:01:12 UTC
 (In reply to comment #10)
> Looks like the ppc keyword was removed so that apbs could be pulled in? 
I can't do testing for ppc so I drop this always. 
> One other detail I noticed:
> there's a new dosym to /usr/share/data. 
Still a relict from bug 215602.

> I couldn't actually open numerous PDBs, because my CPU pegged at 100% in
> /usr/lib64/python2.5/site-packages/pymol/_cmd.so:ObjectMoleculeConnect. I could
> open some small ones, but not a few larger ones (for example PDB ID 3GRS).
I do testing on python 2.5 on x86. Here everything is right (also for 3GRS). Upstream?
> 
> Here's an updated ebuild, with the above changes remaining to be looked into.
I did some more cosmetic changes.
Comment 12 Justin Lecher (RETIRED) gentoo-dev 2008-06-28 09:06:12 UTC
Created attachment 158733 [details]
diff to Donnies Version

Cosmtic things which I learned from my studies in /usr/portage/eclass!
Comment 13 Donnie Berkholz (RETIRED) gentoo-dev 2008-06-30 17:38:29 UTC
Looks mostly good, although that diff format is nearly unreadable. Please create diffs with 'diff -u' in the future. The only thing I saw is that the one you uploaded has DEPEND=$DEPEND (set equal to itself).
Comment 14 Justin Lecher (RETIRED) gentoo-dev 2008-06-30 17:50:10 UTC
 (In reply to comment #13)
>Please create diffs with 'diff -u' in the future. 
Sorry, thats the way I normally do, but this time.. I don't know.
> The only thing I saw is that the one you
> uploaded has DEPEND=$DEPEND (set equal to itself).
Think this was a typo.
Comment 15 Donnie Berkholz (RETIRED) gentoo-dev 2008-06-30 21:11:34 UTC
My 100% CPU issue was some ricer CFLAGS, so ignore that.

je_fro, do you want to commit this?
Comment 16 Jeffrey Gardner (RETIRED) gentoo-dev 2008-07-01 05:49:29 UTC
Yeah, I'll test and commit :)
Thanks!
Comment 17 Jeffrey Gardner (RETIRED) gentoo-dev 2008-07-06 03:00:48 UTC
In science overlay for a week of testing, then it's off to the tree :)

Thanks justin!
Comment 18 Justin Lecher (RETIRED) gentoo-dev 2008-07-06 07:41:28 UTC
Every time again, je_fro!
Comment 19 Justin Lecher (RETIRED) gentoo-dev 2008-07-15 11:55:13 UTC
Created attachment 160432 [details, diff]
branch from upstream

Hi je_fro,
upstream did branch the code, which include two fixes. I updated the ESVN_URI. Perhaps you like two include it again.

Thanks,
justin
Comment 20 hodak 2008-08-12 09:31:18 UTC
Please add this to the portage tree, the week of testing should be over now.

Not seeing this in th tree, I went into trouble of building my own ebuild and update all related files, I have only now realized that it was wasted effort. Please add soon. 

This bug should not have been closed, since it never made it all the way into the official tree. 
Comment 21 Donnie Berkholz (RETIRED) gentoo-dev 2008-08-13 03:56:03 UTC
... yet another reason why a "third level" of testing sucks. (overlay, ~arch, stable)
Comment 22 Jeffrey Gardner (RETIRED) gentoo-dev 2008-08-13 05:44:49 UTC
mmkay, I'm sold... give me a couple days I'll commit it.
slammed at work atm...

Sorry for the hassle!
je_fro