msys2 / msys2/msys2-runtime

Unexpected change in path conversion behaviour after #143

Open
#150 4 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

pathconv
Dominant language
C
Stars
240
Forks
60
Avg merge
2d 3h
Merged PRs (30d)
1

Description

After #143, my xmllint command has broken:

(msys2-runtime 3.4.6-2)

$ xmllint --xpath 'string(settings/servers/server[id = "gitlab"]/configuration/httpHeaders/property/name)' ~/.m2/settings.xml
XPath error : Invalid expression
string(settings/servers/server[id = "gitlab"]C:/msys64_b/configuration/httpHeaders/property/name)
                                             ^
XPath evaluation failure

because the space in the argument somehow means that the path conversion behaviour is triggered for the next part of the argument, despite the whole thing being quoted.

I've managed to reduce it to a minimal example below:

image

It's not quite clear from #143 whether this was intentional or not, certainly the behaviour is a little strange. If so, what's the correct work around? I suppose I could set MSYS2_ARG_CONV_EXCL=* for just that command, but that feels hacky at best. Is there something more permanent I can set in my environment that will work for "all" commands?

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the minimal xmllint reproduction in the issue and compare the path-conversion behavior introduced by #143. The payload names no source files or tests; done means determining whether the quoted XPath behavior is intended and fixing it or documenting a supported permanent workaround.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
operating-systems
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.