[Biopython-dev] [Bug 2616] BioSQL support for Psycopg2

[bugzilla-daemon at portal.open-bio.org]

http://bugzilla.open-bio.org/show_bug.cgi?id=2616





------- Comment #8 from biopython-bugzilla at maubp.freeserve.co.uk  2008-10-16 06:18 EST -------
(In reply to comment #7)
> (In reply to comment #6)
> > 
> > In BioSQL/BioSeqDatabase.py there is a comment about building a DSN being
> > required for older releases of psycopg - do we still need this for psycopg2?
> 
> Apparently not (tests run after editing out code...).
> I assume we want to maintain support for the older psycopg driver.
> Unfortunately, I cant get the old drivers to compile on my box - they
> configure, but gcc chokes on the make... I cant see why the patch should
> effect the operation of the old driver, but it would be worth checking.

OK - but I'll leave it in rather than risk breaking the old psycopg driver.

> > > (NB: the buglet in preserving qualifier order - sorted the lists in
> > > test_BioSQL_SeqIO.py in order to pass test).
> > 
> > I suggest you check in this patch, plus the tweak to test_BioSQL_SeqIO.py
> > (conditional on the database driver) with a "TODO" comment next to it
> > about checking why the order wasn't preserved.
> 
> Unfortunately, I dont have direct CVS access - proxy hassles - I
> downloaded the cvs tarball previously. Besides I'm not a developer.
> Perhaps, someone else would check it in.

Sorry - I knew Frank had CVS access and had assumed you did too.  If you think
you'll need CVS access, send an email on the dev-list.  In the meantime, I'm
happy to check this in on your behalf.

Peter


-- 
Configure bugmail: http://bugzilla.open-bio.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.