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.web10.5982.1630371900177312386 for ; Mon, 30 Aug 2021 18:05:02 -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 ; Tue, 31 Aug 2021 09:04:55 +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: "'Tan, Dun'" , 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> In-Reply-To: Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gW1BBVENIIDIvMl0gLmF6dXJlcGlwZWxpbmVzOiBBZGQgVWVmaVBheWxvYWRQa2cgaW4gZ2F0ZS1idWlsZC1qb2IueW1sIGFuZCBDSVNldHRpbmcucHk=?= Date: Tue, 31 Aug 2021 09:04:56 +0800 Message-ID: <021801d79e04$37003fe0$a500bfa0$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQJPuknib+gWyDmBfT4vV22OaqIm1AMJlD09AjYZtNwCYvydSgHQCp3RqlD9caA= Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn Dun: This PR also includes one additional commit to fix the build issue. Have = you send the patch for it?=20 For this patch set, Reviewed-by: Liming Gao . I= t 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 ; 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 > 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 >=20 > 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: Ad= d UefiPayloadPkg > in gate-build-job.yml and CISetting.py >=20 > Dun: > I don't see the issues to enable UefiPayloadPkg in CI. Have you created= the > private PR to verify this change? >=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=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 Barke= lew > > ; Kinney, Michael D > > ; Liming Gao > > =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 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