From: "Oliver Smith-Denny" <osd@smith-denny.com>
To: devel@edk2.groups.io, rebecca@bsdio.com,
Michael D Kinney <michael.d.kinney@intel.com>,
Liming Gao <gaoliming@byosoft.com.cn>,
Zhiguang Liu <zhiguang.liu@intel.com>,
Bob Feng <bob.c.feng@intel.com>,
Yuwei Chen <yuwei.chen@intel.com>, Andrew Fish <afish@apple.com>,
Leif Lindholm <quic_llindhol@quicinc.com>,
Ard Biesheuvel <ardb+tianocore@kernel.org>,
Jordan Justen <jordan.l.justen@intel.com>,
Gerd Hoffmann <kraxel@redhat.com>
Subject: Re: [edk2-devel] [PATCH 0/9] BaseTools,CryptoPkg,MdePkg,OvmfPkg: Delete CLANG35,CLANG38 and update CLANGDWARF, delete VS 2008-2013, EBC
Date: Wed, 22 Mar 2023 09:37:09 -0700 [thread overview]
Message-ID: <fadea40b-edd8-9f18-a01e-7d85fe9c6cd8@smith-denny.com> (raw)
In-Reply-To: <20230322013040.3322865-1-rebecca@bsdio.com>
Thanks for the cleanup!
Reviewed-by: Oliver Smith-Denny <osd@smith-denny.com>
On 3/21/2023 6:30 PM, Rebecca Cran wrote:
> 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.
>
> - 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.
>
> Personal GitHub PR: https://github.com/tianocore/edk2/pull/4158
> GitHub branch: https://github.com/bcran/edk2/tree/clangdwarf
>
> Rebecca Cran (9):
> OvmfPkg: Replace static struct initialization with ZeroMem call
> CryptoPkg: Add CLANGDWARF and remove CLANG35 and CLANG38 compiler
> flags
> BaseTools: Update CLANGDWARF toolchain and remove CLANG35 and CLANG38
> 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
>
> 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 | 3 +-
> CryptoPkg/Library/OpensslLib/OpensslLibAccel.inf | 3 +-
> CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf | 3 +-
> CryptoPkg/Library/OpensslLib/OpensslLibFull.inf | 3 +-
> CryptoPkg/Library/OpensslLib/OpensslLibFullAccel.inf | 3 +-
> MdePkg/Include/Ia32/ProcessorBind.h | 8 +-
> MdePkg/Include/X64/ProcessorBind.h | 8 +-
> OvmfPkg/Library/PlatformInitLib/MemDetect.c | 4 +-
> BaseTools/Conf/tools_def.template | 1572 +++-----------------
> BaseTools/Scripts/SetVisualStudio.bat | 22 +-
> BaseTools/Scripts/ShowEnvironment.bat | 44 -
> BaseTools/get_vsvars.bat | 13 -
> BaseTools/set_vsprefix_envs.bat | 64 -
> BaseTools/toolsetup.bat | 24 +-
> edksetup.bat | 6 +-
> 21 files changed, 192 insertions(+), 1606 deletions(-)
>
next prev parent reply other threads:[~2023-03-22 16:37 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-03-22 1:30 [PATCH 0/9] BaseTools,CryptoPkg,MdePkg,OvmfPkg: Delete CLANG35,CLANG38 and update CLANGDWARF, delete VS 2008-2013, EBC Rebecca Cran
2023-03-22 1:30 ` [PATCH 1/9] OvmfPkg: Replace static struct initialization with ZeroMem call Rebecca Cran
2023-03-22 5:55 ` Gerd Hoffmann
2023-03-22 1:30 ` [PATCH 2/9] CryptoPkg: Add CLANGDWARF and remove CLANG35 and CLANG38 compiler flags Rebecca Cran
2023-03-22 1:30 ` [PATCH 3/9] BaseTools: Update CLANGDWARF toolchain and remove CLANG35 and CLANG38 Rebecca Cran
2023-03-22 4:57 ` [edk2-devel] " Guo, Gua
2023-03-22 13:10 ` Rebecca Cran
2023-03-22 22:07 ` Guo, Gua
2023-03-22 22:23 ` Rebecca Cran
2023-03-22 22:46 ` Guo, Gua
2023-03-27 22:15 ` Rebecca Cran
2023-03-22 11:49 ` Ard Biesheuvel
2023-03-22 12:28 ` [edk2-devel] " Rebecca Cran
2023-03-22 12:32 ` Ard Biesheuvel
2023-03-22 13:03 ` Gerd Hoffmann
2023-03-22 13:05 ` Ard Biesheuvel
2023-03-23 1:30 ` Rebecca Cran
2023-03-23 9:04 ` Ard Biesheuvel
2023-03-23 13:14 ` Ard Biesheuvel
2023-03-22 1:30 ` [PATCH 4/9] BaseTools: Remove VS2008, 2010, 2012 and 2013 toolchain definitions Rebecca Cran
2023-03-22 1:30 ` [PATCH 5/9] BaseTools: Remove VS2008-VS2013 remnants Rebecca Cran
2023-03-22 1:30 ` [PATCH 6/9] MdePkg: " Rebecca Cran
2023-03-22 1:30 ` [PATCH 7/9] edksetup.bat: " Rebecca Cran
2023-03-22 1:30 ` [PATCH 8/9] BaseTools: Remove unused IPHONE_TOOLS and SOURCERY_CYGWIN_TOOLS defs Rebecca Cran
2023-03-22 1:30 ` [PATCH 9/9] BaseTools: Remove EBC (EFI Byte Code) compiler definitions Rebecca Cran
2023-03-22 9:04 ` [PATCH 0/9] BaseTools,CryptoPkg,MdePkg,OvmfPkg: Delete CLANG35,CLANG38 and update CLANGDWARF, delete VS 2008-2013, EBC Ard Biesheuvel
2023-03-22 16:37 ` Oliver Smith-Denny [this message]
2023-03-23 1:05 ` 回复: [edk2-devel] " gaoliming
2023-03-23 1:10 ` Rebecca Cran
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=fadea40b-edd8-9f18-a01e-7d85fe9c6cd8@smith-denny.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox