Re: [HACKERS] Re: Problems compiling version 7

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
Thread:
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

Responses

Browse pgsql-general by date

  From Date Subject
Next Message Hitesh Patel 2000-05-10 16:07:39 Re: textpos() function
Previous Message Patrick FICHE 2000-05-10 15:48:27 ISAM Api for Postgres

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2000-05-10 16:09:29 Re: pg_hba.conf && ident ...
Previous Message The Hermit Hacker 2000-05-10 15:58:56 Re: pg_hba.conf && ident ...