From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from wout2-smtp.messagingengine.com (wout2-smtp.messagingengine.com [64.147.123.25]) by mx.groups.io with SMTP id smtpd.web10.1855.1682529141775135841 for ; Wed, 26 Apr 2023 10:12:21 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="signature has expired" header.i=@bsdio.com header.s=fm1 header.b=Wqu6umDu; spf=pass (domain: bsdio.com, ip: 64.147.123.25, mailfrom: rebecca@bsdio.com) Received: from compute2.internal (compute2.nyi.internal [10.202.2.46]) by mailout.west.internal (Postfix) with ESMTP id 69A393200956; Wed, 26 Apr 2023 13:12:19 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute2.internal (MEProxy); Wed, 26 Apr 2023 13:12:21 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdio.com; h=cc :cc:content-transfer-encoding:content-type:content-type:date :date:from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:sender:subject:subject:to:to; s=fm1; t= 1682529139; x=1682615539; bh=aurwaydDGaum+zqDRPRdYZnO6K3TaoNyBid J7uVQdCY=; b=Wqu6umDua6AdgCL5JFmJB+WskgtrkeUOGoSt+drTkaawi0phLVd wElPrFnTYkaPgTqjR+haplqRcUX7VOOTMFJPie/oqN97phPBOmA5IBluPnOW0mzE DiEz3tA37zMlTWRGO4cvy/+wlxFhgjsqDhqx+ABq492pYIACiqm4yfuoowo3Fmcb EpSeIBt9HJwsdZcreqOqX9BcECwpyYfWoddkWT2GkszNFGej90N0PoiIVpk2TuGl rLQTla8MKdjOgAQEHkSij8DdyRCkkMmePZ7PLoy4mI//rbPq+tZrNTe0RI58dYwx iOrdo/JGMePDMsOjAM6owXldJZGG7pkRGcQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:content-type:date:date:feedback-id:feedback-id :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:sender:subject:subject:to:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm3; t= 1682529139; x=1682615539; bh=aurwaydDGaum+zqDRPRdYZnO6K3TaoNyBid J7uVQdCY=; b=H1zzfMld2HxXdUHYM6kebIDbn226d7irAvi8G5H9KyiusHGtHCl qu903tlvdg0vK+DsXEVOVUex9VYkGXO24RNjTELZ78Z3Op0F7q9iHqIuy590sRQ/ msfrPF6BYirUROXzRwAPo4hXsusoQg3yvInyr1ShBUYECrc+IEThSOz3W6E77Da1 v53dX3D+rLlcP6Avc85umAc9zVQ3GBfvtYuFv4g3Dgs0J08p4Eky8WA6J6I3AVFB tKauoDTjkBAqqwkbXKV/qZamoEZCH+8dXXp6Oy3odBD4ipCqrVbVlUdkGtBsYwlQ UPPcb5lHVzARak2u9mhAkaTA9LFDWzvMA9g== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvhedrfedugedguddtlecutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfgh necuuegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmd enucfjughrpefkffggfgfuvfevfhfhjggtgfesthekredttdefjeenucfhrhhomheptfgv sggvtggtrgcuvehrrghnuceorhgvsggvtggtrgessghsughiohdrtghomheqnecuggftrf grthhtvghrnhepgeefgeevudehjefgudfffefhheelheeigffhtedvjedvlefhueekhffh vdegheevnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomh eprhgvsggvtggtrgessghsughiohdrtghomh X-ME-Proxy: Feedback-ID: i5b994698:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Wed, 26 Apr 2023 13:12:16 -0400 (EDT) Message-ID: <88212bf3-7f76-7436-51eb-bcc37013e900@bsdio.com> Date: Wed, 26 Apr 2023 11:12:15 -0600 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0 Subject: Re: [edk2-devel] [PATCH v5 00/13] BaseTools,CryptoPkg,EmulatorPkg,MdePkg,others: Delete CLANG35,CLANG38,VS2008-2013,EBC, deprecate GCC48,GCC49,GCC5, add GCC and GCCNOLTO, update CLANGDWARF To: "Kinney, Michael D" , "devel@edk2.groups.io" , "Gao, Liming" , "Feng, Bob C" , "Chen, Christine" , Michael Kubacki , Sean Brogan , "Chiu, Chasel" , "Desimone, Nathaniel L" , "Zeng, Star" , Andrew Fish Cc: "Ni, Ray" , Leif Lindholm , "Liu, Zhiguang" , "Wang, Jian J" , "Lu, Xiaoyu1" , "Jiang, Guomin" , "Guo, Gua" , Ard Biesheuvel , Pedro Falcato , Gerd Hoffmann , =?UTF-8?Q?Marvin_H=c3=a4user?= References: <20230421171508.73336-1-rebecca@bsdio.com> <5d45df06-6ccf-4cdb-73d0-29faf518f0bc@bsdio.com> <9ffb35fb-6d2f-5348-befd-4e8a1e861d51@bsdio.com> From: "Rebecca Cran" In-Reply-To: Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit I've not been able to replicate either of those problems, so it sounds like I'll need to leave this until you can figure it out. -- Rebecca Cran On 4/26/23 09:09, Kinney, Michael D wrote: > Yes. > > Actually 2 issues. Working on ways to reproduce both in edk2 repo. > > One is in build.py that is generating a python traceback when a tool chain tag is used that will be retired in the future. > > The 2nd one is in CLANGDWARF changes. Main break is that one of the PATHs used to map to echo and how it is mapped to a path objcopy-llvm. There are other differences and need to determine if they are on purpose or not. > > Mike > >> -----Original Message----- >> From: Rebecca Cran >> Sent: Wednesday, April 26, 2023 6:28 AM >> To: devel@edk2.groups.io; Kinney, Michael D ; Gao, Liming ; Feng, Bob C >> ; Chen, Christine ; Michael Kubacki ; Sean Brogan >> ; Chiu, Chasel ; Desimone, Nathaniel L ; Zeng, >> Star ; Andrew Fish >> Cc: Ni, Ray ; Leif Lindholm ; Liu, Zhiguang ; Wang, Jian J >> ; Lu, Xiaoyu1 ; Jiang, Guomin ; Guo, Gua >> ; Ard Biesheuvel ; Pedro Falcato ; Gerd Hoffmann >> ; Marvin Häuser >> Subject: Re: [edk2-devel] [PATCH v5 00/13] BaseTools,CryptoPkg,EmulatorPkg,MdePkg,others: Delete CLANG35,CLANG38,VS2008- >> 2013,EBC, deprecate GCC48,GCC49,GCC5, add GCC and GCCNOLTO, update CLANGDWARF >> >> Do you have an update? >> >> >> -- >> >> Rebecca Cran >> >> >> On 4/25/23 09:26, Michael D Kinney wrote: >>> Made some progress on root cause. Related to rename of CLANG38 to CLANGDWARF definitions for IA32. >>> >>> Wil give an update in a few hours after some more testing. >>> >>> Mike >>> >>>> -----Original Message----- >>>> From: devel@edk2.groups.io On Behalf Of Rebecca Cran >>>> Sent: Tuesday, April 25, 2023 6:10 AM >>>> To: devel@edk2.groups.io; Kinney, Michael D ; Gao, Liming ; Feng, Bob C >>>> ; Chen, Christine ; Michael Kubacki ; Sean Brogan >>>> ; Chiu, Chasel ; Desimone, Nathaniel L ; >> Zeng, >>>> Star ; Andrew Fish >>>> Cc: Ni, Ray ; Leif Lindholm ; Liu, Zhiguang ; Wang, Jian >> J >>>> ; Lu, Xiaoyu1 ; Jiang, Guomin ; Guo, Gua >>>> ; Ard Biesheuvel ; Pedro Falcato ; Gerd Hoffmann >>>> ; Marvin Häuser >>>> Subject: Re: [edk2-devel] [PATCH v5 00/13] BaseTools,CryptoPkg,EmulatorPkg,MdePkg,others: Delete CLANG35,CLANG38,VS2008- >>>> 2013,EBC, deprecate GCC48,GCC49,GCC5, add GCC and GCCNOLTO, update CLANGDWARF >>>> >>>> I've merged the OBJCOPY fix. My understanding is that you're seeing a >>>> problem on Windows with an internal build: if you could share part of >>>> the log file with filenames etc. changed to protect proprietary >>>> information that could be useful so we can try and find where the >>>> program is trying to be run that needs quotes around it. >>>> >>>> >>>> -- >>>> >>>> Rebecca Cran >>>> >>>> >>>> On 4/24/23 00:14, Michael D Kinney wrote: >>>>> Hi Rebecca, >>>>> >>>>> I have seen some issues with CLANGDWARF and need a little time to investigate further. >>>>> >>>>> Mike >>>>> >>>>>> -----Original Message----- >>>>>> From: Rebecca Cran >>>>>> Sent: Friday, April 21, 2023 10:15 AM >>>>>> To: devel@edk2.groups.io; Gao, Liming ; Feng, Bob C ; Chen, Christine >>>>>> ; Kinney, Michael D ; Michael Kubacki ; >> Sean >>>>>> Brogan ; Chiu, Chasel ; Desimone, Nathaniel L >>>>>> ; Zeng, Star ; Andrew Fish >>>>>> Cc: Rebecca Cran ; Ni, Ray ; Leif Lindholm ; Liu, Zhiguang >>>>>> ; Wang, Jian J ; Lu, Xiaoyu1 ; Jiang, Guomin >>>>>> ; Guo, Gua ; Ard Biesheuvel ; Pedro Falcato >>>>>> ; Gerd Hoffmann ; Marvin Häuser >>>>>> Subject: [PATCH v5 00/13] BaseTools,CryptoPkg,EmulatorPkg,MdePkg,others: Delete CLANG35,CLANG38,VS2008-2013,EBC, deprecate >>>>>> GCC48,GCC49,GCC5, add GCC and GCCNOLTO, update CLANGDWARF >>>>>> >>>>>> Update the toolchain definitions: >>>>>> >>>>>> - Delete the CLANG35 and CLANG38 toolchains, and replace CLANG38 with >>>>>> CLANGDWARF, updating it to support ARM and AARCH64 in addition to X64 >>>>>> and IA32. >>>>>> >>>>>> - Mark GCC48, GCC49 and GCC5 as deprecated. >>>>>> >>>>>> - Add GCC and GCCNOLTO toolchain definitions. >>>>>> >>>>>> - Remove VS2008, VS2010, VS2012 and VS2013. >>>>>> >>>>>> - Remove EBC compiler definitions. Full removal of EBC support from the >>>>>> various packages etc. will be done in a follow-up patch series. >>>>>> >>>>>> - Remove unused IPHONE_TOOLS and SOURCERY_CYGWIN_TOOLS definitions. >>>>>> >>>>>> - Remove unused CYGWIN_ definitions. >>>>>> >>>>>> Personal GitHub PR: https://github.com/tianocore/edk2/pull/4240 >>>>>> GitHub branch: https://github.com/bcran/edk2/tree/clangdwarf >>>>>> >>>>>> Note: CI is now passing. >>>>>> >>>>>> Changes from v1 to v2 >>>>>> ===================== >>>>>> >>>>>> - Added a commit to delete GCC48 and GCC49, rename GCC5 to GCC and >>>>>> update the flags for other toolchains to work with the new GCC >>>>>> definitions. >>>>>> >>>>>> - Bumped VERSION from 2.00 to 3.00 to inform users that they should >>>>>> update their Conf/tools_def.txt. >>>>>> >>>>>> Changes from v2 to v3 >>>>>> ===================== >>>>>> >>>>>> - Keep GCC48, GCC49 and GCC5 but mark them deprecated, including with >>>>>> warnings at the start and end of running a build. >>>>>> >>>>>> - Dropped the commit fixing the IA32 build of UefiPayloadPkg, because it >>>>>> causes CI to fail due to building both IA32 and X64 at the same time: >>>>>> >>>>>> "Module built under multiple ARCHs [IA32, X64]. Not able to determine which output to put into flash for Module..." >>>>>> >>>>>> - Added more information to the VERSION section in tools_def.template. >>>>>> >>>>>> - Various fixes to the CLANGDWARF flags. >>>>>> >>>>>> Changes from v3 to v4 >>>>>> ===================== >>>>>> >>>>>> - Use lld for -a ARM -t CLANGDWARF. >>>>>> >>>>>> Changes from v4 to v5 >>>>>> ===================== >>>>>> >>>>>> - Fixed CLANGDWARF on Windows when llvm is installed with the llvm.org >>>>>> installer and isn't installed in Visual Studio: use llvm-objcopy and the >>>>>> tools prefix. >>>>>> >>>>>> - Added flags for GCCNOLTO to UnitTestFrameworkPkgHost.dsc.inc >>>>>> >>>>>> - Added GCC flags to CryptoPkg/Library/OpensslLib/*.inf >>>>>> >>>>>> Rebecca Cran (13): >>>>>> BaseTools,CryptoPkg: Update CLANGDWARF, remove CLANG 35/38 toolchains >>>>>> BaseTools: Remove VS2008, 2010, 2012 and 2013 toolchain definitions >>>>>> BaseTools: Remove VS2008-VS2013 remnants >>>>>> MdePkg: Remove VS2008-VS2013 remnants >>>>>> edksetup.bat: Remove VS2008-VS2013 remnants >>>>>> BaseTools: Remove unused IPHONE_TOOLS and SOURCERY_CYGWIN_TOOLS defs >>>>>> BaseTools: Remove EBC (EFI Byte Code) compiler definitions >>>>>> BaseTools: Update VS toolchain descriptions in tools_def.txt.template >>>>>> BaseTools/Conf/tools_def.template: Add GCC and GCCNOLTO toolchains >>>>>> BaseTools: Only call LoadConfiguration once in build.py >>>>>> BaseTools: Add a deprecation warning for GCC48,GCC49,GCC5 to build.py >>>>>> BaseTools/Conf/tools_def.template: Add section for deprecated >>>>>> toolchains >>>>>> BaseTools/Conf/tools_def.template: Bump VERSION to 3.00 >>>>>> >>>>>> UnitTestFrameworkPkg/UnitTestFrameworkPkgHost.dsc.inc | 4 ++++ >>>>>> IntelFsp2Pkg/Tools/Tests/QemuFspPkg.dsc | 9 ++++++++- >>>>>> CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf | 3 +-- >>>>>> CryptoPkg/Library/BaseCryptLib/PeiCryptLib.inf | 3 +-- >>>>>> CryptoPkg/Library/BaseCryptLib/RuntimeCryptLib.inf | 3 +-- >>>>>> CryptoPkg/Library/BaseCryptLib/SecCryptLib.inf | 3 +-- >>>>>> CryptoPkg/Library/BaseCryptLib/SmmCryptLib.inf | 3 +-- >>>>>> CryptoPkg/Library/BaseCryptLib/UnitTestHostBaseCryptLib.inf | 3 +-- >>>>>> CryptoPkg/Library/OpensslLib/OpensslLib.inf | 4 ++-- >>>>>> CryptoPkg/Library/OpensslLib/OpensslLibAccel.inf | 4 ++-- >>>>>> CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf | 4 ++-- >>>>>> CryptoPkg/Library/OpensslLib/OpensslLibFull.inf | 4 ++-- >>>>>> CryptoPkg/Library/OpensslLib/OpensslLibFullAccel.inf | 5 +++-- >>>>>> EmulatorPkg/Unix/Host/Host.inf | 2 ++ >>>>>> MdePkg/Include/Ia32/ProcessorBind.h | 8 ++------ >>>>>> MdePkg/Include/X64/ProcessorBind.h | 8 ++------ >>>>>> BaseTools/Conf/tools_def.template | 1993 >>>>>> >> ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ >> ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ >> ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ >> ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ >>>>>> ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++----------------------------------------------------- >> -- >>>> --- >>>>>> --------------------------------------------------------------------------------------------------------------------------- >> -- >>>> --- >>>>>> --------------------------------------------------------------------------------------------------------------------------- >> -- >>>> --- >>>>>> --------------------------------------------------------------------------------------------- >>>>>> --------------------------------------------------------------------------------------------------------------------------- >> -- >>>> --- >>>>>> --------------------------------------------------------------------------------------------------------------------------- >> -- >>>> --- >>>>>> --------------------------------------------------------------------------------------------------------------------------- >> -- >>>> --- >>>>>> --------------------------------------------------------------------------------------------------------------------------- >> -- >>>> --- >>>>>> --------------------------------------------------------------------------------------------------------------------------- >> -- >>>> --- >>>>>> --------------------------------------------------------------------------------------------------------------------------- >> -- >>>> --- >>>>>> --------------------------------------------------------------------------------------------------------------------------- >> -- >>>> --- >>>>>> --------------------------------------------------------------------------------------------- >>>>>> --------------- >>>>>> BaseTools/Scripts/SetVisualStudio.bat | 22 +--------------------- >>>>>> BaseTools/Scripts/ShowEnvironment.bat | 44 -------------------------------------------- >>>>>> BaseTools/Source/Python/build/build.py | 18 +++++++++++++++--- >>>>>> BaseTools/get_vsvars.bat | 13 ------------- >>>>>> BaseTools/set_vsprefix_envs.bat | 64 ----------------------------------------------------- >> --- >>>> ---- >>>>>> ---- >>>>>> BaseTools/toolsetup.bat | 24 +----------------------- >>>>>> edksetup.bat | 6 +----- >>>>>> 24 files changed, 635 insertions(+), 1619 deletions(-) >>>>>> >>>>>> -- >>>>>> 2.39.2 >>>>> >>>>> >>>>> >>>> >>>> >>> >>> >>> >>>