Re: File.toURL() and org.apache.tools.ant.launch.Locator.fileToURL(File)

2012-08-22 Thread Jesse Glick
On 08/22/2012 02:30 PM, Nicolas Lalevée wrote: I could not think we can put 4 slashes file:server/share/path is the broken result of new File("server\\share\\path").toURI(). While new File(URI) works to round-trip it (IIRC), if you call URI.normalize() or otherwise do any kind of inter

Re: File.toURL() and org.apache.tools.ant.launch.Locator.fileToURL(File)

2012-08-22 Thread Nicolas Lalevée
Le 22 août 2012 à 20:07, Jesse Glick a écrit : >> As far I understand, such path reference some file on a remote computer. But >> then why the host part of the URI being build is left to null? > > Line #3059 (JDK 7) would not be executed for a UNC path because you would > have file://server/sha

Re: File.toURL() and org.apache.tools.ant.launch.Locator.fileToURL(File)

2012-08-22 Thread Jesse Glick
On 08/22/2012 08:50 AM, Nicolas Lalevée wrote: Don't understand the encoding issue there, since we are manipulating Strings. But String's which represent pathnames on an OS which might natively represent pathnames as octet sequences in some encoding. Again I am not sure what the defining examp

Re: File.toURL() and org.apache.tools.ant.launch.Locator.fileToURL(File)

2012-08-22 Thread Nicolas Lalevée
Le 22 août 2012 à 00:04, Jesse Glick a écrit : > On 08/21/2012 03:13 PM, Nicolas Lalevée wrote: >> Or is there some code which rely on the broken behavior? > > Not that I know of. See the various commented-out code blocks and "XXX" > comments in this class; it has a long and messy history. I do

Re: File.toURL() and org.apache.tools.ant.launch.Locator.fileToURL(File)

2012-08-21 Thread Jesse Glick
On 08/21/2012 03:13 PM, Nicolas Lalevée wrote: Or is there some code which rely on the broken behavior? Not that I know of. See the various commented-out code blocks and "XXX" comments in this class; it has a long and messy history. I do not know whether the current code successfully works aro

Re: File.toURL() and org.apache.tools.ant.launch.Locator.fileToURL(File)

2012-08-21 Thread Jesse Glick
On 08/21/2012 03:41 PM, Martin Gainty wrote: Make sure you catch IllegalArgumentException - If this URL is not absolute Should never happen in this case. MalformedURLException - If a protocol handler for the URL could not be found, or if some other error occurred while constructing

RE: File.toURL() and org.apache.tools.ant.launch.Locator.fileToURL(File)

2012-08-21 Thread Martin Gainty
ligatoire. Étant donné que les email peuvent facilement être sujets à la manipulation, nous ne pouvons accepter aucune responsabilité pour le contenu fourni. > From: nicolas.lale...@hibnet.org > Subject: File.toURL() and org.apache.tools.ant.launch.Locator.fileToURL(File) > Date: Tu

File.toURL() and org.apache.tools.ant.launch.Locator.fileToURL(File)

2012-08-21 Thread Nicolas Lalevée
When trying to setup the build matrix for Ivy, I got hit by the spaces in the paths for the tests. It was due to the well known broken File.toURL(). I have done a broadcast search on Ivy and fix it (it was mainly the unit tests which were broken). So decided to look into IvyDE too and I have fou