You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by js...@apache.org on 2008/04/26 04:23:54 UTC

svn commit: r651788 - /incubator/tuscany/java/sca/tools/maven/maven-incremental-build/src/main/java/org/apache/tuscany/sca/tools/incremental/build/plugin/IncrementalBuildMojo.java

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