public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [edk2-BuildSpecification PATCH] Update Precedence of PCD Values
@ 2017-05-27  3:43 Michael Kinney
  2017-05-27  3:43 ` Michael Kinney
  0 siblings, 1 reply; 3+ messages in thread
From: Michael Kinney @ 2017-05-27  3:43 UTC (permalink / raw)
  To: edk2-devel; +Cc: Liming Gao, Yonghong Zhu, Kevin W Shaw

https://bugzilla.tianocore.org/show_bug.cgi?id=518

GitHub branch for review:

* https://github.com/mdkinney/edk2-BuildSpecification/tree/Bugzilla_518_UpdatePrecedenceOfPcdValues

GitHub word diff view of the patches in this series:

* [1/1] https://github.com/mdkinney/edk2-BuildSpecification/commit/db9050f6eea825a6c307a037638ff1fc010c0e69?w=1

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>

Michael Kinney (1):
  Update Precedence of PCD Values

 8_pre-build_autogen_stage/82_auto-generation_process.md | 7 +++++--
 README.md                                               | 3 ++-
 2 files changed, 7 insertions(+), 3 deletions(-)

-- 
2.6.3.windows.1



^ permalink raw reply	[flat|nested] 3+ messages in thread

* [edk2-BuildSpecification PATCH] Update Precedence of PCD Values
  2017-05-27  3:43 [edk2-BuildSpecification PATCH] Update Precedence of PCD Values Michael Kinney
@ 2017-05-27  3:43 ` Michael Kinney
  2017-05-31  2:48   ` Zhu, Yonghong
  0 siblings, 1 reply; 3+ messages in thread
From: Michael Kinney @ 2017-05-27  3:43 UTC (permalink / raw)
  To: edk2-devel; +Cc: Liming Gao, Yonghong Zhu, Kevin W Shaw

https://bugzilla.tianocore.org/show_bug.cgi?id=518

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>
---
 8_pre-build_autogen_stage/82_auto-generation_process.md | 7 +++++--
 README.md                                               | 3 ++-
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/8_pre-build_autogen_stage/82_auto-generation_process.md b/8_pre-build_autogen_stage/82_auto-generation_process.md
index 5c2022b..7bca366 100644
--- a/8_pre-build_autogen_stage/82_auto-generation_process.md
+++ b/8_pre-build_autogen_stage/82_auto-generation_process.md
@@ -856,10 +856,10 @@ from different locations and different meta-data files. The following provides
 the precedence (high to low) to assign a value to a PCD.
 
 * Command-line, `--pcd` flags (left most has higher priority)
-* FDF file, SET statements within a section
+* DSC file, Component INF `<Pcd*>` section statements
 * FDF file, grammar describing automatic assignment of PCD values
+* FDF file, SET statements within a section
 * FDF file, SET statement in the [Defines] section
-* DSC file, Component INF `<Pcd*>` section statements
 * DSC file, global [Pcd*] sections
 * INF file, PCD sections, Default Values
 * DEC file, PCD sections, Default Values
@@ -1083,6 +1083,9 @@ II module:
 
   - Command-line, `--pcd` flags (left most has higher priority)
   - The DSC file's component INF scoping `<Pcds*>` sections
+  - FDF file, grammar describing automatic assignment of PCD values
+  - FDF file, SET statements within a section
+  - FDF file, SET statement in the [Defines] section
   - The DSC file's `[Pcd*.arch.skuid]` sections
   - The DSC file's `[Pcd*.common.skuid]` sections
   - The DSC file's `[Pcd*.arch]` sections
diff --git a/README.md b/README.md
index 16fcc4a..43022c7 100644
--- a/README.md
+++ b/README.md
@@ -200,7 +200,7 @@ Copyright (c) 2008-2017, Intel Corporation. All rights reserved.
 |            | Remove Unicode file storage requirement; refer to the Multi-String UNI File Format Specification instead.                                                                                                                                                                                                                                                           |               |
 |            | Clarify BUILDRULEORDER                                                                                                                                                                                                                                                                                                                                              |               |
 |            | Add support for INF statement in an FD region.                                                                                                                                                                                                                                                                                                                      |               |
-| 1.27       | Convert to Gitbook                                                                                                                                                                                                                                                                                                                                                  | April 2017    |
+| 1.27       | Convert to Gitbook                                                                                                                                                                                                                                                                                                                                                  | May 2017      |
 |            | [#471](https://bugzilla.tianocore.org/show_bug.cgi?id=471) Build spec: only copy the "TianoCore" Userextension section into "As Built" INF                                                                                                                                                                                                                          |               |
 |            | [#472](https://bugzilla.tianocore.org/show_bug.cgi?id=472) [Build Spec] Extend macro usage in the !include statements for DSC/FDF files                                                                                                                                                                                                                             |               |
 |            | [#483](https://bugzilla.tianocore.org/show_bug.cgi?id=483) Build Spec: add description for merge Pre-build and Post-build into build process                                                                                                                                                                                                                        |               |
@@ -212,3 +212,4 @@ Copyright (c) 2008-2017, Intel Corporation. All rights reserved.
 |            | [#504](https://bugzilla.tianocore.org/show_bug.cgi?id=504) Update build report to support -Y HASH option                                                                                                                                                                                                                                                            |               |
 |            | [#508](https://bugzilla.tianocore.org/show_bug.cgi?id=508) Build Spec: Add sections to PCD report for PCDs only used in conditions PCDs that are not used anywhere                                                                                                                                                                                                  |               |
 |            | [#480](https://bugzilla.tianocore.org/show_bug.cgi?id=480) Build spec: add description for Override PCD value on the command line                                                                                                                                                                                                                                   |               |
+|            | [#518](https://bugzilla.tianocore.org/show_bug.cgi?id=518) Build Spec: Update Precedence of PCD Values                                                                                                                                                                                                                                                              |               |
-- 
2.6.3.windows.1



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [edk2-BuildSpecification PATCH] Update Precedence of PCD Values
  2017-05-27  3:43 ` Michael Kinney
@ 2017-05-31  2:48   ` Zhu, Yonghong
  0 siblings, 0 replies; 3+ messages in thread
From: Zhu, Yonghong @ 2017-05-31  2:48 UTC (permalink / raw)
  To: Kinney, Michael D, edk2-devel@lists.01.org
  Cc: Gao, Liming, Shaw, Kevin W, Zhu, Yonghong

Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> 

Best Regards,
Zhu Yonghong


-----Original Message-----
From: Kinney, Michael D 
Sent: Saturday, May 27, 2017 11:44 AM
To: edk2-devel@lists.01.org
Cc: Gao, Liming <liming.gao@intel.com>; Zhu, Yonghong <yonghong.zhu@intel.com>; Shaw, Kevin W <kevin.w.shaw@intel.com>
Subject: [edk2-BuildSpecification PATCH] Update Precedence of PCD Values

https://bugzilla.tianocore.org/show_bug.cgi?id=518

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>
---
 8_pre-build_autogen_stage/82_auto-generation_process.md | 7 +++++--
 README.md                                               | 3 ++-
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/8_pre-build_autogen_stage/82_auto-generation_process.md b/8_pre-build_autogen_stage/82_auto-generation_process.md
index 5c2022b..7bca366 100644
--- a/8_pre-build_autogen_stage/82_auto-generation_process.md
+++ b/8_pre-build_autogen_stage/82_auto-generation_process.md
@@ -856,10 +856,10 @@ from different locations and different meta-data files. The following provides  the precedence (high to low) to assign a value to a PCD.
 
 * Command-line, `--pcd` flags (left most has higher priority)
-* FDF file, SET statements within a section
+* DSC file, Component INF `<Pcd*>` section statements
 * FDF file, grammar describing automatic assignment of PCD values
+* FDF file, SET statements within a section
 * FDF file, SET statement in the [Defines] section
-* DSC file, Component INF `<Pcd*>` section statements
 * DSC file, global [Pcd*] sections
 * INF file, PCD sections, Default Values
 * DEC file, PCD sections, Default Values @@ -1083,6 +1083,9 @@ II module:
 
   - Command-line, `--pcd` flags (left most has higher priority)
   - The DSC file's component INF scoping `<Pcds*>` sections
+  - FDF file, grammar describing automatic assignment of PCD values
+  - FDF file, SET statements within a section
+  - FDF file, SET statement in the [Defines] section
   - The DSC file's `[Pcd*.arch.skuid]` sections
   - The DSC file's `[Pcd*.common.skuid]` sections
   - The DSC file's `[Pcd*.arch]` sections diff --git a/README.md b/README.md index 16fcc4a..43022c7 100644
--- a/README.md
+++ b/README.md
@@ -200,7 +200,7 @@ Copyright (c) 2008-2017, Intel Corporation. All rights reserved.
 |            | Remove Unicode file storage requirement; refer to the Multi-String UNI File Format Specification instead.                                                                                                                                                                                                                                                           |               |
 |            | Clarify BUILDRULEORDER                                                                                                                                                                                                                                                                                                                                              |               |
 |            | Add support for INF statement in an FD region.                                                                                                                                                                                                                                                                                                                      |               |
-| 1.27       | Convert to Gitbook                                                                                                                                                                                                                                                                                                                                                  | April 2017    |
+| 1.27       | Convert to Gitbook                                                                                                                                                                                                                                                                                                                                                  | May 2017      |
 |            | [#471](https://bugzilla.tianocore.org/show_bug.cgi?id=471) Build spec: only copy the "TianoCore" Userextension section into "As Built" INF                                                                                                                                                                                                                          |               |
 |            | [#472](https://bugzilla.tianocore.org/show_bug.cgi?id=472) [Build Spec] Extend macro usage in the !include statements for DSC/FDF files                                                                                                                                                                                                                             |               |
 |            | [#483](https://bugzilla.tianocore.org/show_bug.cgi?id=483) Build Spec: add description for merge Pre-build and Post-build into build process                                                                                                                                                                                                                        |               |
@@ -212,3 +212,4 @@ Copyright (c) 2008-2017, Intel Corporation. All rights reserved.
 |            | [#504](https://bugzilla.tianocore.org/show_bug.cgi?id=504) Update build report to support -Y HASH option                                                                                                                                                                                                                                                            |               |
 |            | [#508](https://bugzilla.tianocore.org/show_bug.cgi?id=508) Build Spec: Add sections to PCD report for PCDs only used in conditions PCDs that are not used anywhere                                                                                                                                                                                                  |               |
 |            | [#480](https://bugzilla.tianocore.org/show_bug.cgi?id=480) Build spec: add description for Override PCD value on the command line                                                                                                                                                                                                                                   |               |
+|            | [#518](https://bugzilla.tianocore.org/show_bug.cgi?id=518) Build Spec: Update Precedence of PCD Values                                                                                                                                                                                                                                                              |               |
--
2.6.3.windows.1



^ permalink raw reply related	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-05-31  2:47 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-05-27  3:43 [edk2-BuildSpecification PATCH] Update Precedence of PCD Values Michael Kinney
2017-05-27  3:43 ` Michael Kinney
2017-05-31  2:48   ` Zhu, Yonghong

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox