]> git.proxmox.com Git - mirror_edk2.git/blobdiff - BaseTools/Source/Python/Workspace/InfBuildData.py
BaseTools: Use absolute import in Workspace
[mirror_edk2.git] / BaseTools / Source / Python / Workspace / InfBuildData.py
index 7ea9b56d5decba82b52e742d15851d1c63ce6619..f79ffe28d804b148e4bc554e6f9c73f648283143 100644 (file)
@@ -1,7 +1,7 @@
 ## @file\r
 # This file is used to create a database used by build tool\r
 #\r
-# Copyright (c) 2008 - 2017, Intel Corporation. All rights reserved.<BR>\r
+# Copyright (c) 2008 - 2018, Intel Corporation. All rights reserved.<BR>\r
 # (C) Copyright 2016 Hewlett Packard Enterprise Development LP<BR>\r
 # This program and the accompanying materials\r
 # are licensed and made available under the terms and conditions of the BSD License\r
 # WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
 #\r
 \r
-from Common.String import *\r
+from __future__ import absolute_import\r
+from Common.StringUtils import *\r
 from Common.DataType import *\r
 from Common.Misc import *\r
 from types import *\r
-from MetaFileParser import *\r
+from .MetaFileParser import *\r
+from collections import OrderedDict\r
 \r
 from Workspace.BuildClassObject import ModuleBuildClassObject, LibraryClassObject, PcdClassObject\r
 ## Module build information from INF file\r
@@ -27,17 +29,17 @@ from Workspace.BuildClassObject import ModuleBuildClassObject, LibraryClassObjec
 class InfBuildData(ModuleBuildClassObject):\r
     # dict used to convert PCD type in database to string used by build tool\r
     _PCD_TYPE_STRING_ = {\r
-        MODEL_PCD_FIXED_AT_BUILD        :   "FixedAtBuild",\r
-        MODEL_PCD_PATCHABLE_IN_MODULE   :   "PatchableInModule",\r
-        MODEL_PCD_FEATURE_FLAG          :   "FeatureFlag",\r
-        MODEL_PCD_DYNAMIC               :   "Dynamic",\r
-        MODEL_PCD_DYNAMIC_DEFAULT       :   "Dynamic",\r
-        MODEL_PCD_DYNAMIC_HII           :   "DynamicHii",\r
-        MODEL_PCD_DYNAMIC_VPD           :   "DynamicVpd",\r
-        MODEL_PCD_DYNAMIC_EX            :   "DynamicEx",\r
-        MODEL_PCD_DYNAMIC_EX_DEFAULT    :   "DynamicEx",\r
-        MODEL_PCD_DYNAMIC_EX_HII        :   "DynamicExHii",\r
-        MODEL_PCD_DYNAMIC_EX_VPD        :   "DynamicExVpd",\r
+        MODEL_PCD_FIXED_AT_BUILD        :   TAB_PCDS_FIXED_AT_BUILD,\r
+        MODEL_PCD_PATCHABLE_IN_MODULE   :   TAB_PCDS_PATCHABLE_IN_MODULE,\r
+        MODEL_PCD_FEATURE_FLAG          :   TAB_PCDS_FEATURE_FLAG,\r
+        MODEL_PCD_DYNAMIC               :   TAB_PCDS_DYNAMIC,\r
+        MODEL_PCD_DYNAMIC_DEFAULT       :   TAB_PCDS_DYNAMIC,\r
+        MODEL_PCD_DYNAMIC_HII           :   TAB_PCDS_DYNAMIC_HII,\r
+        MODEL_PCD_DYNAMIC_VPD           :   TAB_PCDS_DYNAMIC_VPD,\r
+        MODEL_PCD_DYNAMIC_EX            :   TAB_PCDS_DYNAMIC_EX,\r
+        MODEL_PCD_DYNAMIC_EX_DEFAULT    :   TAB_PCDS_DYNAMIC_EX,\r
+        MODEL_PCD_DYNAMIC_EX_HII        :   TAB_PCDS_DYNAMIC_EX_HII,\r
+        MODEL_PCD_DYNAMIC_EX_VPD        :   TAB_PCDS_DYNAMIC_EX_VPD,\r
     }\r
 \r
     # dict used to convert part of [Defines] to members of InfBuildData directly\r
@@ -65,32 +67,12 @@ class InfBuildData(ModuleBuildClassObject):
         TAB_COMPONENTS_SOURCE_OVERRIDE_PATH         : "_SourceOverridePath",\r
     }\r
 \r
-    # dict used to convert Component type to Module type\r
-    _MODULE_TYPE_ = {\r
-        "LIBRARY"               :   "BASE",\r
-        "SECURITY_CORE"         :   "SEC",\r
-        "PEI_CORE"              :   "PEI_CORE",\r
-        "COMBINED_PEIM_DRIVER"  :   "PEIM",\r
-        "PIC_PEIM"              :   "PEIM",\r
-        "RELOCATABLE_PEIM"      :   "PEIM",\r
-        "PE32_PEIM"             :   "PEIM",\r
-        "BS_DRIVER"             :   "DXE_DRIVER",\r
-        "RT_DRIVER"             :   "DXE_RUNTIME_DRIVER",\r
-        "SAL_RT_DRIVER"         :   "DXE_SAL_DRIVER",\r
-        "DXE_SMM_DRIVER"        :   "DXE_SMM_DRIVER",\r
-    #    "SMM_DRIVER"            :   "DXE_SMM_DRIVER",\r
-    #    "BS_DRIVER"             :   "DXE_SMM_DRIVER",\r
-    #    "BS_DRIVER"             :   "UEFI_DRIVER",\r
-        "APPLICATION"           :   "UEFI_APPLICATION",\r
-        "LOGO"                  :   "BASE",\r
-    }\r
-\r
     # regular expression for converting XXX_FLAGS in [nmake] section to new type\r
     _NMAKE_FLAG_PATTERN_ = re.compile("(?:EBC_)?([A-Z]+)_(?:STD_|PROJ_|ARCH_)?FLAGS(?:_DLL|_ASL|_EXE)?", re.UNICODE)\r
     # dict used to convert old tool name used in [nmake] section to new ones\r
     _TOOL_CODE_ = {\r
         "C"         :   "CC",\r
-        "LIB"       :   "SLINK",\r
+        BINARY_FILE_TYPE_LIB       :   "SLINK",\r
         "LINK"      :   "DLINK",\r
     }\r
 \r
@@ -106,7 +88,7 @@ class InfBuildData(ModuleBuildClassObject):
     #   @param      Platform        The name of platform employing this module\r
     #   @param      Macros          Macros used for replacement in DSC file\r
     #\r
-    def __init__(self, FilePath, RawData, BuildDatabase, Arch='COMMON', Target=None, Toolchain=None):\r
+    def __init__(self, FilePath, RawData, BuildDatabase, Arch=TAB_ARCH_COMMON, Target=None, Toolchain=None):\r
         self.MetaFile = FilePath\r
         self._ModuleDir = FilePath.Dir\r
         self._RawData = RawData\r
@@ -114,7 +96,7 @@ class InfBuildData(ModuleBuildClassObject):
         self._Arch = Arch\r
         self._Target = Target\r
         self._Toolchain = Toolchain\r
-        self._Platform = 'COMMON'\r
+        self._Platform = TAB_COMMON\r
         self._SourceOverridePath = None\r
         if FilePath.Key in GlobalData.gOverrideDir:\r
             self._SourceOverridePath = GlobalData.gOverrideDir[FilePath.Key]\r
@@ -156,7 +138,7 @@ class InfBuildData(ModuleBuildClassObject):
         self._ModuleUnloadImageList = None\r
         self._ConstructorList       = None\r
         self._DestructorList        = None\r
-        self._Defs                  = None\r
+        self._Defs                  = OrderedDict()\r
         self._Binaries              = None\r
         self._Sources               = None\r
         self._LibraryClasses        = None\r
@@ -166,7 +148,7 @@ class InfBuildData(ModuleBuildClassObject):
         self._Ppis                  = None\r
         self._PpiComments           = None\r
         self._Guids                 = None\r
-        self._GuidsUsedByPcd        = sdict()\r
+        self._GuidsUsedByPcd        = OrderedDict()\r
         self._GuidComments          = None\r
         self._Includes              = None\r
         self._Packages              = None\r
@@ -179,7 +161,7 @@ class InfBuildData(ModuleBuildClassObject):
 \r
     ## Get current effective macros\r
     def _GetMacros(self):\r
-        if self.__Macros == None:\r
+        if self.__Macros is None:\r
             self.__Macros = {}\r
             # EDK_GLOBAL defined macros can be applied to EDK module\r
             if self.AutoGenVersion < 0x00010005:\r
@@ -246,23 +228,21 @@ class InfBuildData(ModuleBuildClassObject):
             # items defined _PROPERTY_ don't need additional processing\r
             if Name in self:\r
                 self[Name] = Value\r
-                if self._Defs == None:\r
-                    self._Defs = sdict()\r
                 self._Defs[Name] = Value\r
                 self._Macros[Name] = Value\r
             # some special items in [Defines] section need special treatment\r
             elif Name in ('EFI_SPECIFICATION_VERSION', 'UEFI_SPECIFICATION_VERSION', 'EDK_RELEASE_VERSION', 'PI_SPECIFICATION_VERSION'):\r
                 if Name in ('EFI_SPECIFICATION_VERSION', 'UEFI_SPECIFICATION_VERSION'):\r
                     Name = 'UEFI_SPECIFICATION_VERSION'\r
-                if self._Specification == None:\r
-                    self._Specification = sdict()\r
+                if self._Specification is None:\r
+                    self._Specification = OrderedDict()\r
                 self._Specification[Name] = GetHexVerValue(Value)\r
-                if self._Specification[Name] == None:\r
+                if self._Specification[Name] is None:\r
                     EdkLogger.error("build", FORMAT_NOT_SUPPORTED,\r
                                     "'%s' format is not supported for %s" % (Value, Name),\r
                                     File=self.MetaFile, Line=Record[-1])\r
             elif Name == 'LIBRARY_CLASS':\r
-                if self._LibraryClass == None:\r
+                if self._LibraryClass is None:\r
                     self._LibraryClass = []\r
                 ValueList = GetSplitValueList(Value)\r
                 LibraryClass = ValueList[0]\r
@@ -272,30 +252,30 @@ class InfBuildData(ModuleBuildClassObject):
                     SupModuleList = SUP_MODULE_LIST\r
                 self._LibraryClass.append(LibraryClassObject(LibraryClass, SupModuleList))\r
             elif Name == 'ENTRY_POINT':\r
-                if self._ModuleEntryPointList == None:\r
+                if self._ModuleEntryPointList is None:\r
                     self._ModuleEntryPointList = []\r
                 self._ModuleEntryPointList.append(Value)\r
             elif Name == 'UNLOAD_IMAGE':\r
-                if self._ModuleUnloadImageList == None:\r
+                if self._ModuleUnloadImageList is None:\r
                     self._ModuleUnloadImageList = []\r
                 if not Value:\r
                     continue\r
                 self._ModuleUnloadImageList.append(Value)\r
             elif Name == 'CONSTRUCTOR':\r
-                if self._ConstructorList == None:\r
+                if self._ConstructorList is None:\r
                     self._ConstructorList = []\r
                 if not Value:\r
                     continue\r
                 self._ConstructorList.append(Value)\r
             elif Name == 'DESTRUCTOR':\r
-                if self._DestructorList == None:\r
+                if self._DestructorList is None:\r
                     self._DestructorList = []\r
                 if not Value:\r
                     continue\r
                 self._DestructorList.append(Value)\r
             elif Name == TAB_INF_DEFINES_CUSTOM_MAKEFILE:\r
                 TokenList = GetSplitValueList(Value)\r
-                if self._CustomMakefile == None:\r
+                if self._CustomMakefile is None:\r
                     self._CustomMakefile = {}\r
                 if len(TokenList) < 2:\r
                     self._CustomMakefile['MSFT'] = TokenList[0]\r
@@ -307,8 +287,6 @@ class InfBuildData(ModuleBuildClassObject):
                                         File=self.MetaFile, Line=Record[-1])\r
                     self._CustomMakefile[TokenList[0]] = TokenList[1]\r
             else:\r
-                if self._Defs == None:\r
-                    self._Defs = sdict()\r
                 self._Defs[Name] = Value\r
                 self._Macros[Name] = Value\r
 \r
@@ -329,22 +307,22 @@ class InfBuildData(ModuleBuildClassObject):
                 EdkLogger.error("build", FORMAT_NOT_SUPPORTED,\r
                                 "MODULE_TYPE %s is not supported for EDK II, valid values are:\n %s" % (self._ModuleType, ' '.join(l for l in SUP_MODULE_LIST)),\r
                                 File=self.MetaFile, Line=LineNo)\r
-            if (self._Specification == None) or (not 'PI_SPECIFICATION_VERSION' in self._Specification) or (int(self._Specification['PI_SPECIFICATION_VERSION'], 16) < 0x0001000A):\r
+            if (self._Specification is None) or (not 'PI_SPECIFICATION_VERSION' in self._Specification) or (int(self._Specification['PI_SPECIFICATION_VERSION'], 16) < 0x0001000A):\r
                 if self._ModuleType == SUP_MODULE_SMM_CORE:\r
                     EdkLogger.error("build", FORMAT_NOT_SUPPORTED, "SMM_CORE module type can't be used in the module with PI_SPECIFICATION_VERSION less than 0x0001000A", File=self.MetaFile)\r
-            if (self._Specification == None) or (not 'PI_SPECIFICATION_VERSION' in self._Specification) or (int(self._Specification['PI_SPECIFICATION_VERSION'], 16) < 0x00010032):\r
+            if (self._Specification is None) or (not 'PI_SPECIFICATION_VERSION' in self._Specification) or (int(self._Specification['PI_SPECIFICATION_VERSION'], 16) < 0x00010032):\r
                 if self._ModuleType == SUP_MODULE_MM_CORE_STANDALONE:\r
                     EdkLogger.error("build", FORMAT_NOT_SUPPORTED, "MM_CORE_STANDALONE module type can't be used in the module with PI_SPECIFICATION_VERSION less than 0x00010032", File=self.MetaFile)\r
                 if self._ModuleType == SUP_MODULE_MM_STANDALONE:\r
                     EdkLogger.error("build", FORMAT_NOT_SUPPORTED, "MM_STANDALONE module type can't be used in the module with PI_SPECIFICATION_VERSION less than 0x00010032", File=self.MetaFile)\r
-            if self._Defs and 'PCI_DEVICE_ID' in self._Defs and 'PCI_VENDOR_ID' in self._Defs \\r
+            if 'PCI_DEVICE_ID' in self._Defs and 'PCI_VENDOR_ID' in self._Defs \\r
                and 'PCI_CLASS_CODE' in self._Defs and 'PCI_REVISION' in self._Defs:\r
                 self._BuildType = 'UEFI_OPTIONROM'\r
                 if 'PCI_COMPRESS' in self._Defs:\r
                     if self._Defs['PCI_COMPRESS'] not in ('TRUE', 'FALSE'):\r
                         EdkLogger.error("build", FORMAT_INVALID, "Expected TRUE/FALSE for PCI_COMPRESS: %s" % self.MetaFile)\r
 \r
-            elif self._Defs and 'UEFI_HII_RESOURCE_SECTION' in self._Defs \\r
+            elif 'UEFI_HII_RESOURCE_SECTION' in self._Defs \\r
                and self._Defs['UEFI_HII_RESOURCE_SECTION'] == 'TRUE':\r
                 self._BuildType = 'UEFI_HII'\r
             else:\r
@@ -357,7 +335,7 @@ class InfBuildData(ModuleBuildClassObject):
                 if ErrorCode != 0:\r
                     EdkLogger.error('build', ErrorCode, ExtraData=ErrorInfo,\r
                                     File=self.MetaFile, Line=LineNo)\r
-                if self.Sources == None:\r
+                if self.Sources is None:\r
                     self._Sources = []\r
                 self._Sources.append(File)\r
         else:\r
@@ -365,9 +343,9 @@ class InfBuildData(ModuleBuildClassObject):
                 EdkLogger.error("build", ATTRIBUTE_NOT_AVAILABLE,\r
                                 "COMPONENT_TYPE is not given", File=self.MetaFile)\r
             self._BuildType = self._ComponentType.upper()\r
-            if self._ComponentType in self._MODULE_TYPE_:\r
-                self._ModuleType = self._MODULE_TYPE_[self._ComponentType]\r
-            if self._ComponentType == 'LIBRARY':\r
+            if self._ComponentType in COMPONENT_TO_MODULE_MAP_DICT:\r
+                self._ModuleType = COMPONENT_TO_MODULE_MAP_DICT[self._ComponentType]\r
+            if self._ComponentType == EDK_COMPONENT_TYPE_LIBRARY:\r
                 self._LibraryClass = [LibraryClassObject(self._BaseName, SUP_MODULE_LIST)]\r
             # make use some [nmake] section macros\r
             Macros = self._Macros\r
@@ -377,7 +355,7 @@ class InfBuildData(ModuleBuildClassObject):
             for Name, Value, Dummy, Arch, Platform, ID, LineNo in RecordList:\r
                 Value = ReplaceMacro(Value, Macros, True)\r
                 if Name == "IMAGE_ENTRY_POINT":\r
-                    if self._ModuleEntryPointList == None:\r
+                    if self._ModuleEntryPointList is None:\r
                         self._ModuleEntryPointList = []\r
                     self._ModuleEntryPointList.append(Value)\r
                 elif Name == "DPX_SOURCE":\r
@@ -387,7 +365,7 @@ class InfBuildData(ModuleBuildClassObject):
                     if ErrorCode != 0:\r
                         EdkLogger.error('build', ErrorCode, ExtraData=ErrorInfo,\r
                                         File=self.MetaFile, Line=LineNo)\r
-                    if self.Sources == None:\r
+                    if self.Sources is None:\r
                         self._Sources = []\r
                     self._Sources.append(File)\r
                 else:\r
@@ -397,8 +375,8 @@ class InfBuildData(ModuleBuildClassObject):
 #                        EdkLogger.warn("build", "Don't know how to do with macro [%s]" % Name,\r
 #                                       File=self.MetaFile, Line=LineNo)\r
                     else:\r
-                        if self._BuildOptions == None:\r
-                            self._BuildOptions = sdict()\r
+                        if self._BuildOptions is None:\r
+                            self._BuildOptions = OrderedDict()\r
 \r
                         if ToolList[0] in self._TOOL_CODE_:\r
                             Tool = self._TOOL_CODE_[ToolList[0]]\r
@@ -424,7 +402,7 @@ class InfBuildData(ModuleBuildClassObject):
 \r
     ## Retrieve file version\r
     def _GetInfVersion(self):\r
-        if self._AutoGenVersion == None:\r
+        if self._AutoGenVersion is None:\r
             RecordList = self._RawData[MODEL_META_DATA_HEADER, self._Arch, self._Platform]\r
             for Record in RecordList:\r
                 if Record[1] == TAB_INF_DEFINES_INF_VERSION:\r
@@ -436,90 +414,90 @@ class InfBuildData(ModuleBuildClassObject):
                     else:\r
                         self._AutoGenVersion = int(Record[2], 0)\r
                     break\r
-            if self._AutoGenVersion == None:\r
+            if self._AutoGenVersion is None:\r
                 self._AutoGenVersion = 0x00010000\r
         return self._AutoGenVersion\r
 \r
     ## Retrieve BASE_NAME\r
     def _GetBaseName(self):\r
-        if self._BaseName == None:\r
-            if self._Header_ == None:\r
+        if self._BaseName is None:\r
+            if self._Header_ is None:\r
                 self._GetHeaderInfo()\r
-            if self._BaseName == None:\r
+            if self._BaseName is None:\r
                 EdkLogger.error('build', ATTRIBUTE_NOT_AVAILABLE, "No BASE_NAME name", File=self.MetaFile)\r
         return self._BaseName\r
 \r
     ## Retrieve DxsFile\r
     def _GetDxsFile(self):\r
-        if self._DxsFile == None:\r
-            if self._Header_ == None:\r
+        if self._DxsFile is None:\r
+            if self._Header_ is None:\r
                 self._GetHeaderInfo()\r
-            if self._DxsFile == None:\r
+            if self._DxsFile is None:\r
                 self._DxsFile = ''\r
         return self._DxsFile\r
 \r
     ## Retrieve MODULE_TYPE\r
     def _GetModuleType(self):\r
-        if self._ModuleType == None:\r
-            if self._Header_ == None:\r
+        if self._ModuleType is None:\r
+            if self._Header_ is None:\r
                 self._GetHeaderInfo()\r
-            if self._ModuleType == None:\r
-                self._ModuleType = 'BASE'\r
+            if self._ModuleType is None:\r
+                self._ModuleType = SUP_MODULE_BASE\r
             if self._ModuleType not in SUP_MODULE_LIST:\r
-                self._ModuleType = "USER_DEFINED"\r
+                self._ModuleType = SUP_MODULE_USER_DEFINED\r
         return self._ModuleType\r
 \r
     ## Retrieve COMPONENT_TYPE\r
     def _GetComponentType(self):\r
-        if self._ComponentType == None:\r
-            if self._Header_ == None:\r
+        if self._ComponentType is None:\r
+            if self._Header_ is None:\r
                 self._GetHeaderInfo()\r
-            if self._ComponentType == None:\r
-                self._ComponentType = 'USER_DEFINED'\r
+            if self._ComponentType is None:\r
+                self._ComponentType = SUP_MODULE_USER_DEFINED\r
         return self._ComponentType\r
 \r
     ## Retrieve "BUILD_TYPE"\r
     def _GetBuildType(self):\r
-        if self._BuildType == None:\r
-            if self._Header_ == None:\r
+        if self._BuildType is None:\r
+            if self._Header_ is None:\r
                 self._GetHeaderInfo()\r
             if not self._BuildType:\r
-                self._BuildType = "BASE"\r
+                self._BuildType = SUP_MODULE_BASE\r
         return self._BuildType\r
 \r
     ## Retrieve file guid\r
     def _GetFileGuid(self):\r
-        if self._Guid == None:\r
-            if self._Header_ == None:\r
+        if self._Guid is None:\r
+            if self._Header_ is None:\r
                 self._GetHeaderInfo()\r
-            if self._Guid == None:\r
+            if self._Guid is None:\r
                 self._Guid = '00000000-0000-0000-0000-000000000000'\r
         return self._Guid\r
 \r
     ## Retrieve module version\r
     def _GetVersion(self):\r
-        if self._Version == None:\r
-            if self._Header_ == None:\r
+        if self._Version is None:\r
+            if self._Header_ is None:\r
                 self._GetHeaderInfo()\r
-            if self._Version == None:\r
+            if self._Version is None:\r
                 self._Version = '0.0'\r
         return self._Version\r
 \r
     ## Retrieve PCD_IS_DRIVER\r
     def _GetPcdIsDriver(self):\r
-        if self._PcdIsDriver == None:\r
-            if self._Header_ == None:\r
+        if self._PcdIsDriver is None:\r
+            if self._Header_ is None:\r
                 self._GetHeaderInfo()\r
-            if self._PcdIsDriver == None:\r
+            if self._PcdIsDriver is None:\r
                 self._PcdIsDriver = ''\r
         return self._PcdIsDriver\r
 \r
     ## Retrieve SHADOW\r
     def _GetShadow(self):\r
-        if self._Shadow == None:\r
-            if self._Header_ == None:\r
+        if self._Shadow is None:\r
+            if self._Header_ is None:\r
                 self._GetHeaderInfo()\r
-            if self._Shadow != None and self._Shadow.upper() == 'TRUE':\r
+            if self._Shadow is not None and self._Shadow.upper() == 'TRUE':\r
                 self._Shadow = True\r
             else:\r
                 self._Shadow = False\r
@@ -527,79 +505,76 @@ class InfBuildData(ModuleBuildClassObject):
 \r
     ## Retrieve CUSTOM_MAKEFILE\r
     def _GetMakefile(self):\r
-        if self._CustomMakefile == None:\r
-            if self._Header_ == None:\r
+        if self._CustomMakefile is None:\r
+            if self._Header_ is None:\r
                 self._GetHeaderInfo()\r
-            if self._CustomMakefile == None:\r
+            if self._CustomMakefile is None:\r
                 self._CustomMakefile = {}\r
         return self._CustomMakefile\r
 \r
     ## Retrieve EFI_SPECIFICATION_VERSION\r
     def _GetSpec(self):\r
-        if self._Specification == None:\r
-            if self._Header_ == None:\r
+        if self._Specification is None:\r
+            if self._Header_ is None:\r
                 self._GetHeaderInfo()\r
-            if self._Specification == None:\r
+            if self._Specification is None:\r
                 self._Specification = {}\r
         return self._Specification\r
 \r
     ## Retrieve LIBRARY_CLASS\r
     def _GetLibraryClass(self):\r
-        if self._LibraryClass == None:\r
-            if self._Header_ == None:\r
+        if self._LibraryClass is None:\r
+            if self._Header_ is None:\r
                 self._GetHeaderInfo()\r
-            if self._LibraryClass == None:\r
+            if self._LibraryClass is None:\r
                 self._LibraryClass = []\r
         return self._LibraryClass\r
 \r
     ## Retrieve ENTRY_POINT\r
     def _GetEntryPoint(self):\r
-        if self._ModuleEntryPointList == None:\r
-            if self._Header_ == None:\r
+        if self._ModuleEntryPointList is None:\r
+            if self._Header_ is None:\r
                 self._GetHeaderInfo()\r
-            if self._ModuleEntryPointList == None:\r
+            if self._ModuleEntryPointList is None:\r
                 self._ModuleEntryPointList = []\r
         return self._ModuleEntryPointList\r
 \r
     ## Retrieve UNLOAD_IMAGE\r
     def _GetUnloadImage(self):\r
-        if self._ModuleUnloadImageList == None:\r
-            if self._Header_ == None:\r
+        if self._ModuleUnloadImageList is None:\r
+            if self._Header_ is None:\r
                 self._GetHeaderInfo()\r
-            if self._ModuleUnloadImageList == None:\r
+            if self._ModuleUnloadImageList is None:\r
                 self._ModuleUnloadImageList = []\r
         return self._ModuleUnloadImageList\r
 \r
     ## Retrieve CONSTRUCTOR\r
     def _GetConstructor(self):\r
-        if self._ConstructorList == None:\r
-            if self._Header_ == None:\r
+        if self._ConstructorList is None:\r
+            if self._Header_ is None:\r
                 self._GetHeaderInfo()\r
-            if self._ConstructorList == None:\r
+            if self._ConstructorList is None:\r
                 self._ConstructorList = []\r
         return self._ConstructorList\r
 \r
     ## Retrieve DESTRUCTOR\r
     def _GetDestructor(self):\r
-        if self._DestructorList == None:\r
-            if self._Header_ == None:\r
+        if self._DestructorList is None:\r
+            if self._Header_ is None:\r
                 self._GetHeaderInfo()\r
-            if self._DestructorList == None:\r
+            if self._DestructorList is None:\r
                 self._DestructorList = []\r
         return self._DestructorList\r
 \r
     ## Retrieve definies other than above ones\r
     def _GetDefines(self):\r
-        if self._Defs == None:\r
-            if self._Header_ == None:\r
-                self._GetHeaderInfo()\r
-            if self._Defs == None:\r
-                self._Defs = sdict()\r
+        if len(self._Defs) == 0 and self._Header_ is None:\r
+            self._GetHeaderInfo()\r
         return self._Defs\r
 \r
     ## Retrieve binary files\r
     def _GetBinaries(self):\r
-        if self._Binaries == None:\r
+        if self._Binaries is None:\r
             self._Binaries = []\r
             RecordList = self._RawData[MODEL_EFI_BINARY_FILE, self._Arch, self._Platform]\r
             Macros = self._Macros\r
@@ -608,7 +583,7 @@ class InfBuildData(ModuleBuildClassObject):
             for Record in RecordList:\r
                 FileType = Record[0]\r
                 LineNo = Record[-1]\r
-                Target = 'COMMON'\r
+                Target = TAB_COMMON\r
                 FeatureFlag = []\r
                 if Record[2]:\r
                     TokenList = GetSplitValueList(Record[2], TAB_VALUE_SPLIT)\r
@@ -646,7 +621,7 @@ class InfBuildData(ModuleBuildClassObject):
             self._Sources = []\r
             return self._Sources\r
 \r
-        if self._Sources == None:\r
+        if self._Sources is None:\r
             self._Sources = []\r
             RecordList = self._RawData[MODEL_EFI_SOURCE_FILE, self._Arch, self._Platform]\r
             Macros = self._Macros\r
@@ -687,8 +662,8 @@ class InfBuildData(ModuleBuildClassObject):
 \r
     ## Retrieve library classes employed by this module\r
     def _GetLibraryClassUses(self):\r
-        if self._LibraryClasses == None:\r
-            self._LibraryClasses = sdict()\r
+        if self._LibraryClasses is None:\r
+            self._LibraryClasses = OrderedDict()\r
             RecordList = self._RawData[MODEL_EFI_LIBRARY_CLASS, self._Arch, self._Platform]\r
             for Record in RecordList:\r
                 Lib = Record[0]\r
@@ -700,7 +675,7 @@ class InfBuildData(ModuleBuildClassObject):
 \r
     ## Retrieve library names (for Edk.x style of modules)\r
     def _GetLibraryNames(self):\r
-        if self._Libraries == None:\r
+        if self._Libraries is None:\r
             self._Libraries = []\r
             RecordList = self._RawData[MODEL_EFI_LIBRARY_INSTANCE, self._Arch, self._Platform]\r
             for Record in RecordList:\r
@@ -716,15 +691,15 @@ class InfBuildData(ModuleBuildClassObject):
         return self._ProtocolComments\r
     ## Retrieve protocols consumed/produced by this module\r
     def _GetProtocols(self):\r
-        if self._Protocols == None:\r
-            self._Protocols = sdict()\r
-            self._ProtocolComments = sdict()\r
+        if self._Protocols is None:\r
+            self._Protocols = OrderedDict()\r
+            self._ProtocolComments = OrderedDict()\r
             RecordList = self._RawData[MODEL_EFI_PROTOCOL, self._Arch, self._Platform]\r
             for Record in RecordList:\r
                 CName = Record[0]\r
                 Value = ProtocolValue(CName, self.Packages, self.MetaFile.Path)\r
-                if Value == None:\r
-                    PackageList = "\n\t".join([str(P) for P in self.Packages])\r
+                if Value is None:\r
+                    PackageList = "\n\t".join(str(P) for P in self.Packages)\r
                     EdkLogger.error('build', RESOURCE_NOT_AVAILABLE,\r
                                     "Value of Protocol [%s] is not found under [Protocols] section in" % CName,\r
                                     ExtraData=PackageList, File=self.MetaFile, Line=Record[-1])\r
@@ -741,15 +716,15 @@ class InfBuildData(ModuleBuildClassObject):
         return self._PpiComments\r
     ## Retrieve PPIs consumed/produced by this module\r
     def _GetPpis(self):\r
-        if self._Ppis == None:\r
-            self._Ppis = sdict()\r
-            self._PpiComments = sdict()\r
+        if self._Ppis is None:\r
+            self._Ppis = OrderedDict()\r
+            self._PpiComments = OrderedDict()\r
             RecordList = self._RawData[MODEL_EFI_PPI, self._Arch, self._Platform]\r
             for Record in RecordList:\r
                 CName = Record[0]\r
                 Value = PpiValue(CName, self.Packages, self.MetaFile.Path)\r
-                if Value == None:\r
-                    PackageList = "\n\t".join([str(P) for P in self.Packages])\r
+                if Value is None:\r
+                    PackageList = "\n\t".join(str(P) for P in self.Packages)\r
                     EdkLogger.error('build', RESOURCE_NOT_AVAILABLE,\r
                                     "Value of PPI [%s] is not found under [Ppis] section in " % CName,\r
                                     ExtraData=PackageList, File=self.MetaFile, Line=Record[-1])\r
@@ -766,15 +741,15 @@ class InfBuildData(ModuleBuildClassObject):
         return self._GuidComments\r
     ## Retrieve GUIDs consumed/produced by this module\r
     def _GetGuids(self):\r
-        if self._Guids == None:\r
-            self._Guids = sdict()\r
-            self._GuidComments = sdict()\r
+        if self._Guids is None:\r
+            self._Guids = OrderedDict()\r
+            self._GuidComments = OrderedDict()\r
             RecordList = self._RawData[MODEL_EFI_GUID, self._Arch, self._Platform]\r
             for Record in RecordList:\r
                 CName = Record[0]\r
                 Value = GuidValue(CName, self.Packages, self.MetaFile.Path)\r
-                if Value == None:\r
-                    PackageList = "\n\t".join([str(P) for P in self.Packages])\r
+                if Value is None:\r
+                    PackageList = "\n\t".join(str(P) for P in self.Packages)\r
                     EdkLogger.error('build', RESOURCE_NOT_AVAILABLE,\r
                                     "Value of Guid [%s] is not found under [Guids] section in" % CName,\r
                                     ExtraData=PackageList, File=self.MetaFile, Line=Record[-1])\r
@@ -788,16 +763,13 @@ class InfBuildData(ModuleBuildClassObject):
 \r
     ## Retrieve include paths necessary for this module (for Edk.x style of modules)\r
     def _GetIncludes(self):\r
-        if self._Includes == None:\r
+        if self._Includes is None:\r
             self._Includes = []\r
             if self._SourceOverridePath:\r
                 self._Includes.append(self._SourceOverridePath)\r
 \r
             Macros = self._Macros\r
-            if 'PROCESSOR' in GlobalData.gEdkGlobal.keys():\r
-                Macros['PROCESSOR'] = GlobalData.gEdkGlobal['PROCESSOR']\r
-            else:\r
-                Macros['PROCESSOR'] = self._Arch\r
+            Macros['PROCESSOR'] = GlobalData.gEdkGlobal.get('PROCESSOR', self._Arch)\r
             RecordList = self._RawData[MODEL_EFI_INCLUDE, self._Arch, self._Platform]\r
             for Record in RecordList:\r
                 if Record[0].find('EDK_SOURCE') > -1:\r
@@ -845,7 +817,7 @@ class InfBuildData(ModuleBuildClassObject):
 \r
     ## Retrieve packages this module depends on\r
     def _GetPackages(self):\r
-        if self._Packages == None:\r
+        if self._Packages is None:\r
             self._Packages = []\r
             RecordList = self._RawData[MODEL_META_DATA_PACKAGE, self._Arch, self._Platform]\r
             Macros = self._Macros\r
@@ -868,9 +840,9 @@ class InfBuildData(ModuleBuildClassObject):
         return self._PcdComments\r
     ## Retrieve PCDs used in this module\r
     def _GetPcds(self):\r
-        if self._Pcds == None:\r
-            self._Pcds = sdict()\r
-            self._PcdComments = sdict()\r
+        if self._Pcds is None:\r
+            self._Pcds = OrderedDict()\r
+            self._PcdComments = OrderedDict()\r
             self._Pcds.update(self._GetPcd(MODEL_PCD_FIXED_AT_BUILD))\r
             self._Pcds.update(self._GetPcd(MODEL_PCD_PATCHABLE_IN_MODULE))\r
             self._Pcds.update(self._GetPcd(MODEL_PCD_FEATURE_FLAG))\r
@@ -880,8 +852,8 @@ class InfBuildData(ModuleBuildClassObject):
 \r
     ## Retrieve build options specific to this module\r
     def _GetBuildOptions(self):\r
-        if self._BuildOptions == None:\r
-            self._BuildOptions = sdict()\r
+        if self._BuildOptions is None:\r
+            self._BuildOptions = OrderedDict()\r
             RecordList = self._RawData[MODEL_META_DATA_BUILD_OPTION, self._Arch, self._Platform]\r
             for Record in RecordList:\r
                 ToolChainFamily = Record[0]\r
@@ -897,30 +869,30 @@ class InfBuildData(ModuleBuildClassObject):
 \r
     ## Retrieve dependency expression\r
     def _GetDepex(self):\r
-        if self._Depex == None:\r
+        if self._Depex is None:\r
             self._Depex = tdict(False, 2)\r
             RecordList = self._RawData[MODEL_EFI_DEPEX, self._Arch]\r
 \r
             # If the module has only Binaries and no Sources, then ignore [Depex]\r
-            if self.Sources == None or self.Sources == []:\r
-                if self.Binaries != None and self.Binaries != []:\r
+            if self.Sources is None or self.Sources == []:\r
+                if self.Binaries is not None and self.Binaries != []:\r
                     return self._Depex\r
 \r
             # PEIM and DXE drivers must have a valid [Depex] section\r
             if len(self.LibraryClass) == 0 and len(RecordList) == 0:\r
-                if self.ModuleType == 'DXE_DRIVER' or self.ModuleType == 'PEIM' or self.ModuleType == 'DXE_SMM_DRIVER' or \\r
-                    self.ModuleType == 'DXE_SAL_DRIVER' or self.ModuleType == 'DXE_RUNTIME_DRIVER':\r
+                if self.ModuleType == SUP_MODULE_DXE_DRIVER or self.ModuleType == SUP_MODULE_PEIM or self.ModuleType == SUP_MODULE_DXE_SMM_DRIVER or \\r
+                    self.ModuleType == SUP_MODULE_DXE_SAL_DRIVER or self.ModuleType == SUP_MODULE_DXE_RUNTIME_DRIVER:\r
                     EdkLogger.error('build', RESOURCE_NOT_AVAILABLE, "No [Depex] section or no valid expression in [Depex] section for [%s] module" \\r
                                     % self.ModuleType, File=self.MetaFile)\r
 \r
-            if len(RecordList) != 0 and self.ModuleType == 'USER_DEFINED':\r
+            if len(RecordList) != 0 and self.ModuleType == SUP_MODULE_USER_DEFINED:\r
                 for Record in RecordList:\r
-                    if Record[4] not in ['PEIM', 'DXE_DRIVER', 'DXE_SMM_DRIVER']:\r
+                    if Record[4] not in [SUP_MODULE_PEIM, SUP_MODULE_DXE_DRIVER, SUP_MODULE_DXE_SMM_DRIVER]:\r
                         EdkLogger.error('build', FORMAT_INVALID,\r
                                         "'%s' module must specify the type of [Depex] section" % self.ModuleType,\r
                                         File=self.MetaFile)\r
 \r
-            Depex = sdict()\r
+            Depex = OrderedDict()\r
             for Record in RecordList:\r
                 DepexStr = ReplaceMacro(Record[0], self._Macros, False)\r
                 Arch = Record[3]\r
@@ -930,24 +902,24 @@ class InfBuildData(ModuleBuildClassObject):
                     Depex[Arch, ModuleType] = []\r
                 DepexList = Depex[Arch, ModuleType]\r
                 for Token in TokenList:\r
-                    if Token in DEPEX_SUPPORTED_OPCODE:\r
+                    if Token in DEPEX_SUPPORTED_OPCODE_SET:\r
                         DepexList.append(Token)\r
                     elif Token.endswith(".inf"):  # module file name\r
                         ModuleFile = os.path.normpath(Token)\r
                         Module = self.BuildDatabase[ModuleFile]\r
-                        if Module == None:\r
+                        if Module is None:\r
                             EdkLogger.error('build', RESOURCE_NOT_AVAILABLE, "Module is not found in active platform",\r
                                             ExtraData=Token, File=self.MetaFile, Line=Record[-1])\r
                         DepexList.append(Module.Guid)\r
                     else:\r
                         # get the GUID value now\r
                         Value = ProtocolValue(Token, self.Packages, self.MetaFile.Path)\r
-                        if Value == None:\r
+                        if Value is None:\r
                             Value = PpiValue(Token, self.Packages, self.MetaFile.Path)\r
-                            if Value == None:\r
+                            if Value is None:\r
                                 Value = GuidValue(Token, self.Packages, self.MetaFile.Path)\r
-                        if Value == None:\r
-                            PackageList = "\n\t".join([str(P) for P in self.Packages])\r
+                        if Value is None:\r
+                            PackageList = "\n\t".join(str(P) for P in self.Packages)\r
                             EdkLogger.error('build', RESOURCE_NOT_AVAILABLE,\r
                                             "Value of [%s] is not found in" % Token,\r
                                             ExtraData=PackageList, File=self.MetaFile, Line=Record[-1])\r
@@ -958,10 +930,10 @@ class InfBuildData(ModuleBuildClassObject):
 \r
     ## Retrieve depedency expression\r
     def _GetDepexExpression(self):\r
-        if self._DepexExpression == None:\r
+        if self._DepexExpression is None:\r
             self._DepexExpression = tdict(False, 2)\r
             RecordList = self._RawData[MODEL_EFI_DEPEX, self._Arch]\r
-            DepexExpression = sdict()\r
+            DepexExpression = OrderedDict()\r
             for Record in RecordList:\r
                 DepexStr = ReplaceMacro(Record[0], self._Macros, False)\r
                 Arch = Record[3]\r
@@ -979,7 +951,7 @@ class InfBuildData(ModuleBuildClassObject):
         return self._GuidsUsedByPcd\r
     ## Retrieve PCD for given type\r
     def _GetPcd(self, Type):\r
-        Pcds = sdict()\r
+        Pcds = OrderedDict()\r
         PcdDict = tdict(True, 4)\r
         PcdList = []\r
         RecordList = self._RawData[Type, self._Arch, self._Platform]\r
@@ -989,8 +961,8 @@ class InfBuildData(ModuleBuildClassObject):
             # get the guid value\r
             if TokenSpaceGuid not in self.Guids:\r
                 Value = GuidValue(TokenSpaceGuid, self.Packages, self.MetaFile.Path)\r
-                if Value == None:\r
-                    PackageList = "\n\t".join([str(P) for P in self.Packages])\r
+                if Value is None:\r
+                    PackageList = "\n\t".join(str(P) for P in self.Packages)\r
                     EdkLogger.error('build', RESOURCE_NOT_AVAILABLE,\r
                                     "Value of Guid [%s] is not found under [Guids] section in" % TokenSpaceGuid,\r
                                     ExtraData=PackageList, File=self.MetaFile, Line=LineNo)\r
@@ -1003,10 +975,11 @@ class InfBuildData(ModuleBuildClassObject):
             self._PcdComments[TokenSpaceGuid, PcdCName] = Comments\r
 \r
         # resolve PCD type, value, datum info, etc. by getting its definition from package\r
+        _GuidDict = self.Guids.copy()\r
         for PcdCName, TokenSpaceGuid in PcdList:\r
             PcdRealName = PcdCName\r
             Setting, LineNo = PcdDict[self._Arch, self.Platform, PcdCName, TokenSpaceGuid]\r
-            if Setting == None:\r
+            if Setting is None:\r
                 continue\r
             ValueList = AnalyzePcdData(Setting)\r
             DefaultValue = ValueList[0]\r
@@ -1054,12 +1027,13 @@ class InfBuildData(ModuleBuildClassObject):
                 # if platform doesn't give its type, use 'lowest' one in the\r
                 # following order, if any\r
                 #\r
-                #   "FixedAtBuild", "PatchableInModule", "FeatureFlag", "Dynamic", "DynamicEx"\r
+                #   TAB_PCDS_FIXED_AT_BUILD, TAB_PCDS_PATCHABLE_IN_MODULE, TAB_PCDS_FEATURE_FLAG, TAB_PCDS_DYNAMIC, TAB_PCDS_DYNAMIC_EX\r
                 #\r
+                _GuidDict.update(Package.Guids)\r
                 PcdType = self._PCD_TYPE_STRING_[Type]\r
                 if Type == MODEL_PCD_DYNAMIC:\r
                     Pcd.Pending = True\r
-                    for T in ["FixedAtBuild", "PatchableInModule", "FeatureFlag", "Dynamic", "DynamicEx"]:\r
+                    for T in PCD_TYPE_LIST:\r
                         if (PcdRealName, TokenSpaceGuid) in GlobalData.MixedPcd:\r
                             for item in GlobalData.MixedPcd[(PcdRealName, TokenSpaceGuid)]:\r
                                 if str(item[0]).endswith(T) and (item[0], item[1], T) in Package.Pcds:\r
@@ -1095,7 +1069,7 @@ class InfBuildData(ModuleBuildClassObject):
                     #\r
                     # Check whether the token value exist or not.\r
                     #\r
-                    if Pcd.TokenValue == None or Pcd.TokenValue == "":\r
+                    if Pcd.TokenValue is None or Pcd.TokenValue == "":\r
                         EdkLogger.error(\r
                                 'build',\r
                                 FORMAT_INVALID,\r
@@ -1108,7 +1082,7 @@ class InfBuildData(ModuleBuildClassObject):
                     #\r
                     ReIsValidPcdTokenValue = re.compile(r"^[0][x|X][0]*[0-9a-fA-F]{1,8}$", re.DOTALL)\r
                     if Pcd.TokenValue.startswith("0x") or Pcd.TokenValue.startswith("0X"):\r
-                        if ReIsValidPcdTokenValue.match(Pcd.TokenValue) == None:\r
+                        if ReIsValidPcdTokenValue.match(Pcd.TokenValue) is None:\r
                             EdkLogger.error(\r
                                     'build',\r
                                     FORMAT_INVALID,\r
@@ -1143,12 +1117,12 @@ class InfBuildData(ModuleBuildClassObject):
                     Pcd.DatumType = PcdInPackage.DatumType\r
                     Pcd.MaxDatumSize = PcdInPackage.MaxDatumSize\r
                     Pcd.InfDefaultValue = Pcd.DefaultValue\r
-                    if Pcd.DefaultValue in [None, '']:\r
+                    if not Pcd.DefaultValue:\r
                         Pcd.DefaultValue = PcdInPackage.DefaultValue\r
                     else:\r
                         try:\r
-                            Pcd.DefaultValue = ValueExpressionEx(Pcd.DefaultValue, Pcd.DatumType, self.Guids)(True)\r
-                        except BadExpression, Value:\r
+                            Pcd.DefaultValue = ValueExpressionEx(Pcd.DefaultValue, Pcd.DatumType, _GuidDict)(True)\r
+                        except BadExpression as Value:\r
                             EdkLogger.error('Parser', FORMAT_INVALID, 'PCD [%s.%s] Value "%s", %s' %(TokenSpaceGuid, PcdRealName, Pcd.DefaultValue, Value),\r
                                             File=self.MetaFile, Line=LineNo)\r
                     break\r
@@ -1158,7 +1132,7 @@ class InfBuildData(ModuleBuildClassObject):
                             FORMAT_INVALID,\r
                             "PCD [%s.%s] in [%s] is not found in dependent packages:" % (TokenSpaceGuid, PcdRealName, self.MetaFile),\r
                             File=self.MetaFile, Line=LineNo,\r
-                            ExtraData="\t%s" % '\n\t'.join([str(P) for P in self.Packages])\r
+                            ExtraData="\t%s" % '\n\t'.join(str(P) for P in self.Packages)\r
                             )\r
             Pcds[PcdCName, TokenSpaceGuid] = Pcd\r
 \r