On Tue, 15 Aug 2023 at 10:10, Ard Biesheuvel <a...@kernel.org> wrote: > > On Mon, 14 Aug 2023 at 10:15, Corvin Köhne <corv...@freebsd.org> wrote: > > > > On Thu, 2023-08-10 at 15:54 -0400, Michael Kubacki wrote: > > > I'm not sure why test results are not being reported that may be a > > > larger issue. > > > > > > I pulled the branch locally, run uncrustify, and it automatically > > > fixed > > > the code to pass. > > > > > > The issue is the following function call: > > > > > > BuildPlatformInfoHob(); > > > > > > Should be: > > > > > > BuildPlatformInfoHob (); > > > > > > > I already run uncrustify locally but it doesn't complain about it. > > Thank you! > > > > Should I resend an updated patch? > > Can this change make it into edk-stable202308? Otherwise, fwcfg won't > > be usable by bhyve in this version. > > > > It is up to Liming to decide but given that > > - it is low risk as it only touches bhyve platform code > - i would have merged it already if bugs in the CI wouldn't have > prevented me from doing so > > I would be in favour of including this change
I accidentally pushed this change to master by pushing it to a branch that was already associated with a PR with the push label set. I will leave it to Liming and the stewards to decide whether to leave it or revert it. Apologies - I wasn't expecting this to be permitted by the CI rules during the freeze. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#107851): https://edk2.groups.io/g/devel/message/107851 Mute This Topic: https://groups.io/mt/100327419/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-