From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: redhat.com, ip: 209.132.183.28, mailfrom: lersek@redhat.com) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by groups.io with SMTP; Tue, 16 Apr 2019 09:53:31 -0700 Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.phx2.redhat.com [10.5.11.16]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 20EC6F74A7; Tue, 16 Apr 2019 16:53:31 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-120-171.rdu2.redhat.com [10.10.120.171]) by smtp.corp.redhat.com (Postfix) with ESMTP id 432105C1B4; Tue, 16 Apr 2019 16:53:30 +0000 (UTC) Subject: Re: [edk2-devel] Building current OVMF master with multiple workspaces on Windows/VS2017 fails due to missing includes? To: devel@edk2.groups.io, ryszard.knop@linux.intel.com References: From: "Laszlo Ersek" Message-ID: <37e1fe57-91fb-5b70-473c-ce9295624188@redhat.com> Date: Tue, 16 Apr 2019 18:53:29 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.79 on 10.5.11.16 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.38]); Tue, 16 Apr 2019 16:53:31 +0000 (UTC) Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: quoted-printable On 04/16/19 16:56, Ryszard Knop wrote: > I have a bit odd issues with building OVMF from the current master > branch (commit 0eccea3...), with the following setup using multiple > workspaces (this is important): >=20 > - Windows 10, 64-bit > - Visual Studio 2017 build tools (conf'd with VS2017_PREFIX envvar) > - NASM 2.14.02 (conf'd with NASM_PREFIX envvar) > - Python 3.7.3 running the latest BaseTools > - WORKSPACE envvar points at a dir with extra modules to be included > with OVMF (C:\Env\OvmfBuild) > - PACKAGES_PATH points at the EDK2 repo (C:\Env\Workspace\edk2) >=20 > All envvars are set before edksetup. On Linux anyway, the above has worked for me, with two finer points: - In $WORKSPACE, the Conf/ directory is created manually (as an empty directory), just before sourcing edksetup.sh - (not sure if this matters, but) the "--platform" option of "build" refers to a standalone DSC file under $WORKSPACE, not to one of the OVMF DSC files under $PACKAGES_PATH. The use case for the above is different from yours though -- it's not for building OVMF with extra modules included, but for building standalone UEFI applications with various libraries from edk2. ... It's also possible, I guess, that the fix for simply forgot about PACKAGES_PATH. If you can provide a stripped-down reproducer (with basically "empty" additional driver modules etc), I'd suggest packaging it all up in a zip file, and reporting a new TianoCore BZ for BaseTools, with the zip file attached. (The zip file is particularly helpful because it can preserve the directory structure too, which individual BZ attachments couldn't.) Thanks Laszlo > Trying to build -a X64 -p > OvmfPkg/OvmfPkgX64.dsc -b RELEASE -t VS2017 fails with the following: >=20 > "C:\Env\BuildTools\MSVS2017_SDK_WDK_Windows10_16299\Program > Files\Microsoft Visual > Studio\2017\BuildTools\VC\Tools\MSVC\14.10.25017\bin\Hostx86\x64\cl.exe" > /nologo /E /TC /FIAutoGen.h > /IC:\Env\Workspace\edk2\MdePkg\Library\BaseLib\X64 > /IC:\Env\Workspace\edk2\MdePkg\Library\BaseLib > /Ic:\Env\OvmfBuild\Build\OvmfX64\RELEASE_VS2017\X64\MdePkg\Library\BaseL= ib\BaseLib\DEBUG > /IC:\Env\Workspace\edk2\MdePkg /IC:\Env\Workspace\edk2\MdePkg\Include > /IC:\Env\Workspace\edk2\MdePkg\Include\X64 > C:\Env\Workspace\edk2\MdePkg\Library\BaseLib\X64\LongJump.nasm > > c:\Env\OvmfBuild\Build\OvmfX64\RELEASE_VS2017\X64\MdePkg\Library\BaseLib= \BaseLib\OUTPUT\X64\LongJump.i >=20 > LongJump.nasm > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 Trim --trim-long --source-cod= e -o > c:\Env\OvmfBuild\Build\OvmfX64\RELEASE_VS2017\X64\MdePkg\Library\BaseLib= \BaseLib\OUTPUT\X64\LongJump.iii > c:\Env\OvmfBuild\Build\OvmfX64\RELEASE_VS2017\X64\MdePkg\Library\BaseLib= \BaseLib\OUTPUT\X64\LongJump.i >=20 > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 "C:\Env\BuildTools\NASM\2.14.= 02\nasm" > -IC:\Env\Workspace\edk2\MdePkg\Library\BaseLib\X64\ -Ox -f win64 -o > c:\Env\OvmfBuild\Build\OvmfX64\RELEASE_VS2017\X64\MdePkg\Library\BaseLib= \BaseLib\OUTPUT\X64\LongJump.obj > c:\Env\OvmfBuild\Build\OvmfX64\RELEASE_VS2017\X64\MdePkg\Library\BaseLib= \BaseLib\OUTPUT\X64\LongJump.iii >=20 > c:\Env\OvmfBuild\Build\OvmfX64\RELEASE_VS2017\X64\MdePkg\Library\BaseLib= \BaseLib\OUTPUT\X64\LongJump.iii:16: > fatal: unable to open include file `Nasm.inc' > NMAKE : fatal error U1077: 'C:\Env\BuildTools\NASM\2.14.02\nasm.EXE' : > return code '0x1' >=20 > Missing Nasm.inc, which is present in MdePkg/Include/X64/Nasm.inc - but > the weird thing is, adding extra NASM include paths was added recently > (in BaseTools/Source/Python/AutoGen/GenMake.py there's a block that > replaces NASM_INC with appropriate directories, build_rule.template > should generate NASM calls with these, as of commit 7c3a1ef). After > adding a few prints to the GenMake I can see it does find the proper > include paths, but none of them land in the final NASM call in the > Makefile, even though it knows about the Nasm.inc dependency - the > NASM_INC macro seems to be completely missing from the generated Makefil= e: >=20 > $(OUTPUT_DIR)\X64\LongJump.obj : $(MAKE_FILE) > $(OUTPUT_DIR)\X64\LongJump.obj : > C:\Env\Workspace\edk2\MdePkg\Include\Base.h > $(OUTPUT_DIR)\X64\LongJump.obj : > C:\Env\Workspace\edk2\MdePkg\Include\X64\Nasm.inc > $(OUTPUT_DIR)\X64\LongJump.obj : $(DEBUG_DIR)\AutoGen.h > $(OUTPUT_DIR)\X64\LongJump.obj : > C:\Env\Workspace\edk2\MdePkg\Include\X64\ProcessorBind.h > $(OUTPUT_DIR)\X64\LongJump.obj : > C:\Env\Workspace\edk2\MdePkg\Include\Library\PcdLib.h > $(OUTPUT_DIR)\X64\LongJump.obj : > C:\Env\Workspace\edk2\MdePkg\Library\BaseLib\X64\LongJump.nasm > =C2=A0=C2=A0=C2=A0 "$(PP)" $(PP_FLAGS) $(INC) > C:\Env\Workspace\edk2\MdePkg\Library\BaseLib\X64\LongJump.nasm > > c:\Env\OvmfBuild\Build\OvmfX64\RELEASE_VS2017\X64\MdePkg\Library\BaseLib= \BaseLib\OUTPUT\X64\LongJump.i >=20 > =C2=A0=C2=A0=C2=A0 Trim --trim-long --source-code -o > c:\Env\OvmfBuild\Build\OvmfX64\RELEASE_VS2017\X64\MdePkg\Library\BaseLib= \BaseLib\OUTPUT\X64\LongJump.iii > c:\Env\OvmfBuild\Build\OvmfX64\RELEASE_VS2017\X64\MdePkg\Library\BaseLib= \BaseLib\OUTPUT\X64\LongJump.i >=20 > =C2=A0=C2=A0=C2=A0 "$(NASM)" -IC:\Env\Workspace\edk2\MdePkg\Library\Base= Lib\X64\ > $(NASM_FLAGS) -o > c:\Env\OvmfBuild\Build\OvmfX64\RELEASE_VS2017\X64\MdePkg\Library\BaseLib= \BaseLib\OUTPUT\X64\LongJump.obj > c:\Env\OvmfBuild\Build\OvmfX64\RELEASE_VS2017\X64\MdePkg\Library\BaseLib= \BaseLib\OUTPUT\X64\LongJump.iii >=20 >=20 > The nasty workaround is to copy Nasm.inc into MdePkg/Library/BaseLib/X64 > and everything builds just fine after that. >=20 > Another interesting thing is that if I don't use WORKSPACE and > PACKAGES_PATH (both contain no spaces and are on local file systems) but > rather build everything directly in the EDK2 directory, OVMF also builds > fine (Makefile contains the NASM_INC definition which is included in > NASM calls). Have anyone also hit this issue? >=20 > Thanks, Richard >=20 >=20 >=20 >=20