From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web12.58258.1598894061821948464 for ; Mon, 31 Aug 2020 10:14:22 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: ard.biesheuvel@arm.com) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 0B94230E; Mon, 31 Aug 2020 10:14:21 -0700 (PDT) Received: from [192.168.1.205] (unknown [10.37.8.74]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 22B053F66F; Mon, 31 Aug 2020 10:14:16 -0700 (PDT) Subject: Re: [edk2-devel] [PATCH V2 2/2] BaseTools: Factorize GCC flags To: Laszlo Ersek , Leif Lindholm Cc: Pierre Gondois , "devel@edk2.groups.io" , "bob.c.feng@intel.com" , "liming.gao@intel.com" , Tomas Pilar , nd References: <20200707083522.138944-1-pierre.gondois@arm.com> <20200707083522.138944-3-pierre.gondois@arm.com> <879fda8a-37bd-a902-6028-c879ed37fa28@redhat.com> <22b94ad5-db03-7280-4032-6ebf8dfc1d49@redhat.com> <518916e0-53cc-732b-cf1b-1e1b8d10a3b3@redhat.com> <20200827152511.GX1191@vanye> <42cb48da-b932-7006-475b-d5259bcd0d8a@redhat.com> <20200828191515.GC1191@vanye> <75feab5d-c7c9-972f-4201-0dd6c4280569@arm.com> <0420cc47-ed41-54b9-b5cf-cf722c40ebfe@redhat.com> From: "Ard Biesheuvel" Message-ID: Date: Mon, 31 Aug 2020 20:14:13 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit On 8/31/20 6:27 PM, Laszlo Ersek wrote: > On 08/31/20 18:18, Laszlo Ersek wrote: > >> Obviously the following form would be preferred: >> >> scl enable example -- less --version >> >> Because this form would not require quoting, just prefixing. > > I need to slow down when reading manual pages. Now that I've actually > *tried* the above, it works! And upon re-reading the manual, it's even > *documented*; just not in the examples section. > > Where's my brown paper bag again... > :-) Happy to hear that you managed, and that you are on board with deprecating GCC48. Given our successful efforts to make the GCC5 profile work with all GCC versions since the 5.x series, I think we should consider dropping GCC48 *and* GCC49, as well as CLANG35 (which only supports ARM anyway, and is superseded by CLANG38 which should also support any Clang version released since) To Liming's point, renaming the profiles may have additional impact which we are not prepared to deal with, so that could be a separate matter. But removing GCC48, GCC49 and CLANG35 reduces the CI effort, and allows us to focus our support on profiles that are actually in wide use.