[ 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