git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH] t7610 test for mktemp existence

From
Armin Kunaschik <megabreit@googlemail.com>
Date
Jul 2, 2016, 19:01 UTC
Message-ID
<57780F9F.3050802@googlemail.com>
Subject: t7610: test for mktemp before test execution

mktemp is not available on all platforms, so the test 'temporary filenames are used with mergetool.writeToTemp' fails there. This patch does not replace mktemp but just disables the test that otherwise would fail. mergetool checks itself before executing mktemp and reports an error.

Signed-off-by: Armin Kunaschik <megabreit@googlemail.com>
---
diff --git a/t/t7610-mergetool.sh b/t/t7610-mergetool.sh
index 76306cf..9279bf5 100755
--- a/t/t7610-mergetool.sh
+++ b/t/t7610-mergetool.sh
@@ -589,7 +589,12 @@ test_expect_success 'filenames seen by tools start with ./' '
 	git reset --hard master >/dev/null 2>&1
 '

-test_expect_success 'temporary filenames are used with mergetool.writeToTemp' '
+test_lazy_prereq MKTEMP '
+	tempdir=$(mktemp -d -t foo.XXX) &&
+	test -d "$tempdir"
+'
+
+test_expect_success MKTEMP 'temporary filenames are used with mergetool.writeToTemp' '
 	git checkout -b test16 branch1 &&
 	test_config mergetool.writeToTemp true &&
 	test_config mergetool.myecho.cmd "echo \"\$LOCAL\"" &&
Next: Junio C Hamano
Message 1 of 4 in “t7610 test for mktemp existence”
  1. t7610 test for mktemp existenceArmin Kunaschik, Jul 2, 2016
  2. Junio C HamanoJul 6, 2016
  3. Jeff KingJul 6, 2016
  4. Junio C HamanoJul 6, 2016

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.