You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by da...@apache.org on 2020/03/22 09:07:19 UTC

[camel] 12/13: CAMEL-14762: Fixed CS

This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git

commit 94c9c3f65a5d4aa8fb03c1c954ba2cf3f65bf3c8
Author: Claus Ibsen <cl...@gmail.com>
AuthorDate: Sun Mar 22 10:01:07 2020 +0100

    CAMEL-14762: Fixed CS
---
 .../camel/maven/packaging/MainConfigurerMojo.java   | 21 ++++++++++-----------
 1 file changed, 10 insertions(+), 11 deletions(-)

diff --git a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/MainConfigurerMojo.java b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/MainConfigurerMojo.java
index 0cc5d5a..42cfc82 100644
--- a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/MainConfigurerMojo.java
+++ b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/MainConfigurerMojo.java
@@ -39,6 +39,13 @@ import org.springframework.util.ReflectionUtils;
 @Mojo(name = "generate-main-configurer", threadSafe = true, defaultPhase = LifecyclePhase.PROCESS_CLASSES)
 public class MainConfigurerMojo extends AbstractGeneratorMojo {
 
+    private static final String[] CLASS_NAMES = new String[]{
+        "org.apache.camel.main.MainConfigurationProperties",
+        "org.apache.camel.main.HystrixConfigurationProperties",
+        "org.apache.camel.main.Resilience4jConfigurationProperties",
+        "org.apache.camel.main.RestConfigurationProperties",
+        "org.apache.camel.ExtendedCamelContext"};
+
     /**
      * The output directory for generated java source code
      */
@@ -50,18 +57,10 @@ public class MainConfigurerMojo extends AbstractGeneratorMojo {
 
     private DynamicClassLoader projectClassLoader;
 
-    private static final String[] CLASS_NAMES = new String[]{
-            "org.apache.camel.main.MainConfigurationProperties",
-            "org.apache.camel.main.HystrixConfigurationProperties",
-            "org.apache.camel.main.Resilience4jConfigurationProperties",
-            "org.apache.camel.main.RestConfigurationProperties",
-            "org.apache.camel.ExtendedCamelContext"};
-
     private static class Option extends BaseOptionModel {
-        // we just use name, type
-
 
         public Option(String name, Class type) {
+            // we just use name, type
             setName(name);
             setJavaType(type.getName());
         }
@@ -139,11 +138,11 @@ public class MainConfigurerMojo extends AbstractGeneratorMojo {
         }
     }
 
-    protected void generateMetaInfConfigurer(String fqn) {
+    private void generateMetaInfConfigurer(String fqn) {
         String name = fqn.substring(fqn.lastIndexOf('.') + 1);
         try (Writer w = new StringWriter()) {
             w.append("# " + GENERATED_MSG + "\n");
-            w.append("class=").append(fqn + "Configurer").append("\n");
+            w.append("class=").append(fqn).append("Configurer").append("\n");
             updateResource(resourcesOutputDir.toPath(), "META-INF/services/org/apache/camel/configurer/" + name, w.toString());
         } catch (IOException e) {
             throw new RuntimeException(e);