Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Better URI-to-File conversion #185

Closed
wants to merge 1 commit into from
Closed

Conversation

@rdeltour
Copy link
Contributor

@rdeltour rdeltour commented Dec 24, 2014

This should notably better handle UNC paths, which are often
represented as file URIs with an authority component.

I tested a few I/O-involving steps using this test file.
There might be some URI-to-File conversion leftovers that are not properly handled (e.g. I haven't touched XProcConfiguration), but this PR should cover most of the step implementations.

This should notably better handle UNC paths, which are often
represented as file URIs with an authority component.
rdeltour added a commit to daisy/osgi-libs that referenced this pull request Dec 24, 2014
 * Calabash sources are now fetched from Git via `maven-scm-plugin`
 * Patches in `src/main/patches` are applied
 * 1.0.23 is patched to fix the UNC path issue, see:
   ndw/xmlcalabash1#185
rdeltour added a commit to daisy/osgi-libs that referenced this pull request Jan 13, 2015
 * Calabash sources are now fetched from Git via `maven-scm-plugin`
 * Patches in `src/main/patches` are applied
 * 1.0.23 is patched to fix the UNC path issue, see:
   ndw/xmlcalabash1#185
bertfrees pushed a commit to daisy/xmlcalabash1 that referenced this pull request Mar 30, 2017
 * Calabash sources are now fetched from Git via `maven-scm-plugin`
 * Patches in `src/main/patches` are applied
 * 1.0.23 is patched to fix the UNC path issue, see:
   ndw#185
ndw added a commit that referenced this pull request Mar 4, 2018
ndw added a commit that referenced this pull request Mar 4, 2018
ndw added a commit that referenced this pull request Mar 4, 2018
@ndw ndw closed this Mar 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants