[ http://jira.codehaus.org/browse/MECLIPSE-108?page=all ]

Andres Martinez updated MECLIPSE-108:
-------------------------------------

    Attachment: MECLIPSE-108-maven-eclipse-plugin.patch

The problem is that resolveServletVersion() is calling getArtifacts() which is 
null unless @requiresDependencyResolution is set.  This used to be set to test 
scope as part of MNG-424 until removed in revision 390686.  I assume this was 
inadvertent and have attached a patch to put it back in.

> .wtpmodules with version 2.4 for javax.servlet:servlet-api:2.3
> --------------------------------------------------------------
>
>                 Key: MECLIPSE-108
>                 URL: http://jira.codehaus.org/browse/MECLIPSE-108
>             Project: Maven 2.x Eclipse Plugin
>          Issue Type: Bug
>          Components: WTP support
>    Affects Versions: 2.2
>            Reporter: Daniel Schulz
>         Attachments: MECLIPSE-108-maven-eclipse-plugin.patch
>
>
> specified the following dependency:
>                 <!-- Servlet API (2.3)-->
>               <dependency>
>                       <artifactId>servlet-api</artifactId>
>                       <groupId>javax.servlet</groupId>
>                       <version>2.3</version>
>               </dependency>
> mvn eclipse:clean eclipse:eclipse produces the following .wtpmodules snippet:
>  <module-type module-type-id="jst.web">
>       <version>2.4</version>
>       <property name="context-root" value="wui"/>
>     </module-type>
> think, it should be  <version>2.3</version>.  looks like it was the intention 
> of the plugin to detect the servlet-api version by processing the dependency 
> list. there is a method resolveServletVersion() in 
> org.apache.maven.plugin.eclipse.writers.AbstractWtpResourceWriter.

-- 
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

        

Reply via email to