Changed spelling to manifest

[mirror_edk2.git] / Tools / Source / GenBuild / org / tianocore / build / global / OnDependency.java

diff --git a/Tools/Source/GenBuild/org/tianocore/build/global/OnDependency.java b/Tools/Source/GenBuild/org/tianocore/build/global/OnDependency.java
index ba388a100086a7be64b0ebe037a7daec53ed0c73..678bfb869bf63cb111f1d46482a89c30f2cc6272 100644 (file)
--- a/Tools/Source/GenBuild/org/tianocore/build/global/OnDependency.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/global/OnDependency.java
@@ -14,13 +14,13 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 package org.tianocore.build.global;\r
 \r
 import java.io.File;\r
-import java.util.HashMap;\r
 import java.util.Iterator;\r
-import java.util.Map;\r
 \r
 import org.apache.tools.ant.BuildException;\r
 import org.apache.tools.ant.Task;\r
 import org.apache.tools.ant.taskdefs.Sequential;\r
+import org.tianocore.common.logger.EdkLog;\r
+import org.tianocore.common.cache.FileTimeStamp;\r
 \r
 /**\r
  Class OnDepdendency is used to check the timestamp between source files and\r
@@ -28,46 +28,57 @@ import org.apache.tools.ant.taskdefs.Sequential;
  be re-generated from source files.\r
  **/\r
 public class OnDependency extends Task {\r
-    ///\r
-    /// cache the modified timestamp of files accessed, to speed up the depencey check\r
-    /// \r
-    private static Map<String, Long> timeStampCache = new HashMap<String, Long>();\r
-    ///\r
-    /// source files list\r
-    ///\r
+    //\r
+    // source files list\r
+    //\r
     private DpFileList sources = null;\r
-    ///\r
-    /// target files list\r
-    ///\r
+\r
+    //\r
+    // target files list\r
+    //\r
     private DpFileList targets = null;\r
-    ///\r
-    /// tasks to be performed to generate target files\r
-    ///\r
+\r
+    //\r
+    // tasks to be performed to generate target files\r
+    //\r
     private Sequential  task = null;\r
 \r
-    ///\r
-    /// An empty constructor for an ANT task can avoid some potential issues\r
-    ///\r
+    /**\r
+     An empty constructor for an ANT task can avoid some potential issues\r
+     **/\r
     public OnDependency(){\r
     }\r
 \r
     /**\r
      Standard execute method of ANT task\r
      **/\r
-    public void execute() {\r
+    public void execute() throws BuildException {\r
         if (isOutOfDate() && task != null) {\r
             task.perform();\r
         }\r
+\r
+        //\r
+        // Update the time stamp of target files since they are just re-generated\r
+        // \r
+        for (Iterator dstIt = targets.nameList.iterator(); dstIt.hasNext();) {\r
+            FileTimeStamp.update((String)dstIt.next());\r
+        }\r
     }\r
 \r
-    ///\r
-    /// check if the target files are outofdate\r
-    ///\r
+    //\r
+    // check if the target files are outofdate\r
+    //\r
     private boolean isOutOfDate() {\r
         ///\r
         /// if no source files specified, take it as a fresh start\r
         ///\r
         if (sources.nameList.size() == 0) {\r
+            EdkLog.log(this, EdkLog.EDK_VERBOSE, "No source file spcified!");\r
+            return true;\r
+        }\r
+\r
+        if (targets.nameList.size() == 0) {\r
+            EdkLog.log(this, EdkLog.EDK_VERBOSE, "No target file found!");\r
             return true;\r
         }\r
 \r
@@ -76,32 +87,31 @@ public class OnDependency extends Task {
             String dstFileName = (String)dstIt.next();\r
             File dstFile = new File(dstFileName);\r
             if (!dstFile.exists()) {\r
+                EdkLog.log(this, EdkLog.EDK_VERBOSE, "Target file [" + dstFileName + "] doesn't exist!");\r
                 return true;\r
             }\r
 \r
-            long dstTimeStamp = dstFile.lastModified();\r
+            long dstTimeStamp = FileTimeStamp.get(dstFileName);\r
             Iterator srcIt = sources.nameList.iterator();\r
             while (srcIt.hasNext()) {\r
                 String srcFileName = (String)srcIt.next();\r
-                long srcTimeStamp;\r
-\r
-                if (timeStampCache.containsKey(srcFileName)) {\r
-                    srcTimeStamp = ((Long)timeStampCache.get(srcFileName)).longValue();\r
-                } else {\r
-                    File srcFile = new File(srcFileName);\r
-                    if (!srcFile.exists()) {\r
-                        throw new BuildException(srcFileName + " doesn't exist !!!");\r
-                    }\r
-                    srcTimeStamp = srcFile.lastModified();\r
-                    timeStampCache.put(srcFileName, new Long(srcTimeStamp));\r
+                long srcTimeStamp = FileTimeStamp.get(srcFileName);\r
+\r
+                if (srcTimeStamp == 0) {\r
+                    //\r
+                    // time stamp 0 means that the file doesn't exist\r
+                    // \r
+                    throw new BuildException("Source File name: " + srcFileName + " doesn't exist!!!");\r
                 }\r
 \r
                 if (dstTimeStamp < srcTimeStamp) {\r
+                    EdkLog.log(this, EdkLog.EDK_VERBOSE, "Source file [" + srcFileName + "] has been changed since last build!");\r
                     return true;\r
                 }\r
             }\r
         }\r
 \r
+        EdkLog.log(this, EdkLog.EDK_VERBOSE, "Target files are up-to-date!");\r
         return false;\r
     }\r
 \r