gnodet opened a new pull request, #12563:
URL: https://github.com/apache/maven/pull/12563
## Summary
- Fixes the 22 test assertion failures in `test-mvn-path-conversion.sh` that
affect all macOS CI jobs in PR #12546
- **Root cause:** On macOS, `$TMPDIR` ends with a trailing `/` (e.g.
`/var/folders/.../T/`), so `mktemp -d "${TMPDIR}/..."` produces a path with a
double slash (`/T//mvn-path-...`). The `pwd` builtin inside the `mvn` launcher
normalizes this to a single slash, causing every path assertion in the test to
mismatch.
- **Fix:** Normalize `work_dir` through `cd/pwd` immediately after `mktemp`,
so the test's reference paths match what the launcher produces.
This is a one-line fix (plus comment) that resolves all 22 failures without
changing the launcher script itself.
Applies on top of PR #12546.
## Test plan
- [x] All 29 assertions pass locally (Linux)
- [ ] CI should now pass on macOS (was failing on all 6 macOS jobs)
- [ ] Ubuntu and Windows jobs should continue to pass (unaffected by this
change)
🤖 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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
