Maven 1.x EJB Plugin

ejb:install-client and ejb:deploy-client uses wrong file name

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.6
  • Fix Version/s: 1.7
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    3

Description

When running ejb:install-client or ejb:deploy-client, the "-client" part of the jar file name is not preserved in the installed/deployed file.

See also http://jira.codehaus.org/browse/MPARTIFACT-35

I have created a solution based on Brett's dist plugin

  1. EJBArtifactTypeHandler.java
    15/Oct/04 4:53 AM
    2 kB
    Håvard Bjåstad
  2. plugin.jelly
    15/Oct/04 4:51 AM
    9 kB
    Håvard Bjåstad
  3. project.xml
    15/Oct/04 4:52 AM
    3 kB
    Håvard Bjåstad

Issue Links

Activity

Hide
Håvard Bjåstad added a comment -

Updated plugin.jelly file

Show
Håvard Bjåstad added a comment - Updated plugin.jelly file
Hide
Håvard Bjåstad added a comment -

Updated project.xml (note: version should be changed before committed)

Show
Håvard Bjåstad added a comment - Updated project.xml (note: version should be changed before committed)
Hide
Håvard Bjåstad added a comment -

New file src/main/org/apache/maven/ejb/EJBArtifactTypeHandler.java

Show
Håvard Bjåstad added a comment - New file src/main/org/apache/maven/ejb/EJBArtifactTypeHandler.java
Hide
Timothy A. Herrick added a comment -

If implementing this with MPEJB-14 the proposed solution works except when deploying snapshots to a remote repository. The jar is fine but the ${artifactId}-snapshot-version file is still overwritten.

Show
Timothy A. Herrick added a comment - If implementing this with MPEJB-14 the proposed solution works except when deploying snapshots to a remote repository. The jar is fine but the ${artifactId}-snapshot-version file is still overwritten.
Hide
Vincent Massol added a comment -

Applied thanks. Next time please submit a real patch (diff -u). That makes it easier to apply. Also, I have added unit tests ase they were missing and I've modifed the output location: the ejb jars go in ejbs/ and not in jars/.

Show
Vincent Massol added a comment - Applied thanks. Next time please submit a real patch (diff -u). That makes it easier to apply. Also, I have added unit tests ase they were missing and I've modifed the output location: the ejb jars go in ejbs/ and not in jars/.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: