Re: [PATCH] Regression tests in windows ignore white space

From: David Rowley <dgrowleyml(at)gmail(dot)com>
To: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCH] Regression tests in windows ignore white space
Date: 2013-12-30 05:38:50
Message-ID: CAApHDvoaREwCpU6gpybMkSnZFdLhLU8uo0VZ3bNwBdic-=0phg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Dec 30, 2013 at 6:02 PM, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>wrote:

> On Sun, Dec 29, 2013 at 2:06 AM, David Rowley <dgrowleyml(at)gmail(dot)com>
> wrote:
> > On Sun, Dec 29, 2013 at 2:29 AM, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
> > wrote:
> >>
> >> When I tried with your patch on windows, it results into following:
> >>
> >> ============== running regression test queries ==============
> >> test tablespace ... diff: unrecognized option
> >> `--strip-trailing-cr
> >> '
> >> Which version of diff you are using?
> >>
> >> Version of diff on my m/c is:
> >> diff - GNU diffutils version 2.7
> >>
> >
> > I had a bit of a look around on the git repository for diffutils and I've
> > found at least part of the commit which introduced --strip-trailing-cr
> >
> >
> http://git.savannah.gnu.org/cgit/diffutils.git/commit/?id=eefb9adae1642dcb0e2ac523c79998f466e94e77
> >
> > Although here I can only see that they've added the command line args and
> > not the actual code which implements stripping the carriage returns.
> Going
> > by that it seems that was added back in 2001, but the version you're
> using
> > is a bit older than than, it seems 2.7 is 19 years old!
>
> For Windows build, I am using whatever latest Git provides rather than
> downloading
> individual components which might not be good, but I find it
> convenient. The latest
> Git (1.8.4) download on windows still provides 2.7, which is the
> reason I am on older
> version. However I agree that it is better to use latest version.
>

It looks like the diff version I'm using is from msys and msys is what is
in my search path rather than the git\bin path. To be honest I didn't
realise that git for windows came with bison and flex, (at least I see
bison.exe and flex.exe in the git\bin path.. I don't remember me putting
them there)

I don't seem to even have git\bin in my %path% environment variable at all,
so all those tools are being picked up from the msys path. I'd need to
remind myself about the msys install process, but since we're already
saying in the docs that msys should be installed, then would the fix for
this not just be as simple as my patch plus a note in the docs to say to
ensure msys\bin occurs before git\bin in the %path% environment var,
minimum supported diff version is 2.8.

Did you install msys? if so does it have a later version of diff?

Regards

David Rowley

>
> > http://git.savannah.gnu.org/cgit/diffutils.git/refs/tags
> >
> > I'm on 2.8.7 which is only 4 years old.
> >
> > I know we don't normally go with bleeding edge, but I wondering if we
> could
> > make the minimum supported diff version at least 2.8 (at least for
> windows)
> > which was released in 2002.
>
> I have checked that for some of the other components like bison, flex,
> ActiveState TCL we specify minimum version required, but not for diff.
> http://www.postgresql.org/docs/devel/static/install-windows-full.html
>
> +1, for minimum diff version as 2.8.
>
> With Regards,
> Amit Kapila.
> EnterpriseDB: http://www.enterprisedb.com
>

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Marko Kreen 2013-12-30 10:02:54 Re: pgcrypto missing header file inclusions
Previous Message Amit Kapila 2013-12-30 05:02:57 Re: [PATCH] Regression tests in windows ignore white space