Skip to content

Stop wagon-http from publishing a dependency-reduced POM - #973

Merged
slachiewicz merged 1 commit into
wagon-3.xfrom
agent/wagon-http-dependency-reduced-pom
Sep 3, 2026
Merged

Stop wagon-http from publishing a dependency-reduced POM#973
slachiewicz merged 1 commit into
wagon-3.xfrom
agent/wagon-http-dependency-reduced-pom

Conversation

@slachiewicz

Copy link
Copy Markdown
Member

wagon-http's main jar is unshaded and needs wagon-http-shared and httpclient at runtime, but the shade execution was reducing them out of the installed POM, leaving HttpWagon unable to load AbstractHttpClientWagon. Fixes #972.

Verified: mvn install on the module → installed POM declares wagon-http-shared, httpclient and httpcore again; both the plain and -shaded jars still produced; no dependency-reduced-pom.xml in the basedir, so #750 stays fixed. A rebuilt Maven 3.9.17-SNAPSHOT distribution ships wagon-http-shared again, and -Dmaven.resolver.transport=wagon selects WagonTransporter for both a file:// repository and central, matching released 3.9.16.

master needs the same change.

This change was created with AI assistance.

The shaded jar is attached under a classifier, so the main jar keeps
needing wagon-http-shared and httpclient; reducing the POM stripped them
and left HttpWagon unloadable. Do not restore dependencyReducedPomLocation
to fix WAGON-636 / #750 -- disabling the reduced POM outright means none
is written to the basedir either.
@slachiewicz slachiewicz added bug Something isn't working build Pull requests that change the build process labels Sep 3, 2026
@slachiewicz
slachiewicz marked this pull request as ready for review September 3, 2026 22:13
@slachiewicz
slachiewicz merged commit e8d5972 into wagon-3.x Sep 3, 2026
13 checks passed
@slachiewicz
slachiewicz deleted the agent/wagon-http-dependency-reduced-pom branch September 3, 2026 22:36
@github-actions github-actions Bot added this to the 3.5.4 milestone Sep 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working build Pull requests that change the build process mvn3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant