From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-hackers(at)postgreSQL(dot)org |
Subject: | Re: [COMMITTERS] pgsql: Remove -w (--ignore-all-space) option from pg_regress's diff |
Date: | 2009-11-23 05:58:18 |
Message-ID: | 4B0A247A.4010800@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Tom Lane wrote:
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>
>> Tom Lane wrote:
>>
>>> Remove -w (--ignore-all-space) option from pg_regress's diff calls.
>>>
>
>
>> Looks like this has broken on Windows due to different line endings,
>> which -w hid from us.
>>
>
> Yeah. I was waiting for brown_bat to report in before bringing this
> up on the list, because I don't entirely understand what's happening.
> So far it appears that the MSVC builds are fine and the MinGW builds
> are not. How can that be? Aren't they using the same diff program?
>
>
>
They might not be using the same CVS programs, though. It appears that
Windows CVS (which, for example, red_bat uses) translates line endings
to CRLF, which is why it passed the regression tests, but MinGW CVS does
not, which I think is is why narwahl and vaquita failed and why dawn_bat
will probably fail next go round. brown_bat is on Cygwin and we should
not expect a change there.
cheers
andrew
From | Date | Subject | |
---|---|---|---|
Next Message | User Okbob | 2009-11-23 08:04:36 | orafce - orafce: fixing formating |
Previous Message | Tom Lane | 2009-11-23 05:30:43 | Re: [COMMITTERS] pgsql: Remove -w (--ignore-all-space) option from pg_regress's diff |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2009-11-23 07:20:24 | Re: operator exclusion constraints |
Previous Message | Scott Marlowe | 2009-11-23 05:58:06 | Re: [HACKERS] Updating column on row update |