RFR: Fix 2 failing tests due to URI syntax errors

Erik Duveblad via github.com duke at openjdk.java.net
Thu Aug 22 13:10:35 UTC 2019


On Thu, 22 Aug 2019 13:01:59 GMT, Jorn Vernee via github.com <duke at openjdk.java.net> wrote:

> There were 2 tests failing due to incorrect URI syntax resulting from directly using a path string as URI, which on Windows will contain incorrect path separators.
> 
> This PR forces the path separators to be unix ones, and inserts an explicit leading slash before the local path starts (both are needed according to the URI spec).
> 
> ----------------
> 
> Commits:
>  - 3ea02c85:	Force unix line endings when creating URI, and use three slashes which is needed if no hostname is in the URI
> 
> Pull request:
> https://git.openjdk.java.net/skara/pull/69
> 
> Webrev:
> https://webrevs.openjdk.java.net/skara/69/webrev.00
> 
> Patch:
> https://git.openjdk.java.net/skara/pull/69.diff
> 
> Fetch command:
> git fetch https://git.openjdk.java.net/skara pull/69/head:pull/69

This PR has been reviewed by Erik Duveblad via github.com - changes are approved. Review comment:

Looks good, thanks for fixing!

PR: https://git.openjdk.java.net/skara/pull/69


More information about the skara-dev mailing list