]> git.proxmox.com Git - mirror_edk2.git/blobdiff - Tools/Source/GenBuild/org/tianocore/build/GenBuildTask.java
Backup original properties to avoid property overriding message. This change is for...
[mirror_edk2.git] / Tools / Source / GenBuild / org / tianocore / build / GenBuildTask.java
index 62304774452967fc01cfedebf814e11b3e80d19e..bb9a2dfbcfb2d4cb2e8686391e2872fec8e002d2 100644 (file)
@@ -16,6 +16,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 package org.tianocore.build;\r
 \r
 import java.io.File;\r
+import java.util.Hashtable;\r
 import java.util.Iterator;\r
 import java.util.LinkedHashSet;\r
 import java.util.List;\r
@@ -26,6 +27,8 @@ import java.util.regex.Matcher;
 import java.util.regex.Pattern;\r
 \r
 import org.apache.tools.ant.BuildException;\r
+import org.apache.tools.ant.BuildListener;\r
+import org.apache.tools.ant.Project;\r
 import org.apache.tools.ant.taskdefs.Ant;\r
 import org.apache.tools.ant.taskdefs.Property;\r
 import org.apache.xmlbeans.XmlObject;\r
@@ -38,7 +41,6 @@ import org.tianocore.build.fpd.FpdParserTask;
 import org.tianocore.build.global.GenBuildLogger;\r
 import org.tianocore.build.global.GlobalData;\r
 import org.tianocore.build.global.OutputManager;\r
-import org.tianocore.build.global.PropertyManager;\r
 import org.tianocore.build.global.SurfaceAreaQuery;\r
 import org.tianocore.build.id.FpdModuleIdentification;\r
 import org.tianocore.build.id.ModuleIdentification;\r
@@ -125,22 +127,24 @@ public class GenBuildTask extends Ant {
               From module build, exception from module surface area invalid.\r
     **/\r
     public void execute() throws BuildException {\r
-       //\r
+       if (!FrameworkBuildTask.multithread) {\r
+            cleanupProperties();\r
+        }\r
+        \r
+        //\r
         // set Logger\r
         //\r
         GenBuildLogger logger = new GenBuildLogger(getProject());\r
         EdkLog.setLogLevel(getProject().getProperty("env.LOGLEVEL"));\r
         EdkLog.setLogger(logger);\r
 \r
-        PropertyManager.setProject(getProject());\r
-        PropertyManager.save();\r
         //\r
         // Enable all specified properties\r
         //\r
         Iterator<Property> iter = properties.iterator();\r
         while (iter.hasNext()) {\r
             Property item = iter.next();\r
-            PropertyManager.setProperty(item.getName(), item.getValue());\r
+            getProject().setProperty(item.getName(), item.getValue());\r
         }\r
 \r
         //\r
@@ -181,14 +185,6 @@ public class GenBuildTask extends Ant {
             //\r
             prepareSingleModuleBuild();\r
         } else {\r
-            //\r
-            // Platform build. Restore the platform related info\r
-            //\r
-            String filename = getProject().getProperty("PLATFORM_FILE");\r
-            PlatformIdentification platformId = GlobalData.getPlatform(filename);\r
-            PropertyManager.setProperty("PLATFORM_DIR", platformId.getFpdFile().getParent().replaceAll("(\\\\)", "/"));\r
-            PropertyManager.setProperty("PLATFORM_RELATIVE_DIR", platformId.getPlatformRelativeDir().replaceAll("(\\\\)", "/"));\r
-\r
             String packageGuid = getProject().getProperty("PACKAGE_GUID");\r
             String packageVersion = getProject().getProperty("PACKAGE_VERSION");\r
             PackageIdentification packageId = new PackageIdentification(packageGuid, packageVersion);\r
@@ -236,7 +232,7 @@ public class GenBuildTask extends Ant {
 \r
         for (int k = 0; k < archList.length; k++) {\r
 \r
-            PropertyManager.setProperty("ARCH", archList[k]);\r
+            getProject().setProperty("ARCH", archList[k]);\r
 \r
             FpdModuleIdentification fpdModuleId = new FpdModuleIdentification(moduleId, archList[k]);\r
 \r
@@ -261,7 +257,7 @@ public class GenBuildTask extends Ant {
                 // Prepare for target related common properties\r
                 // TARGET\r
                 //\r
-                PropertyManager.setProperty("TARGET", targetList[i]);\r
+                getProject().setProperty("TARGET", targetList[i]);\r
                 String[] toolchainList = GlobalData.getToolChainInfo().getTagnames();\r
                 for(int j = 0; j < toolchainList.length; j ++){\r
                     //\r
@@ -277,7 +273,7 @@ public class GenBuildTask extends Ant {
                     // Prepare for toolchain related common properties\r
                     // TOOLCHAIN\r
                     //\r
-                    PropertyManager.setProperty("TOOLCHAIN", toolchainList[j]);\r
+                    getProject().setProperty("TOOLCHAIN", toolchainList[j]);\r
 \r
                     System.out.println("Build " + moduleId + " start >>>");\r
                     System.out.println("Target: " + targetList[i] + " Tagname: " + toolchainList[j] + " Arch: " + archList[k]);\r
@@ -308,8 +304,6 @@ public class GenBuildTask extends Ant {
                 }\r
             }\r
         }\r
-        \r
-        PropertyManager.restore();\r
     }\r
 \r
     /**\r
@@ -347,17 +341,30 @@ public class GenBuildTask extends Ant {
         FpdParserTask fpdParser = new FpdParserTask();\r
         fpdParser.setProject(getProject());\r
         fpdParser.parseFpdFile(platformId.getFpdFile());\r
-        PropertyManager.setProperty("ARCH", fpdParser.getAllArchForModule(moduleId));\r
-\r
-        //\r
-        // Prepare for Platform related common properties\r
-        // PLATFORM, PLATFORM_DIR, PLATFORM_RELATIVE_DIR\r
-        //\r
-        PropertyManager.setProperty("PLATFORM", platformId.getName());\r
-        PropertyManager.setProperty("PLATFORM_DIR", platformId.getFpdFile().getParent().replaceAll("(\\\\)", "/"));\r
-        PropertyManager.setProperty("PLATFORM_RELATIVE_DIR", platformId.getPlatformRelativeDir().replaceAll("(\\\\)", "/"));\r
+        getProject().setProperty("ARCH", fpdParser.getAllArchForModule(moduleId));\r
     }\r
 \r
+    private void cleanupProperties() {\r
+        Project newProject = new Project();\r
+\r
+        Hashtable<String, String> passdownProperties = FrameworkBuildTask.originalProperties;\r
+        Iterator<String> iter = passdownProperties.keySet().iterator();\r
+        while (iter.hasNext()) {\r
+            String item = iter.next();\r
+            newProject.setProperty(item, passdownProperties.get(item));\r
+        }\r
+\r
+        newProject.setInputHandler(getProject().getInputHandler());\r
+\r
+        Iterator listenerIter = getProject().getBuildListeners().iterator();\r
+        while (listenerIter.hasNext()) {\r
+            newProject.addBuildListener((BuildListener) listenerIter.next());\r
+        }\r
+\r
+        getProject().initSubProject(newProject);\r
+\r
+        setProject(newProject);\r
+    }\r
 \r
     /**\r
       Set Module-Related information to properties.\r
@@ -370,29 +377,29 @@ public class GenBuildTask extends Ant {
         // PACKAGE, PACKAGE_GUID, PACKAGE_VERSION, PACKAGE_DIR, PACKAGE_RELATIVE_DIR\r
         //\r
         PackageIdentification packageId = moduleId.getPackage();\r
-        PropertyManager.setProperty("PACKAGE", packageId.getName());\r
-        PropertyManager.setProperty("PACKAGE_GUID", packageId.getGuid());\r
-        PropertyManager.setProperty("PACKAGE_VERSION", packageId.getVersion());\r
-        PropertyManager.setProperty("PACKAGE_DIR", packageId.getPackageDir().replaceAll("(\\\\)", "/"));\r
-        PropertyManager.setProperty("PACKAGE_RELATIVE_DIR", packageId.getPackageRelativeDir().replaceAll("(\\\\)", "/"));\r
+        getProject().setProperty("PACKAGE", packageId.getName());\r
+        getProject().setProperty("PACKAGE_GUID", packageId.getGuid());\r
+        getProject().setProperty("PACKAGE_VERSION", packageId.getVersion());\r
+        getProject().setProperty("PACKAGE_DIR", packageId.getPackageDir().replaceAll("(\\\\)", "/"));\r
+        getProject().setProperty("PACKAGE_RELATIVE_DIR", packageId.getPackageRelativeDir().replaceAll("(\\\\)", "/"));\r
 \r
         //\r
         // MODULE or BASE_NAME, GUID or FILE_GUID, VERSION, MODULE_TYPE\r
         // MODULE_DIR, MODULE_RELATIVE_DIR\r
         //\r
-        PropertyManager.setProperty("MODULE", moduleId.getName());\r
+        getProject().setProperty("MODULE", moduleId.getName());\r
         String baseName = saq.getModuleOutputFileBasename();\r
         if (baseName == null) {\r
-            PropertyManager.setProperty("BASE_NAME", moduleId.getName());\r
+            getProject().setProperty("BASE_NAME", moduleId.getName());\r
         } else {\r
-            PropertyManager.setProperty("BASE_NAME", baseName);\r
+            getProject().setProperty("BASE_NAME", baseName);\r
         }\r
-        PropertyManager.setProperty("GUID", moduleId.getGuid());\r
-        PropertyManager.setProperty("FILE_GUID", moduleId.getGuid());\r
-        PropertyManager.setProperty("VERSION", moduleId.getVersion());\r
-        PropertyManager.setProperty("MODULE_TYPE", moduleId.getModuleType());\r
-        PropertyManager.setProperty("MODULE_DIR", moduleId.getMsaFile().getParent().replaceAll("(\\\\)", "/"));\r
-        PropertyManager.setProperty("MODULE_RELATIVE_DIR", moduleId.getModuleRelativePath().replaceAll("(\\\\)", "/"));\r
+        getProject().setProperty("GUID", moduleId.getGuid());\r
+        getProject().setProperty("FILE_GUID", moduleId.getGuid());\r
+        getProject().setProperty("VERSION", moduleId.getVersion());\r
+        getProject().setProperty("MODULE_TYPE", moduleId.getModuleType());\r
+        getProject().setProperty("MODULE_DIR", moduleId.getMsaFile().getParent().replaceAll("(\\\\)", "/"));\r
+        getProject().setProperty("MODULE_RELATIVE_DIR", moduleId.getModuleRelativePath().replaceAll("(\\\\)", "/"));\r
 \r
         //\r
         // SUBSYSTEM\r
@@ -418,18 +425,18 @@ public class GenBuildTask extends Ant {
                 break ;\r
             }\r
         }\r
-        PropertyManager.setProperty("SUBSYSTEM", subsystem);\r
+        getProject().setProperty("SUBSYSTEM", subsystem);\r
 \r
         //\r
         // ENTRYPOINT\r
         //\r
         if (arch.equalsIgnoreCase("EBC")) {\r
-            PropertyManager.setProperty("ENTRYPOINT", "EfiStart");\r
+            getProject().setProperty("ENTRYPOINT", "EfiStart");\r
         } else {\r
-            PropertyManager.setProperty("ENTRYPOINT", "_ModuleEntryPoint");\r
+            getProject().setProperty("ENTRYPOINT", "_ModuleEntryPoint");\r
         }\r
 \r
-        PropertyManager.setProperty("OBJECTS", "");\r
+        getProject().setProperty("OBJECTS", "");\r
     }\r
 \r
     private void getCompilerFlags(String target, String toolchain, FpdModuleIdentification fpdModuleId) throws EdkException {\r
@@ -444,7 +451,7 @@ public class GenBuildTask extends Ant {
             key[4] = ToolDefinitions.TOOLS_DEF_ATTRIBUTE_NAME;\r
             String cmdName = GlobalData.getCommandSetting(key, fpdModuleId);\r
             File cmdFile = new File(cmdPath + File.separatorChar + cmdName);\r
-            PropertyManager.setProperty(cmd[m], cmdFile.getPath().replaceAll("(\\\\)", "/"));\r
+            getProject().setProperty(cmd[m], cmdFile.getPath().replaceAll("(\\\\)", "/"));\r
 \r
             //\r
             // set CC_FLAGS\r
@@ -454,7 +461,7 @@ public class GenBuildTask extends Ant {
             Set<String> addset = new LinkedHashSet<String>();\r
             Set<String> subset = new LinkedHashSet<String>();\r
             putFlagsToSet(addset, cmdFlags);\r
-            PropertyManager.setProperty(cmd[m] + "_FLAGS", getProject().replaceProperties(getFlags(addset, subset)));\r
+            getProject().setProperty(cmd[m] + "_FLAGS", getProject().replaceProperties(getFlags(addset, subset)));\r
 \r
             //\r
             // Set CC_EXT\r
@@ -462,9 +469,9 @@ public class GenBuildTask extends Ant {
             key[4] = ToolDefinitions.TOOLS_DEF_ATTRIBUTE_EXT;\r
             String extName = GlobalData.getCommandSetting(key, fpdModuleId);\r
             if ( extName != null && ! extName.equalsIgnoreCase("")) {\r
-                PropertyManager.setProperty(cmd[m] + "_EXT", extName);\r
+                getProject().setProperty(cmd[m] + "_EXT", extName);\r
             } else {\r
-                PropertyManager.setProperty(cmd[m] + "_EXT", "");\r
+                getProject().setProperty(cmd[m] + "_EXT", "");\r
             }\r
 \r
             //\r
@@ -473,7 +480,7 @@ public class GenBuildTask extends Ant {
             key[4] = ToolDefinitions.TOOLS_DEF_ATTRIBUTE_FAMILY;\r
             String toolChainFamily = GlobalData.getCommandSetting(key, fpdModuleId);\r
             if (toolChainFamily != null) {\r
-                PropertyManager.setProperty(cmd[m] + "_FAMILY", toolChainFamily);\r
+                getProject().setProperty(cmd[m] + "_FAMILY", toolChainFamily);\r
             }\r
 \r
             //\r
@@ -482,9 +489,9 @@ public class GenBuildTask extends Ant {
             key[4] = ToolDefinitions.TOOLS_DEF_ATTRIBUTE_SPATH;\r
             String spath = GlobalData.getCommandSetting(key, fpdModuleId);\r
             if (spath != null) {\r
-                PropertyManager.setProperty(cmd[m] + "_SPATH", spath.replaceAll("(\\\\)", "/"));\r
+                getProject().setProperty(cmd[m] + "_SPATH", spath.replaceAll("(\\\\)", "/"));\r
             } else {\r
-                PropertyManager.setProperty(cmd[m] + "_SPATH", "");\r
+                getProject().setProperty(cmd[m] + "_SPATH", "");\r
             }\r
 \r
             //\r
@@ -493,9 +500,9 @@ public class GenBuildTask extends Ant {
             key[4] = ToolDefinitions.TOOLS_DEF_ATTRIBUTE_DPATH;\r
             String dpath = GlobalData.getCommandSetting(key, fpdModuleId);\r
             if (dpath != null) {\r
-                PropertyManager.setProperty(cmd[m] + "_DPATH", dpath.replaceAll("(\\\\)", "/"));\r
+                getProject().setProperty(cmd[m] + "_DPATH", dpath.replaceAll("(\\\\)", "/"));\r
             } else {\r
-                PropertyManager.setProperty(cmd[m] + "_DPATH", "");\r
+                getProject().setProperty(cmd[m] + "_DPATH", "");\r
             }\r
         }\r
     }\r
@@ -566,7 +573,7 @@ public class GenBuildTask extends Ant {
         for (int i = 0; i < libinstances.length; i++) {\r
             propertyLibs += " " + getProject().getProperty("BIN_DIR") + File.separatorChar + libinstances[i].getName() + ".lib";\r
         }\r
-        PropertyManager.setProperty("LIBS", propertyLibs.replaceAll("(\\\\)", "/"));\r
+        getProject().setProperty("LIBS", propertyLibs.replaceAll("(\\\\)", "/"));\r
 \r
         //\r
         // Get all includepath and set to INCLUDE_PATHS\r
@@ -768,7 +775,7 @@ public class GenBuildTask extends Ant {
             includePaths.append(iter.next());\r
             includePaths.append("; ");\r
         }\r
-        PropertyManager.setProperty("INCLUDE_PATHS", getProject().replaceProperties(includePaths.toString()).replaceAll("(\\\\)", "/"));\r
+        getProject().setProperty("INCLUDE_PATHS", getProject().replaceProperties(includePaths.toString()).replaceAll("(\\\\)", "/"));\r
         \r
         return includes.toArray(new String[includes.size()]);\r
     }\r