You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by GitBox <gi...@apache.org> on 2022/04/08 14:08:06 UTC
[GitHub] [maven] cstamas opened a new pull request, #710: [MNG-7454] Include resolver-transport-http in Maven
cstamas opened a new pull request, #710:
URL: https://github.com/apache/maven/pull/710
But keep Wagon as default transport. This PR merely includes
resolver http and file transport and switches wagon-http
to non-shaded one.
Changes:
* switch to non-shaded wagon-http (as httpClient is now shared)
* include resolver http and file transport
* override resolver default behaviour (native transport preferred over wagon, when both on classpath)
* provide simplistic means to choose transport
The chosen transport can be seen in debug (-X) output on line
`[DEBUG] Using transporter XXX...`
The `-Dmaven.traport` simplistic switch can be used to choose tranport:
* not set: default, that is Wagon
* `wagon`: explicitly sets Wagon
* `resolver`: explicitly sets resolver native transports (file and http)
* `auto`: relies on resolver "auto discovery" (priorities, etc). This is MUST to keep transport pluggable with 3rd party transports.
Note: resolver by default, if both wagon and native transports are present on classpath,
would prefer native ones. Hence, to retain "wagon as default" this extra config
bit is a must in Maven.
--
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
[GitHub] [maven] cstamas merged pull request #710: [MNG-7454] Include resolver-transport-http in Maven 3.9.x
Posted by GitBox <gi...@apache.org>.
cstamas merged PR #710:
URL: https://github.com/apache/maven/pull/710
--
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
[GitHub] [maven] cstamas commented on pull request #710: [MNG-7454] Include resolver-transport-http in Maven
Posted by GitBox <gi...@apache.org>.
cstamas commented on PR #710:
URL: https://github.com/apache/maven/pull/710#issuecomment-1092934252
As master is broken until https://github.com/apache/maven/pull/709 merged, MNG6090 ITs are expected to fail, but Wagon ancient (plexus XML and other) ITs should succeed.
--
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
[GitHub] [maven] cstamas commented on pull request #710: [MNG-7454] Include resolver-transport-http in Maven
Posted by GitBox <gi...@apache.org>.
cstamas commented on PR #710:
URL: https://github.com/apache/maven/pull/710#issuecomment-1093084247
Branch maven-3.9.x is fixed in the mean time, so now all ITs needs to pass.
--
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