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

orpiske pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel-kafka-connector.git

commit 6db14871295c3068f1ab1a3282d4fd639996b101
Author: Otavio Rodolfo Piske <[email protected]>
AuthorDate: Wed Feb 24 11:50:20 2021 +0100

    Avoid hard-coded separators and they can cause portability problems
---
 .../camel/kafkaconnector/maven/CamelKafkaConnectorUpdateMojo.java   | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/tooling/camel-kafka-connector-generator-maven-plugin/src/main/java/org/apache/camel/kafkaconnector/maven/CamelKafkaConnectorUpdateMojo.java
 
b/tooling/camel-kafka-connector-generator-maven-plugin/src/main/java/org/apache/camel/kafkaconnector/maven/CamelKafkaConnectorUpdateMojo.java
index 6ae3e1a..8a20072 100644
--- 
a/tooling/camel-kafka-connector-generator-maven-plugin/src/main/java/org/apache/camel/kafkaconnector/maven/CamelKafkaConnectorUpdateMojo.java
+++ 
b/tooling/camel-kafka-connector-generator-maven-plugin/src/main/java/org/apache/camel/kafkaconnector/maven/CamelKafkaConnectorUpdateMojo.java
@@ -447,7 +447,7 @@ public class CamelKafkaConnectorUpdateMojo extends 
AbstractCamelKafkaConnectorMo
 
         confMethod.setBody(confMethod.getBody() + "return conf;");
 
-        String javaClassConnectorConfigFileName = 
packageName.replaceAll("\\.", "\\/") + "/" + javaClassConnectorConfigName + 
".java";
+        String javaClassConnectorConfigFileName = 
packageName.replaceAll("\\.", "\\/") + File.separator + 
javaClassConnectorConfigName + ".java";
         MavenUtils.writeSourceIfChanged(javaClassConnectorConfig, 
javaClassConnectorConfigFileName, false, connectorDir, 
rm.getResourceAsFile(javaFilesHeader));
 
         // Camel{sanitizedName}{Sink,Source}Task.java
@@ -475,7 +475,7 @@ public class CamelKafkaConnectorUpdateMojo extends 
AbstractCamelKafkaConnectorMo
         }
         getDefaultConfigMethod.setBody(getDefaultConfigMethod.getBody() + 
"}};\n");
         getDefaultConfigMethod.addAnnotation(Override.class);
-        String javaClassTaskFileName = packageName.replaceAll("\\.", "\\/") + 
"/" + javaClassTaskName + ".java";
+        String javaClassTaskFileName = packageName.replaceAll("\\.", "\\/") + 
File.separator + javaClassTaskName + ".java";
         MavenUtils.writeSourceIfChanged(javaClassTask, javaClassTaskFileName, 
false, connectorDir, rm.getResourceAsFile(javaFilesHeader));
 
         // Camel{sanitizedName}{Sink,Source}Connector.java
@@ -495,7 +495,7 @@ public class CamelKafkaConnectorUpdateMojo extends 
AbstractCamelKafkaConnectorMo
         
javaClassConnector.addMethod().setConstructor(false).setName("taskClass").setPublic().setReturnType("Class<?
 extends Task>")
             .setBody("return Camel" + 
StringUtils.capitalize(sanitizedName.replace("-", "")) + ctCapitalizedName + 
"Task.class;").addAnnotation(Override.class);
 
-        String javaClassConnectorFileName = packageName.replaceAll("\\.", 
"\\/") + "/" + javaClassConnectorName + ".java";
+        String javaClassConnectorFileName = packageName.replaceAll("\\.", 
"\\/") + File.separator + javaClassConnectorName + ".java";
         MavenUtils.writeSourceIfChanged(javaClassConnector, 
javaClassConnectorFileName, false, connectorDir, 
rm.getResourceAsFile(javaFilesHeader));
 
         List<String> convertersList = new ArrayList<>();

Reply via email to