Skip to content

Forward port 2.1.x main - #1188

Merged
Kehrlann merged 4 commits into
mainfrom
forward-port-2.1.x-main
Oct 15, 2026
Merged

Kehrlann merged 4 commits into
mainfrom
forward-port-2.1.x-main

Conversation

@Kehrlann

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Daniel Garnier-Moiroux <git@garnier.wf>
mcp's module-info refers to sibling modules by their JPMS names, which
only exist once those modules are packaged as jars. Before the package
phase, the reactor passes bare target/classes directories, which can't
be automatic modules. So mcp is compiled at prepare-package.

javadoc:aggregate forks a compile-phase build of the whole reactor, so
the fork never produces mcp classes. Modules that depend on mcp at
compile scope (the conformance tests) then fall back to Maven Central,
where the release version doesn't exist yet.

The conformance tests are not deployed, so leave them out of the
release build.

Signed-off-by: Daniel Garnier-Moiroux <git@garnier.wf>
@Kehrlann
Kehrlann force-pushed the forward-port-2.1.x-main branch from 502644c to 8512a84 Compare October 15, 2026 16:47
The ubuntu-24.04 runner image now ships Maven 3.10, where the staging
metadata is written as maven-metadata-local.xml. central-publishing
0.7.0 only removes maven-metadata-central-staging.xml before bundling,
so the leftover files end up in the bundle and Central rejects it
("Bundle has content that does NOT have a .pom file").

Use ./mvnw everywhere, so CI runs the Maven version pinned in
maven-wrapper.properties instead of whatever the runner image ships.

Signed-off-by: Daniel Garnier-Moiroux <git@garnier.wf>
Signed-off-by: Daniel Garnier-Moiroux <git@garnier.wf>
@Kehrlann
Kehrlann force-pushed the forward-port-2.1.x-main branch from 8512a84 to 05d6ebf Compare October 15, 2026 17:06
@Kehrlann
Kehrlann merged commit 5112339 into main Oct 15, 2026
24 checks passed
@Kehrlann
Kehrlann deleted the forward-port-2.1.x-main branch October 15, 2026 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant