[MNG-5035] \ -> / on Unix for java.io.File-typed mojo param

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Not A Problem
• Affects Version/s: 3.0.3
• Component/s: Plugin API
• Environment: Ubuntu 10.04, JDK 6
• Assignee: Benjamin Bentmann
• Reporter: Jesse N. Glick
• Created: 07/Mar/11 10:06
• Updated: 07/Mar/11 12:44
• Resolved: 07/Mar/11 11:49

Description

I created a file /tmp/a\b/c (note backslash). I created a quickstart app with

    public static void main( String[] args )
    {
        System.out.println( System.getProperty("user.dir") );
        System.out.println(Arrays.toString(new File("").listFiles()));
    }

and then run it with

mvn -Dexec.workingdir=/tmp/a\\b "-Dexec.args=-classpath %classpath ....App" -Dexec.executable=.../java process-classes org.codehaus.mojo:exec-maven-plugin:1.2:exec -X

I see

[DEBUG] Configuring mojo 'org.codehaus.mojo:exec-maven-plugin:1.2:exec' with basic configurator -->
[DEBUG]   (f) workingDirectory = /tmp/a/b
[DEBUG] Making working directory '/tmp/a/b'.
/tmp/a/b
null

where I expected to see

[DEBUG] Configuring mojo 'org.codehaus.mojo:exec-maven-plugin:1.2:exec' with basic configurator -->
[DEBUG]   (f) workingDirectory = /tmp/a\b
/tmp/a\b
[/tmp/a\b/c]

I believe the guilty code is in org.codehaus.plexus.component.configurator.converters.basic.FileConverter:

new File( str.replace( '\\', File.separatorChar ).replace( '/', File.separatorChar ) )

should probably replace only / with \ and only on Windows. Not sure what compatibility regressions there might be, if some POM written by a Windows developer uses \ as the separator character for relative paths.

Issue Links

• is related to: MNG-4464 Improve handling of relative paths with backslashes