Classpath too long on windows with useManifestOnlyJar=false

If the useMandifestOnlyJar=false, then then jnuit won't run on windows because the classpath is to long. The better way to fork a java process is to set the CLASSPATH env variable instead of passing it on the command line.

This patch fixes the issue.

Index: src/main/java/org/apache/maven/plugin/surefire/booterclient/ForkConfiguration.java
===================================================================
— src/main/java/org/apache/maven/plugin/surefire/booterclient/ForkConfiguration.java (revision 1092789)
+++ src/main/java/org/apache/maven/plugin/surefire/booterclient/ForkConfiguration.java (working copy)
@@ -221,13 +221,13 @@
}
else

{ - cli.createArg().setValue( "-classpath" ); - - cli.createArg().setValue( StringUtils.join( classPath.iterator(), File.pathSeparator ) ); +// cli.createArg().setValue( "-classpath" ); +// cli.createArg().setValue( StringUtils.join( classPath.iterator(), File.pathSeparator ) ); + cli.addEnvironment("CLASSPATH",StringUtils.join( classPath.iterator(), File.pathSeparator )); + + final String forkedBooter = ForkedBooter.class.getName(); - final String forkedBooter = ForkedBooter.class.getName(); - - cli.createArg().setValue( shadefire ? new Relocator( ).relocate( forkedBooter ) : forkedBooter); + cli.createArg().setValue( shadefire ? new Relocator( ).relocate( forkedBooter ) : forkedBooter); }

cli.setWorkingDirectory( workingDirectory.getAbsolutePath() );