[Biopython-dev] [Biopython - Bug #2619] Bio.PDB.MMCIFParser component MMCIFlex commented out in setup.py

[redmine at redmine.open-bio.org]

Issue #2619 has been updated by Peter Cock.


Probably it would be best to leave PLY as an external dependency. If this works nicely we can define it in the setup.py as a requirement for automatic installation via easy_install or pip. Note ply is in pypi http://pypi.python.org/pypi/ply/ and supports Python 2 and 3 already.

If using PLY turns out to be non-trivial, then a pure Python CIF parser might be advantageous.

----------------------------------------
Bug #2619: Bio.PDB.MMCIFParser component MMCIFlex commented out in setup.py
https://redmine.open-bio.org/issues/2619

Author: Chris Oldfield
Status: New
Priority: Normal
Assignee: Biopython Dev Mailing List
Category: Main Distribution
Target version: 1.48
URL: 


MMCIFParser is a documented feature of Bio.PDB, but it is broken by default because the MMCIFlex build is commented out in the distribution setup.py.  According to  

http://osdir.com/ml/python.bio.devel/2006-02/msg00038.html

this is because it doesn't compile on Windows.  Though the function is documented, the changes need to enable are not, so this seems like an installation bug to me.

The fix on linux is to uncomment setup.py lines 486 on.  A general work around might be to condition the compile on the os.sys.platform variable. I'd offer a diff, but I'm new to biopython and python in general, so please forgive my ignorance.

Source install of version 1.48, gentoo linux 2008, x86_64.


-- 
You have received this notification because you have either subscribed to it, or are involved in it.
To change your notification preferences, please click here and login: http://redmine.open-bio.org