Re: Geometry patch for freebsd alpha - Mailing list pgsql-patches

From Tom Lane
Subject Re: Geometry patch for freebsd alpha
Date
Msg-id 8669.1008862513@sss.pgh.pa.us
Whole thread Raw
In response to Geometry patch for freebsd alpha  ("Christopher Kings-Lynne" <chriskl@familyhealth.com.au>)
List pgsql-patches
"Christopher Kings-Lynne" <chriskl@familyhealth.com.au> writes:
> *** src/test/regress/resultmap    2001/12/19 17:25:55    1.52
> --- src/test/regress/resultmap    2001/12/20 02:45:56
> ***************
> *** 13,19 ****
>   geometry/.*-bsdi=geometry-bsdi-precision
>   geometry/.*-darwin=geometry-powerpc-darwin
>   geometry/.*-freebsd=geometry-positive-zeros
> ! geometry/.*-freebsd4=geometry-positive-zeros-bsd
>   geometry/i386-.*-openbsd=geometry-positive-zeros-bsd
>   geometry/sparc-.*-openbsd=geometry-positive-zeros
>   geometry/.*-irix6=geometry-irix
> --- 13,20 ----
>   geometry/.*-bsdi=geometry-bsdi-precision
>   geometry/.*-darwin=geometry-powerpc-darwin
>   geometry/.*-freebsd=geometry-positive-zeros
> ! geometry/i.86-.*-freebsd4=geometry-positive-zeros-bsd
> ! geometry/alpha-.*-freebsd4=geometry-positive-zeros
>   geometry/i386-.*-openbsd=geometry-positive-zeros-bsd
>   geometry/sparc-.*-openbsd=geometry-positive-zeros
>   geometry/.*-irix6=geometry-irix

Hmm.  I think it's bad practice to have two lines in resultmap that
can match, and here "geometry/.*-freebsd" will certainly match all the
same platforms that your two lines do.  My inclination is to replace
all three lines with

geometry/i.86-.*-freebsd=geometry-positive-zeros-bsd
geometry/alpha-.*-freebsd=geometry-positive-zeros

and wait to see if anyone using version 3 complains.

            regards, tom lane

pgsql-patches by date:

Previous
From: Thomas Lockhart
Date:
Subject: Re: SunOS patch for memcmp()
Next
From: Karel Zak
Date:
Subject: Czech NLS