Re: [PATCH] Regression tests in windows ignore white space - Mailing list pgsql-hackers
| From | David Rowley |
|---|---|
| Subject | Re: [PATCH] Regression tests in windows ignore white space |
| Date | |
| Msg-id | CAApHDvoaREwCpU6gpybMkSnZFdLhLU8uo0VZ3bNwBdic-=0phg@mail.gmail.com Whole thread Raw |
| In response to | Re: [PATCH] Regression tests in windows ignore white space (Amit Kapila <amit.kapila16@gmail.com>) |
| Responses |
Re: [PATCH] Regression tests in windows ignore white space
|
| List | pgsql-hackers |
On Mon, Dec 30, 2013 at 6:02 PM, Amit Kapila <amit.kapila16@gmail.com> wrote:
On Sun, Dec 29, 2013 at 2:06 AM, David Rowley <dgrowleyml@gmail.com> wrote:
> On Sun, Dec 29, 2013 at 2:29 AM, Amit Kapila <amit.kapila16@gmail.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?For Windows build, I am using whatever latest Git provides rather than
>>
>> 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!
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
I have checked that for some of the other components like bison, flex,
> 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.
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.
pgsql-hackers by date: