Jeff King <peff@xxxxxxxx> writes: > On Mon, Mar 25, 2013 at 01:03:52PM -0500, Hans Brigman wrote: > >> "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. >> @@ -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 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html