Re: [PORTS] Re: [BUGS] 6.5.2 broken on alpha/Tru64 Unix 4.0d/cc

From: Uncle George <gatgul(at)voicenet(dot)com>
To: PostgreSQL bugs mailing list <pgsql-bugs(at)postgreSQL(dot)org>
Cc: PostgreSQL ports mailing list <pgsql-ports(at)postgreSQL(dot)org>
Subject: Re: [PORTS] Re: [BUGS] 6.5.2 broken on alpha/Tru64 Unix 4.0d/cc
Date: 1999-10-10 11:38:13
Message-ID: 38007AA5.FCDCA567@voicenet.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-ports

egcs, also defines both __alpha & __alpha__
( if u havent't test there already ) I believe this covers all avail compiled
on the Digital Alpha box
gat
Tom Lane wrote:

> "Pedro J. Lobo" <pjlobo(at)euitt(dot)upm(dot)es> writes:
> > Someone has changed the "#if defined(__alpha)" tests that existed in
> > include/storage/s_lock.h in 6.5.1 to "#if defined(__alpha__)". The problem
> > is that DEC (¿Compaq?) C only defines __alpha, (gcc defines both __alpha
> > and __alpha__) so it doesn't work anymore.
>
> I thought that was bogus when it was done. Should be
> #if defined(__alpha) || defined(__alpha__)
> to cover both compilers.
>
> > In fact, there are two places where the test is used, one inside a "#if
> > defined(__GNUC__)" and the other one inside the corresponding "#else". It
> > seems that the easiest sollution is to change the check inside the "#else"
> > back to "#if defined(__alpha)".
>
> ... which would break whichever compiler is being used by the person who
> submitted the patch. I don't think it was changed on a whim.
>
> regards, tom lane
>
> ************

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Tom Lane 1999-10-10 16:13:07 Re: [BUGS] Three Regression Test Problems involving dates
Previous Message Duncan Kinder 1999-10-09 23:24:35 Three Regression Test Problems involving dates

Browse pgsql-ports by date

  From Date Subject
Next Message Brian Edmonds 1999-10-10 15:26:05 Linux 2.2.12 SMP build success report
Previous Message Peter Schaefer 1999-10-09 16:18:14 'current'::datetime