From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Joachim Wieland <joe(at)mcknight(dot)de> |
Cc: | Andrew Dunstan <andrew(at)dunslane(dot)net>, Michael Meskes <meskes(at)postgresql(dot)org>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Dave Page <dpage(at)postgresql(dot)org> |
Subject: | Re: ecpg regression broken on mingw |
Date: | 2007-06-14 13:08:00 |
Message-ID: | 20070614130800.GC29032@svr2.hagander.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, Jun 14, 2007 at 02:35:29PM +0200, Magnus Hagander wrote:
> On Thu, Jun 14, 2007 at 02:28:26PM +0200, Joachim Wieland wrote:
> >
> >
> > On Thu, Jun 14, 2007 at 08:17:05AM -0400, Andrew Dunstan wrote:
> > > Maybe it uses a different CVS or something. Who knows? Aren't our diffs
> > > supposed to be done ignoring whitespace?
> >
> > pg_init() in pg_regress_ecpg.c has:
> >
> > /* no reason to set -w for ecpg checks, except for when on windows */
> > if (strstr(host_platform, "-win32"))
> > basic_diff_opts = "-w";
> > else
> > basic_diff_opts = "";
> >
> > What value does host_platform have on MinGW?
>
> That could certainly be it - on the failing machine, it's:
> i686-pc-mingw32
>
> Dave, any chance you can change that check to look for both -win32 and
> -mingw32 and give it a test run on vaquita?
>
> (I'll see if I have a vmware around that can run the build meanwhile, but
> if it's easy for you to do..)
This seems to fix most of the problems - I still see 3 tests failing. I'll
commit this change for now.
The interesting thing is that the failed ones seem to *still* be because of
line endings...
//Magnus
From | Date | Subject | |
---|---|---|---|
Next Message | Magnus Hagander | 2007-06-14 13:12:11 | Re: ecpg regression broken on mingw |
Previous Message | Magnus Hagander | 2007-06-14 12:35:29 | Re: ecpg regression broken on mingw |