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.web10.5180.1639622813881556044 for ; Wed, 15 Dec 2021 18:46:54 -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, 16 Dec 2021 10:46:46 +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: <20211208102701.10815-1-pedro.falcato@gmail.com> <01f701d7f175$aa37a4c0$fea6ee40$@byosoft.com.cn> In-Reply-To: Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gW1BBVENIIHYyIDAvMl0gVXBkYXRlIGJyb3RsaSB0byB0aGUgbGF0ZXN0IGNvbW1pdA==?= Date: Thu, 16 Dec 2021 10:46:52 +0800 Message-ID: <011e01d7f227$2e365d00$8aa31700$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQGV3yuTTaPwzRpLMKHCXK8ZucPq/QHB8zxlAM+JfgWspAQAUA== Content-Type: multipart/alternative; boundary="----=_NextPart_000_011F_01D7F26A.3C5A6050" Content-Language: zh-cn ------=_NextPart_000_011F_01D7F26A.3C5A6050 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable That=E2=80=99s great. Reviewed-by: Liming Gao =20 =E5=8F=91=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io = =E4=BB=A3=E8=A1=A8 Pedro Falcato =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2021=E5=B9=B412=E6=9C=8815=E6=97=A5 1= 8:18 =E6=94=B6=E4=BB=B6=E4=BA=BA: edk2-devel-groups-io ; L= iming Gao =E4=B8=BB=E9=A2=98: Re: [edk2-devel] [PATCH v2 0/2] Update brotli to the la= test commit =20 Liming, =20 Yes, yes it does. No need to turn off Werror now. =20 Thanks,=20 Pedro =20 On Wed, 15 Dec 2021, 05:36 gaoliming, > wrote: Pedro: Does Brotli fix GCC warning in the latest version?=20 Thanks Liming > -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6----- > =E5=8F=91=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io > =E4=BB=A3=E8= =A1=A8 Pedro Falcato > =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2021=E5=B9=B412=E6=9C=888=E6=97=A5 = 18:27 > =E6=94=B6=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io =20 > =E4=B8=BB=E9=A2=98: [edk2-devel] [PATCH v2 0/2] Update brotli to the late= st commit >=20 > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3D3D3417 >=20 > This patch set updates the brotli submodule to the latest commit, which > fixes a warning that triggers build failures for both BaseTools and > MdeModulePkg/BrotliCustomDecompressLib in GCC 11 compilers. >=20 > Pedro Falcato (2): > BaseTools: Update brotli to the latest upstream commit > MdeModulePkg/BrotliCustomDecompressLib: Update brotli >=20 > BaseTools/Source/C/BrotliCompress/GNUmakefile > | 7 +++++++ > BaseTools/Source/C/BrotliCompress/Makefile > | 12 +++++++++++- > BaseTools/Source/C/BrotliCompress/brotli > | 2 +- >=20 > MdeModulePkg/Library/BrotliCustomDecompressLib/BrotliCustomDecompre > ssLib.inf | 3 +++ > MdeModulePkg/Library/BrotliCustomDecompressLib/brotli > | 2 +- > 5 files changed, 23 insertions(+), 3 deletions(-) >=20 > -- > 2.34.1 >=20 >=20 >=20 >=20 >=20 ------=_NextPart_000_011F_01D7F26A.3C5A6050 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable

That=E2=80=99s great. Reviewed-by: Liming Gao <gaolimin= g@byosoft.com.cn>

&nb= sp;

=E5=8F=91=E4=BB=B6=E4= =BA=BA: devel@edk2.groups.io <dev= el@edk2.groups.io> =E4=BB=A3=E8=A1=A8 Pedro Falcato
=E5=8F= =91=E9=80=81=E6=97=B6=E9=97=B4: 2021= =E5= =B9=B412=E6=9C=8815=E6= =97=A5 18:18
=E6=94=B6=E4=BB=B6=E4=BA=BA: edk2-devel-groups-io <de= vel@edk2.groups.io>; Liming Gao <gaoliming@byosoft.com.cn>
=E4=B8=BB=E9=A2=98: = Re: [edk2-devel] [PATCH v2 0/2] Update brotli to the latest commit

Limin= g,

&= nbsp;

Ye= s, yes it does. No need to turn off Werror now.

=

Thanks,

Pedro

=

On Wed, = 15 Dec 2021, 05:36 gaoliming, <gaoliming@byosoft.com.cn> wrote:

Pedro:
Does Brotli f= ix GCC warning in the latest version?

Thanks
Liming
> ----= -=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6-----
>= ; =E5=8F=91=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io <devel@edk2.groups.io> =E4=BB=A3=E8=A1=A8 Pedro Falcato
> =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2021= =E5=B9=B412=E6=9C=888= =E6=97=A5 18:27
> =E6=94=B6=E4=BB=B6=E4=BA= =BA: devel@edk2.groups.io
> =E4=B8=BB=E9=A2=98: [edk2-devel] [PATCH v2 0/2] Update brotli to the latest commit>
> BZ: https://bugzilla.tianocore.org/show_bug.cgi?= id=3D3D3417
>
> This patch set updates the brotli submodul= e to the latest commit, which
> fixes a warning that triggers build f= ailures for both BaseTools and
> MdeModulePkg/BrotliCustomDecompressL= ib in GCC 11 compilers.
>
> Pedro Falcato (2):
> &= nbsp;BaseTools: Update brotli to the latest upstream commit
> &= nbsp;MdeModulePkg/BrotliCustomDecompressLib: Update brotli
>
>= BaseTools/Source/C/BrotliCompress/GNUmakefile
> | 7 ++++= +++
> BaseTools/Source/C/BrotliCompress/Makefile
> | 12 += ++++++++++-
> BaseTools/Source/C/BrotliCompress/brotli
> = | 2 +-
>
> MdeModulePkg/Library/BrotliCustomDecompressLi= b/BrotliCustomDecompre
> ssLib.inf | 3 +++
> MdeMod= ulePkg/Library/BrotliCustomDecompressLib/brotli
> | 2 +-
>= ; 5 files changed, 23 insertions(+), 3 deletions(-)
>
> = --
> 2.34.1
>
>
>
>
>

<= br>

------=_NextPart_000_011F_01D7F26A.3C5A6050--