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.web08.4190.1630476331019095293 for ; Tue, 31 Aug 2021 23:05:32 -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 ; Wed, 01 Sep 2021 14:05:26 +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: Wed, 1 Sep 2021 14:05:27 +0800 Message-ID: <004e01d79ef7$5cc7aee0$16570ca0$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQJPuknib+gWyDmBfT4vV22OaqIm1AMJlD09AjYZtNwCYvydSgHQCp3RAogaH4IBnMUduqoxvXOA Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn Create PR https://github.com/tianocore/edk2/pull/1940 for this patch set.= =20 > -----=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=881=E6=97=A5 1= 1:32 > =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 > 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