[openssl-commits] [tools] master update

[Matt Caswell]

The branch master has been updated
       via  9c2a1bcbb06f1075307267fe16bd34e43f750cd4 (commit)
      from  c1388cc424c5c1aaab2b1193e7d9a82310a0e6de (commit)


- Log -----------------------------------------------------------------
commit 9c2a1bcbb06f1075307267fe16bd34e43f750cd4
Author: Matt Caswell <matt at openssl.org>
Date:   Fri Dec 8 09:48:09 2017 +0000

    Ignore case when checking author against cladb
    
    Reviewed-by: Richard Levitte <levitte at openssl.org>
    (Merged from https://github.com/openssl/openssl/pull/5)

-----------------------------------------------------------------------

Summary of changes:
 review-tools/gitaddrev | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/review-tools/gitaddrev b/review-tools/gitaddrev
index a53d9ad..2beb48a 100755
--- a/review-tools/gitaddrev
+++ b/review-tools/gitaddrev
@@ -30,7 +30,7 @@ my @unknown_reviewers;
 my $skip_reviewer;
 my $omccount = 0;
 sub try_add_reviewer {
-    my $id = shift;
+    my $id = lc(shift);
     my $rc = undef;
     my $id2 = $id =~ /^\@(.*)$/ ? { github => $1 } : $id;
     my $rev = $query->find_person_tag($id2, 'rev');
@@ -123,7 +123,7 @@ if (my $rev = try_add_reviewer($ENV{GIT_AUTHOR_EMAIL})) {
     # In case the author is unknown to our databases or is lacking a CLA,
     # we need to be extra careful to check if this is supposed to be a
     # trivial commit.
-    my $author = $ENV{GIT_AUTHOR_EMAIL};
+    my $author = lc($ENV{GIT_AUTHOR_EMAIL});
 
     # Note: it really should be enough to check if $author is unknown, since
     # the databases are supposed to be consistent with each other.  However,