You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "gnodet (via GitHub)" <gi...@apache.org> on 2026/07/28 09:55:47 UTC
[PR] [3.10.x] Convert paths for MSYS2 and jvm.config placeholders in bin/mvn [maven]
gnodet opened a new pull request, #12567:
URL: https://github.com/apache/maven/pull/12567
## Summary
Backport of #12546 to the `maven-3.10.x` branch, adapted for the different script location (`src/bin/mvn` vs `src/assembly/maven/bin/mvn`) and the simpler jvm.config handling (no `JvmConfigParser` on this branch).
- Detect MSYS2 (`MSYS_NT-*`) alongside MinGW in `uname` matching
- Guard against missing `cygpath(1)` — warn and fall back to POSIX paths
- Convert all paths (`MAVEN_HOME`, `CLASSWORLDS_JAR`, `MAVEN_PROJECTBASEDIR`, jline-native) for both Cygwin and MinGW/MSYS2, not just Cygwin
- Convert jline-native path separately to avoid mixed separators (`C:\maven/lib/jline-native`)
- Use `printf` instead of `echo` for values holding paths (avoid backslash expansion in dash/ash)
- Add shell test covering Cygwin, MinGW, MSYS2, and missing-cygpath fallback, wired into the build via `exec-maven-plugin`
## Test plan
- [x] Shell test passes locally (21/21 assertions)
- [ ] CI passes on all platforms (Linux, macOS, Windows)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [3.10.x] Convert paths for MSYS2 and jvm.config placeholders in bin/mvn [maven]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #12567:
URL: https://github.com/apache/maven/pull/12567#issuecomment-5109029726
@gnodet Please assign appropriate label to PR according to the type of change.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [3.10.x] Convert paths for MSYS2 and jvm.config placeholders in bin/mvn [maven]
Posted by "gnodet (via GitHub)" <gi...@apache.org>.
gnodet merged PR #12567:
URL: https://github.com/apache/maven/pull/12567
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org