From: "Liming Gao" <liming.gao@intel.com>
To: "Zhang, Shenglei" <shenglei.zhang@intel.com>,
"devel@edk2.groups.io" <devel@edk2.groups.io>
Cc: "Wang, Jian J" <jian.j.wang@intel.com>,
"Wu, Hao A" <hao.a.wu@intel.com>,
"Feng, Bob C" <bob.c.feng@intel.com>
Subject: Re: [edk2-devel] [PATCH 0/2] Use submodule way to access brotli
Date: Mon, 9 Mar 2020 05:21:29 +0000 [thread overview]
Message-ID: <2d878908f7224d98a5689495d244cfba@intel.com> (raw)
In-Reply-To: <C0706E73DB8C124D9B9C38AA364E5D5E06041F84@SHSMSX104.ccr.corp.intel.com>
Shenglei:
Module INF should include all used source files in its [Sources] section. So, please also include the wrap header files, such as stddef.h into its [Sources] section.
And, please use REF: BZ link url to include BZ in the commit message.
Thanks
Liming
-----Original Message-----
From: Zhang, Shenglei <shenglei.zhang@intel.com>
Sent: 2020年3月5日 16:21
To: devel@edk2.groups.io; Zhang, Shenglei <shenglei.zhang@intel.com>
Cc: Gao, Liming <liming.gao@intel.com>; Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A <hao.a.wu@intel.com>; Feng, Bob C <bob.c.feng@intel.com>
Subject: RE: [edk2-devel] [PATCH 0/2] Use submodule way to access brotli
The patch is too big. So I attach the patch link.
https://github.com/shenglei10/edk2/commits/brotli
Thanks,
Shenglei
> -----Original Message-----
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Zhang, Shenglei
> Sent: Monday, March 2, 2020 11:20 AM
> To: devel@edk2.groups.io
> Cc: Gao, Liming <liming.gao@intel.com>; Wang, Jian J
> <jian.j.wang@intel.com>; Wu, Hao A <hao.a.wu@intel.com>; Feng, Bob C
> <bob.c.feng@intel.com>
> Subject: [edk2-devel] [PATCH 0/2] Use submodule way to access brotli
>
> Currently brotli is used and customized by edk2 in BaseTools and
> MdeModulePkg. These two patches make brotli a submodule in edk2.
> https://bugzilla.tianocore.org/show_bug.cgi?id=2558
> https://bugzilla.tianocore.org/show_bug.cgi?id=2559
>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Hao A Wu <hao.a.wu@intel.com>
> Cc: Bob Feng <bob.c.feng@intel.com>
> Shenglei Zhang (2):
> MdeModulePkg/BrotliCustomDecompressLib: Make brotli a submodule
> BaseTools: Make brotli a submodule
>
> .../C/BrotliCompress/common/dictionary.c | 5905 -----------------
> .../C/BrotliCompress/common/transform.c | 235 -
> .../Source/C/BrotliCompress/dec/bit_reader.c | 48 -
> .../Source/C/BrotliCompress/dec/decode.c | 2497 -------
> .../Source/C/BrotliCompress/dec/huffman.c | 356 -
> BaseTools/Source/C/BrotliCompress/dec/state.c | 164 -
> .../BrotliCompress/enc/backward_references.c | 144 -
> .../enc/backward_references_hq.c | 830 ---
> .../Source/C/BrotliCompress/enc/bit_cost.c | 35 -
> .../C/BrotliCompress/enc/block_splitter.c | 194 -
> .../C/BrotliCompress/enc/brotli_bit_stream.c | 1331 ----
> .../Source/C/BrotliCompress/enc/cluster.c | 56 -
> .../C/BrotliCompress/enc/compress_fragment.c | 790 ---
> .../enc/compress_fragment_two_pass.c | 645 --
> .../C/BrotliCompress/enc/dictionary_hash.c | 1120 ----
> .../Source/C/BrotliCompress/enc/encode.c | 1864 ------
> .../C/BrotliCompress/enc/encoder_dict.c | 32 -
> .../C/BrotliCompress/enc/entropy_encode.c | 501 --
> .../Source/C/BrotliCompress/enc/histogram.c | 100 -
> .../C/BrotliCompress/enc/literal_cost.c | 175 -
> .../Source/C/BrotliCompress/enc/memory.c | 170 -
> .../Source/C/BrotliCompress/enc/metablock.c | 667 --
> .../Source/C/BrotliCompress/enc/static_dict.c | 486 --
> .../Source/C/BrotliCompress/enc/utf8_util.c | 85 -
> .../Source/C/BrotliCompress/tools/brotli.c | 1143 ----
> .../BrotliDecUefiSupport.c | 31 +
> .../common/dictionary.c | 5905 -----------------
> .../common/transform.c | 235 -
> .../dec/bit_reader.c | 48 -
> .../BrotliCustomDecompressLib/dec/decode.c | 2500 -------
> .../BrotliCustomDecompressLib/dec/huffman.c | 356 -
> .../BrotliCustomDecompressLib/dec/state.c | 164 -
> .gitmodules | 6 +
> BaseTools/Source/C/BrotliCompress/GNUmakefile | 54 +-
> BaseTools/Source/C/BrotliCompress/LICENSE | 19 -
> BaseTools/Source/C/BrotliCompress/Makefile | 52 +-
> BaseTools/Source/C/BrotliCompress/README.md | 26 -
> BaseTools/Source/C/BrotliCompress/ReadMe.txt | 2 -
> BaseTools/Source/C/BrotliCompress/brotli | 1 +
> .../C/BrotliCompress/common/constants.h | 64 -
> .../Source/C/BrotliCompress/common/context.h | 261 -
> .../C/BrotliCompress/common/dictionary.h | 64 -
> .../Source/C/BrotliCompress/common/platform.h | 558 --
> .../C/BrotliCompress/common/transform.h | 80 -
> .../Source/C/BrotliCompress/common/version.h | 26 -
> .../Source/C/BrotliCompress/dec/bit_reader.h | 309 -
> .../Source/C/BrotliCompress/dec/huffman.h | 72 -
> .../Source/C/BrotliCompress/dec/prefix.h | 750 ---
> BaseTools/Source/C/BrotliCompress/dec/state.h | 258 -
> .../brotli-comparison-study-2015-09-22.pdf | Bin 215208 -> 0 bytes
> .../BrotliCompress/enc/backward_references.h | 38 -
> .../enc/backward_references_hq.h | 93 -
> .../enc/backward_references_inc.h | 153 -
> .../Source/C/BrotliCompress/enc/bit_cost.h | 63 -
> .../C/BrotliCompress/enc/bit_cost_inc.h | 127 -
> .../C/BrotliCompress/enc/block_encoder_inc.h | 34 -
> .../C/BrotliCompress/enc/block_splitter.h | 51 -
> .../C/BrotliCompress/enc/block_splitter_inc.h | 431 --
> .../C/BrotliCompress/enc/brotli_bit_stream.h | 84 -
> .../Source/C/BrotliCompress/enc/cluster.h | 48 -
> .../Source/C/BrotliCompress/enc/cluster_inc.h | 317 -
> .../Source/C/BrotliCompress/enc/command.h | 190 -
> .../C/BrotliCompress/enc/compress_fragment.h | 61 -
> .../enc/compress_fragment_two_pass.h | 54 -
> .../C/BrotliCompress/enc/dictionary_hash.h | 24 -
> .../C/BrotliCompress/enc/encoder_dict.h | 41 -
> .../C/BrotliCompress/enc/entropy_encode.h | 122 -
> .../enc/entropy_encode_static.h | 539 --
> .../Source/C/BrotliCompress/enc/fast_log.h | 147 -
> .../C/BrotliCompress/enc/find_match_length.h | 80 -
> BaseTools/Source/C/BrotliCompress/enc/hash.h | 497 --
> .../C/BrotliCompress/enc/hash_composite_inc.h | 133 -
> .../enc/hash_forgetful_chain_inc.h | 254 -
> .../enc/hash_longest_match64_inc.h | 266 -
> .../enc/hash_longest_match_inc.h | 258 -
> .../enc/hash_longest_match_quickly_inc.h | 235 -
> .../C/BrotliCompress/enc/hash_rolling_inc.h | 215 -
> .../enc/hash_to_binary_tree_inc.h | 327 -
> .../Source/C/BrotliCompress/enc/histogram.h | 63 -
> .../C/BrotliCompress/enc/histogram_inc.h | 51 -
> .../C/BrotliCompress/enc/literal_cost.h | 30 -
> .../Source/C/BrotliCompress/enc/memory.h | 102 -
> .../Source/C/BrotliCompress/enc/metablock.h | 105 -
> .../C/BrotliCompress/enc/metablock_inc.h | 183 -
> .../Source/C/BrotliCompress/enc/params.h | 44 -
> .../Source/C/BrotliCompress/enc/prefix.h | 53 -
> .../Source/C/BrotliCompress/enc/quality.h | 165 -
> .../Source/C/BrotliCompress/enc/ringbuffer.h | 164 -
> .../Source/C/BrotliCompress/enc/static_dict.h | 40 -
> .../C/BrotliCompress/enc/static_dict_lut.h | 5864 ----------------
> .../Source/C/BrotliCompress/enc/utf8_util.h | 32 -
> .../Source/C/BrotliCompress/enc/write_bits.h | 85 -
> .../C/BrotliCompress/include/brotli/decode.h | 344 -
> .../C/BrotliCompress/include/brotli/encode.h | 427 --
> .../C/BrotliCompress/include/brotli/port.h | 274 -
> .../C/BrotliCompress/include/brotli/types.h | 83 -
> .../Source/C/BrotliCompress/tools/brotli.md | 107 -
> .../BrotliCustomDecompressLib.inf | 42 +-
> .../BrotliDecUefiSupport.h | 43 +
> .../BrotliDecompressLibInternal.h | 23 +-
> .../Library/BrotliCustomDecompressLib/LICENSE | 19 -
> .../BrotliCustomDecompressLib/README.md | 26 -
> .../BrotliCustomDecompressLib/ReadMe.txt | 2 -
> .../Library/BrotliCustomDecompressLib/brotli | 1 +
> .../BrotliCustomDecompressLib/brotli/decode.h | 344 -
> .../BrotliCustomDecompressLib/brotli/port.h | 274 -
> .../BrotliCustomDecompressLib/brotli/types.h | 92 -
> .../common/constants.h | 64 -
> .../common/context.h | 261 -
> .../common/dictionary.h | 64 -
> .../common/platform.h | 559 --
> .../common/transform.h | 80 -
> .../common/version.h | 26 -
> .../dec/bit_reader.h | 309 -
> .../BrotliCustomDecompressLib/dec/huffman.h | 72 -
> .../BrotliCustomDecompressLib/dec/prefix.h | 750 ---
> .../BrotliCustomDecompressLib/dec/state.h | 258 -
> .../brotli-comparison-study-2015-09-22.pdf | Bin 215208 -> 0 bytes
> .../BrotliCustomDecompressLib/stddef.h | 9 +
> .../BrotliCustomDecompressLib/stdint.h | 9 +
> .../BrotliCustomDecompressLib/stdlib.h | 9 +
> .../BrotliCustomDecompressLib/string.h | 9 +
> MdeModulePkg/MdeModulePkg.dec | 2 +
> Readme.md | 6 +-
> 124 files changed, 202 insertions(+), 47663 deletions(-) delete mode
> 100644 BaseTools/Source/C/BrotliCompress/common/dictionary.c
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/common/transform.c
> delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/bit_reader.c
> delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/decode.c
> delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/huffman.c
> delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/state.c
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/backward_references.c
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/backward_references_hq.c
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/bit_cost.c
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/block_splitter.c
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/brotli_bit_stream.c
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/cluster.c
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/compress_fragment.c
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/compress_fragment_two_pass.c
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/dictionary_hash.c
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/encode.c
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/encoder_dict.c
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/entropy_encode.c
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/histogram.c
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/literal_cost.c
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/memory.c
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/metablock.c
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/static_dict.c
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/utf8_util.c
> delete mode 100644 BaseTools/Source/C/BrotliCompress/tools/brotli.c
> create mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/BrotliDecUefiSupport.c
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/common/dictionary.c
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/common/transform.c
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/dec/bit_reader.c
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/dec/decode.c
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/dec/huffman.c
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/dec/state.c
> delete mode 100644 BaseTools/Source/C/BrotliCompress/LICENSE
> delete mode 100644 BaseTools/Source/C/BrotliCompress/README.md
> delete mode 100644 BaseTools/Source/C/BrotliCompress/ReadMe.txt
> create mode 160000 BaseTools/Source/C/BrotliCompress/brotli
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/common/constants.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/common/context.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/common/dictionary.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/common/platform.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/common/transform.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/common/version.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/bit_reader.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/huffman.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/prefix.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/state.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/docs/brotli-
> comparison-study-2015-09-22.pdf
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/backward_references.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/backward_references_hq.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/backward_references_inc.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/bit_cost.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/bit_cost_inc.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/block_encoder_inc.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/block_splitter.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/block_splitter_inc.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/brotli_bit_stream.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/cluster.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/cluster_inc.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/command.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/compress_fragment.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/compress_fragment_two_pass.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/dictionary_hash.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/encoder_dict.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/entropy_encode.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/entropy_encode_static.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/fast_log.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/find_match_length.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/hash.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/hash_composite_inc.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/hash_forgetful_chain_inc.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/hash_longest_match64_inc.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/hash_longest_match_inc.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/hash_longest_match_quickly_inc.
> h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/hash_rolling_inc.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/hash_to_binary_tree_inc.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/histogram.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/histogram_inc.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/literal_cost.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/memory.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/metablock.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/metablock_inc.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/params.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/prefix.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/quality.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/ringbuffer.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/static_dict.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/enc/static_dict_lut.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/utf8_util.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/write_bits.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/include/brotli/decode.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/include/brotli/encode.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/include/brotli/port.h
> delete mode 100644
> BaseTools/Source/C/BrotliCompress/include/brotli/types.h
> delete mode 100644 BaseTools/Source/C/BrotliCompress/tools/brotli.md
> create mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/BrotliDecUefiSupport.
> h
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/LICENSE
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/README.md
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/ReadMe.txt
> create mode 160000
> MdeModulePkg/Library/BrotliCustomDecompressLib/brotli
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/brotli/decode.h
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/brotli/port.h
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/brotli/types.h
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/common/constants.h
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/common/context.h
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/common/dictionary.h
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/common/platform.h
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/common/transform.h
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/common/version.h
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/dec/bit_reader.h
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/dec/huffman.h
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/dec/prefix.h
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/dec/state.h
> delete mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/docs/brotli-
> comparison-study-2015-09-22.pdf
> create mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/stddef.h
> create mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/stdint.h
> create mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/stdlib.h
> create mode 100644
> MdeModulePkg/Library/BrotliCustomDecompressLib/string.h
>
> --
> 2.18.0.windows.1
>
>
>
prev parent reply other threads:[~2020-03-09 5:21 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <15F85E85523672EB.20252@groups.io>
2020-03-05 8:20 ` [edk2-devel] [PATCH 0/2] Use submodule way to access brotli Zhang, Shenglei
2020-03-09 5:21 ` Liming Gao [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2d878908f7224d98a5689495d244cfba@intel.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox