Re: [edk2-devel] [PATCH] BaseTools/Brotli: update to latest brotli submodule

2022-01-11 Thread Pedro Falcato
iming > Sent: Friday, December 24, 2021 9:42 AM > To: 'Ross Burton' ; Kinney, Michael D < > michael.d.kin...@intel.com>; 'Pedro Falcato' > Cc: devel@edk2.groups.io; Feng, Bob C ; Chen, > Christine > Subject: 回复: [edk2-devel] [PATCH] BaseTools/Brotli: up

Re: [edk2-devel] [PATCH] BaseTools/Brotli: update to latest brotli submodule

2022-01-11 Thread Bob Feng
7; ; Kinney, Michael D ; 'Pedro Falcato' Cc: devel@edk2.groups.io; Feng, Bob C ; Chen, Christine Subject: 回复: [edk2-devel] [PATCH] BaseTools/Brotli: update to latest brotli submodule Ross: I know the reason. Your patch is to update brotli version to the latest one e83c7b8e8fb8

回复: [edk2-devel] [PATCH] BaseTools/Brotli: update to latest brotli submodule

2021-12-23 Thread gaoliming
t; ; Chen, Christine > 主题: Re: [edk2-devel] [PATCH] BaseTools/Brotli: update to latest brotli > submodule > > My original PR has passed CI: https://github.com/tianocore/edk2/pull/2231 > > Ross > > On Thu, 23 Dec 2021 at 22:35, Kinney, Michael D > wrote: > > >

Re: [edk2-devel] [PATCH] BaseTools/Brotli: update to latest brotli submodule

2021-12-23 Thread Ross Burton
878035Z CRITICAL - Build failed > > Can you please resolve and update the PR. > > Thanks, > > Mike > > > -Original Message----- > > From: devel@edk2.groups.io On Behalf Of Ross Burton > > Sent: Thursday, December 23, 2021 2:02 AM > > To: Gao, Liming &g

Re: [edk2-devel] [PATCH] BaseTools/Brotli: update to latest brotli submodule

2021-12-23 Thread Michael D Kinney
PR. Thanks, Mike > -Original Message- > From: devel@edk2.groups.io On Behalf Of Ross Burton > Sent: Thursday, December 23, 2021 2:02 AM > To: Gao, Liming > Cc: Pedro Falcato ; edk2-devel-groups-io > ; Feng, Bob C ; > Chen, Christine > Subject: Re: [edk2-devel] [

Re: [edk2-devel] [PATCH] BaseTools/Brotli: update to latest brotli submodule

2021-12-23 Thread Ross Burton
On Thu, 23 Dec 2021 at 00:48, gaoliming wrote: > Create PR https://github.com/tianocore/edk2/pull/2346 for this patch. There are now *three* PRs on GitHub to solve this issue, which is a build breaker for anyone using a new GCC. Can someone please merge the fix? Ross -=-=-=-=-=-=-=-=-=-=-=- G

回复: [edk2-devel] [PATCH] BaseTools/Brotli: update to latest brotli submodule

2021-12-22 Thread gaoliming
Create PR https://github.com/tianocore/edk2/pull/2346 for this patch. Thanks Liming 发件人: Pedro Falcato 发送时间: 2021年12月21日 1:02 收件人: edk2-devel-groups-io ; r...@burtonini.com 抄送: Bob Feng ; Liming Gao ; Yuwei Chen 主题: Re: [edk2-devel] [PATCH] BaseTools/Brotli: update to latest brotli

Re: [edk2-devel] [PATCH] BaseTools/Brotli: update to latest brotli submodule

2021-12-20 Thread Pedro Falcato
Ross, This is a duplicate of https://edk2.groups.io/g/devel/message/84497, which has already been Rb'd and is just waiting for a maintainer to merge it. Best regards, Pedro On Mon, Dec 20, 2021 at 4:55 PM Ross Burton wrote: > Update to the latest Brotli commit, and extend the makefiles as need

[edk2-devel] [PATCH] BaseTools/Brotli: update to latest brotli submodule

2021-12-20 Thread Ross Burton
Update to the latest Brotli commit, and extend the makefiles as needed. Specifically, this is needed for 0a3944 in brotli, to fix VLA parameter warnings with new compilers. Signed-off-by: Ross Burton Change-Id: I36d28cadb9bf2d9e01ac0341e69509fa1b8d6969 --- BaseTools/Source/C/BrotliCompress/GNUm