Re: [PATCH] log: make "show --show-signature" use gpg.program setting

Jeff King [off-list ref] writes:

On Mon, Mar 25, 2013 at 01:03:52PM -0500, Hans Brigman wrote:

quoted

"show --show-signature" doesn't currently use the gpg.program setting.  Commit signing, tag signing, and tag verification currently use this setting properly, so the logic has been added to handle it here as well.

Please wrap your commit messages at something reasonable (70 is probably
as high as you want to go, given that log output is often shown
indented).

Thanks for pointing out Documentation/SubmittingPatches.

Also please do not send multipart/mixed (or alternative).  Send
patches in text/plain.

quoted

@@ -364,7 +365,8 @@ static int git_log_config(const char *var, const char *value, void *cb)
 		use_mailmap_config = git_config_bool(var, value);
 		return 0;
 	}
-
+	if (!prefixcmp(var, "gpg."))
+		return git_gpg_config(var, value, NULL); 
 	if (grep_config(var, value, cb) < 0)
 		return -1;

The gpg config can also be other places than "gpg.*". Right now it is
just user.signingkey, which log would not care about, but it feels like
we are depending an unnecessary detail here. We also don't know whether
it would care about the callback data. Is there a reason not to do:

  if (git_gpg_config(var, value, cb) < 0)
          return -1;

just like the grep_config call below?

-Peff