the contrib tree clean up - Mailing list pgsql-hackers

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)/modulesexec-binary    - $(POSTGRESDIR)/bin*.doc
- $(POSTGRESDIR)/docIn 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
"Tipshow 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:arraydatetimefindoidjoinsfulltextindexisbn_issnlomiscutilodbcpg_dumplopgbenchspi        string
soundex   toolsuser_locksvacuumlo
 

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?