This is an automated email from the ASF dual-hosted git repository. davsclaus pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/camel.git
commit 15168a5f28fdf905796d72d7f91a667144163052 Author: Claus Ibsen <[email protected]> AuthorDate: Sun Feb 20 10:32:07 2022 +0100 CAMEL-17665: camel-package-maven-plugin - False WARN about missing doc --- .../maven/packaging/PrepareCamelMainDocMojo.java | 6 +++-- .../camel/maven/packaging/PrepareExampleMojo.java | 6 +++-- .../camel/maven/packaging/UpdateReadmeMojo.java | 30 ++++++++++++++-------- 3 files changed, 28 insertions(+), 14 deletions(-) diff --git a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PrepareCamelMainDocMojo.java b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PrepareCamelMainDocMojo.java index 68c7a2c..4a7b092 100644 --- a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PrepareCamelMainDocMojo.java +++ b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PrepareCamelMainDocMojo.java @@ -88,8 +88,10 @@ public class PrepareCamelMainDocMojo extends AbstractGeneratorMojo { if (updated) { getLog().info("Updated doc file: " + file); - } else if (exists && getLog().isDebugEnabled()) { - getLog().debug("No changes to doc file: " + file); + } else if (exists) { + if (getLog().isDebugEnabled()) { + getLog().debug("No changes to doc file: " + file); + } } else { getLog().warn("No main doc file: " + file); } diff --git a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PrepareExampleMojo.java b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PrepareExampleMojo.java index 2f15b64..a0e305a 100644 --- a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PrepareExampleMojo.java +++ b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PrepareExampleMojo.java @@ -140,8 +140,10 @@ public class PrepareExampleMojo extends AbstractMojo { if (updated) { getLog().info("Updated readme.adoc file: " + file); - } else if (exists && getLog().isDebugEnabled()) { - getLog().debug("No changes to readme.adoc file: " + file); + } else if (exists) { + if (getLog().isDebugEnabled()) { + getLog().debug("No changes to readme.adoc file: " + file); + } } else { getLog().warn("No readme.adoc file: " + file); } diff --git a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/UpdateReadmeMojo.java b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/UpdateReadmeMojo.java index 5f6ab1f..45d4959 100644 --- a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/UpdateReadmeMojo.java +++ b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/UpdateReadmeMojo.java @@ -191,8 +191,10 @@ public class UpdateReadmeMojo extends AbstractGeneratorMojo { if (updated) { getLog().info("Updated doc file: " + file); - } else if (exists && getLog().isDebugEnabled()) { - getLog().debug("No changes to doc file: " + file); + } else if (exists) { + if (getLog().isDebugEnabled()) { + getLog().debug("No changes to doc file: " + file); + } } else { getLog().warn("No component doc file: " + file); if (isFailFast()) { @@ -237,8 +239,10 @@ public class UpdateReadmeMojo extends AbstractGeneratorMojo { if (updated) { getLog().info("Updated doc file: " + file); - } else if (exists && getLog().isDebugEnabled()) { - getLog().debug("No changes to doc file: " + file); + } else if (exists) { + if (getLog().isDebugEnabled()) { + getLog().debug("No changes to doc file: " + file); + } } else { getLog().warn("No component doc file: " + file); if (isFailFast()) { @@ -299,8 +303,10 @@ public class UpdateReadmeMojo extends AbstractGeneratorMojo { if (updated) { getLog().info("Updated doc file: " + file); - } else if (exists && getLog().isDebugEnabled()) { - getLog().debug("No changes to doc file: " + file); + } else if (exists) { + if (getLog().isDebugEnabled()) { + getLog().debug("No changes to doc file: " + file); + } } else { getLog().warn("No dataformat doc file: " + file); if (isFailFast()) { @@ -356,8 +362,10 @@ public class UpdateReadmeMojo extends AbstractGeneratorMojo { if (updated) { getLog().info("Updated doc file: " + file); - } else if (exists && getLog().isDebugEnabled()) { - getLog().debug("No changes to doc file: " + file); + } else if (exists) { + if (getLog().isDebugEnabled()) { + getLog().debug("No changes to doc file: " + file); + } } else { getLog().warn("No language doc file: " + file); if (isFailFast()) { @@ -415,8 +423,10 @@ public class UpdateReadmeMojo extends AbstractGeneratorMojo { if (updated) { getLog().info("Updated doc file: " + file); - } else if (exists && getLog().isDebugEnabled()) { - getLog().debug("No changes to doc file: " + file); + } else if (exists) { + if (getLog().isDebugEnabled()) { + getLog().debug("No changes to doc file: " + file); + } } else { getLog().warn("No eip doc file: " + file); if (isFailFast()) {
