Tags: patch

Patch as agreed with Emmanuel Bourg.

Test cases to be considered.

Christopher
diff --git a/maven-packager-utils/src/main/java/org/debian/maven/packager/util/IgnoreDependencyQuestions.java b/maven-packager-utils/src/main/java/org/debian/maven/packager/util/IgnoreDependencyQuestions.java
index e061a8d..33ab512 100644
--- a/maven-packager-utils/src/main/java/org/debian/maven/packager/util/IgnoreDependencyQuestions.java
+++ b/maven-packager-utils/src/main/java/org/debian/maven/packager/util/IgnoreDependencyQuestions.java
@@ -163,9 +163,12 @@ public class IgnoreDependencyQuestions {
     }
 
     private boolean askIgnoreDependency(String sourcePomLoc, Dependency dependency, String message, boolean defaultToIgnore) {
-        if (!interactive || notIgnoredDependencies.contains(dependency)) {
+        if (notIgnoredDependencies.contains(dependency)) {
             return false;
         }
+        if (!interactive) {
+            return defaultToIgnore;
+        }
         String question = "\n" + "In " + sourcePomLoc + ": " + message + "  " + dependency;
         boolean ignore = new YesNoQuestion(question, defaultToIgnore).ask();
         if (!ignore) {
@@ -230,4 +233,4 @@ public class IgnoreDependencyQuestions {
         return "";
     }
 }
- 
\ No newline at end of file
+ 

Reply via email to