Fix get_dirent_type() for symlinks on MinGW/MSYS.
authorThomas Munro
Thu, 28 Jul 2022 02:13:37 +0000 (14:13 +1200)
committerThomas Munro
Thu, 28 Jul 2022 02:27:28 +0000 (14:27 +1200)
commitfc4e5af30749eadc2acf98de426fa51910f8766b
treebcf1a2a6f355bd979c119475702344a6b50bb524
parent1d01d198536f24995f2d5271f55c73be644438fc
Fix get_dirent_type() for symlinks on MinGW/MSYS.

On Windows with MSVC, get_dirent_type() was recently made to return
DT_LNK for junction points by commit 9d3444dc, which fixed some
defective dirent.c code.

On Windows with Cygwin, get_dirent_type() already worked for symlinks,
as it does on POSIX systems, because Cygwin has its own fake symlinks
that behave like POSIX (on closer inspection, Cygwin's dirent has the
BSD d_type extension but it's probably always DT_UNKNOWN, so we fall
back to lstat(), which understands Cygwin symlinks with S_ISLNK()).

On Windows with MinGW/MSYS, we need extra code, because the MinGW
runtime has its own readdir() without d_type, and the lstat()-based
fallback has no knowledge of our convention for treating junctions as
symlinks.

Back-patch to 14, where get_dirent_type() landed.

Reported-by: Andrew Dunstan
Discussion: https://postgr.es/m/b9ddf605-6b36-f90d-7c30-7b3e95c46276%40dunslane.net
src/common/file_utils.c