[jira] Commented: (SUREFIRE-30) Wrong classpath separator

["Brett Porter (JIRA)" <ji...@codehaus.org>]

    [ http://jira.codehaus.org/browse/SUREFIRE-30?page=comments#action_60513 ] 

Brett Porter commented on SUREFIRE-30:
--------------------------------------

that's what I used in the 2.0 version. As you can see, it is already marked fixed.

> Wrong classpath separator
> -------------------------
>
>          Key: SUREFIRE-30
>          URL: http://jira.codehaus.org/browse/SUREFIRE-30
>      Project: surefire
>         Type: Bug

>     Versions: 1.5.2, 1.5.3
>  Environment: Only Windows
>     Reporter: Marcin Cetnarski
>     Assignee: Brett Porter
>      Fix For: 2.0
>  Attachments: SurefireBooter.patch
>
>
> In SurefireBooter when use fork mode elements of classpath are separated by colon. This works on Linux but not on Windows. I sugest to use semicolon.
>    private static ClassLoader createForkingClassLoader( String basedir )
>         throws Exception
>     {
>         Properties p = loadProperties( basedir, CLASSLOADER_PROPERTIES );
>         String cp = p.getProperty( "classpath" );
>         boolean childDelegation = "true".equals( p.getProperty( "childDelegation", "false" ) );
>         List urls = Arrays.asList( cp.split( ";" ) );  // was List urls = Arrays.asList( cp.split( ":" ) );
>         return createClassLoader( urls, childDelegation );
>     }
> and 
>    private void getForkArgs( String batteryConfig )
>         throws Exception
>     {
> .....
>             else
>             {
>                 if ( cp.length() == 0 )
>                     cp = url;
>                 else
>                     cp += ";" + url; // was cp += ":" + url;
>             }

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira