Merge pull request #1288 from jojansen/KARAF-6992
KARAF-6992: maven-release-plugin: Only append config settings if config file exist
diff --git a/profile/src/main/java/org/apache/karaf/profile/assembly/ConfigInstaller.java b/profile/src/main/java/org/apache/karaf/profile/assembly/ConfigInstaller.java
index df928f5..4f78059 100644
--- a/profile/src/main/java/org/apache/karaf/profile/assembly/ConfigInstaller.java
+++ b/profile/src/main/java/org/apache/karaf/profile/assembly/ConfigInstaller.java
@@ -65,7 +65,8 @@
for (Config config : content.getConfig()) {
if (pidMatching(config.getName())) {
Path configFile = etcDirectory.resolve(config.getName() + ".cfg");
- if (!config.isAppend() && Files.exists(configFile)) {
+ boolean configFileExist = Files.exists(configFile);
+ if (!config.isAppend() && configFileExist) {
LOGGER.info(" not changing existing config file: {}", homeDirectory.relativize(configFile));
continue;
}
@@ -84,8 +85,13 @@
});
} else {
if (config.isAppend()) {
- LOGGER.info(" appending to config file: {}", homeDirectory.relativize(configFile));
- Files.write(configFile, config.getValue().getBytes(), StandardOpenOption.APPEND);
+ if (configFileExist) {
+ LOGGER.info(" appending to config file: {}", homeDirectory.relativize(configFile));
+ Files.write(configFile, config.getValue().getBytes(), StandardOpenOption.APPEND);
+ }
+ else
+ LOGGER.warn(" Could not append, because config file does not exist: {}", homeDirectory.relativize(configFile));
+
} else {
LOGGER.info(" adding config file: {}", homeDirectory.relativize(configFile));
Files.write(configFile, config.getValue().getBytes());