A small patch from Andrew for the linux port in v1.09

remove KERBEROS config cruft from Makefile.global
- don't advertise something that jus tdoes'nt work

Fix the ordering of variables in the memset() call

Pointed out by: wieck@sapserv.debis.de

More btree fixes from Massimo Dal Zotto <dz@cs.unitn.it>

Fixes:

I found another bug in btree index.  Looking at the code it seems that NULL
keys are never used to build or scan a btree index (see the explain commands
in the example).  However this is not the case when a null key is retrieved
in an outer loop of a join select and used in an index scan of an inner loop.
This bug causes at least three kinds of problems:

1)  the backend crashes when it tries to compare a text string with a null.

2)  it is not possible to find tuples with null keys in a join.

3)  null is considered equal to 0 when the datum is passed by value, see
    the last query.

Somehow my personal changes got commit'd to the tree, and, therefore,
got into 1.09 :(  Fix this

Fixed a botched patch...

Pointed out by Andrew...

Another patch that was put into 2.x and not into 1.x

From Bruce...

Accidently commited Makefile.global with PGPORT set to 6543

There is a bug in the btree insert code which can cause, under very rare
conditions, the corruption of index data and possibly of shared memory data.

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

One thing we said before should go into 1.09 is the fix to the master
make file so it doesn't say "Postgres made - ready to install" when you
do a make install or make clean.

Bryan Henderson <bryanh@giraffe.netgate.net>

Forgot to comment out the USE_READLINE define...fixed now

Okay D'Arcy...found a purpose to Makefile.custom *grin*:

# If your OS requires these libraries, they should be put in
# Makefile.custom with a line similar to:
# X11_LIB+= -lsocket -lnsl

Actually, those should probably go into the ports/Makefile.inc's, but...

Set default SRCDIR/POSTGRESDIR to /usr/local/postgres95

Bring in changes so that PORTNAME is UNDEFINED by default, and error/halt
happens if it isn't changed

Disable READLINE by default

Fixes: Errors when PQexec() in backend creates temp
              relations and transaction is aborted

Submitted by: wieck@sapserv.debis.de (Jan Wieck)

Fixes: Growing backend when using nested function calls
Submitted by: wieck@sapserv.debis.de

Added in prototype for: extern void setMaxOid(FILE *fout);

The patch got applied to 2.x, not 1.x :(

Fixes:

It's bug in nodeAgg.c on lines 241, 242:

                null_array = malloc(nagg);
                for (i=0;i<nagg;i++)
                    null_array[i] = 'n';
                oneTuple = heap_formtuple(tupType, tupValue, null_array);

- your query has not only aggregates but also 'group by-ed' fields and so
null_array should contain tupType->natts elements (tupType->natts > nagg in
your case).

Patch follows and it's very simple.

VAdim

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 ]

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

This change should have no practical effect but it is the more
correct way to do this.  Theoretically you could have a NULL
pointer that isn't represented internally as all 0 bits.  This
guarantees that it convert correctly.

Submitted by: darcy@druid.com (D'Arcy J.M. Cain)

I have written some patches to the postgres lock manager which allow the
use of long term cooperative locks managed by the user applications.

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

There is a bug in aclitemout which causes a notice to be sent to the client
while the backend is trying to pfree a string not allocated with palloc.

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

Comment out EUROPEAN_DATES by default.  v2.0 moves it into include/config.h

Oops...thought we fixed the bug with the -I ../.. problem :(

fixed it here too...

Oops, wrong message with the other patch...this was the patch for the other
comment, so here is the comment for the other patch *grin*

> > You are right.  I checked the gramar and saw the ability to use the
> > parameter.  I looked at the manual pages, and saw no reference to it.  I
> > tried running it, and found vacuum does nothing when you give it a table
> > name.
> >
> > I checked a debug version of postgres, and the table name is passed to
> > vacuum() in the variable (char *vacrel).  The problem is that the vacuum
> > spans transactions, and the vacrel name gets changed to '<vacuum>',
> > which is the name of the portal that gets created in
> > vacuum.c::_vc_vacuum().  vacuum.c::_vc_init() does a
> > CommitTransactionCommand() which frees the memory allocated to vacrel.
> >
> > Should I change vacuum.c to copy the relation name to a local string
> > variable of vacuum(), or do you recommend we allocate the table name in
> > a different fashion?  You are the man who knows the most about this.
>
> static NameData VacRel;

Done.  Attached is the patch.  I have already applied it to the 2.0
tree. (Marc!)

I tested it and it works.  I also applied documentation patches to go
with it.

So now vacuum can be run for only one table if you wish.

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

> - it excludes system-relation too (relkind == 's'). (Note: Vacuum updates
pg_class
> by overwriting existing tuple for vacrel, so there are no many reasons to
vacuum pg_class).
>
> It can be done somewhere in _vc_getrels - near to checks against archive
relations
> and relations on the write-once storage managers...
>
> Excuse me - I forgot to say about this.
>

Attached is the recently posted fix for this.  Thanks.

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

Here the fix for the first assertion failure I had which killed
my postmaster 1.07.
It's really simple, the loop dealing with all sockets
can't handle more than one ready socket :-)
A simple logic error dealing with lists.
OR IS THERE ANY REASON FOR SETTING curr TO 0?

Submitted by:  Carsten Heyl <Heyl@nads.de>

Added spaces to end of QUERY lines so that createuser actually works...

Quick fix required to fix SED script in postgres.shell.mk

V1.07

change a reference to stdout to point to fout instead...

submitted by: Carsten Heyl <heyl@nads.de>

From: "Kurt J. Lidl" <lidl@va.pubnix.com>
To: pg95-dev@ki.net
Subject: [PG95-DEV] array overrun in postmaster.c

Added a fix for the dgux port

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

*** src/backend/storage/file/fd.c.orig  Thu Sep 12 17:17:21 1996
--- src/backend/storage/file/fd.c       Thu Sep 12 17:23:38 1996
***************
*** 262,268 ****
      Delete(file);

      /* save the seek position */
!     fileP->seekPos = lseek(fileP->fd, 0L, SEEK_CUR);
      Assert( fileP->seekPos != -1);

      /* if we have written to the file, sync it */
--- 262,268 ----
      Delete(file);

      /* save the seek position */
!     fileP->seekPos = (long) lseek(fileP->fd, 0L, SEEK_CUR);
      Assert( fileP->seekPos != -1);

      /* if we have written to the file, sync it */

Submitted by: Randy Terbush <randy@zyzzyva.com>

Had EUROPEAN_DATES turned on by default

Pointed out by Bruce

More declaration mis-match fixes...

Now match the declaration in fe-auth.c to the one in fe-auth.h :)

fix a conflicting declaration problem with fe_setauthsvc()...

slight fix to Makefile.inc so that gram.c/parse.h get put into
backend/obj instead of into backend

there is probably a better fix for this, but looking at the Makefile.inc,
I can't find a reason *why* it isn't putting them into obj...

POSTPORT should be 5432, not 6543

Patches to make POSTPORT changes in scripts

Originally submitted by: ernst.molitor@uni-bonn.de
resubmitted by: D'Arcy Cain

This patch stops the Postgres build from ignoring the fact that yacc
has failed to create gram.c.

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

Hey, I didn't take it out, and its in the 2.0 source tree...*sigh*

A fix for the pg_log bug

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

More sync ups...

Another mis-sync...

Somehow, we got out of sync here

Pointed out by Bryan

The following patch makes postmaster -D work.  -D specifies a different PGDATA
directory.  The code that looks for the pg_hba file doesn't use it, though,
so the postmaster uses the wrong pg_hba file.  Also, when the postmaster
looks in one directory and the user thinks it is looking in another
directory, the error messages don't give enough information to solve the
problem.  I extended the error message for this.

Submitted by: Bryan

|The patch that is applied at the end of the email makes sure that these
|conditions are always met. The patch can be applied to any version
|of Postgres95 from 1.02 to 1.05. After applying the patch, queries
|using indices on bpchar and varchar fields should (hopefully ;-) )
|always return the same tuple set regardless to the fact whether
|indices are used or not.
|

Submitted by: Gerhard Reithofer <tbr_laa@AON.AT>

Take a chance that I'm correct:

attno is used in an if statement, yet isn't defined, nor used
anywhere else (~line 945)

Fix the prototype for pg_dump's dumpClasses

Fixed some commits that were missed dealing with Bryan's recent
patches

This patch for Versions 1 and 2 corrects the following bug:

In a catalog class that has a "name" type attribute, UPDATEing of an
instance of that class may destroy all of the attributes of that
instance that are stored as or after the "name" attribute.

This is caused by the alignment value of the "name" type being set to
"double" in Class pg_type, but "integer" in Class pg_attribute.
Postgres constructs a tuple using double alignment, but interprets it
using integer alignment.

The fix is to change the alignment to integer in pg_type.

Note that this corrects the problem for new Postgres systems.  Existing
databases already contain the error and it can't easily be repaired because
this very bug prevents updating the class that contains it.

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

The patch does several things:

        It adds a WITH OIDS option to the copy command, which allows
dumping and loading of oids.

        If a copy command tried to load in an oid that is greater than
its current system max oid, the system max oid is incremented.  No
checking is done to see if other backends are running and have cached
oids.

        pg_dump as its first step when using the -o (oid) option, will
copy in a dummy row to set the system max oid value so as rows are
loaded in, they are certain to be lower than the system oid.

        pg_dump now creates indexes at the end to speed loading

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

Here's the fix for the problem that Evan Champion reported today.

This presumably corrects a problem of initdb failing on systems that have
an awk that is sensitive to this.

Submitted by:  bryanh@giraffe.netgate.net (Bryan Henderson)

Changed genbki.sh block comment patch to avoid triggering an apparent
bug in GNU sed 3.0.

|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>