Maven 1.x Eclipse Plugin

avoid duplicated builders/natures (patch)

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.9
  • Fix Version/s: 1.10
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    0

Description

The eclipse plugin automatically adds the java nature/builder as the first entry for each project.
Anyway, sometime you will need to add more builders and in a different order. Specifying an addictional java builder in project properties, however, lead to a duplicate entries in .project.

The problem is discussed at:
http://www.eclipse.org/webtools/jst/components/j2ee/scenarios/MavenEclipseIntegration.html

snippet:

"Create a file project.properties under the csrweb folder, the folder where you will find project.xml. Now add the following in this file.

maven.eclipse.projectnatures=org.eclipse.jst.j2ee.web.WebNature,org.eclipse.jdt.core.javanature,org.eclipse.wst.common.modulecore.ModuleCoreNature
maven.eclipse.buildcommands=org.eclipse.wst.common.modulecore.DeployableModuleBuilder,org.eclipse.jdt.core.javabuilder,org.eclipse.jst.j2ee.web.LibDirBuilder,org.eclipse.wst.validation.validationbuilder,org.eclipse.jst.j2ee.LibCopyBuilder,org.eclipse.wst.common.modulecore.LocalDependencyResolver

...

Open the .project and remove the first buildcommand javabuilder and the first nature org.eclipse.jdt.core.javanature.
This is done to ensure that only one entry of javabuilder and org.eclipse.jdt.core.javanature exists.
The correct order of builder and natures are needed to ensure proper functionality
"

This patch solves the problem avoiding addictional javabuilders and javanatures if they are already present in the list specified using project.properties. If maven.eclipse.projectnatures and maven.eclipse.buildcommands don't contain a such entry, everything works as before.

The patch also contains a new specific plugin test.

Activity

Hide
Brett Porter added a comment -

test attachment

Show
Brett Porter added a comment - test attachment
Hide
fabrizio giustina added a comment -

Patch is attached to MPECLIPSE-80

Show
fabrizio giustina added a comment - Patch is attached to MPECLIPSE-80
Hide
fabrizio giustina added a comment -

fixed in svn

Show
fabrizio giustina added a comment - fixed in svn

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved:

Time Tracking

Estimated:
10m
Original Estimate - 10 minutes
Remaining:
10m
Remaining Estimate - 10 minutes
Logged:
Not Specified
Time Spent - Not Specified