From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242]) by mx.groups.io with SMTP id smtpd.web12.1570.1630548584532110219 for ; Wed, 01 Sep 2021 19:09:45 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=none, err=permanent DNS error (domain: byosoft.com.cn, ip: 58.240.74.242, mailfrom: gaoliming@byosoft.com.cn) Received: from DESKTOPS6D0PVI ([58.246.60.130]) (envelope-sender ) by 192.168.6.13 with ESMTP for ; Thu, 02 Sep 2021 10:09:39 +0800 X-WM-Sender: gaoliming@byosoft.com.cn X-Originating-IP: 58.246.60.130 X-WM-AuthFlag: YES X-WM-AuthUser: gaoliming@byosoft.com.cn From: "gaoliming" To: , Cc: "'Sean Brogan'" , "'Bret Barkelew'" , "'Kinney, Michael D'" References: <20210820064337.795-1-dun.tan@intel.com> <20210820064337.795-3-dun.tan@intel.com> <019101d79d4e$d1efe9d0$75cfbd70$@byosoft.com.cn> <021801d79e04$37003fe0$a500bfa0$@byosoft.com.cn> In-Reply-To: Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gW1BBVENIIDIvMl0gLmF6dXJlcGlwZWxpbmVzOiBBZGQgVWVmaVBheWxvYWRQa2cgaW4gZ2F0ZS1idWlsZC1qb2IueW1sIGFuZCBDSVNldHRpbmcucHk=?= Date: Thu, 2 Sep 2021 10:09:41 +0800 Message-ID: <00b701d79f9f$972611f0$c57235d0$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQJPuknib+gWyDmBfT4vV22OaqIm1AMJlD09AjYZtNwCYvydSgHQCp3RAogaH4IBnMUdugFO5ZERqiiWrfA= Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn I upload it again. Now, it has been merged.=20 Thanks Liming > -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6----- > =E5=8F=91=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io = =E4=BB=A3=E8=A1=A8 duntan > =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2021=E5=B9=B49=E6=9C=882=E6=97=A5 9= :24 > =E6=94=B6=E4=BB=B6=E4=BA=BA: gaoliming ; devel@= edk2.groups.io > =E6=8A=84=E9=80=81: 'Sean Brogan' ; 'Bret Bark= elew' > ; Kinney, Michael D > > =E4=B8=BB=E9=A2=98: Re: [edk2-devel] [PATCH 2/2] .azurepipelines: Add Uef= iPayloadPkg in > gate-build-job.yml and CISetting.py >=20 > Hi Liming, >=20 > I have seen the link to push my commits to master of edk2. The result sho= ws > that the Ci test failed in ArmPlatformPkg. But I think my commits will no= t > affect the ArmPlatformPkg. Here is the link. > https://github.com/tianocore/edk2/pull/1940the > So is there anything I need to do? >=20 > Thanks, > Dun > -----Original Message----- > From: Tan, Dun > Sent: Wednesday, September 1, 2021 11:32 AM > To: gaoliming ; devel@edk2.groups.io > Cc: 'Sean Brogan' ; 'Bret Barkelew' > ; Kinney, Michael D > > Subject: RE: [edk2-devel] [PATCH 2/2] .azurepipelines: Add UefiPayloadPkg= in > gate-build-job.yml and CISetting.py >=20 > Hi Liming, >=20 > The patch to fix the build issue has been merged this morning. > Would you please help to merge the two patches to enable core CI for > UefiPayloadPkg? >=20 > Thanks, > Dun > -----Original Message----- > From: gaoliming > Sent: Tuesday, August 31, 2021 9:05 AM > To: Tan, Dun ; devel@edk2.groups.io > Cc: 'Sean Brogan' ; 'Bret Barkelew' > ; Kinney, Michael D > > Subject: =E5=9B=9E=E5=A4=8D: [edk2-devel] [PATCH 2/2] .azurepipelines: Ad= d UefiPayloadPkg > in gate-build-job.yml and CISetting.py >=20 > Dun: > This PR also includes one additional commit to fix the build issue. Hav= e you > send the patch for it? >=20 > For this patch set, Reviewed-by: Liming Gao . > It will be merged after the build issue has been fixed. >=20 > Thanks > Liming > > -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6----- > > =E5=8F=91=E4=BB=B6=E4=BA=BA: Tan, Dun > > =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2021=E5=B9=B48=E6=9C=8830=E6=97= =A5 16:31 > > =E6=94=B6=E4=BB=B6=E4=BA=BA: gaoliming ; deve= l@edk2.groups.io > > =E6=8A=84=E9=80=81: 'Sean Brogan' ; 'Bret Ba= rkelew' > > ; Kinney, Michael D > > > > =E4=B8=BB=E9=A2=98: RE: [edk2-devel] [PATCH 2/2] .azurepipelines: Add U= efiPayloadPkg > > in gate-build-job.yml and CISetting.py > > > > Hi Liming, > > > > Here is the link of the PR to verify the change in Tiano EDKII. It > > passed the CI by GCC and VS both. > > https://github.com/tianocore/edk2/pull/1931 > > > > Thanks, > > Dun > > -----Original Message----- > > From: gaoliming > > Sent: Monday, August 30, 2021 11:26 AM > > To: devel@edk2.groups.io; Tan, Dun > > Cc: 'Sean Brogan' ; 'Bret Barkelew' > > ; Kinney, Michael D > > > > Subject: =E5=9B=9E=E5=A4=8D: [edk2-devel] [PATCH 2/2] .azurepipelines: = Add > > UefiPayloadPkg in gate-build-job.yml and CISetting.py > > > > Dun: > > I don't see the issues to enable UefiPayloadPkg in CI. Have you > > created the private PR to verify this change? > > > > Thanks > > Liming > > > -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6----- > > > =E5=8F=91=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io =E4=BB=A3=E8=A1=A8 duntan > > > =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2021=E5=B9=B48=E6=9C=8824=E6=97= =A5 17:24 > > > =E6=94=B6=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io > > > =E6=8A=84=E9=80=81: Sean Brogan ; Bret Bar= kelew > > > ; Kinney, Michael D > > > ; Liming Gao > > > =E4=B8=BB=E9=A2=98: Re: [edk2-devel] [PATCH 2/2] .azurepipelines: Add= UefiPayloadPkg > > > in gate-build-job.yml and CISetting.py > > > > > > Hi all, > > > Since the CI for UefiPayloadPkg is important to our develop > > > progress, would you please speed up the review process? Thanks a lot! > > > > > > Thanks, > > > Dun Tan > > > -----Original Message----- > > > From: Tan, Dun > > > Sent: Friday, August 20, 2021 2:44 PM > > > To: devel@edk2.groups.io > > > Cc: Sean Brogan ; Bret Barkelew > > > ; Kinney, Michael D > > > ; Liming Gao ; > > > Tan, Dun > > > Subject: [PATCH 2/2] .azurepipelines: Add UefiPayloadPkg in > > > gate-build-job.yml and CISetting.py > > > > > > Add UefiPayloadPkg in gate-build-job.yml to enable Core ci for > > > UefiPayloadPkg. > > > Add UefiPayloadPkg to supported Packages in CISettings. > > > > > > Cc: Sean Brogan > > > Cc: Bret Barkelew > > > Cc: Michael D Kinney > > > Cc: Liming Gao > > > Signed-off-by: DunTan > > > --- > > > .azurepipelines/templates/pr-gate-build-job.yml | 3 +++ > > > .pytool/CISettings.py | 3 ++- > > > 2 files changed, 5 insertions(+), 1 deletion(-) > > > > > > diff --git a/.azurepipelines/templates/pr-gate-build-job.yml > > > b/.azurepipelines/templates/pr-gate-build-job.yml > > > index 207acc7631..d5b16c127f 100644 > > > --- a/.azurepipelines/templates/pr-gate-build-job.yml > > > +++ b/.azurepipelines/templates/pr-gate-build-job.yml > > > @@ -48,6 +48,9 @@ jobs: > > > TARGET_SECURITY: > > > Build.Pkgs: 'SecurityPkg' > > > Build.Targets: 'DEBUG,RELEASE,NO-TARGET' > > > + TARGET_UEFIPAYLOAD: > > > + Build.Pkgs: 'UefiPayloadPkg' > > > + Build.Targets: 'DEBUG,RELEASE,NO-TARGET' > > > TARGET_PLATFORMS: > > > # For Platforms only check code. Leave it to Platform CI > > > # to build them. > > > diff --git a/.pytool/CISettings.py b/.pytool/CISettings.py index > > > 96e6baa519..ce330e2c73 100644 > > > --- a/.pytool/CISettings.py > > > +++ b/.pytool/CISettings.py > > > @@ -67,7 +67,8 @@ class Settings(CiBuildSettingsManager, > > > UpdateSettingsManager, SetupSettingsManag > > > "CryptoPkg", > > > "UnitTestFrameworkPkg", > > > "OvmfPkg", > > > - "RedfishPkg" > > > + "RedfishPkg", > > > + "UefiPayloadPkg" > > > ) > > > > > > def GetArchitecturesSupported(self): > > > -- > > > 2.31.1.windows.1 > > > > > > > > > > > > > > > > > > > >=20 >=20 >=20 >=20 >=20 >=20 >=20