Re: msvc failure in largeobject regression test

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Magnus Hagander <magnus(at)hagander(dot)net>, Jeremy Drake <pgsql(at)jdrake(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: msvc failure in largeobject regression test
Date: 2007-03-10 12:45:00
Message-ID: 45F2A84C.3030106@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Tom Lane wrote:
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>
>> Magnus Hagander wrote:
>> I wonder if this is a line-end issue? Assuming you are working from CVS,
>> does your client turn \n into \r\n ?
>>
>
>
>> I have just run into this today while trying to get buildfarm working
>> for MSVC. After some consideration I think an alternative result file is
>> the best solution. I have looked at switches for cnsnt, but they are
>> likely to be fragile at best.
>>
>
> Are you proposing an alternate result file that has a different linefeed
> style? I would really really rather that we not go there, because it
> will be impossibly fragile to maintain. Or are you willing to accept
> that the Windows builds will break every time someone changes that
> regression test, until someone else with a Windows machine fixes the
> result file?
>
> I would find it preferable to make pg_regress compensate for this
> issue somehow ...
>
>
No, the alternate results file does not use different line feeds, and is
as you would wish not sensitive to line-end style. I was careful (I hope
careful enough) to make sure that the file I checked in didn't have a
single CR in it. It just reflects the results obtained when tenk.data
file is checked out with CRLF line endings and then used for LO input.
The alternative as you say is that we make pg_regress perform some sort
of dos2unix on the data file. I note that tenk.data hasn't changed in 4
years.

cheers

andrew

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Simon Riggs 2007-03-10 13:34:36 Re: [PATCHES] scan_recycle_buffers
Previous Message Pavan Deolasee 2007-03-10 11:42:44 Re: Bug in VACUUM FULL ?