Clean up Makefile(s)

Add #include "postgres.h" as required

Remove #include "c.h" *and* "utils/elog.h" as applicable

Clean up Makefile

Add #include "postgres.h" as required

Remove #include "utils/elog.h"

Clean up Makefile, add #postgres.h as required

Clean up Makefile, make sure that postmaster.c compiles

Continuation of make file simplification; deal with header file shuffle.

Continuation of make file simplification; handle libpq.a dependancy.

Add complete setting of AROPT

Convenience routine for checking superuser status.

Add misc subdirectory.

Fix bug in checking permissions on table being COPY'd.

Allow only superuser to do backend copy.

Comment changes.

More intelligent #include cleanups, as pointed out by Bryan...

Compiled with -Wall -Werror

Okay, following Bryan's (and others) suggestions...cleaning up the
include files properly...yet, slowly...

Include dependencies based on tupdesc.h, as:

access/tupdesc.h
catalog/pg_attribute.h
nodes/pg_list.h
nodes/nodes.h
access/attnum.h

Include file cleanup:

Based on include files require by htup.h, as:

access/htup.h
storage/itemptr.h
storage/block.h
storage/off.h
utils/nabstime.h
<time.h>

Okay, so I was going backwards on my include cleanup...

Pointed out by Bryan (in a subtle way *grin*)

More chanes for bsdi_2_1 removal

Decoupled ASSERT and DEBUG options.

FIx for bsdi.

Removal of bsdi_2_1 port.

typo fix on include file.

Fix prototype.

Fix for gmake clean.

Add #include "postgres.h"

Cleanup Makefiles

Add #include "postgres.h"
and
    #include <sys/types.h>

More Makefile cleanups

Add #include "postgres.h" as required

clean up makefile

add #include "postgres.h"

Clean out makefile

add #include "postgres.h"

more removal of PORTNAME_*

more removals of PORTNAME_*

remove use of PORTNAME_*

add #include "postgres.h", as required by all .c files

remove:
#include "postgres.h"
#include "c.h"

Require superuser privilege to create C function.

More of the same...clean Makefile, add include files

Same clean up of Makefile, and addition of include files

Cleaned out -I../.. -I../../include from Makefile

Added missing include files

Remove -I../.. and -I../../include from Makefile

Add some missing include files

Add a couple of missing includes

Added misc include files

.h files shouldn't include .h files, .c files should contain .h files...

remove postgres.h from fmgr.h creation

Even compile generated .h files should be in ${SRCDIR}/include...

Well, normally there wouldn't be anything in -I../.., and in a minute, there
won't be...

Add these files to ${SRCDIR}/include

Move these files to ${SRCDIR}/include

There is no -I../.. include files...

USE of PORTNAME_* should be discouraged wherever possible.  The code
*should* be intelligent enough that:

#if defined(__FreeBSD__) works, where __FreeBSD__ is actually defined
by the compiler itself.

Makefile.global used to have -DPORTNAME_<port> -D<port> as part of the flags
for gcc while all occurances of PORTNAME_<port> slowly get removed from
the source tree...

Installed new FAQ's after I deleted them accidentally while
removing duplicate names.

Removed duplicate files.

Changed include path.

Added needed include file.

Missing from parser patch.
Added needed include file.

changed define(port) to define(PORTNAME_port)
Added needed include file.

Changed make to gmake.
Added needed include file.

Added prototypes missing from parser patch.
Added needed include file.

Removed false and changed to a commented true.

Fix memset() call, variables being passed in wrong order.

Pointed out by: wieck@sapserv.debis.de

Fixes: Incorrect syntax for copy command
By: Erich Stamberger <eberger@gewi.kfunigraz.ac.at>

Changes to libpgtcl submitted by: wieck@sapserv.debis.de (Jan Wieck)

Adds:

    -lAttributes

        Returns another format of the results attribute list. Per
        attribute a sublist  of  {{attname}  atttype  attlen}  is
        returned  and  an  empty  string  if  no attributes where
        received.

    -numAttrs

        Returns the number of attributes in the result.

commit a newer IRIX FAQ from Andrew

Commit a newer fact for 1.09 from Andrew

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.

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

Fix up INSTALL file

From Andrew

Parser Overhaul

Fix small syntax error.

Fix sigaction again to try for portability.

Remove another snprintf() for Solaris.

Slight changes to INSTALL to point ppl at the Linux/IRIX specific
FAQs

Add in Andrew's FAQs

Use strncpy() and local buffers instead of snprintf(), since not everyone
has snprintf().

Make a local isblank() function, since not all standard C libraries have it.
For sparc_solaris, add inet_aton prototype, since it isn't in the header
files included for the standard C library functions.

Add our own copy of inet_aton() for sparc_solaris, which doesn't have it in
the standard C library.

Simplify make files, add full dependencies.

Correct #if statement so it compiles on Solaris.  Thanks Keith Parks.

D'Arcy's cleanups

Make sure the btree patch gets into 2.0 as well...

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

Add CFLAGS+=-DPOSTPORT=... so we can take it out of the global CFLAGS.

Use EUROPEAN_DATES instead of EUROPEAN_STYLE

E Use EUROPEAN_DATES from config.h instead of -D option

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

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

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

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

Take out the PERFECT_MMGR #ifdefs:

My guess is that the thing had bugs, and the pfree was commented out.
The thing is probabally free'ed anyway at the end, so it was not a bad
thing.

If it does cause a bug, it will generate an error when hit, so I say
unless someone else knows, let's remove it and run the regression test.

-Bruce

Bruce...does this fix it once and for all? *grin*

Major code cleanups from D'arcy (-Wall -Werror)

|I have moved things around so that things happen in the following order:
|
|  1. Set default variables
|  2. Include Makefile.custom to override defaults
|  3. Set CFLAGS, etc. with variables
|
|This fixes the problem of Makefile.custom changes not taking effect.

Submitted by: D'Arcy Cain

Update "planner" directory to "optimizer" in include paths in this dead code.

Cosmetic changes to ordering of #include files

-Wall'd

-Wall'd

-Wall'd ... I missed a subdir under access :(

Make line 159:bp = NULL, to quiet compiler

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

-Wall'd

That finishes access/*

Force user to set PORTNAME.  Remove NAMEDATALEN from CFLAGS (don't need it
there because it's in config.h, which is cleaner).

-Wall'd

-Wall'd

-Wall Cleanup of gist subdirectory

-Wall cleanup of gist subdirectory:
- cleaned out unused variables
- added missing prototype headers

One outstanding -Werror failure:

/home/staff/scrappy/postgres/2.0/cvs/postgres95/src/backend/access/gist/giststra
t.c:117: warning: missing braces around initializer for `GISTEvaluationData.expr
ession'

Define prototypes from giststrat.c required in gistscan.c