From: Michael Kinney <michael.d.kinney@intel.com>
To: edk2-devel@lists.01.org
Cc: Liming Gao <liming.gao@intel.com>,
Yonghong Zhu <yonghong.zhu@intel.com>,
Kevin W Shaw <kevin.w.shaw@intel.com>
Subject: [ edk2-BuildSpecification PATCH] Update build report to support -Y HASH option
Date: Mon, 24 Apr 2017 18:57:35 -0700 [thread overview]
Message-ID: <1493085455-10268-2-git-send-email-michael.d.kinney@intel.com> (raw)
In-Reply-To: <1493085455-10268-1-git-send-email-michael.d.kinney@intel.com>
https://bugzilla.tianocore.org/show_bug.cgi?id=504
Cc: Liming Gao <liming.gao@intel.com>
Cc: Yonghong Zhu <yonghong.zhu@intel.com>
Cc: Kevin W Shaw <kevin.w.shaw@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Michael Kinney <michael.d.kinney@intel.com>
---
13_build_reports/131_build_report_generation_options.md | 4 ++++
13_build_reports/132_sample_launch_steps_nt32_platform.md | 11 ++++++-----
13_build_reports/133_output.md | 2 +-
13_build_reports/138_module_section.md | 4 ++++
README.md | 1 +
appendix_d_buildexe_command/d4_usage.md | 7 ++++---
6 files changed, 20 insertions(+), 9 deletions(-)
diff --git a/13_build_reports/131_build_report_generation_options.md b/13_build_reports/131_build_report_generation_options.md
index fc32590..05d7eb2 100644
--- a/13_build_reports/131_build_report_generation_options.md
+++ b/13_build_reports/131_build_report_generation_options.md
@@ -61,6 +61,10 @@ notification invoking sequence; Also the predicted addresses of module image
loading, entry point and notification functions. Generating this report does
take a significant amount of time, more than 2x the standard build time.
+**_Module Information_**
+
+Details of the module, may include the HASH of the `.efi` file.
+
**********
**Note:** The execution order prediction report output is an html file,
separate from the rest of the reports. All remaining reports are generated in a
diff --git a/13_build_reports/132_sample_launch_steps_nt32_platform.md b/13_build_reports/132_sample_launch_steps_nt32_platform.md
index 33ca1bf..099dc07 100644
--- a/13_build_reports/132_sample_launch_steps_nt32_platform.md
+++ b/13_build_reports/132_sample_launch_steps_nt32_platform.md
@@ -42,17 +42,18 @@ command. The following steps output the build report for NT32 platform:
5. Run **build.exe -a IA32 -p Nt32Pkg\Nt32Pkg.dsc -y ReportFile.txt**
* **-y**: This option specifies the output file name for build report.
* **-Y**: This option specifies flags that control the type of build report.
- It must be from the set of **PCD**, **LIBRARY**, **DEPEX**, **BUILD_FLAGS**,
- **FLASH**, **FIXED_ADDRESS** and **EXECUTION_ORDER**. To specify more than
- one flag, repeat the option on the command line. Example of usage:
+ It must be from the set of **PCD**, **LIBRARY**, **DEPEX**, **HASH**,
+ **BUILD_FLAGS**, **FLASH**, **FIXED_ADDRESS** and **EXECUTION_ORDER**. To
+ specify more than one flag, repeat the option on the command line. Example
+ of usage:
On the command line, append the following arguments:
**-y report_filename.txt -Y PCD -Y FLASH -Y DEPEX**
The default set of flags (if **-Y** is not specified) is: **PCD**,
- **LIBRARY**, **FLASH**, **DEPEX**, **BUILD_FLAGS** and **FIXED_ADDRESS**.
-
+ **LIBRARY**, **FLASH**, **DEPEX**, **HASH**, **BUILD_FLAGS** and
+ **FIXED_ADDRESS**.
[^1] On Microsoft Windows 7, you must be an administrator to create a directory
in the root of the C: drive. It recommended that you checkout edk2 into your
diff --git a/13_build_reports/133_output.md b/13_build_reports/133_output.md
index ce64efd..73afb0c 100644
--- a/13_build_reports/133_output.md
+++ b/13_build_reports/133_output.md
@@ -101,7 +101,7 @@ Target: DEBUG
Output Path: s:\edk2\Build\NT32IA32
Build Environment: Windows-7-6.1.7601-SP1
Build Duration: 00:01:53
-Report Contents: PCD, LIBRARY, BUILD_FLAGS, DEPEX, FLASH, FIXED_ADDRESS
+Report Contents: PCD, LIBRARY, BUILD_FLAGS, DEPEX, HASH, FLASH, FIXED_ADDRESS
>==========================================================================<
Firmware Device (FD)
FD Name: NT32
diff --git a/13_build_reports/138_module_section.md b/13_build_reports/138_module_section.md
index 23e039b..d4aa365 100644
--- a/13_build_reports/138_module_section.md
+++ b/13_build_reports/138_module_section.md
@@ -51,6 +51,8 @@ file GUID, module size, module build time stamp and driver type.
The following entries are options:
+* If using defaults or the `HASH` flag is specified:
+ - SHA1 HASH: %SHA1 HASH% and *%Module .efi file name%
* UEFI Specification Version: %The UEFI specification
version:'`UEFI_SPECIFICATION_VERSION`' in INF `[Defines]` section%
* PI Specification Version: %The PI specification
@@ -75,6 +77,7 @@ Module Name: SmbiosDxe
Module INF Path: MdeModule\Universal\SmbiosDxe\SmbiosDxe.inf
File GUID: F9D88642-0737-49BC-81B5-6889CD57D9EA
Size: 0x7000 (28.00K)
+SHA1 HASH: d94c3f180f25d6b562f477bc4a16b286cb66a8b6 *SmbiosDxe.efi
Build Time Stamp: 1969-12-31 16:00:00
Driver Type: 0x7 (DRIVER)
============================================================================
@@ -91,6 +94,7 @@ Module Name: EbcDxe
Module INF Path: MdeModule\Universal\EbcDxe\EbcDxe.inf
File GUID: 13AC6DD0-73D0-11D4-B06B-00AA00BD6DE7
Size: 0x9000 (36.00K)
+SHA1 HASH: ff4c019345614afe5c88e7fc37219c30a07f4af4 *EbcDxe.efi
Time Stamp: 1969-12-31 16:00:00
Driver Type: 0x7 (DRIVER)
============================================================================
diff --git a/README.md b/README.md
index 89578d0..b6668cc 100644
--- a/README.md
+++ b/README.md
@@ -208,3 +208,4 @@ Copyright (c) 2008-2017, Intel Corporation. All rights reserved.
| | [#479](https://bugzilla.tianocore.org/show_bug.cgi?id=479) Build spec: add description for nmake long command line handling | |
| | [#485](https://bugzilla.tianocore.org/show_bug.cgi?id=485) Build spec: add support for mixed Pcd access method | |
| | [#476](https://bugzilla.tianocore.org/show_bug.cgi?id=476) Build spec: add support for HII image package | |
+| | [#504](https://bugzilla.tianocore.org/show_bug.cgi?id=504) Update build report to support -Y HASH option | |
diff --git a/appendix_d_buildexe_command/d4_usage.md b/appendix_d_buildexe_command/d4_usage.md
index c262f1a..07b9092 100644
--- a/appendix_d_buildexe_command/d4_usage.md
+++ b/appendix_d_buildexe_command/d4_usage.md
@@ -96,11 +96,12 @@ Options:
Create/overwrite the report to the specified filename.
-Y REPORTTYPE, --report-type=REPORTTYPE
Flags that control the type of build report to
- generate. Must be one of: [PCD, LIBRARY, FLASH,
- DEPEX, BUILD_FLAGS, FIXED_ADDRESS, EXECUTION_ORDER].
+ generate. Must be one of: [PCD, LIBRARY, FLASH, DEPEX,
+ HASH, BUILD_FLAGS, FIXED_ADDRESS, EXECUTION_ORDER].
To specify more than one flag, repeat this option on
the command line and the default flag set is [PCD,
- LIBRARY, FLASH, DEPEX, BUILD_FLAGS, FIXED_ADDRESS]
+ LIBRARY, FLASH, DEPEX, HASH, BUILD_FLAGS,
+ FIXED_ADDRESS]
-F FLAG, --flag=FLAG Specify the specific option to parse EDK UNI file.
Must be one of: [-c, -s]. -c is for EDK framework UNI
file, and -s is for EDK UEFI UNI file. This option can
--
2.6.3.windows.1
next prev parent reply other threads:[~2017-04-25 1:57 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-25 1:57 [ edk2-BuildSpecification PATCH] Update build report to support -Y HASH option Michael Kinney
2017-04-25 1:57 ` Michael Kinney [this message]
2017-04-25 2:29 ` Zhu, Yonghong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1493085455-10268-2-git-send-email-michael.d.kinney@intel.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox