Mehul Jain <mehul.jain2029@xxxxxxxxx> writes: > Add a new configuratation variable "log.showSignature" for git-log > and related commands. "log.showSignature=true" will enable user to > see GPG signature by default for git-log and related commands. > > Changes compared to v2: > * A preparatory patch 1/3 has been introduced so that tests > in patches 2/3 and 3/3 can take advantage of it. It is unclear how this change allows the remainder to "take advanrage" to me. Earlier, "signed" branch was created only when the GPG prerequisite is met and with this change the branch is always created, which is the only change as far as I can see. But the tests that are added in 2 and 3 are all protected with the GPG prerequiste. Besides, the invocation of "git commit -S" after this change is no longer protected by the GPG prerequisite and it may even cause the 'setup' step to fail on a host without GPG. What am I missing? I do not quite see any reason to take 1/2; I only see a possible downside without any upside. The main two steps 2&3 looked good. Thanks. > * Mistake regarding branch in [patch v2 2/2] has been > corrected. > * Tight coupling between the tests in [patch v2 2/2] has > been resovled. > > I would like to thanks Eric Sunshine for his feedback on previous > series [1]. > > [1]: http://thread.gmane.org/gmane.comp.version-control.git/297648 > > Mehul Jain (3): > t4202: refactoring of test > log: add "--no-show-signature" command line option > log: add log.showSignature configuration variable > > Documentation/git-log.txt | 4 ++++ > builtin/log.c | 6 ++++++ > revision.c | 2 ++ > t/t4202-log.sh | 32 ++++++++++++++++++++++++++++++-- > t/t7510-signed-commit.sh | 7 +++++++ > 5 files changed, 49 insertions(+), 2 deletions(-) -- 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