Merge pull request #2263 from pjanouse/JENKINS-34228

[JENKINS-34228] Fixed IOException in DeleteBuildsCommand class
jenkinsci · Apr 22, 2016 · ce9de5a · ce9de5a
2 parents e1dc525 + efc1285
commit ce9de5a
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/core/src/main/java/hudson/cli/DeleteBuildsCommand.java b/core/src/main/java/hudson/cli/DeleteBuildsCommand.java
@@ -29,6 +29,7 @@
 
 import java.io.IOException;
 import java.io.PrintStream;
+import java.util.HashSet;
 import java.util.List;
 
 /**
@@ -54,10 +55,16 @@ protected void printUsageSummary(PrintStream stderr) {
     protected int act(List<AbstractBuild<?, ?>> builds) throws IOException {
         job.checkPermission(Run.DELETE);
 
-        for (AbstractBuild build : builds)
-            build.delete();
+        final HashSet<Integer> hsBuilds = new HashSet<Integer>();
 
-        stdout.println("Deleted "+builds.size()+" builds");
+        for (AbstractBuild build : builds) {
+            if (!hsBuilds.contains(build.number)) {
+                build.delete();
+                hsBuilds.add(build.number);
+            }
+        }
+
+        stdout.println("Deleted "+hsBuilds.size()+" builds");
 
         return 0;
     }