(cc Bob) NOTE this affects the stable tag - please see below
On Sun, 21 Aug 2022 at 06:34, Rebecca Cran <rebe...@bsdio.com> wrote: > > I noticed that MdeModulePkg fails to build on my Ubuntu 22.04 system > (and previously on the Ubuntu 20.04 installation). Is this a known bug? > > It shouldn't be relevant, but I'm using gcc version 11.2.0 (Ubuntu > 11.2.0-17ubuntu1). But perhaps more relevant, I'm using Python 3.10.4. > > I'm trying to build commit e2ac68a23b4954d5c0399913a1df3dd9fd90315d. > > > bcran@photon:~/src/tmp/edk2$ build -p MdeModulePkg/MdeModulePkg.dsc -a > AARCH64 -t GCC5 -b RELEASE > Build environment: Linux-5.15.0-46-generic-x86_64-with-glibc2.35 > Build start time: 22:29:18, Aug.20 2022 > > WORKSPACE = /home/bcran/src/tmp/edk2 > EDK_TOOLS_PATH = /home/bcran/src/tmp/edk2/BaseTools > CONF_PATH = /home/bcran/src/tmp/edk2/Conf > PYTHON_COMMAND = /usr/bin/python3 > > > Processing meta-data > .Architecture(s) = AARCH64 > Build target = RELEASE > Toolchain = GCC5 > > Active Platform = > /home/bcran/src/tmp/edk2/MdeModulePkg/MdeModulePkg.dsc > > > build.py... > /home/bcran/src/tmp/edk2/MdeModulePkg/Library/SmmLockBoxLib/SmmLockBoxPeiLib.inf(42): > error 3000: PCD > [gEfiMdeModulePkgTokenSpaceGuid.PcdDxeIplSwitchToLongMode] in > [/home/bcran/src/tmp/edk2/MdeModulePkg/Library/SmmLockBoxLib/SmmLockBoxPeiLib.inf] > is not found in dependent packages: > /home/bcran/src/tmp/edk2/MdePkg/MdePkg.dec > /home/bcran/src/tmp/edk2/MdeModulePkg/MdeModulePkg.dec > > This looks like a BaseTools regression to me - afaik, these packages all used to build for AARCH64 in the past. The following LockBoxLib resolutions appear in MdeModulePkg.dsc (with line numbers) 115-[LibraryClasses.common.PEIM] 119: LockBoxLib|MdeModulePkg/Library/SmmLockBoxLib/SmmLockBoxPeiLib.inf 178-[LibraryClasses.ARM, LibraryClasses.AARCH64] 181: LockBoxLib|MdeModulePkg/Library/LockBoxNullLib/LockBoxNullLib.inf No other [LlibraryClasses] references to SmmLockBoxPeiLib.inf exist in that file, and the [Components] reference does not apply to AARCH64. This means the DSC parser ends up using the earlier definition, which I think is a bug, or at least a change in behavior. Note that doing this silently could potentially break many platforms in subtle ways so I think this should be root caused and preferably fixed before creating the stable tag. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#92611): https://edk2.groups.io/g/devel/message/92611 Mute This Topic: https://groups.io/mt/93156882/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
