From: "Zhu, Yonghong" <yonghong.zhu@intel.com>
To: "Kinney, Michael D" <michael.d.kinney@intel.com>,
"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Cc: "Gao, Liming" <liming.gao@intel.com>,
"Shaw, Kevin W" <kevin.w.shaw@intel.com>,
"Zhu, Yonghong" <yonghong.zhu@intel.com>
Subject: Re: [ edk2-DecSpecification PATCH 0/4] Add support for Private declarations in a package
Date: Tue, 25 Apr 2017 01:40:54 +0000 [thread overview]
Message-ID: <B9726D6DCCFB8B4CA276A9169B02216D51E64130@SHSMSX103.ccr.corp.intel.com> (raw)
In-Reply-To: <1493082606-9348-1-git-send-email-michael.d.kinney@intel.com>
Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com>
Best Regards,
Zhu Yonghong
-----Original Message-----
From: Kinney, Michael D
Sent: Tuesday, April 25, 2017 9:10 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-DecSpecification PATCH 0/4] Add support for Private declarations in a package
https://bugzilla.tianocore.org/show_bug.cgi?id=465
https://bugzilla.tianocore.org/show_bug.cgi?id=482
GitHub word diff view of the patches in this series
* [1/4] https://github.com/mdkinney/edk2-DecSpecification/commit/24a07ae89ef0467c9009633d19d24bb1ba2e115a?w=1
* [2/4] https://github.com/mdkinney/edk2-DecSpecification/commit/65f49d1d86439b6144ff82217349a5145ec42ef0?w=1
* [3/4] https://github.com/mdkinney/edk2-DecSpecification/commit/a28fd82b07521fc7412be75b159a44a0dd3bf1b1?w=1
* [4/4] https://github.com/mdkinney/edk2-DecSpecification/commit/c04e668af9e1aa8c8e49d73fede6df7287b7d454?w=1
Remove trailing spaces from README.md
Update DEC_SPECIFCATION to 0x0001001A / 1.26
Add new syntax to the DEC file for specifying information that can only be used by modules within the package. When modules outside the packages attempt to use this content, the EDK II build system must break with an error regarding content not found.
The four sections, Includes, Ppis, Guids and Protocols headers will be modified with a keyword, Private following the architecture modifier. If Private is not present, then the content is usable by modules outside the package.
Clarify restriction that an element is not allowed to be declared with and without Private modifier.
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 (4):
Remove trailing spaces from README.MD
Update version from 1.25 to 1.26
Add support for Private declarations in a package
Declarations not allowed to be both public and private
2_dec_file_overview/24_[defines]_usage.md | 2 +-
2_dec_file_overview/25_[includes]_usage.md | 24 ++-
2_dec_file_overview/26_[guids]_usage.md | 22 ++-
2_dec_file_overview/27_[protocols]_usage.md | 24 ++-
2_dec_file_overview/28_[ppis]_usage.md | 24 ++-
3_edk_ii_dec_file_format/34_[defines]_section.md | 28 ++-
3_edk_ii_dec_file_format/35_[includes]_sections.md | 30 ++-
3_edk_ii_dec_file_format/36_[guids]_sections.md | 29 ++-
.../37_[protocols]_sections.md | 29 ++-
3_edk_ii_dec_file_format/38_[ppis]_sections.md | 29 ++-
README.md | 211 +++++++++++----------
11 files changed, 329 insertions(+), 123 deletions(-)
--
2.6.3.windows.1
next prev parent reply other threads:[~2017-04-25 1:40 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-25 1:10 [ edk2-DecSpecification PATCH 0/4] Add support for Private declarations in a package Michael Kinney
2017-04-25 1:10 ` [ edk2-DecSpecification PATCH 1/4] Remove trailing spaces from README.MD Michael Kinney
2017-04-25 1:10 ` [ edk2-DecSpecification PATCH 2/4] Update version from 1.25 to 1.26 Michael Kinney
2017-04-25 1:10 ` [ edk2-DecSpecification PATCH 3/4] Add support for Private declarations in a package Michael Kinney
2017-04-25 1:10 ` [ edk2-DecSpecification PATCH 4/4] Declarations not allowed to be both public and private Michael Kinney
2017-04-25 1:40 ` Zhu, Yonghong [this message]
2017-04-25 15:21 ` [ edk2-DecSpecification PATCH 0/4] Add support for Private declarations in a package Laszlo Ersek
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=B9726D6DCCFB8B4CA276A9169B02216D51E64130@SHSMSX103.ccr.corp.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