tag -v: use RUN_GIT_CMD to run verify-tag

jrn · gitster · commit a6ccbbdb66be · 2010-04-17T12:40:19.000-07:00
This is the preferred way to run a git command.

The only obvious observable effects I can think of are that the exec
is properly reported in GIT_TRACE output and that verifying signed
tags will still work if the git-verify-tag hard link in gitexecdir
goes missing.

Helped-by: Johannes Sixt &lt;j6t@kdbg.org&gt;
Signed-off-by: Jonathan Nieder &lt;jrnieder@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/builtin/tag.c b/builtin/tag.c
@@ -147,11 +147,11 @@ static int delete_tag(const char *name, const char *ref,
 static int verify_tag(const char *name, const char *ref,
 				const unsigned char *sha1)
 {
-	const char *argv_verify_tag[] = {"git-verify-tag",
+	const char *argv_verify_tag[] = {"verify-tag",
 					"-v", "SHA1_HEX", NULL};
 	argv_verify_tag[2] = sha1_to_hex(sha1);
 
-	if (run_command_v_opt(argv_verify_tag, 0))
+	if (run_command_v_opt(argv_verify_tag, RUN_GIT_CMD))
 		return error("could not verify the tag '%s'", name);
 	return 0;
 }

Original file line number	Diff line number	Diff line change
`@@ -147,11 +147,11 @@ static int delete_tag(const char name, const char ref,`
`147`	`147`	`static int verify_tag(const char name, const char ref,`
`148`	`148`	`const unsigned char *sha1)`
`149`	`149`	`{`
`150`		`- const char *argv_verify_tag[] = {"git-verify-tag",`
	`150`	`+ const char *argv_verify_tag[] = {"verify-tag",`
`151`	`151`	`"-v", "SHA1_HEX", NULL};`
`152`	`152`	`argv_verify_tag[2] = sha1_to_hex(sha1);`
`153`	`153`
`154`		`- if (run_command_v_opt(argv_verify_tag, 0))`
	`154`	`+ if (run_command_v_opt(argv_verify_tag, RUN_GIT_CMD))`
`155`	`155`	`return error("could not verify the tag '%s'", name);`
`156`	`156`	`return 0;`
`157`	`157`	`}`