public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "gaoliming via groups.io" <gaoliming=byosoft.com.cn@groups.io>
To: "'Joey Vagedes'" <joey.vagedes@gmail.com>, <devel@edk2.groups.io>
Cc: "'Alexei Fedorov'" <Alexei.Fedorov@arm.com>,
	"'Ard Biesheuvel'" <ardb+tianocore@kernel.org>,
	"'Ashraf Ali S'" <ashraf.ali.s@intel.com>,
	"'Chasel Chiu'" <chasel.chiu@intel.com>,
	"'Chen Gang C'" <gang.c.chen@intel.com>,
	"'Daniel Schaefer'" <git@danielschaefer.me>,
	"'Duggapu Chinni B'" <chinni.b.duggapu@intel.com>,
	"'Eric Dong'" <eric.dong@intel.com>,
	"'Gerd Hoffmann'" <kraxel@redhat.com>,
	"'Guo Dong'" <guo.dong@intel.com>,
	"'Guomin Jiang'" <guomin.jiang@intel.com>,
	"'Hao A Wu'" <hao.a.wu@intel.com>,
	"'Igor Kulchytskyy'" <igork@ami.com>,
	"'James Lu'" <james.lu@intel.com>,
	"'Jian J Wang'" <jian.j.wang@intel.com>,
	"'Jiewen Yao'" <jiewen.yao@intel.com>,
	"'Leif Lindholm'" <quic_llindhol@quicinc.com>,
	"'Maciej Rabeda'" <maciej.rabeda@linux.intel.com>,
	"'Michael D Kinney'" <michael.d.kinney@intel.com>,
	"'Michael Kubacki'" <mikuback@linux.microsoft.com>,
	"'Nate DeSimone'" <nathaniel.l.desimone@intel.com>,
	"'Nickle Wang'" <nicklew@nvidia.com>,
	"'Pierre Gondois'" <pierre.gondois@arm.com>,
	"'Rahul Kumar'" <rahul1.kumar@intel.com>,
	"'Ray Han Lim Ng'" <ray.han.lim.ng@intel.com>,
	"'Ray Ni'" <ray.ni@intel.com>,
	"'Sami Mujawar'" <sami.mujawar@arm.com>,
	"'Sean Brogan'" <sean.brogan@microsoft.com>,
	"'Sean Rhodes'" <sean@starlabs.systems>,
	"'Siyuan Fu'" <siyuan.fu@intel.com>,
	"'Star Zeng'" <star.zeng@intel.com>,
	"'Susovan Mohapatra'" <susovan.mohapatra@intel.com>,
	"'Ted Kuo'" <ted.kuo@intel.com>, "'Wei6 Xu'" <wei6.xu@intel.com>,
	"'Xiaoyu Lu'" <xiaoyu1.lu@intel.com>,
	"'Yi Li'" <yi1.li@intel.com>,
	"'Zhichao Gao'" <zhichao.gao@intel.com>,
	"'Zhiguang Liu'" <zhiguang.liu@intel.com>
Subject: [edk2-devel] 回复: [PATCH v1 00/24] Update Edk2-pytools to latest versions
Date: Wed, 2 Aug 2023 11:14:16 +0800	[thread overview]
Message-ID: <002301d9c4ef$6bfcfcb0$43f6f610$@byosoft.com.cn> (raw)
In-Reply-To: <CAKURc-8RCMBBT0=Y1Y9eM3kLx9T1icWV_MQp+POsrCfcNQm4oQ@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 9943 bytes --]

Joey:

 For MdePkg, MdeModulePkg, FmpDevicePkg,  Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>

 

Thanks

Liming

发件人: Joey Vagedes <joey.vagedes@gmail.com> 
发送时间: 2023年8月2日 0:17
收件人: devel@edk2.groups.io
抄送: Alexei Fedorov <Alexei.Fedorov@arm.com>; Ard Biesheuvel <ardb+tianocore@kernel.org>; Ashraf Ali S <ashraf.ali.s@intel.com>; Chasel Chiu <chasel.chiu@intel.com>; Chen Gang C <gang.c.chen@intel.com>; Daniel Schaefer <git@danielschaefer.me>; Duggapu Chinni B <chinni.b.duggapu@intel.com>; Eric Dong <eric.dong@intel.com>; Gerd Hoffmann <kraxel@redhat.com>; Guo Dong <guo.dong@intel.com>; Guomin Jiang <guomin.jiang@intel.com>; Hao A Wu <hao.a.wu@intel.com>; Igor Kulchytskyy <igork@ami.com>; James Lu <james.lu@intel.com>; Jian J Wang <jian.j.wang@intel.com>; Jiewen Yao <jiewen.yao@intel.com>; Leif Lindholm <quic_llindhol@quicinc.com>; Liming Gao <gaoliming@byosoft.com.cn>; Maciej Rabeda <maciej.rabeda@linux.intel.com>; Michael D Kinney <michael.d.kinney@intel.com>; Michael Kubacki <mikuback@linux.microsoft.com>; Nate DeSimone <nathaniel.l.desimone@intel.com>; Nickle Wang <nicklew@nvidia.com>; Pierre Gondois <pierre.gondois@arm.com>; Rahul Kumar <rahul1.kumar@intel.com>; Ray Han Lim Ng <ray.han.lim.ng@intel.com>; Ray Ni <ray.ni@intel.com>; Sami Mujawar <sami.mujawar@arm.com>; Sean Brogan <sean.brogan@microsoft.com>; Sean Rhodes <sean@starlabs.systems>; Siyuan Fu <siyuan.fu@intel.com>; Star Zeng <star.zeng@intel.com>; Susovan Mohapatra <susovan.mohapatra@intel.com>; Ted Kuo <ted.kuo@intel.com>; Wei6 Xu <wei6.xu@intel.com>; Xiaoyu Lu <xiaoyu1.lu@intel.com>; Yi Li <yi1.li@intel.com>; Zhichao Gao <zhichao.gao@intel.com>; Zhiguang Liu <zhiguang.liu@intel.com>
主题: Re: [PATCH v1 00/24] Update Edk2-pytools to latest versions

 

Hello everyone, I'm still looking for reviews for this change. Please remember this has no effect on the package itself, rather it adds additional helps to ensure your package is tested when necessary. The particular situation this change now catches is when a library instance (INF) from outside your package changes and your package consumes that library instance. When this situation occurs, your package will also be built as a part of CI, ensuring other package changes don't break your package in a build breaking way.

 

I still need reviews from the following packages:
- UefiPayloadPkg

- ArmPkg

- UefiCpuPkg

- ArmPlatformPkg

- CryptoPkg

- FatPkg

- FmpDevicePkg

- IntelFsp2Pkg

- CryptoPkg

- SourceLevelDebugPkg

- SignedCapsulePkg

- ShellPkg

- SecurityPkg

- PcAtChipsetPkg

- NetworkPkg

- MdePkg

- MdeModulePkg

- IntelFsp2WrapperPkg

 

Thank you to those that have already reviewed!

 

Thanks,

Joey

 

On Thu, Jul 6, 2023 at 8:22 AM Joey Vagedes <joey.vagedes@gmail.com <mailto:joey.vagedes@gmail.com> > wrote:

Hello everyone,

 

Just following up on this patch series; looking for reviews from each package owner. Just to reiterate, this patch series enables a new policy for detecting which packages should be built for any given PR / Patch on github. The policy is used when an INF is changed. It requires that any package consuming that inf (generally a library instance) be built and tested. I'm happy to help answer any questions!

 

Thanks,

Joey

 

On Mon, Jun 26, 2023 at 2:34 PM Joey Vagedes <joey.vagedes@gmail.com <mailto:joey.vagedes@gmail.com> > wrote:

Updates Edk2-pytools to the latest versions, which introduces a new Policy
for detecting which packages must be built given a set of changes. This policy
is used when a library class instance INF is changed. When this occurs, it will
review the PrEval:DscPath provided in the CI YAML file and determine if the
package uses the library instance or not. If it does, it will build and test
the package.

Due to these changes, each package must now include an entry in it's CI YAML
file (PrEval:DscPath). If the package does not provide this entry, this policy
will be skipped, which could lead to a library instance breaking packages outside
of it's parent package.

Cc: Abner Chang <abner.chang@amd.com <mailto:abner.chang@amd.com> >
Cc: Alexei Fedorov <Alexei.Fedorov@arm.com <mailto:Alexei.Fedorov@arm.com> >
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org <mailto:ardb%2Btianocore@kernel.org> >
Cc: Ashraf Ali S <ashraf.ali.s@intel.com <mailto:ashraf.ali.s@intel.com> >
Cc: Chasel Chiu <chasel.chiu@intel.com <mailto:chasel.chiu@intel.com> >
Cc: Chen Gang C <gang.c.chen@intel.com <mailto:gang.c.chen@intel.com> >
Cc: Daniel Schaefer <git@danielschaefer.me <mailto:git@danielschaefer.me> >
Cc: Duggapu Chinni B <chinni.b.duggapu@intel.com <mailto:chinni.b.duggapu@intel.com> >
Cc: Eric Dong <eric.dong@intel.com <mailto:eric.dong@intel.com> >
Cc: Gerd Hoffmann <kraxel@redhat.com <mailto:kraxel@redhat.com> >
Cc: Guo Dong <guo.dong@intel.com <mailto:guo.dong@intel.com> >
Cc: Guomin Jiang <guomin.jiang@intel.com <mailto:guomin.jiang@intel.com> >
Cc: Hao A Wu <hao.a.wu@intel.com <mailto:hao.a.wu@intel.com> >
Cc: Igor Kulchytskyy <igork@ami.com <mailto:igork@ami.com> >
Cc: James Lu <james.lu@intel.com <mailto:james.lu@intel.com> >
Cc: Jian J Wang <jian.j.wang@intel.com <mailto:jian.j.wang@intel.com> >
Cc: Jiewen Yao <jiewen.yao@intel.com <mailto:jiewen.yao@intel.com> >
Cc: Leif Lindholm <quic_llindhol@quicinc.com <mailto:quic_llindhol@quicinc.com> >
Cc: Liming Gao <gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn> >
Cc: Maciej Rabeda <maciej.rabeda@linux.intel.com <mailto:maciej.rabeda@linux.intel.com> >
Cc: Michael D Kinney <michael.d.kinney@intel.com <mailto:michael.d.kinney@intel.com> >
Cc: Michael Kubacki <mikuback@linux.microsoft.com <mailto:mikuback@linux.microsoft.com> >
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com <mailto:nathaniel.l.desimone@intel.com> >
Cc: Nickle Wang <nicklew@nvidia.com <mailto:nicklew@nvidia.com> >
Cc: Pierre Gondois <pierre.gondois@arm.com <mailto:pierre.gondois@arm.com> >
Cc: Rahul Kumar <rahul1.kumar@intel.com <mailto:rahul1.kumar@intel.com> >
Cc: Ray Han Lim Ng <ray.han.lim.ng@intel.com <mailto:ray.han.lim.ng@intel.com> >
Cc: Ray Ni <ray.ni@intel.com <mailto:ray.ni@intel.com> >
Cc: Sami Mujawar <sami.mujawar@arm.com <mailto:sami.mujawar@arm.com> >
Cc: Sean Brogan <sean.brogan@microsoft.com <mailto:sean.brogan@microsoft.com> >
Cc: Sean Rhodes <sean@starlabs.systems <mailto:sean@starlabs.systems> >
Cc: Siyuan Fu <siyuan.fu@intel.com <mailto:siyuan.fu@intel.com> >
Cc: Star Zeng <star.zeng@intel.com <mailto:star.zeng@intel.com> >
Cc: Susovan Mohapatra <susovan.mohapatra@intel.com <mailto:susovan.mohapatra@intel.com> >
Cc: Ted Kuo <ted.kuo@intel.com <mailto:ted.kuo@intel.com> >
Cc: Wei6 Xu <wei6.xu@intel.com <mailto:wei6.xu@intel.com> >
Cc: Xiaoyu Lu <xiaoyu1.lu@intel.com <mailto:xiaoyu1.lu@intel.com> >
Cc: Yi Li <yi1.li@intel.com <mailto:yi1.li@intel.com> >
Cc: Zhichao Gao <zhichao.gao@intel.com <mailto:zhichao.gao@intel.com> >
Cc: Zhiguang Liu <zhiguang.liu@intel.com <mailto:zhiguang.liu@intel.com> >

Joey Vagedes (24):
  Edk2: edk2-pytools: Update to latest versions
  UnitTestFrameworkPkg: CI: Add PrEval entry
  UefiPayloadPkg: CI: Add PrEval entry
  ArmPkg: CI: Add PrEval entry
  UefiCpuPkg: CI: Add PrEval entry
  ArmPlatformPkg: CI: Add PrEval entry
  CryptoPkg: CI: Add PrEval entry
  DynamicTablesPkg: CI: Add PrEval entry
  EmbeddedPkg: CI: Add PrEval entry
  FatPkg: CI: Add PrEval entry
  FmpDevicePkg: CI: Add PrEval entry
  IntelFsp2Pkg: CI: Add PrEval entry
  CryptoPkg: CI: Add PrEval entry
  SourceLevelDebugPkg: CI: Add PrEval entry
  SignedCapsulePkg: CI: Add PrEval entry
  ShellPkg: CI: Add PrEval entry
  SecurityPkg: CI: Add PrEval entry
  RedfishPkg: CI: Add PrEval entry
  PrmPkg: CI: Add PrEval entry
  PcAtChipsetPkg: CI: Add PrEval entry
  NetworkPkg: CI: Add PrEval entry
  MdePkg: CI: Add PrEval entry
  MdeModulePkg: CI: Add PrEval entry
  IntelFsp2WrapperPkg: CI: Add PrEval entry

 ArmPkg/ArmPkg.ci.yaml                             | 3 +++
 ArmPlatformPkg/ArmPlatformPkg.ci.yaml             | 3 +++
 CryptoPkg/CryptoPkg.ci.yaml                       | 3 +++
 DynamicTablesPkg/DynamicTablesPkg.ci.yaml         | 3 +++
 EmbeddedPkg/EmbeddedPkg.ci.yaml                   | 3 +++
 FatPkg/FatPkg.ci.yaml                             | 3 +++
 FmpDevicePkg/FmpDevicePkg.ci.yaml                 | 3 +++
 IntelFsp2Pkg/IntelFsp2Pkg.ci.yaml                 | 3 +++
 IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.ci.yaml   | 3 +++
 MdeModulePkg/MdeModulePkg.ci.yaml                 | 3 +++
 MdePkg/MdePkg.ci.yaml                             | 3 +++
 NetworkPkg/NetworkPkg.ci.yaml                     | 3 +++
 PcAtChipsetPkg/PcAtChipsetPkg.ci.yaml             | 3 +++
 PrmPkg/PrmPkg.ci.yaml                             | 3 +++
 RedfishPkg/RedfishPkg.ci.yaml                     | 3 +++
 SecurityPkg/SecurityPkg.ci.yaml                   | 3 +++
 ShellPkg/ShellPkg.ci.yaml                         | 3 +++
 SignedCapsulePkg/SignedCapsulePkg.ci.yaml         | 3 +++
 SourceLevelDebugPkg/SourceLevelDebugPkg.ci.yaml   | 3 +++
 StandaloneMmPkg/StandaloneMmPkg.ci.yaml           | 3 +++
 UefiCpuPkg/UefiCpuPkg.ci.yaml                     | 3 +++
 UefiPayloadPkg/UefiPayloadPkg.ci.yaml             | 3 +++
 UnitTestFrameworkPkg/UnitTestFrameworkPkg.ci.yaml | 3 +++
 pip-requirements.txt                              | 4 ++--
 24 files changed, 71 insertions(+), 2 deletions(-)

-- 
2.41.0.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#107440): https://edk2.groups.io/g/devel/message/107440
Mute This Topic: https://groups.io/mt/100498733/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



[-- Attachment #2: Type: text/html, Size: 19478 bytes --]

  reply	other threads:[~2023-08-02  3:14 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-26 21:34 [PATCH v1 00/24] Update Edk2-pytools to latest versions Joey Vagedes
2023-06-26 21:34 ` [PATCH v1 01/24] Edk2: edk2-pytools: Update " Joey Vagedes
2023-07-05 17:33   ` Michael Kubacki
2023-06-26 21:34 ` [PATCH v1 02/24] UnitTestFrameworkPkg: CI: Add PrEval entry Joey Vagedes
2023-07-05 17:34   ` [edk2-devel] " Michael Kubacki
2023-06-26 21:34 ` [PATCH v1 03/24] UefiPayloadPkg: " Joey Vagedes
2023-06-26 21:34 ` [PATCH v1 04/24] ArmPkg: " Joey Vagedes
2023-06-26 21:34 ` [PATCH v1 05/24] UefiCpuPkg: " Joey Vagedes
2023-06-26 21:34 ` [PATCH v1 06/24] ArmPlatformPkg: " Joey Vagedes
2023-06-26 21:34 ` [PATCH v1 07/24] CryptoPkg: " Joey Vagedes
2023-08-02  4:41   ` [edk2-devel] " Li, Yi
     [not found]   ` <1777774354535489.2497@groups.io>
2023-10-07  3:07     ` Li, Yi
2023-10-09 16:26       ` Joey Vagedes via groups.io
2023-06-26 21:34 ` [PATCH v1 08/24] DynamicTablesPkg: " Joey Vagedes
2023-07-06 15:36   ` Sami Mujawar
2023-06-26 21:34 ` [PATCH v1 09/24] EmbeddedPkg: " Joey Vagedes
2023-06-28  6:20   ` Chang, Abner
2023-06-26 21:34 ` [PATCH v1 10/24] FatPkg: " Joey Vagedes
2023-06-26 21:34 ` [PATCH v1 11/24] FmpDevicePkg: " Joey Vagedes
2023-06-26 21:34 ` [PATCH v1 12/24] IntelFsp2Pkg: " Joey Vagedes
2023-08-07 22:57   ` [edk2-devel] " Nate DeSimone
2023-06-26 21:34 ` [PATCH v1 13/24] CryptoPkg: " Joey Vagedes
2023-10-09 16:49   ` [edk2-devel] [PATCH v1 13/24] StandaloneMmPkg: " Joey Vagedes via groups.io
2023-10-10  7:44     ` Ard Biesheuvel
2023-06-26 21:34 ` [PATCH v1 14/24] SourceLevelDebugPkg: " Joey Vagedes
2023-06-26 21:34 ` [PATCH v1 15/24] SignedCapsulePkg: " Joey Vagedes
2023-06-26 21:34 ` [PATCH v1 16/24] ShellPkg: " Joey Vagedes
2023-06-26 21:34 ` [PATCH v1 17/24] SecurityPkg: " Joey Vagedes
2023-06-26 21:34 ` [PATCH v1 18/24] RedfishPkg: " Joey Vagedes
2023-06-28  6:20   ` Chang, Abner
2023-06-26 21:34 ` [PATCH v1 19/24] PrmPkg: " Joey Vagedes
2023-07-05 17:34   ` [edk2-devel] " Michael Kubacki
2023-08-07 22:57   ` Nate DeSimone
2023-06-26 21:34 ` [PATCH v1 20/24] PcAtChipsetPkg: " Joey Vagedes
2023-06-26 21:34 ` [PATCH v1 21/24] NetworkPkg: " Joey Vagedes
2023-06-26 21:34 ` [PATCH v1 22/24] MdePkg: " Joey Vagedes
2023-06-26 21:34 ` [PATCH v1 23/24] MdeModulePkg: " Joey Vagedes
2023-06-26 21:34 ` [PATCH v1 24/24] IntelFsp2WrapperPkg: " Joey Vagedes
2023-08-07 22:57   ` [edk2-devel] " Nate DeSimone
2023-06-27  0:59 ` 回复: [PATCH v1 00/24] Update Edk2-pytools to latest versions gaoliming
2023-06-27 16:58   ` Joey Vagedes
2023-06-29 18:01   ` [edk2-devel] 回复: " Sean
2023-06-30  1:50     ` 回复: " gaoliming
2023-07-06 15:22 ` Joey Vagedes
2023-08-01 16:16   ` [edk2-devel] " Joey Vagedes via groups.io
2023-08-02  3:14     ` gaoliming via groups.io [this message]
2023-08-23 17:14     ` Joey Vagedes via groups.io
2023-10-02 22:20       ` Joey Vagedes via groups.io

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='002301d9c4ef$6bfcfcb0$43f6f610$@byosoft.com.cn' \
    --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