You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ant.apache.org by Charles Duffy <ch...@dyfis.net> on 2013/11/11 19:51:25 UTC
[PATCH] Ivy useOrigin=true support for URLResolver with file:// URLs
This isn't actually so important for URLResolver itself, which can be
replaced with the FileSystem resolver at will -- but without this patch,
one can't get the unique behavior of the IBiblio resolver combined with
useOrigin.
Feedback appreciated.
Re: [PATCH] Ivy useOrigin=true support for URLResolver with file:// URLs
Posted by Charles Duffy <ch...@dyfis.net>.
Thank you kindly for the review!
The attached revision of the patch uses the File(URI) constructor to
perform the necessary decoding.
On Sun, Nov 17, 2013 at 6:08 PM, Antoine Levy Lambert <an...@gmx.de>wrote:
> Hello Charles,
>
> thanks for proposing a patch.
>
> I have read the patch and I see one area for improvement.
>
> Your patch seems to assume that File URLs have exactly the same syntax
> like file system paths except the file: prefix to indicate the protocol.
>
> You need to use URLDecoder or equivalent because a File URL will have some
> special characters encoded such as spaces replaced by %32.
>
> Regards,
>
> Antoine
>
> On Nov 11, 2013, at 1:51 PM, Charles Duffy wrote:
>
> > This isn't actually so important for URLResolver itself, which can be
> replaced with the FileSystem resolver at will -- but without this patch,
> one can't get the unique behavior of the IBiblio resolver combined with
> useOrigin.
> >
> > Feedback appreciated.
> > <url-local-cache-support.diff>
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
> > For additional commands, e-mail: dev-help@ant.apache.org
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
> For additional commands, e-mail: dev-help@ant.apache.org
>
>
Re: [PATCH] Ivy useOrigin=true support for URLResolver with file:// URLs
Posted by Antoine Levy Lambert <an...@gmx.de>.
Hello Charles,
thanks for proposing a patch.
I have read the patch and I see one area for improvement.
Your patch seems to assume that File URLs have exactly the same syntax like file system paths except the file: prefix to indicate the protocol.
You need to use URLDecoder or equivalent because a File URL will have some special characters encoded such as spaces replaced by %32.
Regards,
Antoine
On Nov 11, 2013, at 1:51 PM, Charles Duffy wrote:
> This isn't actually so important for URLResolver itself, which can be replaced with the FileSystem resolver at will -- but without this patch, one can't get the unique behavior of the IBiblio resolver combined with useOrigin.
>
> Feedback appreciated.
> <url-local-cache-support.diff>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
> For additional commands, e-mail: dev-help@ant.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org