]> git.proxmox.com Git - mirror_edk2.git/commitdiff
BaseTools:fix regression issue for platform .map file
authorFan, ZhijuX <zhijux.fan@intel.com>
Fri, 22 Nov 2019 10:54:39 +0000 (18:54 +0800)
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Mon, 25 Nov 2019 06:01:10 +0000 (06:01 +0000)
BZ:https://bugzilla.tianocore.org/show_bug.cgi?id=2363

This patch is to fix a build tool regression issue which was introduced
by commit b8ac0b7f28.This issue caused map file lost the line of IMAGE=***.
For example,in Ovmf.map, there is no line of (IMAGE=<path to efi> ) under
each of modules item.

The path to the efi file generated by each module is written on this line
The purpose of this line is add the debug image full path.
there is no information about the module in the map file other than FVName,
it allows us to quickly know which module this part corresponds to.

In commit b8ac0b7f28,add a line ("self.BuildModules = []") in function,
but it's used to calculate the variable ModuleList in the following code.

Signed-off-by: Zhiju.Fan <zhijux.fan@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Bob Feng <bob.c.feng@intel.com>
Reviewed-by: Bob Feng <bob.c.feng@intel.com>
BaseTools/Source/Python/build/build.py

index 6c8798ef93fbcf3059f4e1d0b99dd17848eee01f..4b31356a4253f5ff4abb875417ffbc06a1517e41 100755 (executable)
@@ -2267,6 +2267,10 @@ class Build():
                 self.CreateAsBuiltInf()\r
                 if GlobalData.gBinCacheDest:\r
                     self.UpdateBuildCache()\r
+                #\r
+                # Get Module List\r
+                #\r
+                ModuleList = {ma.Guid.upper(): ma for ma in self.BuildModules}\r
                 self.BuildModules = []\r
                 self.MakeTime += int(round((time.time() - MakeContiue)))\r
                 #\r
@@ -2285,10 +2289,6 @@ class Build():
                         #\r
                         if (Arch == 'IA32' or Arch == 'ARM') and self.LoadFixAddress != 0xFFFFFFFFFFFFFFFF and self.LoadFixAddress >= 0x100000000:\r
                             EdkLogger.error("build", PARAMETER_INVALID, "FIX_LOAD_TOP_MEMORY_ADDRESS can't be set to larger than or equal to 4G for the platorm with IA32 or ARM arch modules")\r
-                    #\r
-                    # Get Module List\r
-                    #\r
-                    ModuleList = {ma.Guid.upper():ma for ma in self.BuildModules}\r
 \r
                     #\r
                     # Rebase module to the preferred memory address before GenFds\r