* [PATCH 1/1] BaseTools: Update brotli to the latest upstream commit @ 2021-10-29 2:22 Pedro Falcato 2021-12-01 0:38 ` Pedro Falcato 0 siblings, 1 reply; 6+ messages in thread From: Pedro Falcato @ 2021-10-29 2:22 UTC (permalink / raw) To: devel; +Cc: Bob Feng, Liming Gao, Yuwei Chen BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3417 This updates BaseTools's brotli submodule to the latest upstream, which fixes GCC 11's issue building BaseTools. Cc: Bob Feng <bob.c.feng@intel.com> Cc: Liming Gao <gaoliming@byosoft.com.cn> Cc: Yuwei Chen <yuwei.chen@intel.com> Signed-off-by: Pedro Falcato <pedro.falcato@gmail.com> --- BaseTools/Source/C/BrotliCompress/GNUmakefile | 7 +++++++ BaseTools/Source/C/BrotliCompress/Makefile | 12 +++++++++++- BaseTools/Source/C/BrotliCompress/brotli | 2 +- 3 files changed, 19 insertions(+), 2 deletions(-) diff --git a/BaseTools/Source/C/BrotliCompress/GNUmakefile b/BaseTools/Source/C/BrotliCompress/GNUmakefile index b150e5dd2bb9..79d7c405069d 100644 --- a/BaseTools/Source/C/BrotliCompress/GNUmakefile +++ b/BaseTools/Source/C/BrotliCompress/GNUmakefile @@ -10,7 +10,11 @@ APPNAME = BrotliCompress OBJECTS = \ BrotliCompress.o \ + brotli/c/common/constants.o \ + brotli/c/common/context.o \ brotli/c/common/dictionary.o \ + brotli/c/common/platform.o \ + brotli/c/common/shared_dictionary.o \ brotli/c/common/transform.o \ brotli/c/dec/bit_reader.o \ brotli/c/dec/decode.o \ @@ -22,12 +26,15 @@ OBJECTS = \ brotli/c/enc/block_splitter.o \ brotli/c/enc/brotli_bit_stream.o \ brotli/c/enc/cluster.o \ + brotli/c/enc/command.o \ + brotli/c/enc/compound_dictionary.o \ brotli/c/enc/compress_fragment.o \ brotli/c/enc/compress_fragment_two_pass.o \ brotli/c/enc/dictionary_hash.o \ brotli/c/enc/encode.o \ brotli/c/enc/encoder_dict.o \ brotli/c/enc/entropy_encode.o \ + brotli/c/enc/fast_log.o \ brotli/c/enc/histogram.o \ brotli/c/enc/literal_cost.o \ brotli/c/enc/memory.o \ diff --git a/BaseTools/Source/C/BrotliCompress/Makefile b/BaseTools/Source/C/BrotliCompress/Makefile index 038d1ec24226..0ed39d4b5a2c 100644 --- a/BaseTools/Source/C/BrotliCompress/Makefile +++ b/BaseTools/Source/C/BrotliCompress/Makefile @@ -13,7 +13,14 @@ APPNAME = BrotliCompress #LIBS = $(LIB_PATH)\Common.lib -COMMON_OBJ = brotli\c\common\dictionary.obj brotli\c\common\transform.obj +COMMON_OBJ = \ + brotli\c\common\constants.obj \ + brotli\c\common\context.obj \ + brotli\c\common\dictionary.obj \ + brotli\c\common\platform.obj \ + brotli\c\common\shared_dictionary.obj \ + brotli\c\common\transform.obj + DEC_OBJ = \ brotli\c\dec\bit_reader.obj \ brotli\c\dec\decode.obj \ @@ -26,12 +33,15 @@ ENC_OBJ = \ brotli\c\enc\block_splitter.obj \ brotli\c\enc\brotli_bit_stream.obj \ brotli\c\enc\cluster.obj \ + brotli\c\enc\command.obj \ + brotli\c\enc\compound_dictionary.obj \ brotli\c\enc\compress_fragment.obj \ brotli\c\enc\compress_fragment_two_pass.obj \ brotli\c\enc\dictionary_hash.obj \ brotli\c\enc\encode.obj \ brotli\c\enc\encoder_dict.obj \ brotli\c\enc\entropy_encode.obj \ + brotli\c\enc\fast_log.obj \ brotli\c\enc\histogram.obj \ brotli\c\enc\literal_cost.obj \ brotli\c\enc\memory.obj \ diff --git a/BaseTools/Source/C/BrotliCompress/brotli b/BaseTools/Source/C/BrotliCompress/brotli index 666c3280cc11..62662f87cdd9 160000 --- a/BaseTools/Source/C/BrotliCompress/brotli +++ b/BaseTools/Source/C/BrotliCompress/brotli @@ -1 +1 @@ -Subproject commit 666c3280cc11dc433c303d79a83d4ffbdd12cc8d +Subproject commit 62662f87cdd96deda90ac817de94e3c4af75226a -- 2.33.1.windows.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] BaseTools: Update brotli to the latest upstream commit 2021-10-29 2:22 [PATCH 1/1] BaseTools: Update brotli to the latest upstream commit Pedro Falcato @ 2021-12-01 0:38 ` Pedro Falcato 2021-12-01 1:51 ` 回复: " gaoliming 2021-12-02 14:41 ` [edk2-devel] " Ross Burton 0 siblings, 2 replies; 6+ messages in thread From: Pedro Falcato @ 2021-12-01 0:38 UTC (permalink / raw) To: edk2-devel-groups-io; +Cc: Bob Feng, Liming Gao, Yuwei Chen [-- Attachment #1: Type: text/plain, Size: 3850 bytes --] Hi! It's been over a month and it would be great if you could take a look at this, at least from a usability POV as more and more distros/operating systems upgrade their compilers to GCC 11. Thanks, Pedro On Fri, Oct 29, 2021 at 3:22 AM Pedro Falcato <pedro.falcato@gmail.com> wrote: > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3417 > > This updates BaseTools's brotli submodule to the latest upstream, > which fixes GCC 11's issue building BaseTools. > > Cc: Bob Feng <bob.c.feng@intel.com> > Cc: Liming Gao <gaoliming@byosoft.com.cn> > Cc: Yuwei Chen <yuwei.chen@intel.com> > Signed-off-by: Pedro Falcato <pedro.falcato@gmail.com> > --- > BaseTools/Source/C/BrotliCompress/GNUmakefile | 7 +++++++ > BaseTools/Source/C/BrotliCompress/Makefile | 12 +++++++++++- > BaseTools/Source/C/BrotliCompress/brotli | 2 +- > 3 files changed, 19 insertions(+), 2 deletions(-) > > diff --git a/BaseTools/Source/C/BrotliCompress/GNUmakefile > b/BaseTools/Source/C/BrotliCompress/GNUmakefile > index b150e5dd2bb9..79d7c405069d 100644 > --- a/BaseTools/Source/C/BrotliCompress/GNUmakefile > +++ b/BaseTools/Source/C/BrotliCompress/GNUmakefile > @@ -10,7 +10,11 @@ APPNAME = BrotliCompress > > OBJECTS = \ > BrotliCompress.o \ > + brotli/c/common/constants.o \ > + brotli/c/common/context.o \ > brotli/c/common/dictionary.o \ > + brotli/c/common/platform.o \ > + brotli/c/common/shared_dictionary.o \ > brotli/c/common/transform.o \ > brotli/c/dec/bit_reader.o \ > brotli/c/dec/decode.o \ > @@ -22,12 +26,15 @@ OBJECTS = \ > brotli/c/enc/block_splitter.o \ > brotli/c/enc/brotli_bit_stream.o \ > brotli/c/enc/cluster.o \ > + brotli/c/enc/command.o \ > + brotli/c/enc/compound_dictionary.o \ > brotli/c/enc/compress_fragment.o \ > brotli/c/enc/compress_fragment_two_pass.o \ > brotli/c/enc/dictionary_hash.o \ > brotli/c/enc/encode.o \ > brotli/c/enc/encoder_dict.o \ > brotli/c/enc/entropy_encode.o \ > + brotli/c/enc/fast_log.o \ > brotli/c/enc/histogram.o \ > brotli/c/enc/literal_cost.o \ > brotli/c/enc/memory.o \ > diff --git a/BaseTools/Source/C/BrotliCompress/Makefile > b/BaseTools/Source/C/BrotliCompress/Makefile > index 038d1ec24226..0ed39d4b5a2c 100644 > --- a/BaseTools/Source/C/BrotliCompress/Makefile > +++ b/BaseTools/Source/C/BrotliCompress/Makefile > @@ -13,7 +13,14 @@ APPNAME = BrotliCompress > > #LIBS = $(LIB_PATH)\Common.lib > > -COMMON_OBJ = brotli\c\common\dictionary.obj brotli\c\common\transform.obj > +COMMON_OBJ = \ > + brotli\c\common\constants.obj \ > + brotli\c\common\context.obj \ > + brotli\c\common\dictionary.obj \ > + brotli\c\common\platform.obj \ > + brotli\c\common\shared_dictionary.obj \ > + brotli\c\common\transform.obj > + > DEC_OBJ = \ > brotli\c\dec\bit_reader.obj \ > brotli\c\dec\decode.obj \ > @@ -26,12 +33,15 @@ ENC_OBJ = \ > brotli\c\enc\block_splitter.obj \ > brotli\c\enc\brotli_bit_stream.obj \ > brotli\c\enc\cluster.obj \ > + brotli\c\enc\command.obj \ > + brotli\c\enc\compound_dictionary.obj \ > brotli\c\enc\compress_fragment.obj \ > brotli\c\enc\compress_fragment_two_pass.obj \ > brotli\c\enc\dictionary_hash.obj \ > brotli\c\enc\encode.obj \ > brotli\c\enc\encoder_dict.obj \ > brotli\c\enc\entropy_encode.obj \ > + brotli\c\enc\fast_log.obj \ > brotli\c\enc\histogram.obj \ > brotli\c\enc\literal_cost.obj \ > brotli\c\enc\memory.obj \ > diff --git a/BaseTools/Source/C/BrotliCompress/brotli > b/BaseTools/Source/C/BrotliCompress/brotli > index 666c3280cc11..62662f87cdd9 160000 > --- a/BaseTools/Source/C/BrotliCompress/brotli > +++ b/BaseTools/Source/C/BrotliCompress/brotli > @@ -1 +1 @@ > -Subproject commit 666c3280cc11dc433c303d79a83d4ffbdd12cc8d > +Subproject commit 62662f87cdd96deda90ac817de94e3c4af75226a > -- > 2.33.1.windows.1 > > -- Pedro Falcato [-- Attachment #2: Type: text/html, Size: 5038 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* 回复: [PATCH 1/1] BaseTools: Update brotli to the latest upstream commit 2021-12-01 0:38 ` Pedro Falcato @ 2021-12-01 1:51 ` gaoliming 2021-12-01 3:07 ` [edk2-devel] " Pedro Falcato 2021-12-02 14:41 ` [edk2-devel] " Ross Burton 1 sibling, 1 reply; 6+ messages in thread From: gaoliming @ 2021-12-01 1:51 UTC (permalink / raw) To: 'Pedro Falcato', 'edk2-devel-groups-io' Cc: 'Bob Feng', 'Yuwei Chen' [-- Attachment #1: Type: text/plain, Size: 4489 bytes --] Pedro: I am OK to update brotli to new version. Please also update MdeModulePkg\Library\BrotliCustomDecompressLib\brotli to make them be same version. Thanks Liming 发件人: Pedro Falcato <pedro.falcato@gmail.com> 发送时间: 2021年12月1日 8:39 收件人: edk2-devel-groups-io <devel@edk2.groups.io> 抄送: Bob Feng <bob.c.feng@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>; Yuwei Chen <yuwei.chen@intel.com> 主题: Re: [PATCH 1/1] BaseTools: Update brotli to the latest upstream commit Hi! It's been over a month and it would be great if you could take a look at this, at least from a usability POV as more and more distros/operating systems upgrade their compilers to GCC 11. Thanks, Pedro On Fri, Oct 29, 2021 at 3:22 AM Pedro Falcato <pedro.falcato@gmail.com <mailto:pedro.falcato@gmail.com> > wrote: BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3417 This updates BaseTools's brotli submodule to the latest upstream, which fixes GCC 11's issue building BaseTools. Cc: Bob Feng <bob.c.feng@intel.com <mailto:bob.c.feng@intel.com> > Cc: Liming Gao <gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn> > Cc: Yuwei Chen <yuwei.chen@intel.com <mailto:yuwei.chen@intel.com> > Signed-off-by: Pedro Falcato <pedro.falcato@gmail.com <mailto:pedro.falcato@gmail.com> > --- BaseTools/Source/C/BrotliCompress/GNUmakefile | 7 +++++++ BaseTools/Source/C/BrotliCompress/Makefile | 12 +++++++++++- BaseTools/Source/C/BrotliCompress/brotli | 2 +- 3 files changed, 19 insertions(+), 2 deletions(-) diff --git a/BaseTools/Source/C/BrotliCompress/GNUmakefile b/BaseTools/Source/C/BrotliCompress/GNUmakefile index b150e5dd2bb9..79d7c405069d 100644 --- a/BaseTools/Source/C/BrotliCompress/GNUmakefile +++ b/BaseTools/Source/C/BrotliCompress/GNUmakefile @@ -10,7 +10,11 @@ APPNAME = BrotliCompress OBJECTS = \ BrotliCompress.o \ + brotli/c/common/constants.o \ + brotli/c/common/context.o \ brotli/c/common/dictionary.o \ + brotli/c/common/platform.o \ + brotli/c/common/shared_dictionary.o \ brotli/c/common/transform.o \ brotli/c/dec/bit_reader.o \ brotli/c/dec/decode.o \ @@ -22,12 +26,15 @@ OBJECTS = \ brotli/c/enc/block_splitter.o \ brotli/c/enc/brotli_bit_stream.o \ brotli/c/enc/cluster.o \ + brotli/c/enc/command.o \ + brotli/c/enc/compound_dictionary.o \ brotli/c/enc/compress_fragment.o \ brotli/c/enc/compress_fragment_two_pass.o \ brotli/c/enc/dictionary_hash.o \ brotli/c/enc/encode.o \ brotli/c/enc/encoder_dict.o \ brotli/c/enc/entropy_encode.o \ + brotli/c/enc/fast_log.o \ brotli/c/enc/histogram.o \ brotli/c/enc/literal_cost.o \ brotli/c/enc/memory.o \ diff --git a/BaseTools/Source/C/BrotliCompress/Makefile b/BaseTools/Source/C/BrotliCompress/Makefile index 038d1ec24226..0ed39d4b5a2c 100644 --- a/BaseTools/Source/C/BrotliCompress/Makefile +++ b/BaseTools/Source/C/BrotliCompress/Makefile @@ -13,7 +13,14 @@ APPNAME = BrotliCompress #LIBS = $(LIB_PATH)\Common.lib -COMMON_OBJ = brotli\c\common\dictionary.obj brotli\c\common\transform.obj +COMMON_OBJ = \ + brotli\c\common\constants.obj \ + brotli\c\common\context.obj \ + brotli\c\common\dictionary.obj \ + brotli\c\common\platform.obj \ + brotli\c\common\shared_dictionary.obj \ + brotli\c\common\transform.obj + DEC_OBJ = \ brotli\c\dec\bit_reader.obj \ brotli\c\dec\decode.obj \ @@ -26,12 +33,15 @@ ENC_OBJ = \ brotli\c\enc\block_splitter.obj \ brotli\c\enc\brotli_bit_stream.obj \ brotli\c\enc\cluster.obj \ + brotli\c\enc\command.obj \ + brotli\c\enc\compound_dictionary.obj \ brotli\c\enc\compress_fragment.obj \ brotli\c\enc\compress_fragment_two_pass.obj \ brotli\c\enc\dictionary_hash.obj \ brotli\c\enc\encode.obj \ brotli\c\enc\encoder_dict.obj \ brotli\c\enc\entropy_encode.obj \ + brotli\c\enc\fast_log.obj \ brotli\c\enc\histogram.obj \ brotli\c\enc\literal_cost.obj \ brotli\c\enc\memory.obj \ diff --git a/BaseTools/Source/C/BrotliCompress/brotli b/BaseTools/Source/C/BrotliCompress/brotli index 666c3280cc11..62662f87cdd9 160000 --- a/BaseTools/Source/C/BrotliCompress/brotli +++ b/BaseTools/Source/C/BrotliCompress/brotli @@ -1 +1 @@ -Subproject commit 666c3280cc11dc433c303d79a83d4ffbdd12cc8d +Subproject commit 62662f87cdd96deda90ac817de94e3c4af75226a -- 2.33.1.windows.1 -- Pedro Falcato [-- Attachment #2: Type: text/html, Size: 9326 bytes --] ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [edk2-devel] 回复: [PATCH 1/1] BaseTools: Update brotli to the latest upstream commit 2021-12-01 1:51 ` 回复: " gaoliming @ 2021-12-01 3:07 ` Pedro Falcato 2021-12-03 2:10 ` 回复: " gaoliming 0 siblings, 1 reply; 6+ messages in thread From: Pedro Falcato @ 2021-12-01 3:07 UTC (permalink / raw) To: edk2-devel-groups-io, Liming Gao; +Cc: Bob Feng, Yuwei Chen [-- Attachment #1: Type: text/plain, Size: 5192 bytes --] Liming, I've just tested this patch, and it is still broken from a usability POV, as while it fixes the original problem, it introduces new Werrors of the same kind. I must've tested this patch in Windows, which possibly doesn't trigger the error, but GCC still does. Therefore, I think we should wait for a fix (whether it is a consensus on the BZ that we should disable Werror in brotli or wait for an upstream fix of some kind) before touching the submodules. Thanks, Pedro On Wed, Dec 1, 2021 at 1:51 AM gaoliming <gaoliming@byosoft.com.cn> wrote: > Pedro: > > I am OK to update brotli to new version. Please also update > MdeModulePkg\Library\BrotliCustomDecompressLib\brotli to make them be same > version. > > > > Thanks > > Liming > > *发件人:* Pedro Falcato <pedro.falcato@gmail.com> > *发送时间:* 2021年12月1日 8:39 > *收件人:* edk2-devel-groups-io <devel@edk2.groups.io> > *抄送:* Bob Feng <bob.c.feng@intel.com>; Liming Gao < > gaoliming@byosoft.com.cn>; Yuwei Chen <yuwei.chen@intel.com> > *主题:* Re: [PATCH 1/1] BaseTools: Update brotli to the latest upstream > commit > > > > Hi! > > > > It's been over a month and it would be great if you could take a look at > this, at least from a usability POV as more and more distros/operating > systems upgrade their compilers to GCC 11. > > > > Thanks, > > Pedro > > > > On Fri, Oct 29, 2021 at 3:22 AM Pedro Falcato <pedro.falcato@gmail.com> > wrote: > > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3417 > > This updates BaseTools's brotli submodule to the latest upstream, > which fixes GCC 11's issue building BaseTools. > > Cc: Bob Feng <bob.c.feng@intel.com> > Cc: Liming Gao <gaoliming@byosoft.com.cn> > Cc: Yuwei Chen <yuwei.chen@intel.com> > Signed-off-by: Pedro Falcato <pedro.falcato@gmail.com> > --- > BaseTools/Source/C/BrotliCompress/GNUmakefile | 7 +++++++ > BaseTools/Source/C/BrotliCompress/Makefile | 12 +++++++++++- > BaseTools/Source/C/BrotliCompress/brotli | 2 +- > 3 files changed, 19 insertions(+), 2 deletions(-) > > diff --git a/BaseTools/Source/C/BrotliCompress/GNUmakefile > b/BaseTools/Source/C/BrotliCompress/GNUmakefile > index b150e5dd2bb9..79d7c405069d 100644 > --- a/BaseTools/Source/C/BrotliCompress/GNUmakefile > +++ b/BaseTools/Source/C/BrotliCompress/GNUmakefile > @@ -10,7 +10,11 @@ APPNAME = BrotliCompress > > OBJECTS = \ > BrotliCompress.o \ > + brotli/c/common/constants.o \ > + brotli/c/common/context.o \ > brotli/c/common/dictionary.o \ > + brotli/c/common/platform.o \ > + brotli/c/common/shared_dictionary.o \ > brotli/c/common/transform.o \ > brotli/c/dec/bit_reader.o \ > brotli/c/dec/decode.o \ > @@ -22,12 +26,15 @@ OBJECTS = \ > brotli/c/enc/block_splitter.o \ > brotli/c/enc/brotli_bit_stream.o \ > brotli/c/enc/cluster.o \ > + brotli/c/enc/command.o \ > + brotli/c/enc/compound_dictionary.o \ > brotli/c/enc/compress_fragment.o \ > brotli/c/enc/compress_fragment_two_pass.o \ > brotli/c/enc/dictionary_hash.o \ > brotli/c/enc/encode.o \ > brotli/c/enc/encoder_dict.o \ > brotli/c/enc/entropy_encode.o \ > + brotli/c/enc/fast_log.o \ > brotli/c/enc/histogram.o \ > brotli/c/enc/literal_cost.o \ > brotli/c/enc/memory.o \ > diff --git a/BaseTools/Source/C/BrotliCompress/Makefile > b/BaseTools/Source/C/BrotliCompress/Makefile > index 038d1ec24226..0ed39d4b5a2c 100644 > --- a/BaseTools/Source/C/BrotliCompress/Makefile > +++ b/BaseTools/Source/C/BrotliCompress/Makefile > @@ -13,7 +13,14 @@ APPNAME = BrotliCompress > > #LIBS = $(LIB_PATH)\Common.lib > > -COMMON_OBJ = brotli\c\common\dictionary.obj brotli\c\common\transform.obj > +COMMON_OBJ = \ > + brotli\c\common\constants.obj \ > + brotli\c\common\context.obj \ > + brotli\c\common\dictionary.obj \ > + brotli\c\common\platform.obj \ > + brotli\c\common\shared_dictionary.obj \ > + brotli\c\common\transform.obj > + > DEC_OBJ = \ > brotli\c\dec\bit_reader.obj \ > brotli\c\dec\decode.obj \ > @@ -26,12 +33,15 @@ ENC_OBJ = \ > brotli\c\enc\block_splitter.obj \ > brotli\c\enc\brotli_bit_stream.obj \ > brotli\c\enc\cluster.obj \ > + brotli\c\enc\command.obj \ > + brotli\c\enc\compound_dictionary.obj \ > brotli\c\enc\compress_fragment.obj \ > brotli\c\enc\compress_fragment_two_pass.obj \ > brotli\c\enc\dictionary_hash.obj \ > brotli\c\enc\encode.obj \ > brotli\c\enc\encoder_dict.obj \ > brotli\c\enc\entropy_encode.obj \ > + brotli\c\enc\fast_log.obj \ > brotli\c\enc\histogram.obj \ > brotli\c\enc\literal_cost.obj \ > brotli\c\enc\memory.obj \ > diff --git a/BaseTools/Source/C/BrotliCompress/brotli > b/BaseTools/Source/C/BrotliCompress/brotli > index 666c3280cc11..62662f87cdd9 160000 > --- a/BaseTools/Source/C/BrotliCompress/brotli > +++ b/BaseTools/Source/C/BrotliCompress/brotli > @@ -1 +1 @@ > -Subproject commit 666c3280cc11dc433c303d79a83d4ffbdd12cc8d > +Subproject commit 62662f87cdd96deda90ac817de94e3c4af75226a > -- > 2.33.1.windows.1 > > > > -- > > Pedro Falcato > > > -- Pedro Falcato [-- Attachment #2: Type: text/html, Size: 9218 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* 回复: [edk2-devel] 回复: [PATCH 1/1] BaseTools: Update brotli to the latest upstream commit 2021-12-01 3:07 ` [edk2-devel] " Pedro Falcato @ 2021-12-03 2:10 ` gaoliming 0 siblings, 0 replies; 6+ messages in thread From: gaoliming @ 2021-12-03 2:10 UTC (permalink / raw) To: 'Pedro Falcato', 'edk2-devel-groups-io' Cc: 'Bob Feng', 'Yuwei Chen' [-- Attachment #1: Type: text/plain, Size: 5760 bytes --] Pedro: Now, we can disable werror in brotli first. Thanks Liming 发件人: Pedro Falcato <pedro.falcato@gmail.com> 发送时间: 2021年12月1日 11:07 收件人: edk2-devel-groups-io <devel@edk2.groups.io>; Liming Gao <gaoliming@byosoft.com.cn> 抄送: Bob Feng <bob.c.feng@intel.com>; Yuwei Chen <yuwei.chen@intel.com> 主题: Re: [edk2-devel] 回复: [PATCH 1/1] BaseTools: Update brotli to the latest upstream commit Liming, I've just tested this patch, and it is still broken from a usability POV, as while it fixes the original problem, it introduces new Werrors of the same kind. I must've tested this patch in Windows, which possibly doesn't trigger the error, but GCC still does. Therefore, I think we should wait for a fix (whether it is a consensus on the BZ that we should disable Werror in brotli or wait for an upstream fix of some kind) before touching the submodules. Thanks, Pedro On Wed, Dec 1, 2021 at 1:51 AM gaoliming <gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn> > wrote: Pedro: I am OK to update brotli to new version. Please also update MdeModulePkg\Library\BrotliCustomDecompressLib\brotli to make them be same version. Thanks Liming 发件人: Pedro Falcato <pedro.falcato@gmail.com <mailto:pedro.falcato@gmail.com> > 发送时间: 2021年12月1日 8:39 收件人: edk2-devel-groups-io <devel@edk2.groups.io <mailto:devel@edk2.groups.io> > 抄送: Bob Feng <bob.c.feng@intel.com <mailto:bob.c.feng@intel.com> >; Liming Gao <gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn> >; Yuwei Chen <yuwei.chen@intel.com <mailto:yuwei.chen@intel.com> > 主题: Re: [PATCH 1/1] BaseTools: Update brotli to the latest upstream commit Hi! It's been over a month and it would be great if you could take a look at this, at least from a usability POV as more and more distros/operating systems upgrade their compilers to GCC 11. Thanks, Pedro On Fri, Oct 29, 2021 at 3:22 AM Pedro Falcato <pedro.falcato@gmail.com <mailto:pedro.falcato@gmail.com> > wrote: BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3417 This updates BaseTools's brotli submodule to the latest upstream, which fixes GCC 11's issue building BaseTools. Cc: Bob Feng <bob.c.feng@intel.com <mailto:bob.c.feng@intel.com> > Cc: Liming Gao <gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn> > Cc: Yuwei Chen <yuwei.chen@intel.com <mailto:yuwei.chen@intel.com> > Signed-off-by: Pedro Falcato <pedro.falcato@gmail.com <mailto:pedro.falcato@gmail.com> > --- BaseTools/Source/C/BrotliCompress/GNUmakefile | 7 +++++++ BaseTools/Source/C/BrotliCompress/Makefile | 12 +++++++++++- BaseTools/Source/C/BrotliCompress/brotli | 2 +- 3 files changed, 19 insertions(+), 2 deletions(-) diff --git a/BaseTools/Source/C/BrotliCompress/GNUmakefile b/BaseTools/Source/C/BrotliCompress/GNUmakefile index b150e5dd2bb9..79d7c405069d 100644 --- a/BaseTools/Source/C/BrotliCompress/GNUmakefile +++ b/BaseTools/Source/C/BrotliCompress/GNUmakefile @@ -10,7 +10,11 @@ APPNAME = BrotliCompress OBJECTS = \ BrotliCompress.o \ + brotli/c/common/constants.o \ + brotli/c/common/context.o \ brotli/c/common/dictionary.o \ + brotli/c/common/platform.o \ + brotli/c/common/shared_dictionary.o \ brotli/c/common/transform.o \ brotli/c/dec/bit_reader.o \ brotli/c/dec/decode.o \ @@ -22,12 +26,15 @@ OBJECTS = \ brotli/c/enc/block_splitter.o \ brotli/c/enc/brotli_bit_stream.o \ brotli/c/enc/cluster.o \ + brotli/c/enc/command.o \ + brotli/c/enc/compound_dictionary.o \ brotli/c/enc/compress_fragment.o \ brotli/c/enc/compress_fragment_two_pass.o \ brotli/c/enc/dictionary_hash.o \ brotli/c/enc/encode.o \ brotli/c/enc/encoder_dict.o \ brotli/c/enc/entropy_encode.o \ + brotli/c/enc/fast_log.o \ brotli/c/enc/histogram.o \ brotli/c/enc/literal_cost.o \ brotli/c/enc/memory.o \ diff --git a/BaseTools/Source/C/BrotliCompress/Makefile b/BaseTools/Source/C/BrotliCompress/Makefile index 038d1ec24226..0ed39d4b5a2c 100644 --- a/BaseTools/Source/C/BrotliCompress/Makefile +++ b/BaseTools/Source/C/BrotliCompress/Makefile @@ -13,7 +13,14 @@ APPNAME = BrotliCompress #LIBS = $(LIB_PATH)\Common.lib -COMMON_OBJ = brotli\c\common\dictionary.obj brotli\c\common\transform.obj +COMMON_OBJ = \ + brotli\c\common\constants.obj \ + brotli\c\common\context.obj \ + brotli\c\common\dictionary.obj \ + brotli\c\common\platform.obj \ + brotli\c\common\shared_dictionary.obj \ + brotli\c\common\transform.obj + DEC_OBJ = \ brotli\c\dec\bit_reader.obj \ brotli\c\dec\decode.obj \ @@ -26,12 +33,15 @@ ENC_OBJ = \ brotli\c\enc\block_splitter.obj \ brotli\c\enc\brotli_bit_stream.obj \ brotli\c\enc\cluster.obj \ + brotli\c\enc\command.obj \ + brotli\c\enc\compound_dictionary.obj \ brotli\c\enc\compress_fragment.obj \ brotli\c\enc\compress_fragment_two_pass.obj \ brotli\c\enc\dictionary_hash.obj \ brotli\c\enc\encode.obj \ brotli\c\enc\encoder_dict.obj \ brotli\c\enc\entropy_encode.obj \ + brotli\c\enc\fast_log.obj \ brotli\c\enc\histogram.obj \ brotli\c\enc\literal_cost.obj \ brotli\c\enc\memory.obj \ diff --git a/BaseTools/Source/C/BrotliCompress/brotli b/BaseTools/Source/C/BrotliCompress/brotli index 666c3280cc11..62662f87cdd9 160000 --- a/BaseTools/Source/C/BrotliCompress/brotli +++ b/BaseTools/Source/C/BrotliCompress/brotli @@ -1 +1 @@ -Subproject commit 666c3280cc11dc433c303d79a83d4ffbdd12cc8d +Subproject commit 62662f87cdd96deda90ac817de94e3c4af75226a -- 2.33.1.windows.1 -- Pedro Falcato -- Pedro Falcato [-- Attachment #2: Type: text/html, Size: 14802 bytes --] ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH 1/1] BaseTools: Update brotli to the latest upstream commit 2021-12-01 0:38 ` Pedro Falcato 2021-12-01 1:51 ` 回复: " gaoliming @ 2021-12-02 14:41 ` Ross Burton 1 sibling, 0 replies; 6+ messages in thread From: Ross Burton @ 2021-12-02 14:41 UTC (permalink / raw) To: Pedro Falcato, devel Yes, please, I just replicated this patch in GitHub (https://github.com/tianocore/edk2/pull/2231) before realising it was already on the list. Ross ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-12-03 2:10 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2021-10-29 2:22 [PATCH 1/1] BaseTools: Update brotli to the latest upstream commit Pedro Falcato 2021-12-01 0:38 ` Pedro Falcato 2021-12-01 1:51 ` 回复: " gaoliming 2021-12-01 3:07 ` [edk2-devel] " Pedro Falcato 2021-12-03 2:10 ` 回复: " gaoliming 2021-12-02 14:41 ` [edk2-devel] " Ross Burton
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox