Re: 回复: 回复: [edk2-devel] [PATCH v2 00/13] BaseTools,CryptoPkg,MdePkg,OvmfPkg: Delete CLANG35,CLANG38,GCC48,GCC49, rename GCC5 to GCC, update CLANGDWARF, delete VS 2008-2013, EBC

["Michael D Kinney" <michael.d.kinney@intel.com>]

Changing the toolchain tag name from GCC5 to GCC will break 
EDK II CI and likely 100's of downstream developer/CI scripts.

Can we keep GCC5 and add GCC that is identical to GCC5?

Mike

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Rebecca Cran
> Sent: Tuesday, April 4, 2023 10:01 AM
> To: Leif Lindholm <quic_llindhol@quicinc.com>; devel@edk2.groups.io; pedro.falcato@gmail.com
> Cc: Gao, Liming <gaoliming@byosoft.com.cn>; Oliver Smith-Denny <osd@smith-denny.com>; Jiang, Guomin
> <guomin.jiang@intel.com>; Lu, Xiaoyu1 <xiaoyu1.lu@intel.com>; Wang, Jian J <jian.j.wang@intel.com>; Yao, Jiewen
> <jiewen.yao@intel.com>; Ard Biesheuvel <ardb+tianocore@kernel.org>; Justen, Jordan L <jordan.l.justen@intel.com>; Gerd
> Hoffmann <kraxel@redhat.com>; Feng, Bob C <bob.c.feng@intel.com>; Andrew Fish <afish@apple.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>
> Subject: Re: 回复: 回复: [edk2-devel] [PATCH v2 00/13] BaseTools,CryptoPkg,MdePkg,OvmfPkg: Delete
> CLANG35,CLANG38,GCC48,GCC49, rename GCC5 to GCC, update CLANGDWARF, delete VS 2008-2013, EBC
> 
> 
> On 4/4/23 10:32 AM, Leif Lindholm wrote:
> > On Tue, Apr 04, 2023 at 16:57:18 +0100, Pedro Falcato wrote:
> >> On Tue, Apr 4, 2023 at 4:49 PM Rebecca Cran <rebecca@bsdio.com> wrote:
> >>> Okay. I'll rework the patch series to only delete GCC48 and keep GCC49
> >>> and GCC5.
> >> Please rename them? There's no point in keeping their current names
> >> anyway. I guarantee you that most people are not really able to guess
> >> GCC49 to be the "non-LTO" flavor.
> >>
> >> And IMO Ard has already proved the value in non-LTO builds anyway, so
> >> yeah. GCC and GCCNOLTO please.
> > Agreed. I don't think there are any outstanding items that need
> > resolving.
> >
> > I'd be very happy to stop hearing "you need GCC 5 in order to build
> > edk2" misunderstandings.
> 
> Thanks! As a new co-maintainer I was trying to be diplomatic, but I
> really don't see any reason to keep the existing names.
> 
> I'll send out a v3 patch series with all the fixes I and others have
> identified.
> 
> 
> --
> Rebecca Cran
> 
> 
> 
> 
>