Re: [PATCH 1/5] t7300: work around platform-specific behaviour with long paths on MinGW

2 messages, 2 authors, 2024-10-10 · open the first message on its own page

Re: [PATCH 1/5] t7300: work around platform-specific behaviour with long paths on MinGW

From: Junio C Hamano <hidden>
Date: 2024-10-09 18:31:28

Patrick Steinhardt [off-list ref] writes:
quoted hunk
diff --git a/t/t7300-clean.sh b/t/t7300-clean.sh
index 0aae0dee67..12ab25296b 100755
--- a/t/t7300-clean.sh
+++ b/t/t7300-clean.sh
@@ -747,7 +747,7 @@ test_expect_success MINGW 'handle clean & core.longpaths = false nicely' '
 	test_must_fail git clean -xdf 2>.git/err &&
 	# grepping for a strerror string is unportable but it is OK here with
 	# MINGW prereq
I find it amusing that this talks about limiting the test to MINGW
yet we need to accept two variants ;-)
quoted hunk
-	test_grep "too long" .git/err
+	test_grep -e "too long" -e "No such file or directory" .git/err
 '
The solution looks obviously good enough.

Thanks.

Re: [PATCH 1/5] t7300: work around platform-specific behaviour with long paths on MinGW

From: Patrick Steinhardt <hidden>
Date: 2024-10-10 05:56:24

On Wed, Oct 09, 2024 at 11:31:25AM -0700, Junio C Hamano wrote:
Patrick Steinhardt [off-list ref] writes:
quoted
diff --git a/t/t7300-clean.sh b/t/t7300-clean.sh
index 0aae0dee67..12ab25296b 100755
--- a/t/t7300-clean.sh
+++ b/t/t7300-clean.sh
@@ -747,7 +747,7 @@ test_expect_success MINGW 'handle clean & core.longpaths = false nicely' '
 	test_must_fail git clean -xdf 2>.git/err &&
 	# grepping for a strerror string is unportable but it is OK here with
 	# MINGW prereq
I find it amusing that this talks about limiting the test to MINGW
yet we need to accept two variants ;-)
Agreed, I was quite confused by this. I hope that Johannes may shed a
bit more light on this, as I couldn't make much sense of it. Was this a
change on our side? Did MinGW change? Is it something else entirely? No
idea.

Patrick
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help