[ 
http://jira.codehaus.org/browse/MECLIPSE-220?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Arnaud Heritier closed MECLIPSE-220.
------------------------------------

       Resolution: Fixed
    Fix Version/s: 2.4

Patch reviewed and applied. Thx. Issue Fixed. SNAPSHOT deployed.

> Incorrect eclipse facet information when doing mvn eclipse:eclipse for war 
> and ejb projects.
> --------------------------------------------------------------------------------------------
>
>                 Key: MECLIPSE-220
>                 URL: http://jira.codehaus.org/browse/MECLIPSE-220
>             Project: Maven 2.x Eclipse Plugin
>          Issue Type: Bug
>          Components: WTP support
>    Affects Versions: 2.3
>         Environment: maven 2.0.4
> maven-eclipse-plugin 2.3
>            Reporter: Minto van der Sluis
>         Assigned To: Arnaud Heritier
>             Fix For: 2.4
>
>         Attachments: proper-facet.patch
>
>
> When running 'mvn eclipse:eclipse' the results do not take the proper 
> servlet-api en ejb-api into account.
> ejb version is not taken into account (see 
> http://jira.codehaus.org/browse/MECLIPSE-198)
> servlet api version is not taken into account (see 
> http://jira.codehaus.org/browse/MECLIPSE-108)
> I have fixed these problems locally, but looking at MECLIPSE-108 is wonder if 
> what I did is correct . Still my solution works for me. I wonder what the 
> difference is between config.getProject().getArtifacts() and 
> config.getProject().getDependencies() . Why in the existing implementation is 
> resolveServletVersion using getArtifacts() and resolveJ2eeVersion using 
> getDependencies() ? My solutions uses getDependencies() only and works for 
> me. 
> What I actually did is the following:
> - I switched the 2 loops in getDependencyVersion. I think the order of 
> ids/names being looked up is more important than the order of the 
> dependencies. So first look for servlet-api then for servletapi then for 
> geronimo-spec-servlet. (something alike for ejb).
> - When no servlet version was found use the J2EE version (resolveJ2eeVersion) 
> to determine the servlet version to be used.
> - Implemented resolveEjbVersion along the lines of resolveServletVersion.
> - Changed resolveJ2eeVersion to work along the line of resolveServletVersion.
> - Fixed a glitch in resolveJavaVersion, where the getCompilerSourceVersion 
> was retrieved but not used.
> Hope my efforts are usefull for others. ;-)
> regards,
> misl

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