[
http://jira.codehaus.org/browse/MECLIPSE-220?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_86052
]
Minto van der Sluis commented on MECLIPSE-220:
----------------------------------------------
Hmm I also like issue http://jira.codehaus.org/browse/MECLIPSE-207
combined these issue would IMHO become something like:
- only resolve EJB version for EJB project when it is not specified in the pom
(like used in MECLIPSE-207)
- only resolve SERVLET version for WAR project when it is not specified in the
pom (like used in MECLIPSE-207)
The same thing could actually be done for JSF and Doclet as well. Default is
setting these facets based on the version of a dependency, unless the actual
version is specified in the additionalProjectFacets tag.
regards,
misl
> 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
> 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
