Hi Rebecca. I will investigate this for my account. joey.vage...@gmail.com is correct on the mail map as it is my preference. Both my personal and professional emails are linked to Bugzilla and github and joey.vage...@gmail.com is what should be showing up as the author for commits.
Thanks for bringing this to my attention. Joey -----Original Message----- From: Rebecca Cran <rebe...@os.amperecomputing.com> Sent: Thursday, January 4, 2024 4:06 PM To: devel@edk2.groups.io; Jeff Brasen <jbra...@nvidia.com>; j...@nvidia.com; michael.r...@amd.com; Joey Vagedes <joeyvage...@microsoft.com> Subject: [EXTERNAL] Updates to .mailmap needed for Jeff Brasen, Jake Garver, Joey Vagades and Michael Roth? [You don't often get email from rebe...@os.amperecomputing.com. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ] I noticed recent commits by Jeff Brasen, Jake Garver, Joey Vagades and Michael Roth have funky Author lines, which I think means .mailmap needs updated? commit 7a5823f85be99b9a92751fcf4141f7982fa5cc80 Author: Jeff Brasen via groups.io <jbrasen=nvidia....@groups.io> Date: Thu Dec 28 12:47:08 2023 -0800 EmbeddedPkg: Add DtPlatformLoaderLib gmock support Add Google Mock Library for DtPlatformLoaderDtbLib Signed-off-by: Jeff Brasen <jbra...@nvidia.com> commit 9f0061a03b61d282fbc0ba5be22155d06a5e64a1 Author: Joey Vagedes via groups.io <joeyvagedes=microsoft....@groups.io> Date: Wed Dec 6 12:27:02 2023 -0800 BaseTools: Resolve regex syntax warnings Switches regex patterns to raw text to resolve python 3.12 syntax warnings in regards to invalid escape sequences, as is suggested by the re (regex) module in python. Cc: Rebecca Cran <rebe...@bsdio.com> Cc: Liming Gao <gaolim...@byosoft.com.cn> Cc: Bob Feng <bob.c.f...@intel.com> Cc: Yuwei Chen <yuwei.c...@intel.com> Signed-off-by: Joey Vagedes <joey.vage...@gmail.com> Reviewed-by: Rebecca Cran <rebe...@bsdio.com> Author: Jake Garver via groups.io <jake=nvidia....@groups.io> Date: Tue Oct 3 23:04:54 2023 +0800 MdeModulePkg/RegularExpressinoDxe: Fix clang error Ignore old style declaration warnings in oniguruma/src/st.c. This was already ignored for MSFT, but newer versions of clang complain as well. Signed-off-by: Jake Garver <j...@nvidia.com> Reviewed-by: Nhi Pham <n...@os.amperecomputing.com> Tested-by: Nhi Pham <n...@os.amperecomputing.com> Reviewed-by: Liming Gao <gaolim...@byosoft.com.cn> commit f008890ae55929f7f17e7d2f8aff929255007d33 Author: Roth, Michael via groups.io <Michael.Roth=amd....@groups.io> Date: Wed Aug 16 15:11:45 2023 -0500 OvmfPkg/AmdSev: fix BdsPlatform.c assertion failure during boot Booting an SEV guest with AmdSev OVMF package currently triggers the following assertion with QEMU: InstallQemuFwCfgTables: installed 7 tables PcRtc: Write 0x20 to CMOS location 0x32 [Variable]END_OF_DXE is signaled Initialize variable error flag (FF) ASSERT_EFI_ERROR (Status = Not Found) ASSERT [BdsDxe] /home/VT_BUILD/ovmf/OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c(1711): !(((INTN)(RETURN_STATUS)(Status)) < 0) This seems to be due to commit 81dc0d8b4c, which switched to using PlatformBootManagerLib instead of PlatformBootManagerLibGrub. That pulls in a dependency on gEfiS3SaveStateProtocolGuid provider being available (which is asserted for in BdsPlatform.c:PlatformBootManagerBeforeConsole()/SaveS3BootScript()), but the libraries that provide it aren't currently included in the build. Add them similarly to what's done for OvmfPkg. Fixes: 81dc0d8b4c ("OvmfPkg/AmdSev: stop using PlatformBootManagerLibGrub") Signed-off-by: Michael Roth <michael.r...@amd.com> Acked-by: Gerd Hoffmann <kra...@redhat.com> Acked-by: Jiewen Yao <jiewen....@intel.com> -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#113405): https://edk2.groups.io/g/devel/message/113405 Mute This Topic: https://groups.io/mt/103534194/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
