SONAR-1706 Depreciate sonar.reuseExistingRulesConfiguration

plugins/sonar-checkstyle-plugin/src/main/java/org/sonar/plugins/checkstyle/CheckstyleConfiguration.java

@@ -57,6 +57,7 @@ public class CheckstyleConfiguration implements BatchExtension {
 
   public File getXMLDefinitionFile() {
     if (project.getReuseExistingRulesConfig()) {
+      LOG.warn("Reusing existing Checkstyle configuration is deprecated as it's unstable and can not provide meaningful results. This feature will be removed soon.");
       return findExistingXML();
     }
 

plugins/sonar-findbugs-plugin/src/main/java/org/sonar/plugins/findbugs/FindbugsMavenPluginHandler.java

@@ -24,6 +24,8 @@ import java.io.IOException;
 import java.io.StringWriter;
 
 import org.apache.commons.lang.StringUtils;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.sonar.api.CoreProperties;
 import org.sonar.api.batch.maven.MavenPlugin;
 import org.sonar.api.batch.maven.MavenPluginHandler;
@@ -37,6 +39,8 @@ import org.sonar.plugins.findbugs.xml.Match;
 
 public class FindbugsMavenPluginHandler implements MavenPluginHandler {
 
+  private static Logger LOG = LoggerFactory.getLogger(FindbugsMavenPluginHandler.class);
+
   private RulesProfile profile;
   private FindbugsProfileExporter exporter;
 
@@ -89,6 +93,9 @@ public class FindbugsMavenPluginHandler implements MavenPluginHandler {
       String existingIncludeFilterConfig = plugin.getParameter("includeFilterFile");
       String existingExcludeFilterConfig = plugin.getParameter("excludeFilterFile");
       boolean existingConfig = !StringUtils.isBlank(existingIncludeFilterConfig) || !StringUtils.isBlank(existingExcludeFilterConfig);
+      if (project.getReuseExistingRulesConfig()) {
+        LOG.warn("Reusing existing Findbugs configuration is deprecated as it's unstable and can not provide meaningful results. This feature will be removed soon.");
+      }
       if ( !project.getReuseExistingRulesConfig() || (project.getReuseExistingRulesConfig() && !existingConfig)) {
         File includeXmlFile = saveIncludeConfigXml(project);
         plugin.setParameter("includeFilterFile", getPath(includeXmlFile));

plugins/sonar-pmd-plugin/src/main/java/org/sonar/plugins/pmd/PmdConfiguration.java

@@ -25,6 +25,8 @@ import java.io.StringWriter;
 import java.util.Arrays;
 import java.util.List;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.sonar.api.BatchExtension;
 import org.sonar.api.CoreProperties;
 import org.sonar.api.batch.maven.MavenPlugin;
@@ -34,6 +36,8 @@ import org.sonar.api.resources.Project;
 
 public class PmdConfiguration implements BatchExtension {
 
+  private static Logger LOG = LoggerFactory.getLogger(PmdConfiguration.class);
+
   private PmdProfileExporter pmdProfileExporter;
   private RulesProfile rulesProfile;
   private Project project;
@@ -46,6 +50,7 @@ public class PmdConfiguration implements BatchExtension {
 
   public List<String> getRulesets() {
     if (project.getReuseExistingRulesConfig()) {
+      LOG.warn("Reusing existing PMD configuration is deprecated as it's unstable and can not provide meaningful results. This feature will be removed soon.");
       return getDeclaredRulesets();
     }
     return Arrays.asList(saveXmlFile().getAbsolutePath());