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/03/21 12:32:42 UTC
[PR] [MNG-8604] Fix concurrent builder crash with reactor plugins [maven]
gnodet opened a new pull request, #11820:
URL: https://github.com/apache/maven/pull/11820
## Summary
- Fixes a crash (`Step $plan$ not found`) when using `--builder concurrent` on projects where a reactor module is also used as a build plugin (e.g., Apache CXF's `cxf-codegen-plugin`)
- Moves reactor plugin ordering logic from `calculateLifecycleMappings()` to `buildInitialPlan()`, where PLAN/READY steps already exist
- Simplifies plugin resolution to only eagerly resolve non-reactor plugins
## Details
The root cause is that `calculateLifecycleMappings()` calls `plan.requiredStep(project, PLAN)` to set up ordering between reactor plugins and their consuming projects, but the PLAN steps are only created later in `buildInitialPlan()`. Moving this logic to after the PLAN/READY step creation fixes the crash.
Tested successfully on Apache CXF, Maven Resolver, Commons Lang, and Maven Surefire with `-T4 --builder concurrent`.
## Test plan
- [x] Existing `BuildPlanCreatorTest` tests pass
- [x] Verified on Apache CXF (`mvn verify -T4 --builder concurrent`) — previously crashed, now builds successfully
- [x] Verified on Maven Resolver, Commons Lang, Maven Surefire with concurrent builder
🤖 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] [MNG-8604] Fix concurrent builder crash with reactor plugins [maven]
Posted by "gnodet (via GitHub)" <gi...@apache.org>.
gnodet merged PR #11820:
URL: https://github.com/apache/maven/pull/11820
--
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] [MNG-8604] Fix concurrent builder crash with reactor plugins [maven]
Posted by "nkonev (via GitHub)" <gi...@apache.org>.
nkonev commented on PR #11820:
URL: https://github.com/apache/maven/pull/11820#issuecomment-4723901917
milestone 4.0.0-rc-6 but branch is master, not maven-4.0.x
--
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] [MNG-8604] Fix concurrent builder crash with reactor plugins [maven]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #11820:
URL: https://github.com/apache/maven/pull/11820#issuecomment-4723186353
@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