gavioto / m2e-extensions

Automatically exported from code.google.com/p/m2e-extensions
0 stars 0 forks source link

Checkstyle Excludes need to be relative paths #10

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Something like this:

--- 
a/maven/ide/plugins/org.maven.ide.eclipse.extensions.project.configurators.check
style/src/main/java/org/maven/ide/eclipse/extensions/project/configurators/check
style/MavenPluginConfigurationTranslator.java     Sun Feb 28 10:51:01 2010 -0600
+++ 
b/maven/ide/plugins/org.maven.ide.eclipse.extensions.project.configurators.check
style/src/main/java/org/maven/ide/eclipse/extensions/project/configurators/check
style/MavenPluginConfigurationTranslator.java     Mon Sep 27 11:20:20 2010 -0700
@@ -219,9 +219,8 @@
          */
         final List<String> excludePatterns = this.getExcludes();
         for (String folder : sourceFolders) {
-            String folderRelativePath = URIUtils
-                .resolve(this.basedirUri, new File(folder).toURI())
-                .getPath();
+            String folderRelativePath = 
+               this.basedirUri.relativize(new File(folder).toURI()).getPath();
             patterns.addAll(this.normalizePatternsToCheckstyleFileMathcPattern(
                     excludePatterns, 
                     this.convertToEclipseCheckstyleRegExpPath(folderRelativePath), 

Original issue reported on code.google.com by jatw...@linuxstuff.org on 27 Sep 2010 at 6:24

GoogleCodeExporter commented 9 years ago
Committed this patch in f0453cdde3

Original comment by mr.id...@gmail.com on 20 Dec 2010 at 5:27