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);
             }
         }


Reply via email to