[edk2-devel] [Patch] BaseTools: Fix the issue that build report missing Module Size

Liming Gao liming.gao at intel.com
Thu May 9 02:33:21 UTC 2019


Reviewed-by: Liming Gao <liming.gao at intel.com>

> -----Original Message-----
> From: Feng, Bob C
> Sent: Wednesday, May 8, 2019 5:47 PM
> To: devel at edk2.groups.io
> Cc: Feng, Bob C <bob.c.feng at intel.com>; Gao, Liming <liming.gao at intel.com>
> Subject: [Patch] BaseTools: Fix the issue that build report missing Module Size
> 
> BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1786
> 
> commit b1e27d changed GENFW output folder from DEBUG to OUTPUT,
> but that commit missed updating the buildreport.py for FwReportFileName.
> 
> This patch is going to fix this issue.
> 
> Signed-off-by: Bob Feng <bob.c.feng at intel.com>
> Cc: Liming Gao <liming.gao at intel.com>
> ---
>  BaseTools/Source/Python/build/BuildReport.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/BaseTools/Source/Python/build/BuildReport.py b/BaseTools/Source/Python/build/BuildReport.py
> index 942631c879..a3eb3b2383 100644
> --- a/BaseTools/Source/Python/build/BuildReport.py
> +++ b/BaseTools/Source/Python/build/BuildReport.py
> @@ -618,11 +618,11 @@ class ModuleReport(object):
>      # @param ReportType             The kind of report items in the final report file
>      #
>      def GenerateReport(self, File, GlobalPcdReport, GlobalPredictionReport, GlobalDepexParser, ReportType):
>          FileWrite(File, gSectionStart)
> 
> -        FwReportFileName = os.path.join(self._BuildDir, "DEBUG", self.ModuleName + ".txt")
> +        FwReportFileName = os.path.join(self._BuildDir, "OUTPUT", self.ModuleName + ".txt")
>          if os.path.isfile(FwReportFileName):
>              try:
>                  FileContents = open(FwReportFileName).read()
>                  Match = gModuleSizePattern.search(FileContents)
>                  if Match:
> --
> 2.20.1.windows.1


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#40223): https://edk2.groups.io/g/devel/message/40223
Mute This Topic: https://groups.io/mt/31541596/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-





More information about the edk2-devel-archive mailing list