Horton

Re: Is this a bug in pg_current_logfile() on Windows?

Tom Lane <tgl@sss.pgh.pa.us>

View thread View on postgr.es
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
Cc: Thomas Kellerer <shammat@gmx.net>, pgsql-hackers@lists.postgresql.org
Date: 2020-07-09T19:36:10Z
Lists: pgsql-hackers, pgsql-general 
I wrote:
> Cool, I'll go try changing all those conditions to use the msys test.

OK, that worked: all four relevant buildfarm members are now showing
the expected test failure.  So I'll go fix the original bug.

Should we consider back-patching the CRLF filtering changes, ie
91bdf499b + ffb4cee43?  It's not really necessary perhaps, but
I dislike situations where the "same" test on different branches is
testing different things.  Seems like a recipe for future surprises.

			regards, tom lane

Commits

  1. Tighten up Windows CRLF conversion in our TAP test scripts.

  2. Fix pg_current_logfile() to not emit a carriage return on Windows.

  3. Further tighten Windows CRLF conversion in our TAP test scripts.

  4. Add test coverage for pg_current_logfile() function.

  5. Fix failures to ignore \r when reading Windows-style newlines.

  6. Fix TAP infrastructure to support Mingw better