Author: jsdelfino
Date: Fri Apr 25 19:23:51 2008
New Revision: 651788
URL: http://svn.apache.org/viewvc?rev=651788&view=rev
Log:
Some improvements of the maven incremental build plugin. Improved performance
by disabling generation of internal Maven stack traces (which you can get by
running the non-incremental build if needed). Added some logs. Added a check to
consider only source files in a compile source root folder.
Modified:
incubator/tuscany/java/sca/tools/maven/maven-incremental-build/src/main/java/org/apache/tuscany/sca/tools/incremental/build/plugin/IncrementalBuildMojo.java
Modified:
incubator/tuscany/java/sca/tools/maven/maven-incremental-build/src/main/java/org/apache/tuscany/sca/tools/incremental/build/plugin/IncrementalBuildMojo.java
URL:
http://svn.apache.org/viewvc/incubator/tuscany/java/sca/tools/maven/maven-incremental-build/src/main/java/org/apache/tuscany/sca/tools/incremental/build/plugin/IncrementalBuildMojo.java?rev=651788&r1=651787&r2=651788&view=diff
==============================================================================
---
incubator/tuscany/java/sca/tools/maven/maven-incremental-build/src/main/java/org/apache/tuscany/sca/tools/incremental/build/plugin/IncrementalBuildMojo.java
(original)
+++
incubator/tuscany/java/sca/tools/maven/maven-incremental-build/src/main/java/org/apache/tuscany/sca/tools/incremental/build/plugin/IncrementalBuildMojo.java
Fri Apr 25 19:23:51 2008
@@ -175,7 +175,7 @@
// FIXME: The maven invoker doesn't handle the directory names
with spaces
// request.setLocalRepositoryDirectory(new
File(localRepository.getBasedir()));
request.setInteractive(false);
- request.setShowErrors(true);
+ request.setShowErrors(false);
request.setRecursive(false);
// request.setDebug(true);
request.setOffline(settings.isOffline());
@@ -272,7 +272,7 @@
if (resource.getExcludes() != null &&
!resource.getExcludes().isEmpty()) {
scanner.setExcludes((String[])resource.getExcludes().toArray(new String[]{}));
}
-
+
scanner.addDefaultExcludes();
scanner.scan();
@@ -297,13 +297,16 @@
File destinationFile = new File(outputDirectory, destination);
if (!destinationFile.exists()) {
+ getLog().info("Source file " + sourceFile + ".");
getLog().info("Target file " + destinationFile + " could
not be found.");
return true;
} else {
if (sourceFile.lastModified() >
destinationFile.lastModified()) {
+ getLog().info("Source file " + sourceFile + " has
changed.");
getLog().info("Target file " + destinationFile + " is
stale.");
return true;
} else if (sourceFile.lastModified() >
outputFile.lastModified()) {
+ getLog().info("Source file " + sourceFile + " has
changed.");
getLog().info("Target build output file " + outputFile
+ " is stale.");
return true;
} else if (outputFile.lastModified() == 0) {
@@ -324,7 +327,8 @@
private boolean isPOMStale() {
File pom = project.getFile();
if (pom.lastModified() > outputFile.lastModified()) {
- getLog().info("File " + pom + " is stale.");
+ getLog().info("File " + pom + " has changed.");
+ getLog().info("Target build output file " + pom + " is stale.");
return true;
} else if (outputFile.lastModified() == 0) {
getLog().info("Target build output file " + outputFile + " could
not be found.");
@@ -356,6 +360,7 @@
if (new File(root).exists()) {
Resource resource = new Resource();
resource.setDirectory(root);
+ resource.addInclude("*.java");
resources.add(resource);
}
}