[PATCH master] devel/review: fix addition of reviewer from environment

[Klaus Aehlig]

In commit 3dd990, the possibility of taking a default reviewer
from the environment was introduced. Fix two deficits here
- perl does not inherit the environment
- add at all places.

Signed-off-by: Klaus Aehlig <aeh...@google.com>
---
 devel/review | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/devel/review b/devel/review
index e2ddd62..d3f6594 100755
--- a/devel/review
+++ b/devel/review
@@ -38,13 +38,15 @@ add_reviewed_by() {
   grep -q '^Reviewed-by: ' "$msgfile" && return
 
   perl -i -e '
+  my $reviewer = $ENV{"REVIEWER"};
+  defined($reviewer) or $reviewer = "";
   my $sob = 0;
   while (<>) {
     if ($sob == 0 and m/^Signed-off-by:/) {
       $sob = 1;
 
     } elsif ($sob == 1 and not m/^Signed-off-by:/) {
-      print "Reviewed-by: $REVIEWER\n";
+      print "Reviewed-by: $reviewer\n";
       $sob = -1;
     }
 
@@ -52,7 +54,7 @@ add_reviewed_by() {
   }
 
   if ($sob == 1) {
-    print "Reviewed-by: \n";
+    print "Reviewed-by: $reviewer\n";
   }
   ' "$msgfile"
 }
-- 
1.8.2.1