Uploaded image for project: 'Maven Deploy Plugin'
  1. Maven Deploy Plugin
  2. MDEPLOY-311

Regression in 3.0.0: deploy-file: -Dpackaging is not considered during artifact upload

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 3.0.0, 3.1.0, 3.1.1
    • 3.1.3
    • deploy:deploy-file
    • None

    Description

      Regression in https://github.com/apache/maven-deploy-plugin/pull/26

       

      Property passed via -Dpackaging is not considered anymore during upload. Instead, packaging is always derived from the file name.

       

      To reproduce:

       

      $ mvn -Dfile=foo.zip -DgroupId=com.bar -Dclassifier=baz -DrepositoryId=deploy.snapshots -DartifactId=foo -Dpackaging=alterantivepackaging -Dversion=1.0.0-SNAPSHOT deploy:3.0.0-M2:deploy-file -Durl=file:./repo
      ...
      Uploading to deploy.snapshots: file:./repo/com/bar/foo/1.0.0-SNAPSHOT/foo-1.0.0-20230703.093021-2-baz.alterantivepackaging
      ...

       

      $ mvn -Dfile=foo.zip -DgroupId=com.bar -Dclassifier=baz -DrepositoryId=deploy.snapshots -DartifactId=foo -Dpackaging=alterantivepackaging -Dversion=1.0.0-SNAPSHOT deploy:3.0.0:deploy-file -Durl=file:./repo
      ...
      Uploading to deploy.snapshots: file:./repo/com/bar/foo/1.0.0-SNAPSHOT/foo-1.0.0-20230703.093125-3-baz.zip
      ...

       

      The problematic line is

      https://github.com/apache/maven-deploy-plugin/blob/707ab61615606053eb816e7c96d1fc276a80a47f/src/main/java/org/apache/maven/plugins/deploy/DeployFileMojo.java#L278C58-L278C96

      Here,
      packaging
      should be used instead of
      isFilePom ? "pom" : getExtension(file)

      Attachments

        Issue Links

          Activity

            People

              cstamas Tamas Cservenak
              sratz Sebastian Ratz
              Votes:
              1 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: