739351d476
Change log:95336cb92b..191d55580eFull diff:95336cb92b..191d55580eRoll chromium third_party 4e16929f46..3a8f2a9e1e Change log:4e16929f46..3a8f2a9e1eChanged dependencies: * src/tools:c44a3f5eca..f524a53b81DEPS diff:95336cb92b..191d55580e/DEPS No update to Clang. TBR=titovartem@google.com, BUG=None CQ_INCLUDE_TRYBOTS=master.internal.tryserver.corp.webrtc:linux_internal Change-Id: Ic9c4a62b050383646e9fcf5cc07a5653c14ac06e Reviewed-on: https://webrtc-review.googlesource.com/76120 Reviewed-by: Patrik Höglund <phoglund@webrtc.org> Reviewed-by: Karl Wiberg <kwiberg@webrtc.org> Reviewed-by: Artem Titov <titovartem@webrtc.org> Commit-Queue: Artem Titov <titovartem@webrtc.org> Cr-Commit-Position: refs/heads/master@{#23205}
43 lines
1.6 KiB
Plaintext
43 lines
1.6 KiB
Plaintext
== Patches applied on top of zlib ==
|
|
|
|
- 0000-build.patch: changes from the upstream version, mostly related to the
|
|
build.
|
|
- 0001-simd.patch: integrate Intel SIMD optimizations from
|
|
https://github.com/jtkukunas/zlib/
|
|
|
|
== Procedure to create a patch file ==
|
|
|
|
Assuming you are working in a new feature branch:
|
|
- git format-patch master --stdout > foo.patch # where naming follows a growing
|
|
# number plus patch description.
|
|
- git add foo.patch
|
|
- git commit -a -m "Local patch."
|
|
- git rebase -i HEAD~2 # Squashing the second commit
|
|
|
|
As patches created in this way will feature a ChangeLog, there is no longer
|
|
the need to append this file with a description of what the patch does. This
|
|
should help to solve frequent conflicts in pending new patches on
|
|
Chromium's zlib.
|
|
|
|
The plan for the near future is to better insulate the platform specific
|
|
changes to ease update adoption with new releases of zlib. This insulation
|
|
happens by making changes inside contrib/ rather than the root directory
|
|
(where conflicts can happen).
|
|
|
|
If a change modifies enough things inside the root directory that the
|
|
intention is not immediately clear, generate a .patch file to go with your
|
|
change. If the change's modifications in the root directory are small, like:
|
|
|
|
#ifdef FEATURE_FLAG
|
|
use_special_feature();
|
|
#elif
|
|
use_default_behavior();
|
|
#endif
|
|
|
|
then the intent is clear and a .patch file doesn't need to be generated (since
|
|
it would not provide much value).
|
|
|
|
Ideally local changes should have a merge request featured in either:
|
|
- canonical zlib: https://github.com/madler/zlib/
|
|
- zlib-ng: https://github.com/Dead2/zlib-ng
|