Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorIgor Fedorenko2011-10-20 20:25:45 +0000
committerIgor Fedorenko2011-10-20 20:25:48 +0000
commit7eb939c1c3c46d5dc23b763abae0c532d734b51e (patch)
treec9703185ce89d04fe62894c3df2a9d833d12cb25 /org.eclipse.m2e.jdt
parentd3171e7253df9c45561ef649e11d770232c6655c (diff)
downloadm2e-core-7eb939c1c3c46d5dc23b763abae0c532d734b51e.tar.gz
m2e-core-7eb939c1c3c46d5dc23b763abae0c532d734b51e.tar.xz
m2e-core-7eb939c1c3c46d5dc23b763abae0c532d734b51e.zip
361549 better handling of non-existing source folders
Create classpath entries for all compile and test compile roots defined in MavenProject, regardless if corresponding workspace resources exist or not. Mark all source classpath entries as optional, so that JDT does not complain too loudly when corresponding resources get deleted by clean workspace build or by running "mvn clean" from command line. Do shallow refresh of workspace folders that correspond to compile and test compile source roots. This should provide slightly better support for source roots added to the model by <execute/> mapping. Signed-off-by: Igor Fedorenko <igor@ifedorenko.com>
Diffstat (limited to 'org.eclipse.m2e.jdt')
-rw-r--r--org.eclipse.m2e.jdt/src/org/eclipse/m2e/jdt/internal/AbstractJavaProjectConfigurator.java86
1 files changed, 48 insertions, 38 deletions
diff --git a/org.eclipse.m2e.jdt/src/org/eclipse/m2e/jdt/internal/AbstractJavaProjectConfigurator.java b/org.eclipse.m2e.jdt/src/org/eclipse/m2e/jdt/internal/AbstractJavaProjectConfigurator.java
index 1f5f7961..246786ea 100644
--- a/org.eclipse.m2e.jdt/src/org/eclipse/m2e/jdt/internal/AbstractJavaProjectConfigurator.java
+++ b/org.eclipse.m2e.jdt/src/org/eclipse/m2e/jdt/internal/AbstractJavaProjectConfigurator.java
@@ -337,26 +337,42 @@ public abstract class AbstractJavaProjectConfigurator extends AbstractProjectCon
}
private void addSourceDirs(IClasspathDescriptor classpath, IProject project, List<String> sourceRoots,
- IPath outputPath, IPath[] inclusion, IPath[] exclusion, String sourceEncoding, IProgressMonitor monitor) throws CoreException {
-
- for(String sourceRoot : sourceRoots) {
- IFolder sourceFolder = getFolder(project, sourceRoot);
- if(sourceFolder != null && sourceFolder.exists() && sourceFolder.getProject().equals(project)) {
- IClasspathEntryDescriptor enclosing = getEnclosingEntryDescriptor(classpath, sourceFolder.getFullPath());
- if(enclosing == null || getEntryDescriptor(classpath, sourceFolder.getFullPath()) != null) {
- log.info("Adding source folder " + sourceFolder.getFullPath());
- classpath.addSourceEntry(sourceFolder.getFullPath(), outputPath, inclusion, exclusion, false);
- } else {
- log.info("Not adding source folder " + sourceFolder.getFullPath() + " because it overlaps with "
- + enclosing.getPath());
- }
+ IPath outputPath, IPath[] inclusion, IPath[] exclusion, String sourceEncoding, IProgressMonitor monitor)
+ throws CoreException {
+
+ for(int i = 0; i < sourceRoots.size(); i++ ) {
+ IFolder sourceFolder = getFolder(project, sourceRoots.get(i));
+
+ if(sourceFolder == null) {
+ // this cannot actually happen, unless I misunderstand how project.getFolder works
+ continue;
+ }
+
+ // be extra nice to less perfectly written maven plugins, which contribute compile source root to the model
+ // but do not use BuildContext to tell as about the actual resources
+ sourceFolder.refreshLocal(IResource.DEPTH_ZERO, monitor);
+
+ if(sourceFolder.exists() && !sourceFolder.getProject().equals(project)) {
+ // source folders outside of ${project.basedir} are not supported
+ continue;
+ }
- // Set folder encoding (null = platform/container default)
+ // Set folder encoding (null = platform/container default)
+ if(sourceFolder.exists()) {
sourceFolder.setDefaultCharset(sourceEncoding, monitor);
+ }
+
+ IClasspathEntryDescriptor enclosing = getEnclosingEntryDescriptor(classpath, sourceFolder.getFullPath());
+ if(enclosing == null || getEntryDescriptor(classpath, sourceFolder.getFullPath()) != null) {
+ log.info("Adding source folder " + sourceFolder.getFullPath());
+
+ // source folder entries are created even when corresponding resources do not actually exist in workspace
+ // to keep JDT from complaining too loudly about non-existing folders,
+ // all source entries are marked as generated (a.k.a. optional)
+ classpath.addSourceEntry(sourceFolder.getFullPath(), outputPath, inclusion, exclusion, true /*generated*/);
} else {
- if(sourceFolder != null) {
- classpath.removeEntry(sourceFolder.getFullPath());
- }
+ log.info("Not adding source folder " + sourceFolder.getFullPath() + " because it overlaps with "
+ + enclosing.getPath());
}
}
}
@@ -412,8 +428,7 @@ public abstract class AbstractJavaProjectConfigurator extends AbstractProjectCon
"**")} /*exclusion*/, false /*optional*/);
} else {
// resources and sources folders overlap. make sure JDT only processes java sources.
- log.info("Resources folder " + r.getFullPath() + " overlaps with sources folder "
- + cped.getPath());
+ log.info("Resources folder " + r.getFullPath() + " overlaps with sources folder " + cped.getPath());
cped.addInclusionPattern(new Path("**/*.java"));
}
@@ -450,27 +465,22 @@ public abstract class AbstractJavaProjectConfigurator extends AbstractProjectCon
// While "5" and "6" are valid synonyms for Java 5 and Java 6 source,
// Eclipse expects the values 1.5 and 1.6.
- if( source.equals("5") ) {
- source = "1.5";
- }
- if( source.equals("6") ) {
- source = "1.6";
+ if(source.equals("5")) {
+ source = "1.5";
+ } else if(source.equals("6")) {
+ source = "1.6";
+ } else if(source.equals("7")) {
+ source = "1.7";
}
- if( source.equals("7") ) {
- source = "1.7";
- }
-
// While "5" and "6" are valid synonyms for Java 5 and Java 6 target,
// Eclipse expects the values 1.5 and 1.6.
- if( target.equals("5") ) {
- target = "1.5";
- }
- if( target.equals("6") ) {
- target = "1.6";
- }
- if( target.equals("7") ) {
- target = "1.7";
+ if(target.equals("5")) {
+ target = "1.5";
+ } else if(target.equals("6")) {
+ target = "1.6";
+ } else if(target.equals("7")) {
+ target = "1.7";
}
options.put(JavaCore.COMPILER_SOURCE, source);
@@ -481,8 +491,8 @@ public abstract class AbstractJavaProjectConfigurator extends AbstractProjectCon
protected List<MojoExecution> getCompilerMojoExecutions(ProjectConfigurationRequest request, IProgressMonitor monitor)
throws CoreException {
- return request.getMavenProjectFacade().getMojoExecutions(COMPILER_PLUGIN_GROUP_ID,
- COMPILER_PLUGIN_ARTIFACT_ID, monitor, GOAL_COMPILE, GOAL_TESTCOMPILE);
+ return request.getMavenProjectFacade().getMojoExecutions(COMPILER_PLUGIN_GROUP_ID, COMPILER_PLUGIN_ARTIFACT_ID,
+ monitor, GOAL_COMPILE, GOAL_TESTCOMPILE);
}
private String getCompilerLevel(MavenSession session, MojoExecution execution, String parameter, String source,

Back to the top