Correct mailmapping of empty email addresses

`not old_email` doesn't distinguish between `None` and an empty string, causing old emails specified as `<>` to apply to every single commit. Signed-off-by: Riley Iverson <[email protected]>
ReinventorsOfTheWheel · Oct 3, 2022 · 91f16fd · 91f16fd
1 parent fff5f43
commit 91f16fd
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/git-filter-repo b/git-filter-repo
@@ -320,8 +320,8 @@ class MailmapInfo(object):
     for old, new in self.changes.items():
       old_name, old_email = old
       new_name, new_email = new
-      if (not old_email or email.lower() == old_email.lower()) and (
-          name  == old_name or not old_name):
+      if (old_email is None or email.lower() == old_email.lower()) and (
+          name == old_name or not old_name):
         return (new_name or name, new_email or email)
     return (name, email)