[openssl-commits] [tools] master update

Richard Levitte levitte at openssl.org
Tue Jun 20 21:46:00 UTC 2017


The branch master has been updated
       via  ec891a63597c67c0bcbec00ed2d4813dd70eb819 (commit)
      from  010c66d97f13363bd3e449a0a371b81b0630c97c (commit)


- Log -----------------------------------------------------------------
commit ec891a63597c67c0bcbec00ed2d4813dd70eb819
Author: Richard Levitte <levitte at openssl.org>
Date:   Tue Jun 20 23:45:20 2017 +0200

    Read commit message after having parsed the options
    
    ... because --list isn't fed anything

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

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

diff --git a/review-tools/gitaddrev b/review-tools/gitaddrev
index 0596aca..9038137 100755
--- a/review-tools/gitaddrev
+++ b/review-tools/gitaddrev
@@ -56,9 +56,6 @@ sub try_add_reviewer {
     return $rc;
 }
 
-my @commit_message = map { (my $x = $_) =~ s|\R$||; $x } <STDIN>;
-my $trivial = !! grep(/^CLA:\s*Trivial\s*$/i, @commit_message);
-
 foreach (@ARGV) {
     if (/^--list$/) {
 	my %list = ();
@@ -100,6 +97,9 @@ foreach (@ARGV) {
     }
 }
 
+my @commit_message = map { (my $x = $_) =~ s|\R$||; $x } <STDIN>;
+my $trivial = !! grep(/^CLA:\s*Trivial\s*$/i, @commit_message);
+
 # If the author is a registered committer, that identity passes as a reviewer
 # too.  There is a twist, though...  see next comment
 if (my $rev = try_add_reviewer($ENV{GIT_AUTHOR_EMAIL})) {


More information about the openssl-commits mailing list