Hiteshsai007 commented on PR #12563: URL: https://github.com/apache/maven/pull/12563#issuecomment-5101317059
Confirmed — this is a bug in my test harness, not the launcher. I reproduced it locally with TMPDIR=/tmp/foo/ and got exactly the same 22 failures: pwd inside the launcher collapses the // that mktemp returns, so the test's reference paths never matched. Verified your fix passes with a trailing-slash TMPDIR, a normal one, and with TMPDIR unset, across sh/dash/bash/busybox sh. It also still reports failures against the unpatched launcher, so the normalisation hasn't made the test vacuous. Thanks for catching and fixing it — happy for this to be merged in place of #12546. -- 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]
