the contrib tree clean up

Started by Karel Zakalmost 26 years ago4 messageshackers
Jump to latest
#1Karel Zak
zakkr@zf.jcu.cz

I start clean up in the contrib tree. Now I have fixed some Makefiles.
My idea is add to all contrib modules Makefile (and allow install
contrib matter) and to top-contrib-directory Makefile.global that define
some definition relevant for contrib only. This Makefile include standard
../src/Makefile.global too. IMHO is not good idea add some setting
relevant (only) for contrib-tree to standard (src) Makefile.global.

IMHO all in contrib need a little standardize.

A question, how install these files from contrib? (I inspire with debian
PG packages)

*.sql - $(POSTGRESDIR)/sql
*.so - $(POSTGRESDIR)/modules
exec-binary - $(POSTGRESDIR)/bin
*.doc - $(POSTGRESDIR)/doc

In the current contrib is come dead code. What remove this code from
contrib tree to ftp.postgresql.org to some dead-code directory. It will
still accessible on ftp if anyone want it, and the contrib will correct.

Comments?
Karel

The current state in contrib:

apache_logging - Use it anyone? IMHO good candidate for *delete* or remove
to some web page like "Tips how use PostgreSQL"....

bit - impossible compile, last change '2000/04/12 17:14:21'
It is already in main tree. Delete?

earthdistance - I fix it, it is probably ready now.

likeplanning - haven't Makefile, I fix it

linux - haven't Makefile, I fix it

mSQL - Use it anyone?, I haven't idea where install it. It is a
"file.c". Delete?

noupdate - I fix (add Makefile) it, it is probably ready now, but
needs this anyone?

unixdate - haven't Makefile

Probably ready:

array
datetime
findoidjoins
fulltextindex
isbn_issn
lo
miscutil
odbc
pg_dumplo
pgbench
spi
string
soundex
tools
user_locks
vacuumlo

os2client - I fix Makefile, but it's still breaked:

gcc -I. -I../../src/include -DFRONTEND -DTCPIPV4 -DHAVE_CRYPT_H -c
../../src/backend/libpq/pqcomm.c
In file included from ../../src/include/storage/ipc.h:28,
from ../../src/include/storage/bufmgr.h:18,
from ../../src/include/storage/bufpage.h:18,
from ../../src/include/access/htup.h:17,
from ../../src/include/tcop/dest.h:55,
from ../../src/include/libpq/libpq.h:23,
from ../../src/backend/libpq/pqcomm.c:77:
config.h:16: warning: DEF_PGPORT' redefined
../../src/include/config.h:202: warning: this is the location of the
previous definition
config.h:18: warning: HAVE_TERMIOS_H' redefined
../../src/include/config.h:270: warning: this is the location of the
previous definition
config.h:19: warning: HAVE_ENDIAN_H' redefined
../../src/include/config.h:231: warning: this is the location of the
previous definition

----------> Commets?

#2The Hermit Hacker
scrappy@hub.org
In reply to: Karel Zak (#1)
Re: the contrib tree clean up

On Tue, 13 Jun 2000, Karel Zak wrote:

I start clean up in the contrib tree. Now I have fixed some Makefiles.
My idea is add to all contrib modules Makefile (and allow install
contrib matter) and to top-contrib-directory Makefile.global that define
some definition relevant for contrib only. This Makefile include standard
../src/Makefile.global too. IMHO is not good idea add some setting
relevant (only) for contrib-tree to standard (src) Makefile.global.

IMHO all in contrib need a little standardize.

A question, how install these files from contrib? (I inspire with debian
PG packages)

*.sql - $(POSTGRESDIR)/sql
*.so - $(POSTGRESDIR)/modules

ipmeter, when it installs their ports.so file, puts it in
$(POSTGRESDIR)/lib/modules ... sounds like an appropriate place to me ...

#3Karel Zak
zakkr@zf.jcu.cz
In reply to: The Hermit Hacker (#2)
Re: the contrib tree clean up

On Wed, 14 Jun 2000, Peter Eisentraut wrote:

On Tue, 13 Jun 2000, Karel Zak wrote:

Now I have fixed some Makefiles. My idea is add to all contrib modules
Makefile (and allow install contrib matter) and to
top-contrib-directory Makefile.global that define some definition
relevant for contrib only.

Try to keep everything as local as possible, don't include
../src/Makefile.global either if at all possible. Make it a
contrib/Makefile.global.in so you can substitute Autoconf variables into
it.

But I want primary ../src/Makefile.global for compilation, because here
is all defined. I not sure if is good define it (same) again in contrib.
If we will (for example) defined path to LIBPG in two Makefiles we must
maintain these Makefiles. IMHO it is bad --- same data is good share not
define it twice.

A solution is put Makefile.global to top directory (over src) and use it in
all trees (contrib/docs/src). What?

A question, how install these files from contrib? (I inspire with
debian PG packages)

*.sql - $(POSTGRESDIR)/sql

@datadir@ (/usr/local/pgsql/share)

*.so - $(POSTGRESDIR)/modules

@libdir@ (or perhaps libexecdir, depending on what it does)

exec-binary - $(POSTGRESDIR)/bin

@bindir@

Agree.

*.doc - $(POSTGRESDIR)/doc

Is there any documentation in contrib?

It is a little mazy, but yes.

See <http://www.gnu.org/prep/standards.html#SEC40&gt; for the GNU makefile
standards. Using anything else will just conflict with the Makefile
cleanup I'm currently doing and thus be inconsistent again at the end. :)

The current state in contrib:

apache_logging - Use it anyone? IMHO good candidate for *delete* or remove
to some web page like "Tips how use PostgreSQL"....

The latter sounds good but why not keep it?

I can create in contrib dir "tips" and somethig like 'apache_logging' can be
put here.

bit - impossible compile, last change '2000/04/12 17:14:21'
It is already in main tree. Delete?

I think this will be included into the main tree.

I detele it. Who not agree?

earthdistance - I fix it, it is probably ready now.

likeplanning - haven't Makefile, I fix it

This will be merged into the main tree.

I know, but in 7.1 (Tom)?

linux - haven't Makefile, I fix it

This definitely doesn't need a make file.

All need Makefile :-) 'make install'

mSQL - Use it anyone?, I haven't idea where install it. It is a
"file.c". Delete?

I think this is a wrapper library, hence @libdir@.

OK.

datetime

I think this is obsoleted by your to_char() and friends.

I look at it, and if you right, I delete it.

Karel

#4Tom Lane
tgl@sss.pgh.pa.us
In reply to: Karel Zak (#3)
Re: the contrib tree clean up

Karel Zak <zakkr@zf.jcu.cz> writes:

likeplanning - haven't Makefile, I fix it

This will be merged into the main tree.

I know, but in 7.1 (Tom)?

I think we could delete contrib/likeplanning --- there shouldn't be
any need for it in 7.1.

regards, tom lane