Maven 2.x Dependency Plugin

Option to not copy 'provided' scoped jars

Details

  • Type: Wish Wish
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 2.0-alpha-1
  • Component/s: None
  • Labels:
    None
  • Environment:
    Maven2
  • Number of attachments :
    1

Description

It would be useful if there were an option that could be set such that a 'scope=provided' jar would not be copied via copy-dependencies, but I can't see any way of setting such or otherwise excluding such a jar from the copy.

Issue Links

is duplicated by

Wish - General wishlist item MDEP-22 copy of provided dependencies

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Gwyn Evans added a comment -

Just in case you think this is worth doing....

Index: CopyDependenciesMojo.java
===================================================================
— CopyDependenciesMojo.java (revision 1841)
+++ CopyDependenciesMojo.java (working copy)
@@ -60,7 +60,12 @@

for ( Iterator i = artifacts.iterator(); i.hasNext(); )
{

  • copyArtifact( (DefaultArtifact) i.next(), this.stripVersion );
    + DefaultArtifact artifact = (DefaultArtifact) i.next();
    + String aScope = artifact.getScope();
    + // Gwyn: Only copy if not provided & not system
    + if (!Artifact.SCOPE_PROVIDED.equals( aScope ) && !Artifact.SCOPE_SYSTEM.equals( aScope )) { + copyArtifact( artifact, this.stripVersion ); + }

    }
    }

Show
Gwyn Evans added a comment - Just in case you think this is worth doing.... Index: CopyDependenciesMojo.java =================================================================== — CopyDependenciesMojo.java (revision 1841) +++ CopyDependenciesMojo.java (working copy) @@ -60,7 +60,12 @@ for ( Iterator i = artifacts.iterator(); i.hasNext(); ) { copyArtifact( (DefaultArtifact) i.next(), this.stripVersion ); + DefaultArtifact artifact = (DefaultArtifact) i.next(); + String aScope = artifact.getScope(); + // Gwyn: Only copy if not provided & not system + if (!Artifact.SCOPE_PROVIDED.equals( aScope ) && !Artifact.SCOPE_SYSTEM.equals( aScope )) { + copyArtifact( artifact, this.stripVersion ); + } } }
Hide
Permalink
Gwyn Evans added a comment -

It looks as if the line
DefaultArtifact artifact = (DefaultArtifact) i.next();
should be
Artifact artifact = (Artifact) i.next();
by the way...

Show
Gwyn Evans added a comment - It looks as if the line DefaultArtifact artifact = (DefaultArtifact) i.next(); should be Artifact artifact = (Artifact) i.next(); by the way...
Hide
Permalink
Brian Fox added a comment -

Added the ability to specify a scope to exclude. This uses the plexus ScopeArtifactFilter, which defines what scopes are included. For example, since test scope effectively includes all other scopes, then excluding the test scope would exclude everything. (which the plugin won't allow)

Show
Brian Fox added a comment - Added the ability to specify a scope to exclude. This uses the plexus ScopeArtifactFilter, which defines what scopes are included. For example, since test scope effectively includes all other scopes, then excluding the test scope would exclude everything. (which the plugin won't allow)
Hide
Permalink
Mark Struberg added a comment -

test case fixes

Show
Mark Struberg added a comment - test case fixes

People

  • Assignee:
    Brian Fox
    Reporter:
    Gwyn Evans
Vote (3)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: