gnodet opened a new pull request, #12379:
URL: https://github.com/apache/maven/pull/12379

   ## Summary
   
   Backport of #12378 to maven-4.0.x.
   
   - Replace `for /f` with `set /p <file` when reading JvmConfigParser output 
in `mvn.cmd`
   - `for /f` silently skips locked files (e.g. Windows Defender real-time 
scanning), leaving `JVM_CONFIG_MAVEN_OPTS` empty and causing intermittent 
MNG-4559 test failures on Windows CI (~30% failure rate)
   - `set /p` uses different file sharing flags and is far less susceptible to 
lock contention
   - Adds a single retry with brief delay as defense-in-depth
   
   ## Test plan
   
   - [x] 3 consecutive full CI runs on #12378 — all 18 Windows jobs passed (0 
failures)
   - [x] MNG-4559 integration tests (spaces in jvm.config, multiple JVM args) 
pass reliably on 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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to