summaryrefslogtreecommitdiff
path: root/git
diff options
context:
space:
mode:
authorAthos Ribeiro <athos@redhat.com>2020-10-23 12:00:17 +0200
committerSebastian Thiel <sebastian.thiel@icloud.com>2020-10-23 19:28:07 +0800
commit08989071e8c47bb75f3a5f171d821b805380baef (patch)
treef57ebf19469f199389bf9d7cfe2a39de8d267e99 /git
parente30a597b028290c7f703e68c4698499b3362a38f (diff)
downloadgitpython-08989071e8c47bb75f3a5f171d821b805380baef.tar.gz
Fix default actor name handling
In c96476b, the new default_name nested function does not contain a retun statement. This leads to an issue when the environment variables are not present, where the actor name would not be set. Signed-off-by: Athos Ribeiro <athos@redhat.com>
Diffstat (limited to 'git')
-rw-r--r--git/util.py2
1 files changed, 1 insertions, 1 deletions
diff --git a/git/util.py b/git/util.py
index 44eb0e51..04c96789 100644
--- a/git/util.py
+++ b/git/util.py
@@ -591,7 +591,7 @@ class Actor(object):
return user_id
def default_name():
- default_email().split('@')[0]
+ return default_email().split('@')[0]
for attr, evar, cvar, default in (('name', env_name, cls.conf_name, default_name),
('email', env_email, cls.conf_email, default_email)):