|May I suggest to add access to the oid of an inserted
|record, by a small patch to libpq++? At least until the
|feature that will allow dumped oid's to be re-loaded into
|a database becomes available, I need access to the oids
|of newly created records... To this end, I have written a
|three-line wrapper for the PQoidStatus function in libpq and
|named this wrapper OidStatus() (I'd appreciate suggestions for
|a name that would better fit into the general naming scheme).
|
|Regards,
|
|Ernst
|

Here is a patch for Versions 1 and 2 that corrects the following makefile
bugs:

  1) linker flags LDFLAGS are missing on some of the links.

  2) libpq.a is not listed as a dependency for the src/bin programs
     that depend on it.

--
Bryan Henderson                                    Phone 408-227-6803
San Jose, California

Here's a patch for Versions 1 and 2 that fixes the following bug:

When you try to do any UPDATE of the catalog class pg_class, such as
to change ownership of a class, the backend crashes.

This is really two serial bugs: 1) there is a hardcoded copy of the
schema of pg_class in the postgres program, and it doesn't match the
actual class that initdb creates in the database; 2) Parts of postgres
determine whether to pass an attribute value by value or by reference
based on the attbyval attribute of the attribute in class
pg_attribute.  Other parts of postgres have it hardcoded.  For the
relacl[] attribute in class pg_class, attbyval does not match the
hardcoded expectation.

The fix is to correct the hardcoded schema for pg_attribute and to
change the fetchatt macro so it ignores attbyval for all variable
length attributes.  The fix also adds a bunch of logic documentation and
extends genbki.sh so it allows source files to contain such documentation.

--
Bryan Henderson                                    Phone 408-227-6803
San Jose, California

change gawk to awk ... creeped in from a previous patch..

No wonder the Linux version kept screwing up...err() was fixed in
the wrong file...

Pointed out by: Philip Plane <P.J.Plane@massey.ac.nz>

i
Improvements to Makefiles to compensate for errors in compiling

From: bryanh@giraffe.netgate.net (Bryan Henderson)

i
Improvements to Makefiles to compensate for errors in compiling

From: bryanh@giraffe.netgate.net (Bryan Henderson)

Added a comment aruond the HISTORY_LIB/INC defines

Furthre cleanup of -lreadline/-lhistory

Fix a bug with using READLINE that prevents psql from compiling

BSD44_derived required -ltermcap as well...

Change default of HISTORY_LIB so that it is disabled by default

FreeBSD has libreadline,but not libhistory...

Change Makefile so that it can be commented out in Makefile.global

Missed an else from Bryan's patch...

Turn *off* TCL by default

Make the doc directory a conditional...only try to "compile" if it
actually exists...

The err() bug creeped back in again...

Pointed out by:  brianc@saintjoe.edu

Added a SVR4 port

---

below my signature, there are a coupls of diffs and files in a shell
archive, which were needed to build postgres95 1.02 on Siemens Nixdorfs
MIPS based SINIX systems. Except for the compiler switches "-W0" and
"-LD-Blargedynsym" these diffs should also apply for other SVR4 based
systems. The changes in "Makefile.global" and "genbki.sh" can probably
be ignored (I needed gawk, to make the script run).

There is one bugfix thou. In "src/backend/parser/sysfunc.c" the
function in this file didn't honor the EUROPEAN_DATES ifdef.

---

Submitted by:  Frank Ridderbusch <ridderbusch.pad@sni.de>

Here's a minor fix that fixes a casting problem:

-Kurt

More run-time checking errors:

-Kurt

Fixes:

Here's a couple more small fixes that I've made to make my runtime
checker happy with the code.  More along the lines of those that
I sent in the past, ie, a pointer to an array != the name of
an array.  The last patch is that I mailed about yesterday -- I got
two replies of "do it", so it's done.  As far as I can tell, however,
the function in question is never called by pg95, so either way
it can't hurt...

From: "Kurt J. Lidl" <lidl@va.pubnix.com>

Fixes for PQsetdb():

When you connect to a database with PQsetdb, as with psql, depending on
how your uninitialized variables are set, you can get a failure with a
"There is no connection to the backend" message.

The fix is to move a call to PQexec() from inside connectDB() to
PQsetdb() after connectDB() returns to PQsetdb().  That way a connection
doesn't have to be already established in order to establish it!

From:  bryanh@giraffe.netgate.net (Bryan Henderson)

|From: Dan McGuirk <mcguirk@indirect.com>
|
|fixed the spelling of 'nonexistent' in a few places...
|

|This patch fixes a backend crash that happens sometimes when you try to
|join on a field that contains NULL in some rows.  Postgres tries to
|compute a hash value of the field you're joining on, but when the field
|is NULL, the pointer it thinks is pointing to the data is really just
|pointing to random memory.  This forces the hash value of NULL to be 0.
|
|It seems that nothing matches NULL on joins, even other NULL's (with or
|without this patch).  Is that what's supposed to happen?
|

Submitted by: Dan McGuirk <mcguirk@indirect.com>

fixed err() -> err_out() pointed out by David Bennett...

Okay...*last* commit, now to create a release...

README file for regresssion tests from Dr. George

Update to expected.input from Dr. George

Fixes:

make TCL conditinal compilation work

Submitted by: Dr_George_D_Detlefsen <drgeorge@madmax.ilt.com>

Fixes:

Originally, I thought the problem was caused by a function that gets
called as a normal function where we want to return a value, and as a
signal handler where we need to have it accept a parameter (the signal
number) and it returns nothing, I was going to case the function name in
the signal call as (void (*)(int)).

Looking at all the source, it turns out this function only gets used as
a signal handler, so I set an int parameter and return void.

I have removed the Linux defines because they are not needed.  BSD let
this sloppiness slide.  Linux gave a compile error.

Submitted by: Bruce Momjian <maillist@candle.pha.pa.us>

Change the TEMPDIR to be obj instead of /tmp

Suggested by: Michael Babcock <michael@kanji.com>

An actual README file for the regression tests

Submitted by: Dr. George

A newer expected.input file for the regression tests

Submitted by: Dr. Geoge

Move port includes from bin/pg_dump/Makefile to Makefile.global

Standardize locations of TCL related files

Submitted by: Dr_George_D_Detlefsen <drgeorge@madmax.ilt.com>

This prevent gcc from complaining about casting a short to a char * and
fixes another complaint.

More fixes from Bruce...

More patches for BSDi from Bruce

Row count patch from Bruce

Fix for <machine.h> bug

Submitted by: Bruce

Re-arrange KRBVERS postition

Suggested by: Daniel Kalchev <daniel@digsys.bg>

A "lost in the archives" patch from Chris Dunlop <chris@atlas.onthe.net.au>

Everyone seems to suggestion this is something that should be there...

#include <sys/termios.h>

Submitted by: Dr. George

Moved src/extend to contrib

already exists in the contrib directory

Added <sys/termios.h> for i386_solaris port

Submitted by: Dr_George_D_Detlefsen <drgeorge@ilt.com>

Fixes:

In postgres95/src/backend/nodes/readfuncs, lines 1188 and 1189,
local_node->relname is taken to point to a NameType, while its
defined as a pointer to char. Both the casting to Name and the
call of namestrcpy should, IMHO, be changed appropriately (first
patch).

As far as I could see from the Linux signal header file,
a signal handler is declared as

        typedef void (*__sighandler_t)(int);

Few changes to postgres95/src/backend/storage/lmgr/proc.c seem
appropriate to comply with this.

Finally, postgres95/src/bin/pg_version/pg_version.c defines
a function GetDataHome (by default, returning an integer)
and returns NULL in the function, which isn't an integer...

Submitted by:  ernst.molitor@uni-bonn.de

Fixes:

        updates the psql.1 manual page for \ options
        add row count and ties it to the header option
        updated manual pages and comment for above change
        got \? to display in one screen-full (almost, \? scrolls off top)
        moved \r to \E, and \z to \r (for historical reasons with monitor)
        small code alignment cleanup

Submitted by: Bruce Momjian <maillist@candle.pha.pa.us>

Style didn't appeal to some...:)

Submitted by:  Bruce Momjian <maillist@candle.pha.pa.us>

Fixes: Floating point exception in psql
Submitted by: Dan McGuirk <mcguirk@indirect.com>

Fixes:

>   INDEXED searches in some cases DO NOT WORK.
>   Although simple search expressions (i.e. with a constant value on
> the right side of an operator) work, performing a join (by putting
> a field of some other table on the right side of an operator) produces
> empty output.
>   WITHOUT indices, everything works fine.
>

submitted by: "Vadim B. Mikheev" <root@ais.sable.krasnoyarsk.su>

More cleanups by "Kurt J. Lidl" <lidl@va.pubnix.com>

More cleandups from: "Kurt J. Lidl" <lidl@va.pubnix.com>

The decode function in psql.c doesn't return anything, so its
declaration is incorrect.

-Kurt

clean up /tmp behind itself

submitted by: bruce

Change ordering of "user modified variables" so that they are at
the top of th efile instead of scattered throughout

Turned off default behavior of compiling with -g enabled

submitted by; bruce

fix a redeclaration error

submitted by Kurt

Modifications for bsdi from Kurt(sp?)

cosmetic changes: char* x changed to char *x

submitted by: bruce

reversed bruces patch to getopt() loop about optarg

submitted by: bruce

fixes: It appears several routines use lcons to store integers rather than
pointers.

submitted by: bruce

small port fixes for sparc_solaris

submitted by: jason

bugfix: if NOREADLINE was not defined and input was not from a tty, then
        the getopt() was not executed.

Various fixes required for psql

Submitted by: Bruce

and now, the long awaited PAGER patches from Bruce...

Submitted by: Bruce Momjian <maillist@candle.pha.pa.us>

Further modifications for fixing createuser bug...

Submitted by: Rick Weldon <rick@wisetech.com>

This is a patch to pg_dump which fixes varchar and char printing in the
case where the attribute length is variable (stored as -1).  Previously,
you'd get output that looked like:

CREATE TABLE foo (bar varchar(-1));

Monitor and psql don't like this at all :).  Here is a fix:

Submitted by: Adam Sussman <myddryn@vidya.com>

My patch to fe-connect.c introduced a new bug which is triggered only, if
Kerberos is being used (attempt to free static memory).
The error was caused by a confusing doublespeak of fe_getauthname():
Returns a pointer to static memory, if you authenticate via Kerberos,
a pointer to dynamic memory otherwise.

Submitted by: Erich Stamberger <eberger@gewi.kfunigraz.ac.at>

Fixed these script sonce and for all?

Submitted by: Rick Weldon <rick@wisetech.com>

corrects the output for a 'SELECT DISTINCT ON ...' at line 2900

submitted by: Dr_George_D_Detlefsen <drgeorge@ilt.com>

Fix where 'make clean' wipes out everything if no obj directory
exists...

Submitted by: Bruce Momjian <maillist@candle.pha.pa.us>

Missed a small patch fro mBruce for BSDi

should always take mail headers out of Makefile files befor ecommitting :)

Minor bug fix

regression Makefile, rev 2

Submitted by: Dr. George

Add in port to DG/UX

Submitted by: "Brian E. Gallew" <geek+@cmu.edu>

Fix applied for NESTLOOP bug

submitted by: vadim@sable.krasnoyarsk.su (Vadim B. Mikheev)

fixed usuage of -ltermcap for psql

submitted by: Bruce Momjian <maillist@candle.pha.pa.us>

improvements to regression testing

submitted by: dr. george

modified i386_solaris port

submitted by: dr. george

multiple define of 'union semun' removed

submitted by: dr. george

moved the FLEX stuf from Makefile.global to here

removed tas.s refernce

Dr George suggested removing Makefile.custom altogether, to reduce
any confusion...so its gone...

Switched from using monitor to using psql

Submitted by: Rick Weldon <rick@wisetech.com>

Removed monitor from compile/install (will eventually be removed
completely)

Large re-write/enhancement. In pg-101 Jolly only included a smaller part
of my (proff) patch. This is the rest of it, with a few, mainly aesthetic
changes. I've removed a lot of redundency from the original code,
added support for the new PQprint() routines in libpq, expanded tables,
and a few generally nifty ways of massaging data in and out of the
backend. Still needs some good stress testing.

PQprint() routines. The older code is still there for historical
compatibility. There isn't much difference here against my previous
PQprint() code, except that you can add optional arguments to the
<table args> in html.

Ack, -I$(srcdir)/backend *is* required...

comment out USE_TCL...having USE_TCL=true seems to imply that you
have to have TCL to use postgres95, which you don't

there are no includes files in backend, there should be, so why look
there?

Fix compiler not finding libpgtcl.h header file

Added an empty Makefile.custom file...

Ack, missed two files from the merge...looks like a .11 is goin gto have
to go out after all :(

iBrought in a fix for backend crashes

Submitted by: "Vadim B. Mikheev" <vadim@sable.krasnoyarsk.su>

comment out inclue of Makefile.custom, as i tbreaks compiling the
bin directory

Last changes to src from Dr. George *wipes brow*

Alot of "changes" from Dr. George's source tree...

Most of the changes in here look to b epurely cosmetic, and don't
affect anything...

...and some stuff is completely questionable...in that I may have reversed
some of the stuf fwe already had :(

More Merge's from Dr' sourec tree

modifications to regressoin tests

More merge's from Dr. George's sourec tree

spelling error...and correction