Hi *,
This patch series tries to make more evident the committer ident
used in a commit to prevent using the wrong one.
The definition of "wrong committer" could be:
1) user.{name,email} or GIT_COMMITTER_{NAME,EMAIL} is not a wrong committer.
2) automatic without a domain name (user@hostname.(none)) is a wrong committer.
(not handled with this patch series)
3) automatic or partially set ident:
a) wrong committer for some users
b) right committer for others
The different strategies are.. Show the committer:
1) always
2) when user.warn = yes (defaulting to yes)
3) when it is automatic
a) always
b) and different from parent
c) and different from a list of valid committer idents
d) and when user.WarnAutomatic = yes (defaulting to yes)
The first patch implements 3a
The second patch implements 3d
I prefer 3a.
Santi
Santi Béjar (2):
commit: Show committer if automatic
user.warnautomatic: add config to control if the committer ident is
shown
Documentation/config.txt | 4 ++++
builtin-commit.c | 40 +++++++++++++++++++++++++++++++++++++---
cache.h | 2 ++
config.c | 9 +++++++++
environment.c | 2 ++
ident.c | 3 +++
t/t7502-commit.sh | 31 +++++++++++++++++++++++++++++--
7 files changed, 86 insertions(+), 5 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
