geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ivan <xhh...@gmail.com>
Subject Re: A dependency in the geronimo-tomcat6 pom.xml
Date Fri, 04 Sep 2009 07:29:37 GMT
I have had a close look at it recently, and  seems no code to do the
replacement, so those artifacts are not in the dependencies list, and I did
not think that they could be ignored, IMO, if we ignored them, those
dependencies will not be seen by the classloader of the car package, except
they could be load from its parents.

Also, following the guide from David, I checked the AbstractCarMojo, the
resolve logic is, first it uses the default collector and dependency tree
listener to build a node tree, then it uses the Scanner class to build the
dependency list. I could see two places to do the changes, one is in the
listener, another one is in the Scanner class, but I did not find related
api to get the dependeny list in the dependencyManagement fragment.

Maybe I miss something, Any comment ?

2009/8/26 Ivan <xhhsld@gmail.com>

> Thanks, David, just notice that the groupId is org.apache.geronimo.plugins,
> not org.apache.geronimo.configs. ^_^
>
>
> 2009/8/26 David Jencks <david_jencks@yahoo.com>
>
>
>> On Aug 25, 2009, at 11:28 PM, Ivan wrote:
>>
>>  Hi, I found a dependency in the pom.xml file of the geronimo-tomcat6
>>>
>>> <dependency>
>>>            <groupId>org.apache.geronimo.plugins</groupId>
>>>            <artifactId>clustering</artifactId>
>>>            <version>${version}</version>
>>>            <type>pom</type>
>>>            <scope>import</scope>
>>>        </dependency>
>>>
>>> Due to this dependency (Not quite sure :-)), in the binary build, we have
>>> a pom file in the repository/org/apache/geronimo/plugins/clustering folder,
>>> and sometimes, I got a Zip related exception.
>>> I wonder that shall we change the type to car ? Or ...
>>>
>>
>> we don't want to change the type to car, since it isn't a car :-)
>>
>> We need to update the code that turns maven dependencies into geronimo
>> dependencies so it leaves  out any pom dependencies.  The only way a pom
>> dependency can mean anything is if it has scope import, and this is pretty
>> new maven feature.  I could be wrong but I don't think leaving these out
>> will cause any problems.  I think the relevant code is in AbstractCarMojo.
>>
>> thanks for noticing this problem !
>>
>> david jencks
>>
>>  Thanks !
>>>
>>> --
>>> Ivan
>>>
>>
>>
>
>
> --
> Ivan
>



-- 
Ivan

Mime
View raw message