Fix an Inccorect Error Message...

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

applied kurt's patch to fix unlikely, but potential, string underflow
problem in psql

Fixes:

Also, I think that an extra source of noise in the diff of regress.out and
expected.out is caused by not substituting the shared library file
extension in the regression.input file (much like the paths and the
usernames are sub'ed). This seems to be fixed with the following patches
to regression.input and the Makefile... If I'm off base here, please tell!

Submitted by:  Wayde Nie <niew@phoenix.cis.mcmaster.ca>

Fixes:

I've enclosed two patches.  The first affects Solaris compilability.  The
bug stems from netdb.h (where MAXHOSTNAMELEN is defined on a stock
system).  If the user has installed the header files from BIND 4.9.x,
there will be no definition of MAXHOSTNAMELEN.  The patch will, if all
else fails, try to include <arpa/nameser.h> and set MAXHOSTNAMELEN to
MAXDNAME, which is 256 (just like MAXHOSTNAMELEN on a stock system).

The second patch adds aliases for "ISNULL" to "IS NULL" and likewise for
"NOTNULL" to "IS NOT NULL".  I have not removed the postgres specific
ISNULL and NOTNULL.  I noticed this on the TODO list, and figured it would
be easy to remove.

The full semantics are:
        [ expression IS NULL ]
        [ expression IS NOT NULL ]

--Jason

Submitted by: Jason Wright <jason@oozoo.vnet.net>

Fixes:

Previously Postgres95 wouldn't accept 'order by' clauses with fields
referred to as '<table>.<field>', e.g.:

        select t1.field1, t2.field2 from table1 t1, table2 t2
                order by t2.field2;

This syntax is required by the ODBC SQL spec.

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

Fixes:

While a normal SELECT statement can contain a GROUP BY clause, a cursor
declaration cannot. This was not the case in PG-1.0. Was there a good
reason why this was changed? Are cursors being phased out? Is there any way
to get data with just a SELECT (and without a DECLARE CURSOR ...)?

The patch below seems to fix things. If anyone can see a problem with it,
please let me know. Thanks.

Submitted by:  David Smith <dasmith@perseus.tufts.edu>

Fixes for:

Here are a few minor fixes to Postgres95.  Mostly I have added const
to some of the char pointers.  There was also a missing header file
and a place where it looks like "==" was used when "=" was meant.
I also changed some variables from Pfin and Pfout tp pfin and pfout
because the latter shadow global variables and that just seems like
an unsafe practice which I like to avoid.

Submitted by:  "D'Arcy J.M. Cain" <darcy@druid.druid.com>

Had a space in CFLAGS+= -I ..

Submitted by:  Andrew Martin <martin@biochemistry.ucl.ac.uk>

added patch from kurt that fixes memory leak (didn't free line buffer
for slash commands)

Fixes:

Someone asked me if the bpchar type could be extended to do
case-insensitive regular expression searches.

Submitted by: "Alistair G. Crooks" <azcb0@juts.ccc.amdahl.com>

Fix a bug in pg_class

submitted by: "Peter Daum" <gator@cs.tu-berlin.de>

Quick patch for compiling under BSD/OS 2.0 from Bruce

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

Put in a new FAQ

Merged in Dr. George's src/bin tree (possibly broke monitor/psql in process)

reversed julian's patch back in again...othere things cause monitor
to not compile now...

Finished merging in src/backend from Dr. George's source tree

More of Dr. George's changes...
        - src/backend/catalog/*
                - no changes
        - src/backend/executor/*
                - change how nodeHash.c handles running out of memory
        - src/backend/optimizer/*
                - mostly cosmetic changes

More merges from Dr. George's tree...
        - src/backend/tcop/*
                - cosmetic changes to OPENLINK patches
        - src/backend/storage/*
                - more changes, mostly cosmetic
        - src/backend/ports/*
                - merge in patches for aix and i386_solaris

- merging in Dr. George's tree with ours
        - src/backend/access
                - no changes
        - src/backend/utils
                - mostly cosmetic changes
                - ESCAPE_PATCH Added
        - src/Makefile.global changes merged

Brought in David Bennett's (dave@bensoft.com) changes to pg_dump

Backed out the changes to monitor.c that seem to be affecting the
ability for createuser to actually create one...

- added LD_ADD+=-ltermcap to postgres.mk.bsdi_2_1

- submitted by: Bruce Momjian (root@candle.pha.pa.us)

quick fix to src/bin/Makefile.global so that compiling doesn't look for a
libpq that is already installed...

submitted by: Bruce Momjian (root@candle.pha.pa.us)

Fix for:

>
> We did some testing and found that if we name the table 'Inv' with
> anything appended to it, the table does not appear in the '\d' table list.
> It appears to be the capital I as a table named 'invItemsL' is created
> and displayed properly.
>

- submitted by: Jason Wright <jason@shiloh.vnet.net>

Brought in NEOSOFT's port to i386_solaris

Submitted by: Randy Kunkee <kunkee@Starbase.NeoSoft.COM>

Fixes:

Hash indices for some data types don't work, for example for time and date.

- submitted by: Massimo Dal Zotto <dz@cs.unitn.it>