Bugzilla Bug 22235

New version of pymol is out (0.88).  Appears you can just rename the ebuild as
before and it works...

http://www.pymol.org

Created an attachment (id=13102) [details]
pymol-0.88 ebuild

Here is an ebuild for version 0.88.
I don't know whether or not simply renaming the 0.86 ebuild works, but this new
ebuild uses the distutils support that exists in version 0.88, so it is quite a
bit different than the 0.86 ebuild.

Cheers,

Ben

Tested out the ebuild, compiles fine, but if you try to run it you get:

[/home/geders ]$ pymol
/usr/bin/python: can't open file '/var/tmp/portage/pymol-0.88/image/usr/lib/python2.2/site-packages/pymol/__init__.py'

Looks like a path problem...

Created an attachment (id=13114) [details]
'emerge pymol' output

Created an attachment (id=13120) [details]
pymol-0.88.ebuild 

Yes it was a path problem.  It should be fixed in the new attachment.  Believe
it or not I did test the original before uploading it ;-)

After looking at the code for the `dosed' command I realized two things: (1) I
had been using it wrong in the first upload, and (2) it had worked for me
because I hadn't clean out /var/tmp/portage/pymol-0.88/image prior to testing
the first ebuild.  Ooops.

On a more general note, `dosed' seems rather broken to me.  I would image that
one most frequently uses it when presented with the situation I had when
writing the pymol ebuild --- namely, you have a path like this

/var/tmp/portage/pymol-0.88/image/usr/lib/python2.2/site-packages/pymol/__init__.py


and you want to strip off the image dir portion.  Well, dosed strips out ${D}
for you, but ${D} includes a trailing slash (/).  So you get this

usr/lib/python2.2/site-packages/pymol/__init__.py

Not good.

Ben

Created an attachment (id=13364) [details]
pymol-0.88 ebuild

Small fix that causes a failure when you attempt to re-emerge the package.

s/fix/fix for a bug/

latest ebuild works for me now...

Hi guys.

Ben: Thanks for the reworked ebuild. However could you please submit ebuild and the patch[es] as a separate attachmentes (text files) and not a tar file? It is much easier to handle it this way (do not worry about digests and Manifest - I will regenerate them anyway).
Also, I noticed that you created 0.88 dir under files to keep patches. Is this really necessary just for the two of them? You can add $PV to their names, or if you want to keep the count of files down, even combine them in a single patch.
Could you please reformat the patches as well?

George

Created an attachment (id=13580) [details]
pymol 0.88 ebuild

Created an attachment (id=13581) [details]
Turns off opening picture of Warren (sorry Warren).

Created an attachment (id=13582) [details]
Get installation script to play nice with portage

George,

Done.  

Yes, I guess the ${PV} subdirectory was unnecessary; it's gone now.  Please let me know if you need anything else.

Cheers,

Ben

Thanks for quick responce.
I have processed and committed the ebuild.
Please test.

George

Ebuild in portage is working for our dept

Closing this bug as fixed.