From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242]) by mx.groups.io with SMTP id smtpd.web08.2226.1613619551065916748 for ; Wed, 17 Feb 2021 19:39:11 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=none, err=permanent DNS error (domain: byosoft.com.cn, ip: 58.240.74.242, mailfrom: gaoliming@byosoft.com.cn) Received: from DESKTOPS6D0PVI ([58.246.60.130]) (envelope-sender ) by 192.168.6.13 with ESMTP for ; Thu, 18 Feb 2021 11:39:04 +0800 X-WM-Sender: gaoliming@byosoft.com.cn X-Originating-IP: 58.246.60.130 X-WM-AuthFlag: YES X-WM-AuthUser: gaoliming@byosoft.com.cn From: "gaoliming" To: , , References: <20210212204926.1114-1-bret.barkelew@microsoft.com> <9dd900ea-5b9f-aa54-04b9-cddf4a0f3ede@redhat.com> In-Reply-To: <9dd900ea-5b9f-aa54-04b9-cddf4a0f3ede@redhat.com> Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gW1BBVENIIHYxIDEvMV0gQmFzZVRvb2xzOiBVcGRhdGUgdGhlIE5BU00gZGVwZW5kZW5jeQ==?= Date: Thu, 18 Feb 2021 11:39:06 +0800 Message-ID: <000f01d705a7$9c1c59e0$d4550da0$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQJAeFucT3w2Afa0rtyinI99RZKPxAFe/vLgqX9lkJA= Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn Laszlo: > -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6----- > =E5=8F=91=E4=BB=B6=E4=BA=BA: bounce+27952+71686+4905953+8761045@groups.i= o > =E4=BB=A3=E8=A1=A8 Laszlo= Ersek > =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2021=E5=B9=B42=E6=9C=8816=E6=97=A5= 5:00 > =E6=94=B6=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io; bret@corthon.com > =E4=B8=BB=E9=A2=98: Re: [edk2-devel] [PATCH v1 1/1] BaseTools: Update th= e NASM > dependency >=20 > Hi Bret, >=20 > On 02/12/21 21:49, Bret Barkelew wrote: > > This is primarily used by CI builds, but may also be used by platforms= . > > > > Update NASM to support newer structures and features, as agreed in rec= ent > > community meetings: > > https://edk2.groups.io/g/devel/message/71289 > > --- > > BaseTools/Bin/nasm_ext_dep.yaml | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/BaseTools/Bin/nasm_ext_dep.yaml > b/BaseTools/Bin/nasm_ext_dep.yaml > > index 60b1f71b56aa..56703d4c18e7 100644 > > --- a/BaseTools/Bin/nasm_ext_dep.yaml > > +++ b/BaseTools/Bin/nasm_ext_dep.yaml > > @@ -13,6 +13,6 @@ > > "type": "nuget", > > "name": "mu_nasm", > > "source": "https://api.nuget.org/v3/index.json", > > - "version": "2.14.02", > > + "version": "2.15.5", > > "flags": ["set_path", "host_specific"] > > } > > >=20 > (1) I'm confused by the "patchlevel" number being "5" and not "05", in > the "2.15.5" version. Per > , my understanding has > been that the latest release is "2.15.05". >=20 > (2) The minimum required version, according to > , is "2.15.01". In the > RHEL build environments that matter to me, I can satisfy a NASM > dependency up to 2.15.03, at this time, but not beyond. And this matters > because... >=20 I understand CI environment tries to use the latest tool version for verif= ication.=20 > (3) ... arguably, the NASM requirement in > "BaseTools/Conf/tools_def.template" should be updated too, in the same > patch: >=20 > # - NASM 2.10 or later for use with the GCC toolchain family > # - NASM 2.12.01 or later for use with all other toolchain families >=20 This is edk2 build environment requirement. It will be updated when the so= urce code change is added.=20 Thanks Liming > Thanks > Laszlo >=20 >=20 >=20 >=20 >=20