Maven 2.x and 3.x Deploy Plugin

deploy:file favors parent groupId over the child's one

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Critical Critical
  • Resolution: Fixed
  • Affects Version/s: 2.2
  • Fix Version/s: 2.2.1
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    2

Description

Fix attached.

  1. File
    fix.diff
    26/Jan/06 9:31 AM
    0.6 kB
    Jerome Lacoste
  2. File
    MDEPLOY23.diff
    27/Jan/06 7:40 AM
    13 kB
    Jerome Lacoste

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Jerome Lacoste added a comment -

And related to this issue, you can't override the pom values....

E.g.

If I have a pom that repacks a zip file. The pom uses the assembly plugin but is of packaging pom.

mvn deploy:deploy-file -DpomFile=pom.xml -Dpackaging=zip -DrepositoryId=... -Durl=... -Dfile=target/file.zip

the file gets deployed as pom packaging...

Will prepare a new patch.

Issue should be renamed as:

deploy-file isn't using a proper properties overriding order when reading the pom

Show
Jerome Lacoste added a comment - And related to this issue, you can't override the pom values.... E.g. If I have a pom that repacks a zip file. The pom uses the assembly plugin but is of packaging pom. mvn deploy:deploy-file -DpomFile=pom.xml -Dpackaging=zip -DrepositoryId=... -Durl=... -Dfile=target/file.zip the file gets deployed as pom packaging... Will prepare a new patch. Issue should be renamed as: deploy-file isn't using a proper properties overriding order when reading the pom
Hide
Permalink
Jerome Lacoste added a comment -

That fixes both issues for me. Including test cases.

Note that the deployer code doesn't seem to handle classifier (See also MNG-1875).

Show
Jerome Lacoste added a comment - That fixes both issues for me. Including test cases. Note that the deployer code doesn't seem to handle classifier (See also MNG-1875 ).
Hide
Permalink
Jerome Lacoste added a comment -

MDEPLOY-19 is now fixed on top of that one. So don't rearrange too much the code, if you don't want to break it...

Show
Jerome Lacoste added a comment - MDEPLOY-19 is now fixed on top of that one. So don't rearrange too much the code, if you don't want to break it...
Hide
Permalink
Allan Ramirez added a comment -

Applied patch. Thanks

Show
Allan Ramirez added a comment - Applied patch. Thanks
Hide
Permalink
John Casey added a comment -

updating fix version

Show
John Casey added a comment - updating fix version

People

  • Assignee:
    Allan Ramirez
    Reporter:
    Jerome Lacoste
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: