You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@maven.apache.org by "Hudson (Jira)" <ji...@apache.org> on 2020/06/11 20:08:00 UTC

[jira] [Commented] (MSHARED-922) Remove checksum creation from DefaultProjectDeployer

    [ https://issues.apache.org/jira/browse/MSHARED-922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17133653#comment-17133653 ] 

Hudson commented on MSHARED-922:
--------------------------------

Build succeeded in Jenkins: Maven TLP » maven-artifact-transfer » master #107

See https://builds.apache.org/job/maven-box/job/maven-artifact-transfer/job/master/107/

> Remove checksum creation from DefaultProjectDeployer
> ----------------------------------------------------
>
>                 Key: MSHARED-922
>                 URL: https://issues.apache.org/jira/browse/MSHARED-922
>             Project: Maven Shared Components
>          Issue Type: Improvement
>          Components: maven-artifact-transfer
>    Affects Versions: maven-artifact-transfer-0.12.0
>            Reporter: Konrad Windszus
>            Assignee: Michael Osipov
>            Priority: Major
>             Fix For: maven-artifact-transfer-0.13.1
>
>
> Currently there are checksums (MD5/SHA1) being created at https://github.com/apache/maven-artifact-transfer/blob/87842f9c712eb6496b32d4aff09307d0970d9f7b/src/main/java/org/apache/maven/shared/transfer/project/deploy/internal/DefaultProjectDeployer.java#L130 which is used e.g. from the maven-deploy-plugin.
> IMHO only the Maven-Resolver should ever create checksums as this is also the only code leveraging the checksums. There is already code for that at https://github.com/apache/maven-resolver/blob/47edcfe69c4e52ced4cb93d65b7348b5645cdd68/maven-resolver-connector-basic/src/main/java/org/eclipse/aether/connector/basic/BasicRepositoryConnector.java#L557.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)