I also see many references to VS2015, VS2017, and GCC5 in the edk2 repo outside of toolsdef.
https://github.com/search?q=repo%3Atianocore%2Fedk2+VS2015&type=code https://github.com/search?q=repo%3Atianocore%2Fedk2+VS2017&type=code https://github.com/search?q=repo%3Atianocore%2Fedk2+GCC5&type=code These should be addressed in an independent set of PRs before removing the toolchains from toolsdef. The UefiPayloadPkg comments suggest you should discuss with those Maintainers to make sure the validation has moved to newer tool chains. Similar searches should be performed on all other TianoCore repos and updated as needed. Mike > -----Original Message----- > From: Kinney, Michael D <[email protected]> > Sent: Friday, November 22, 2024 4:02 PM > To: [email protected]; [email protected]; Rebecca Cran > <[email protected]>; [email protected] > Cc: Kinney, Michael D <[email protected]> > Subject: RE: [edk2-devel] Removal of VS2015 (and maybe VS2017) and > deprecated toolchains (GCC48, GCC49, GCC5) > > VS1015, VS2017, GCC5 > > Mike > > > > -----Original Message----- > > From: [email protected] <[email protected]> On Behalf Of Rebecca > > Cran via groups.io > > Sent: Friday, November 22, 2024 2:30 PM > > To: [email protected]; Kinney, Michael D > > <[email protected]>; Rebecca Cran <[email protected]>; > > [email protected] > > Subject: Re: [edk2-devel] Removal of VS2015 (and maybe VS2017) and > > deprecated toolchains (GCC48, GCC49, GCC5) > > > > All of them including GCC 4.8 and 4.9? > > > > > > Rebecca > > > > > > On 11/22/2024 2:08 PM, Michael D Kinney via groups.io wrote: > > > Hi Rebecca, > > > > > > Yes. I have collected feedback. I have found some use of these > > toolchains. > > > > > > I do not have a date when the move to newer toolchains will be > > completed. > > > > > > Mike > > > > > >> -----Original Message----- > > >> From: Rebecca Cran <[email protected]> > > >> Sent: Thursday, November 21, 2024 9:05 PM > > >> To: Kinney, Michael D <[email protected]>; > > >> [email protected]; [email protected] > > >> Subject: Re: [edk2-devel] Removal of VS2015 (and maybe VS2017) and > > >> deprecated toolchains (GCC48, GCC49, GCC5) > > >> > > >> Hi Mike, > > >> > > >> > > >> I was wondering if you'd been able to collect any feedback or > review > > the > > >> proposal? > > >> > > >> > > >> Rebecca > > >> > > >> > > >> On 10/23/24 9:23 PM, Kinney, Michael D wrote: > > >>> Hi, > > >>> > > >>> I will need more time to review this impacts of this proposal. > > >>> > > >>> Can we extend the timeline for comments to mid November. > > >>> > > >>> Thanks, > > >>> > > >>> Mike > > >>> > > >>>> -----Original Message----- > > >>>> From: [email protected] <[email protected]> On Behalf Of > Leif > > >>>> Lindholm via groups.io > > >>>> Sent: Tuesday, October 22, 2024 5:15 AM > > >>>> To: [email protected]; [email protected] > > >>>> Subject: Re: [edk2-devel] Removal of VS2015 (and maybe VS2017) > and > > >>>> deprecated toolchains (GCC48, GCC49, GCC5) > > >>>> > > >>>> On Tue, 22 Oct 2024 at 12:19, Rebecca Cran <[email protected]> > > wrote: > > >>>>> In addition, since GCC48, GCC49 and GCC5 have been marked > > deprecated > > >>>> for > > >>>>> over a year, I'd like to proceed with removing them as well. > > >>>>> > > >>>>> Please provide feedback by Thursday October 31st. In the absence > > of > > >>>>> feedback I'll plan to proceed with removing VS2015, GCC48, GCC49 > > and > > >>>> GCC5. > > >>>> > > >>>> No comment on the VS ones, but for the GCC ones, I'm all for it. > > >>>> > > >>>> We might also want to raise an issue on deleting workarounds put > > into > > >>>> place for ancient toolchains. > > >>>> For gcc48/49 that was redundant initialization due to broken > > >>>> use-uninitialized detection. > > >>>> e.g. > > >>>> > > >> > > > https://github.com/tianocore/edk2/blob/master/OvmfPkg/Library/X86QemuLoa > > >>>> dImageLib/X86QemuLoadImageLib.c#L346 > > >>>> > > >>>> / > > >>>> Leif > > >>>> > > >>>> > > >>>> > > >>>> > > > > > > > > > > > > > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#120825): https://edk2.groups.io/g/devel/message/120825 Mute This Topic: https://groups.io/mt/109150018/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
