Re: [HACKERS] Re: Problems compiling version 7

Lists: pgsql-generalpgsql-hackers
From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: Travis Bauer <trbauer(at)indiana(dot)edu>, pgsql-general(at)postgresql(dot)org, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [HACKERS] Re: Problems compiling version 7
Date: 2000-05-10 16:02:43
Message-ID: 18522.957974563@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-general pgsql-hackers

Peter Eisentraut <e99re41(at)DoCS(dot)UU(dot)SE> writes:
> On Tue, 9 May 2000, Tom Lane wrote:
>> dnl Check tr flags to convert from lower to upper case

>> Does anyone recall why this test is in there to begin with?

> I don't see the results of this test being used anywhere at all, so I'd
> say yank it. If your system doesn't support tr '[A-Z]' '[a-z]' the
> configure script will fail to run anyway, as it uses this contruct
> indiscriminately.

The results *are* used, in backend/utils/Gen_fmgrtab.sh.in (and
apparently nowhere else). But the data being processed there is just
builtin function names, so I'm at a loss why someone thought that it'd
be worth testing for a locale-specific variant of 'tr'. I agree, I'm
pretty strongly tempted to yank it.

But we haven't yet figured out Travis' problem: why is the configure
test failing? Useless or not, I don't see why it's falling over...

regards, tom lane


From: Travis Bauer <trbauer(at)indiana(dot)edu>
To: pgsql-general(at)postgresql(dot)org
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: [HACKERS] Re: Problems compiling version 7
Date: 2000-05-10 17:11:43
Message-ID: Pine.GSO.4.21.0005101208270.17696-100000@school.cs.indiana.edu
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-general pgsql-hackers

I've found part of the problem. I put "echo $TR" right before the line
that is failing. $TR is set to "/usr/ucb/tr" during the
configuration. The directory ucb does not exist. If execute "which
tr" at the command line, I get "/usr/bin//tr." Why is $TR getting set to
/usr/ucb/tr?

By the way, I do not get this problem when compiling the last version of
Postgresql on this same machine. I'm in the processof upgrading. That
compile was fine.

Thanks,

----------------------------------------------------------------
Travis Bauer | CS Grad Student | IU |www.cs.indiana.edu/~trbauer
----------------------------------------------------------------

>
> But we haven't yet figured out Travis' problem: why is the configure
> test failing? Useless or not, I don't see why it's falling over...
>
> regards, tom lane


From: Peter Eisentraut <peter_e(at)gmx(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Travis Bauer <trbauer(at)indiana(dot)edu>, pgsql-general(at)postgresql(dot)org, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [HACKERS] Re: Problems compiling version 7
Date: 2000-05-10 21:25:29
Message-ID: Pine.LNX.4.21.0005101848070.30279-100000@localhost.localdomain
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-general pgsql-hackers

Tom Lane writes:

> >> dnl Check tr flags to convert from lower to upper case

> The results *are* used, in backend/utils/Gen_fmgrtab.sh.in (and
> apparently nowhere else).

Ah, I see. Substituting into source files directly from configure ... very
evil...

(Before you ask why: What if I change Gen_fmgrtab.sh.in, do I have to
re-configure?)

> But we haven't yet figured out Travis' problem: why is the configure
> test failing? Useless or not, I don't see why it's falling over...

Unfortunately he cut off the line where it says `checking for tr'.

--
Peter Eisentraut Sernanders väg 10:115
peter_e(at)gmx(dot)net 75262 Uppsala
http://yi.org/peter-e/ Sweden


From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: Travis Bauer <trbauer(at)indiana(dot)edu>, pgsql-general(at)postgresql(dot)org, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [HACKERS] Re: Problems compiling version 7
Date: 2000-05-10 22:24:19
Message-ID: 20094.957997459@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-general pgsql-hackers

Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
> Ah, I see. Substituting into source files directly from configure ... very
> evil...
> (Before you ask why: What if I change Gen_fmgrtab.sh.in, do I have to
> re-configure?)

Yup, or at least re-run config.status. I've griped that configure
writes far too many files myself. But I didn't have much luck
convincing the other developers that it's a bad idea to set things up
that way, rather than writing just a small number of config files.

We're going to have to deal with the problem though if we ever want
to be able to build in a separate directory tree...

regards, tom lane