[PATCHES] trivial cleanup: ExecProcAppend

2004-09-23 Thread Neil Conway
ExecProcAppend() wasn't called ExecAppend() because the latter name was formerly used in execMain. Since that is no longer the case, this patch renames ExecProcAppend() to ExecAppend() for the sake of consistency. Barring any objection, I intend to apply this to HEAD within 24 hours. -Neil Index

Re: [PATCHES] Xid in log_line_prefix

2004-09-23 Thread Bruce Momjian
Neil Conway wrote: > On Sat, 2004-09-11 at 13:44, Alvaro Herrera wrote: > > This trivial patch adds the transaction Id to the list of escapes > > available to log_line_prefix. > > Patch applied to HEAD. Thanks. > > Bruce, this patch was/is on the pending patches list. I don't know of a > way to r

Re: [PATCHES] doc patch for ssl in server

2004-09-23 Thread Bruce Momjian
Patch applied. Thanks. Your documentation changes can be viewed in five minutes using links at the bottom of the developer's page, http://developer.postgresql.org/index.php. --- Dominic Mitchell wrote: > This patch attem

Re: [PATCHES] SSL Error Code logging.

2004-09-23 Thread Bruce Momjian
Patch applied. Thanks. --- Dominic Mitchell wrote: > This patch logs the error code in the default case, so that the user > stands a chance of looking it up. "Unrecognised error" is always > disheartening. :-) > > -Dom

Re: [PATCHES] psql: rollback only last query on error

2004-09-23 Thread Michael Paesold
Alvaro Herrera wrote: > On Tue, Sep 21, 2004 at 02:30:17PM +0200, Michael Paesold wrote: > > http://archives.postgresql.org/pgsql-hackers/2004-09/msg00576.php > > One potential problem I see with the patch is that it opens lots of > savepoints but does not release any. In this mode, given autocom

Re: [PATCHES] psql: rollback only last query on error

2004-09-23 Thread Tom Lane
"Michael Paesold" <[EMAIL PROTECTED]> writes: > Alvaro Herrera wrote: >> One potential problem I see with the patch is that it opens lots of >> savepoints but does not release any. > Well, given that EXCEPTION blocks in Pl/pgSQL and the like also never > release savepoints That statement is flat

Re: [PATCHES] doc patch for ssl in server

2004-09-23 Thread Tom Lane
Dominic Mitchell <[EMAIL PROTECTED]> writes: > + If verification of client certificates is required, place the > + certificates of the CA you wish to check for in > + the file root.crt in the data directory. When > + present, a client certificate will be requested from the client > + mak

Re: [PATCHES] doc patch for ssl in server

2004-09-23 Thread Dominic Mitchell
On Thu, Sep 23, 2004 at 04:37:52PM -0400, Tom Lane wrote: > Dominic Mitchell <[EMAIL PROTECTED]> writes: > > + If verification of client certificates is required, place the > > + certificates of the CA you wish to check for in > > + the file root.crt in the data directory. When > > + prese

Re: [PATCHES] doc patch for ssl in server

2004-09-23 Thread Tom Lane
[EMAIL PROTECTED] (Dominic Mitchell) writes: > On Thu, Sep 23, 2004 at 04:37:52PM -0400, Tom Lane wrote: >> That last statement is not actually correct, is it? AFAICS we do tell >> SSL to enforce certificates if we find a valid root.crt file. > According to the docs[1], you also need > SSL_VERIFY

Re: [PATCHES] ALTER TABLE .. OWNER TO and sequences

2004-09-23 Thread Tom Lane
Alvaro Herrera <[EMAIL PROTECTED]> writes: > Here is a patch regarding ALTER TABLE ... OWNER and the sequences that > were created by a SERIAL column. Applied with some changes. I tightened up the checks on the pg_depend entries, just for paranoia's sake. (At the moment I don't think there could

Re: [PATCHES] trivial cleanup: ExecProcAppend

2004-09-23 Thread Neil Conway
On Thu, 2004-09-23 at 17:42, Neil Conway wrote: > ExecProcAppend() wasn't called ExecAppend() because the latter name was > formerly used in execMain. Since that is no longer the case, this patch > renames ExecProcAppend() to ExecAppend() for the sake of consistency. Patch applied. -Neil -

[PATCHES] Support LDFLAGS_SL on most ports

2004-09-23 Thread Oliver Jowett
This patch includes LDFLAGS_SL in SHLIB_LINK on most ports (previously it was only used on AIX and BeOS), and adds support for specifying it in configure. This lets you do something like: ./configure LDFLAGS=-static-libgcc LDFLAGS_SL=-static-libgcc to produce binaries that do not depend on lib

[PATCHES] cast pid_t to int when using *printf

2004-09-23 Thread Oliver Jowett
gcc (3.2.3 on Solaris 9) warns about a couple of places where a pid_t is formatted with %d by a printf-family function. This patch explicitly casts to int to suppress the warning. -O Index: src/backend/postmaster/pgstat.c === RCS fi