Sean, This is the reason that OvmfPkg was kept in the edk2 repo so only a single repo is required for local dev testing and CI testing. Same reason for the EmulatorPkg.
The current rule for the edk2 repo is common firmware packages and virtual/emulated platforms. The fact that there has not been bandwidth to implement the CI testing for OvmfPkg or EmulatorPkg is not a reasonable reason to remove them. Mike From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Sean via Groups.Io Sent: Tuesday, March 10, 2020 12:11 PM To: Ard Biesheuvel <ard.biesheu...@linaro.org>; devel@edk2.groups.io Subject: Re: [edk2-devel] Adding Bhyve support into upstream EDK2 On Tue, Mar 10, 2020 at 10:54 AM, Ard Biesheuvel wrote: This means we can reasonably require any contributor not to regress on those platforms, given how they have full access to it, and this is actually where i would like us to take the next step when it comes to ci automation, i.e., automatically flag PRs that break the boot on a selected set of qemu based configs. I appreciate the viewpoint but I don't think edk2 has "required" this yet and I would push back on that if it was brought up for community discussion. As you know I believe contributing to edk2 is much too hard already, and filled with custom edk2 workflows, and this one would only make it more of a burden to contribute. I do however agree that if this is important to the edk2 community, systems/automation should be setup to run this type of thing. If that happens, i don't think it is much harder to do it for multi-repo than single repo and if done right for multi-repo then other platforms can opt in as well and the efforts to build this infrastructure will scale well beyond a single virtual platform. Thanks Sean -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#55721): https://edk2.groups.io/g/devel/message/55721 Mute This Topic: https://groups.io/mt/71776477/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
