Skip to content

Replace quotes for xmllint on Windows Git Bash #16

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

Merged
merged 1 commit into from
Nov 10, 2017

Conversation

imagejan
Copy link
Member

Fixes #15 on Windows.

@ctrueden could you please test if this still works for you?

Surprisingly, xmllint in Git Bash only accepted this way of quoting, otherwise it would return an empty XPath set.

@ctrueden
Copy link
Member

ctrueden commented Nov 10, 2017

Should be OK. Worked in my tests. Actually, I think it's generally safer/better because everything inside the single quotes is literal, not intended for expansion by any shell. Thanks for the fix!

@ctrueden ctrueden merged commit a048aeb into scijava:master Nov 10, 2017
@imagejan imagejan deleted the fix-xmllint-on-windows branch November 10, 2017 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants