Re: [COMMITTERS] pgsql: Remove -w (--ignore-all-space) option from pg_regress's diff

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

In response to

Responses

Browse pgsql-committers by date

  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

Browse pgsql-hackers by date

  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