Re: tests fail on windows with default git settings
Nazir Bilal Yavuz <byavuz81@gmail.com>
From: Nazir Bilal Yavuz <byavuz81@gmail.com>
To: Andrew Dunstan <andrew@dunslane.net>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Dave Page <dpage@pgadmin.org>, Andres Freund <andres@anarazel.de>, pgsql-hackers@postgresql.org
Date: 2024-07-11T08:59:57Z
Lists: pgsql-hackers
Hi, On Wed, 10 Jul 2024 at 17:04, Andrew Dunstan <andrew@dunslane.net> wrote: > > > On 2024-07-10 We 9:25 AM, Tom Lane wrote: > > Dave Page <dpage@pgadmin.org> writes: > >> On Wed, 10 Jul 2024 at 12:12, Andrew Dunstan <andrew@dunslane.net> wrote: > >>> As I was looking at this I wondered if there might be anywhere else that > >>> needed adjustment. One thing that occurred to me was that that maybe we > >>> should replace the use of "-w" in pg_regress.c with this rather less > >>> dangerous flag, so instead of ignoring any white space difference we would > >>> only ignore line end differences. The use of "-w" apparently dates back to > >>> 2009. > >> That seems like a good improvement to me. > > +1 > > > > > > > OK, done. It looks like Postgres CI did not like this change. 'Windows - Server 2019, VS 2019 - Meson & ninja' [1] task started to fail after this commit, there is one extra space at the end of line in regress test's output. [1] https://cirrus-ci.com/task/6753781205958656 -- Regards, Nazir Bilal Yavuz Microsoft
Commits
-
Use diff --strip-trailing-cr in pg_regress.c
- 291c42074721 18.0 landed
-
Use diff's --strip-trailing-cr flag where appropriate on Windows
- 4506d18a9891 17.0 landed
- 628c1d1f2c82 18.0 landed