Justin Pryzby <pryzby@telsasoft.com> writes:
> I suggest to leave stat() alone in your patch for stable releases. I think
> it's okay if we change behavior so that a broken symlink is skipped instead of
> erroring (as a side effect of skipping ENOENT with stat()). But not okay if we
> change pg_ls_logdir() to hide symlinks in back braches.
Meh. I'm not really convinced, but in the absence of anyone expressing
support for my position, I'll do it that way. I don't think it's worth
doing both a stat and lstat to tell the difference between file-is-gone
and file-is-a-broken-symlink.
regards, tom lane