Merge pull request #163 from policeman-tools/dev/emptyclasseswarning

Hides the warning if the classes directory/fileset is missing
policeman-tools · Apr 13, 2020 · b3bfa4c · b3bfa4c
2 parents 4416a82 + 8c5bb00
commit b3bfa4c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/src/main/java/de/thetaphi/forbiddenapis/ant/AntTask.java b/src/main/java/de/thetaphi/forbiddenapis/ant/AntTask.java
@@ -83,7 +83,7 @@ public void error(String msg) {
       @Override
       public void warn(String msg) {
         // ANT has no real log levels printed, so prefix with "WARNING":
-        log("WARNING: " + msg, Project.MSG_WARN);
+        log("WARNING: ".concat(msg), Project.MSG_WARN);
       }
 
       @Override
@@ -188,8 +188,7 @@ public void info(String msg) {
         }
         if (!foundClass) {
           if (ignoreEmptyFileset) {
-            log.warn("There is no <fileset/> or other resource collection given, or the collection does not contain any class files to check.");
-            log.info("Scanned 0 class files.");
+            log.info("Resource collection of class files is empty. Scanned 0 class files.");
             return;
           } else {
             throw new BuildException("There is no <fileset/> or other resource collection given, or the collection does not contain any class files to check.");
@@ -341,7 +340,7 @@ public void setRestrictClassFilename(boolean restrictClassFilename) {
     this.restrictClassFilename = restrictClassFilename;
   }
 
-  /** Ignore empty fileset/resource collection and print a warning instead.
+  /** Ignore empty fileset/resource collection.
    * Defaults to {@code false}.
    */
   public void setIgnoreEmptyFileSet(boolean ignoreEmptyFileset) {

diff --git a/src/main/java/de/thetaphi/forbiddenapis/maven/AbstractCheckMojo.java b/src/main/java/de/thetaphi/forbiddenapis/maven/AbstractCheckMojo.java
@@ -336,7 +336,7 @@ public void info(String msg) {
       log.info("Scanning for classes to check...");
       final File classesDirectory = getClassesDirectory();
       if (!classesDirectory.exists()) {
-        log.warn("Classes directory does not exist, forbiddenapis check skipped: " + classesDirectory);
+        log.info("Classes directory does not exist, forbiddenapis check skipped: " + classesDirectory);
         return;
       }
       final DirectoryScanner ds = new DirectoryScanner();
@@ -348,7 +348,7 @@ public void info(String msg) {
       ds.scan();
       final String[] files = ds.getIncludedFiles();
       if (files.length == 0) {
-        log.warn(String.format(Locale.ENGLISH,
+        log.info(String.format(Locale.ENGLISH,
           "No classes found in '%s' (includes=%s, excludes=%s), forbiddenapis check skipped.",
           classesDirectory.toString(), Arrays.toString(includes), Arrays.toString(excludes)));
         return;