Switched some warnings over to debug, as they always occur when installing a bundle created using the bundle plugin and the log message is confusing (it doesn't tell the user if/why they should be concerned about this issue - in fact I can't really tell why it's a warning). Also tweaked a couple of info messages to give the user better feedback.
git-svn-id: https://svn.apache.org/repos/asf/felix/trunk@579737 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/maven-obr-plugin/src/main/java/org/apache/felix/sandbox/obr/plugin/ObrUpdate.java b/maven-obr-plugin/src/main/java/org/apache/felix/sandbox/obr/plugin/ObrUpdate.java
index 017fe08..6bb0da4 100644
--- a/maven-obr-plugin/src/main/java/org/apache/felix/sandbox/obr/plugin/ObrUpdate.java
+++ b/maven-obr-plugin/src/main/java/org/apache/felix/sandbox/obr/plugin/ObrUpdate.java
@@ -230,6 +230,7 @@
}
// the repository.xml file have been modified, so we save it
+ m_logger.info("Writing OBR metadata");
writeToFile(m_repoFilename, m_root);
}
@@ -264,7 +265,7 @@
try {
fout.createNewFile();
- m_logger.info("Created new repository.xml file in "+fout.getAbsolutePath());
+ m_logger.info("Created "+fout.getAbsolutePath());
} catch (IOException e) {
m_logger.error("Cannot create file " + fout.getAbsolutePath());
e.printStackTrace();
@@ -304,7 +305,7 @@
private Document parseFile(String filename, DocumentBuilder constructor) {
if (constructor == null) { return null; }
// The document is the root of the DOM tree.
- m_logger.info("Try to open: " + filename);
+ m_logger.info("Parsing " + filename);
Document doc = null;
try {
doc = constructor.parse(new File(filename));
diff --git a/maven-obr-plugin/src/main/java/org/apache/felix/sandbox/obr/plugin/ResourcesBundle.java b/maven-obr-plugin/src/main/java/org/apache/felix/sandbox/obr/plugin/ResourcesBundle.java
index 29a164d..5d006f1 100644
--- a/maven-obr-plugin/src/main/java/org/apache/felix/sandbox/obr/plugin/ResourcesBundle.java
+++ b/maven-obr-plugin/src/main/java/org/apache/felix/sandbox/obr/plugin/ResourcesBundle.java
@@ -314,42 +314,42 @@
if (ebi.getPresentationName() != null) {
this.setPresentationName(ebi.getPresentationName());
- if (project.getName() != null) { m_logger.warn("pom property override:<presentationname> " + project.getName()); }
+ if (project.getName() != null) { m_logger.debug("pom property override:<presentationname> " + project.getName()); }
} else {
this.setPresentationName(project.getName());
}
if (ebi.getSymbolicName() != null) {
this.setSymbolicName(ebi.getSymbolicName());
- if (project.getArtifactId() != null) { m_logger.warn("pom property override:<symbolicname> " + project.getArtifactId()); }
+ if (project.getArtifactId() != null) { m_logger.debug("pom property override:<symbolicname> " + project.getArtifactId()); }
} else {
this.setSymbolicName(project.getArtifactId());
}
if (ebi.getVersion() != null) {
this.setVersion(ebi.getVersion());
- if (project.getVersion() != null) { m_logger.warn("pom property override:<version> " + project.getVersion()); }
+ if (project.getVersion() != null) { m_logger.debug("pom property override:<version> " + project.getVersion()); }
} else {
this.setVersion(project.getVersion());
}
if (ebi.getDescription() != null) {
this.setDescription(ebi.getDescription());
- if (project.getDescription() != null) { m_logger.warn("pom property override:<description> " + project.getDescription()); }
+ if (project.getDescription() != null) { m_logger.debug("pom property override:<description> " + project.getDescription()); }
} else {
this.setDescription(project.getDescription());
}
if (ebi.getDocumentation() != null) {
this.setDocumentation(ebi.getDocumentation());
- if (project.getUrl() != null) { m_logger.warn("pom property override:<documentation> " + project.getUrl()); }
+ if (project.getUrl() != null) { m_logger.debug("pom property override:<documentation> " + project.getUrl()); }
} else {
this.setDocumentation(project.getUrl());
}
if (ebi.getSource() != null) {
this.setSource(ebi.getSource());
- if (project.getScm() != null) { m_logger.warn("pom property override:<source> " + project.getScm()); }
+ if (project.getScm() != null) { m_logger.debug("pom property override:<source> " + project.getScm()); }
} else {
String src = null;
if (project.getScm() != null) { src = project.getScm().getUrl(); }
@@ -363,7 +363,7 @@
Iterator it = l.iterator();
while (it.hasNext()) {
if (it.next() != null) {
- m_logger.warn("pom property override:<source> " + lic);
+ m_logger.debug("pom property override:<license> " + lic);
break;
}
}