Re: [PATCH] t1800: correct test to handle Cygwin
From: Johannes Schindelin <hidden>
Date: 2022-09-02 12:27:17
Hi Adam, On Thu, 1 Sep 2022, Adam Dinwoodie wrote:
On Cygwin, when failing to spawn a process using start_command, Git outputs the same error as on Linux systems, rather than using the GIT_WINDOWS_NATIVE-specific error output.
Given this explanation, I would have expected...
quoted hunk ↗ jump to hunk
The WINDOWS test prerequisite is set in both Cygwin and native Windows environments, which means it's not appropriate to use to anticipate the error output from start_command. Instead, use the MINGW test prerequisite, which is only set for Git in native Windows environments, and not for Cygwin. Signed-off-by: Adam Dinwoodie <redacted> --- The job of setting Cygwin up to get Git CI builds, either as part of the main CI builds or as something using separate automation, is rapidly rising up my when-I-have-the-time to-do list... t/t1800-hook.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/t/t1800-hook.sh b/t/t1800-hook.sh index 64096adac7..fae8b2faf9 100755 --- a/t/t1800-hook.sh +++ b/t/t1800-hook.sh@@ -159,7 +159,7 @@ test_expect_success 'git hook run a hook with a bad shebang' ' # TODO: We should emit the same (or at least a more similar) # error on Windows and !Windows. See the OS-specific code in # start_command() - if test_have_prereq !WINDOWS + if test_have_prereq !MINGW
... this to be `test_have_prereq !WINDOWS || test_have_prereq CYGWIN` instead. At least it is my understanding that Cygwin is the special case of the special case that is Windows. That is, if most scenarios on Windows were behaving like Cygwin and the mingw-w64 build was the odd duck, I would understand to restrict this only to MINGW. But I think it is the other way around: with Cygwin, _despite_ being on Windows, we see this Linux-like behavior. Ciao, Dscho
then cat >expect <<-\EOF fatal: cannot run bad-hooks/test-hook: ... -- 2.37.2