I've had a second look at this issue and it seems pretty safe to change this: the URL's are only being fed to the URLClassLoader.
The tests still run smoothly so I'll fix this issue. - Robert > Date: Thu, 15 Jul 2010 22:02:46 +0200 > From: [email protected] > To: [email protected] > Subject: Re: [mojo-dev] exec-maven-plugin 1.1.2 release ? > > Robert Scholte, > > > I know File.toURL() is deprecated, but I've never seen any proof that > > File.toURI().toURL() is enough to fix this problem. > > Well, the difference in output of > > System.out.println( new File( " " ).toURL() ); > System.out.println( new File( " " ).toURI().toURL() ); > > is easy to notice and checking the URL/URI specs, you see that the > literal space character isn't valid. So, passing around a URL with > unquoted space (or other illegal characters) puts one at the mercy of > the other components. Might wore here and fail there. See MNG-3607 for a > related issue in Maven core and MJBOSS-35 for a nasty consequence. > > > Benjamin > > --------------------------------------------------------------------- > To unsubscribe from this list, please visit: > > http://xircles.codehaus.org/manage_email > > _________________________________________________________________ New Windows 7: Find the right PC for you. Learn more. http://windows.microsoft.com/shop
