You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by da...@apache.org on 2012/02/02 14:08:24 UTC

svn commit: r1239595 - /subversion/trunk/tools/dist/backport.pl

Author: danielsh
Date: Thu Feb  2 13:08:23 2012
New Revision: 1239595

URL: http://svn.apache.org/viewvc?rev=1239595&view=rev
Log:
* tools/dist/backport.pl
  (handle_entry): Skip vetoed entries in batch mode.

Modified:
    subversion/trunk/tools/dist/backport.pl

Modified: subversion/trunk/tools/dist/backport.pl
URL: http://svn.apache.org/viewvc/subversion/trunk/tools/dist/backport.pl?rev=1239595&r1=1239594&r2=1239595&view=diff
==============================================================================
--- subversion/trunk/tools/dist/backport.pl (original)
+++ subversion/trunk/tools/dist/backport.pl Thu Feb  2 13:08:23 2012
@@ -180,6 +180,7 @@ sub parse_entry {
 
 sub handle_entry {
   my %entry = parse_entry @_;
+  my @vetoes = grep { /^  -1:/ } @{$entry{votes}};
 
   print "";
   print "\n>>> The $entry{header}:";
@@ -190,11 +191,11 @@ sub handle_entry {
   print "";
   print for @{$entry{votes}};
   print "";
-  print "Vetoes found!" if grep { /^  -1:/ } @{$entry{votes}};
+  print "Vetoes found!" if @vetoes;
 
   # TODO: this changes ./STATUS, which we're reading below, but
   #       on my system the loop in main() doesn't seem to care.
-  merge %entry if $ENV{YES} or prompt;
+  merge %entry if $ENV{YES} ? !@vetoes : prompt;
 
   1;
 }