Hi Benjamin

Thanks for the input, it is intention not to look up beyond project
dependency.  Transitive dependency also turn off with this plugin.

-D

On Sat, Jan 9, 2010 at 3:07 AM, Benjamin Bentmann
<[email protected]> wrote:
> Hi Dan,
>
>> Revision
>>    11607 <http://fisheye.codehaus.org/changelog/mojo/?cs=11607>
>> Author
>>    dantran
>> Date
>>    2010-01-08 23:41:56 -0600 (Fri, 08 Jan 2010)
>>
>>
>>      Log Message
>>
>> Lookup dependency using groupId:artifactId pair using a slower method due
>> to the introduction of "inczip" where ${project.artifactMap} no longer can
>> serve the purpose
>> [...]
>> ---
>> trunk/mojo/maven-native/native-maven-plugin/src/main/java/org/codehaus/mojo/natives/plugin/NativeLinkMojo.java
>>      2010-01-08 19:18:42 UTC (rev 11606)
>> +++
>> trunk/mojo/maven-native/native-maven-plugin/src/main/java/org/codehaus/mojo/natives/plugin/NativeLinkMojo.java
>>      2010-01-09 05:41:56 UTC (rev 11607)
>> @@ -113,14 +113,6 @@
>>     private List linkingOrderLibs;
>>       /**
>> -     * Internal: Map of of project artifacts.
>> -     *
>> -     * @parameter expression="${project.artifactMap}"
>> -     * @since 1.0-alpha-2
>> -     */
>> -    private Map projectArtifactMap;
>> [...]
>> +        Set allDependencyArtifacts = project.getDependencyArtifacts();
>
> I'm not sure whether you are aware of this but project.getArtifactMap() and
> project.getDependencyArtifacts() do not refer to the same set of artifacts.
> getArtifactMap() is derived from getArtifacts() which is a super set of
> getDependencyArtifacts(). getArtifacts() returns all/transitive
> dependencies, whereas getDependencyArtifacts() returns only the direct
> dependencies as given in the POM.
>
>
> Benjamin
>
> ---------------------------------------------------------------------
> To unsubscribe from this list, please visit:
>
>   http://xircles.codehaus.org/manage_email
>
>
>

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email


Reply via email to