Re: PyGreSQL bug - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: PyGreSQL bug
Date
Msg-id 200204231731.g3NHVKE19117@candle.pha.pa.us
Whole thread Raw
List pgsql-hackers
Discription of patch for python.

---------------------------------------------------------------------------


Andrew Johnson wrote:
> On Wed, Apr 17, 2002 at 10:12:10PM -0400, Bruce Momjian wrote:
> > 
> > No one has replied, so I worked up a patch that I will apply in a few
> > days.  Let me know if you don't like it.
> 
> Hmm... problem I have with that is it changes the interface, and might
> break existing code.  Only other option is to use ordered, rather than named
> parameters in the _pg.connect call.
> 
>  
> > Index: src/interfaces/python/pgdb.py
> > ===================================================================
> > RCS file: /cvsroot/pgsql/src/interfaces/python/pgdb.py,v
> > retrieving revision 1.10
> > diff -c -r1.10 pgdb.py
> > *** src/interfaces/python/pgdb.py    19 Mar 2002 02:47:57 -0000    1.10
> > --- src/interfaces/python/pgdb.py    18 Apr 2002 02:10:20 -0000
> > ***************
> > *** 337,343 ****
> >   ### module interface
> >   
> >   # connects to a database
> > ! def connect(dsn = None, user = None, password = None, host = None, database = None):
> >       # first get params from DSN
> >       dbport = -1
> >       dbhost = ""
> > --- 337,343 ----
> >   ### module interface
> >   
> >   # connects to a database
> > ! def connect(dsn = None, user = None, password = None, xhost = None, database = None):
> >       # first get params from DSN
> >       dbport = -1
> >       dbhost = ""
> > ***************
> > *** 364,372 ****
> >           dbpasswd = password
> >       if database != None:
> >           dbbase = database
> > !     if host != None:
> >           try:
> > !             params = string.split(host, ":")
> >               dbhost = params[0]
> >               dbport = int(params[1])
> >           except:
> > --- 364,372 ----
> >           dbpasswd = password
> >       if database != None:
> >           dbbase = database
> > !     if xhost != None:
> >           try:
> > !             params = string.split(xhost, ":")
> >               dbhost = params[0]
> >               dbport = int(params[1])
> >           except:
> 
> 

--  Bruce Momjian                        |  http://candle.pha.pa.us pgman@candle.pha.pa.us               |  (610)
853-3000+  If your life is a hard drive,     |  830 Blythe Avenue +  Christ can be your backup.        |  Drexel Hill,
Pennsylvania19026
 


pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: new food for the contrib/ directory
Next
From: Bruce Momjian
Date:
Subject: Re: My talk at Linuxtag