Erik Bray added the comment:
I agree this has a slight change in behavior which I was at first hesitant
about. But I think the previous behavior was wrong insofar as it was overly
ambiguous. I agree it should apply on MSYS2 as well (I actually thought
__CYGWIN__ was defined on MSYS2 but I
Masayuki Yamamoto added the comment:
This patch has impact to end user, thus I don't agree to apply as far as Cygwin
to avoid user surprise.
I think to need a consistency between other platforms having executable suffix
(e.g. MSYS2), and also the patch needs short documentation of changing
New submission from Erik Bray:
This actually came up previously in #1543469 in the context of test_subprocess,
but it's a more general issue that I thought was worth addressing somehow.
The issue here is that as Cygwin tries to provide a "UNIX-like" experience, any
system interfaces that take
