Re: msvc failure in largeobject regression test - Mailing list pgsql-hackers

From Magnus Hagander
Subject Re: msvc failure in largeobject regression test
Date
Msg-id 20070124093550.GA19748@svr2.hagander.net
Whole thread Raw
In response to Re: msvc failure in largeobject regression test  (Jeremy Drake <pgsql@jdrake.com>)
Responses Re: msvc failure in largeobject regression test  (Bruce Momjian <bruce@momjian.us>)
Re: msvc failure in largeobject regression test  (Andrew Dunstan <andrew@dunslane.net>)
List pgsql-hackers
On Tue, Jan 23, 2007 at 11:39:23AM -0800, Jeremy Drake wrote:
> On Tue, 23 Jan 2007, Magnus Hagander wrote:
> 
> > On Tue, Jan 23, 2007 at 09:31:40AM -0500, Andrew Dunstan wrote:
> > > Magnus Hagander wrote:
> > > >Hi!
> > > >
> > > >I get failures for the largeobject regression tests on my vc++ build. I
> > > >don't think this has ever worked, given that those tests are fairly new.
> > > >Any quick ideas on what's wrong before I dig deeper?
> > > >
> > > >
> > > [snip]
> > >
> > > I wonder if this is a line-end issue? Assuming you are working from CVS,
> > > does your client turn \n into \r\n ? I see that other windows boxes are
> > > happily passing this test on the buildfarm, and of course the mingw cvs
> > > doesn't adjust line endings.
> >
> > Bingo!
> >
> > That's it. I copeid the file in binary mode from a linux box and now it
> > passes.
> 
> I thought about that when I wrote it, and thus tried it under mingw and
> cygwin without issue ;)  I don't think the regression tests were in a
> position of running on the msvc build at the time...  My thought for what
> to do if this did run into a problem would be an alternate output file
> that is also acceptable (I don't know what they're called but other tests
> have them IIRC).

Either that, or we require a checkout using Unix style linefeeds. I've
confirmed that removing the file and checking it back out with "cvs --lf
update tenk.data" works - tests pass fine.

Yet another option might be to flag that file as binary in cvs, in which
case I think cvsnt shouldn't go mess with it.

//Magnus


pgsql-hackers by date:

Previous
From: re-plore
Date:
Subject: Searching some sites explaing about PosgtreSQL source codes
Next
From: "Simon Riggs"
Date:
Subject: Re: Piggybacking vacuum I/O