gnodet opened a new pull request, #12971:
URL: https://github.com/apache/maven/pull/12971
## Summary
Cherry-pick of #12970 to `maven-4.0.x`.
- Force `providers("exec")` in the test's `TerminalBuilder` configuration so
JLine creates a lightweight `ExternalTerminal` instead of a `PosixPtyTerminal`
via the FFM provider
- With `system(false)`, the FFM provider calls `CLibrary.openpty()` creating
a real PTY pair; the subsequent grapheme-cluster probe (`ensureModesProbed()`)
then blocks on the idle PTY master for the full probe timeout, and the PTY's
pump threads can stall `close()` during teardown — making the tests flaky on
slow CI machines
- The exec provider's `ExternalTerminal` avoids all of this: no PTY, no pump
threads, and `supportsGraphemeClusterMode()` returns `false` immediately
**Failed builds:**
https://github.com/apache/maven/actions/runs/33349929091/job/99362003331
## Test plan
- [x] `mvn test -pl impl/maven-jline -Dtest=FastTerminalReentrancyTest`
passes (5/5 tests, ~0.45s)
- [x] Full `impl/maven-jline` module tests pass
- [ ] CI build passes
🤖 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]
