From: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | PostgreSQL Development <pgsql-hackers(at)postgreSQL(dot)org> |
Subject: | Re: Gripe: working on configure now requires Automake installed locally |
Date: | 2000-06-11 22:44:44 |
Message-ID: | Pine.LNX.4.21.0006120036380.353-100000@localhost.localdomain |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Tom Lane writes:
> You seem to be relying on automake's aclocal program to combine the
> config/*.m4 files into aclocal.m4. I can see no value in introducing
> this additional package dependency, since as far as I can tell aclocal
> isn't doing anything for us that 'cat' couldn't do.
We need it if we ever want to use libtool unless you want to include half
of libtool into CVS.
> there's no really good reason not to just keep all the configure code
> in configure.in.
It's really the same as keeping all of the C source code in one file, you
don't do that either. IMHO, it's a matter of logical organization,
modularity etc. I find setups organized like this much more readable. Many
tests in configure don't interface to config.cache properly;
unfortunately, fixing that makes the tests longer. I don't think a 5000
line file of shell and macro processing is really what you want.
> Regression tests are currently failing for me because the int8-related
> configure tests are busted.
I already fixed something there.
Well, I'm not proprietory on any of this, I'm just following the book. Let
me know...
--
Peter Eisentraut Sernanders väg 10:115
peter_e(at)gmx(dot)net 75262 Uppsala
http://yi.org/peter-e/ Sweden
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2000-06-12 01:00:11 | Re: ALTER TABLE DROP COLUMN |
Previous Message | Peter Eisentraut | 2000-06-11 22:34:49 | Re: Install error current CVS |