[openssl-commits] [tools] master update

[Richard Levitte]

The branch master has been updated
       via  4d6363793d47c66069634a58c0d21c5e52e2e3b7 (commit)
       via  63a6cb9771263aff1462a5ad60c30051706db0ea (commit)
      from  9a7c7931d7d8d1b26955026acdd32a1fa447cc95 (commit)


- Log -----------------------------------------------------------------
commit 4d6363793d47c66069634a58c0d21c5e52e2e3b7
Author: Richard Levitte <levitte at openssl.org>
Date:   Sat Nov 11 20:09:44 2017 +0100

    gitaddrev: small typo

commit 63a6cb9771263aff1462a5ad60c30051706db0ea
Author: Richard Levitte <levitte at openssl.org>
Date:   Sat Nov 11 20:09:20 2017 +0100

    gitaddrev: reviewers must be commit or OMC members

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

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

diff --git a/review-tools/gitaddrev b/review-tools/gitaddrev
index 4dfd04b..a53d9ad 100755
--- a/review-tools/gitaddrev
+++ b/review-tools/gitaddrev
@@ -65,12 +65,13 @@ foreach (@ARGV) {
 	    my $rev = $query->find_person_tag($email_id, 'rev');
 	    my $omc = $query->is_member_of($email_id, 'omc');
 	    next unless $query->has_cla($rev);
+	    next unless $query->is_member_of($email_id, 'commit') || $omc;
 	    my @ids =
 		sort grep { $_ =~ /^[a-z]+$/ || $_ =~ /^\@(?:\w|\w-\w)+$/ }
 		map {
 		    if (ref($_) eq "HASH") {
 			my %h = %$_;
-			map { $_ eq "github" ? '@'.$h{$_} : $h{$_} } %h;
+			map { $_ eq "github" ? '@'.$h{$_} : $h{$_} } keys %h;
 		    } else {
 			$_;
 		    }