search

gavioto / m2e-extensions

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

Still possibly incorrect use of URUtils.resolve in PMD #11

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
As seen in issue #5, I'm beginning to think this is still the wrong thing to 
do.  On Windows machines, this generates a String prefixed with a / (eg, 
"/c:/users/someone/project/target/") - which is still suboptimal.

Will attach a patch once I figure out what to do.

Original issue reported on code.google.com by jatw...@linuxstuff.org on 1 Oct 2010 at 9:55

GoogleCodeExporter commented 9 years ago 
As far as I can tell, PMD likes *absolute* paths, so 
MavenPluginConfigurationTranslator.convertFileFoldersToRelativePathStrings 
really does the opposite of what it claims; I'm not a PMD expert though.

Original comment by jatw...@linuxstuff.org on 1 Oct 2010 at 10:02

GoogleCodeExporter commented 9 years ago 
Patched in f0453cdde3fc

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