From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web11.7793.1586167879874386291 for ; Mon, 06 Apr 2020 03:11:19 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: ard.biesheuvel@arm.com) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 42A4130E; Mon, 6 Apr 2020 03:11:19 -0700 (PDT) Received: from [10.37.8.45] (unknown [10.37.8.45]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id F31BC3F52E; Mon, 6 Apr 2020 03:11:17 -0700 (PDT) Subject: Re: [edk2-devel] [PATCH] .azurepipelines: Enable CI for OvmfPkg and EmulatorPkg To: devel@edk2.groups.io, sean.brogan@microsoft.com References: <14537.1585602191842859281@groups.io> <22879.1586068785483835499@groups.io> Cc: Leif Lindholm From: "Ard Biesheuvel" Message-ID: <5d642758-d8f1-ef6d-8ca8-229ff496a893@arm.com> Date: Mon, 6 Apr 2020 12:11:16 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.4.1 MIME-Version: 1.0 In-Reply-To: <22879.1586068785483835499@groups.io> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: quoted-printable On 4/5/20 8:39 AM, Sean via groups.io wrote: > Platform and Core CI patches are ready for review.=C2=A0 I have 14 comm= its here. > https://github.com/spbrogan/edk2/tree/PlatformAndCoreCIForOvmfArmVirtEm= ulatorPackages >=20 > This adds support for "Platform CI" for ArmVirtPkg, OvmfPkg, and=20 > EmulatorPkg. > Each readme has live status and links to the builds as well as details=20 > of how to build and run the same way the CI server will. > ArmVirt:=20 > https://github.com/spbrogan/edk2/blob/PlatformAndCoreCIForOvmfArmVirtEm= ulatorPackages/ArmVirtPkg/README-pytools.md > Emulator:=20 > https://github.com/spbrogan/edk2/blob/PlatformAndCoreCIForOvmfArmVirtEm= ulatorPackages/EmulatorPkg/README-pytools.md > OvmfPkg:=20 > https://github.com/spbrogan/edk2/blob/PlatformAndCoreCIForOvmfArmVirtEm= ulatorPackages/OvmfPkg/README-pytools.md >=20 > It also adds ArmVirtPkg, OvmfPkg, and EmulatorPkg to Core CI just for=20 > the code evaluation tests (not compiling).=C2=A0 Details of those tests= are=20 > here:=20 > https://github.com/spbrogan/edk2/tree/PlatformAndCoreCIForOvmfArmVirtEm= ulatorPackages/.pytool >=20 > ArmVirtPkg, OvmfPkg, and EmulatorPkg maintainers please review and let=20 > me know what the next step for you package is.=C2=A0 This is ready to c= ommit=20 > from my perspective and has already caught a few issues in the last=20 > couple of weeks. >=20 Thanks Sean. This is all looking really good. I think this stuff is=20 ready to be sent to the mailing list for wider review. > Finally there are a few pending bugs that should get fixed and at least= =20 > a couple of gaps I have identified. These are not fixed in the above 14= =20 > commits. >=20 > 1. OvmfPkg > 1. https://bugzilla.tianocore.org/show_bug.cgi?id=3D2662=C2=A0-- B= locking > Core CI > 2. https://bugzilla.tianocore.org/show_bug.cgi?id=3D2661=C2=A0-- B= locking > Platform CI > 3. Spell check in audit mode > 2. EmulatorPkg > 1. https://bugzilla.tianocore.org/show_bug.cgi?id=3D2663=C2=A0-- I= gnore > added to Core CI > 2. https://bugzilla.tianocore.org/show_bug.cgi?id=3D2639=C2=A0-- F= eature > in Platform CI turned off > 3. https://bugzilla.tianocore.org/show_bug.cgi?id=3D2638=C2=A0-- F= eature > in Platform CI turned off > 4. Spell check in audit mode > 3. ArmVirtPkg > 1. No builds on Windows or with VS toolchain -- Feature in Platfor= m > CI turned off Leif has looked into this in the past. As I understand it, it is simply=20 a lack of .asm versions of the various assembler source files in ArmLib,=20 ArmMmuLib and the startup code in ArmPlatformPkg. --=20 Ard.