[ http://jira.codehaus.org/browse/MNG-1310?page=comments#action_50554 ] 

Jerome Lacoste commented on MNG-1310:
-------------------------------------

It looks like this won't be necessary after all. If MNG-1274 and MNG-1462 are 
fixed, then I can reference the correct dependency using the <classifier> node 
in the <dependency> element.
Given that the IncludesArtifactFilter filter implementation only takes into 
account the groupId and artifactId, I don't need to update my include 
dependency in the assembly config.

    public boolean include( Artifact artifact )
    {
        String id = artifact.getGroupId() + ":" + artifact.getArtifactId();

        boolean matched = false;
        for ( Iterator i = patterns.iterator(); i.hasNext() & !matched; )
        {
            // TODO: what about wildcards? Just specifying groups? versions?
            if ( id.equals( i.next() ) )
            {
                matched = true;
            }
        }
        return matched;
    }

So we can close that issue.

> Support installation of 
> ------------------------
>
>          Key: MNG-1310
>          URL: http://jira.codehaus.org/browse/MNG-1310
>      Project: Maven 2
>         Type: New Feature
>   Components: maven-assembly-plugin
>     Versions: 2.0
>     Reporter: Jerome Lacoste
>     Assignee: John Casey
>  Attachments: MNG-1310.diff
>
> Original Estimate: 1 hour
>         Remaining: 1 hour
>
> What about adding a <install>true</install> to the config to install all 
> produced artifacts (as AttachedArtifacts) ?
> Feel free to also review MNG-1251 as I will probably use it for 
> implementation.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to