[Tarantool-patches] [PATCH v1 0/4] Fix app/digest and app/socket on 1.10
Alexander V. Tikhonov
avtikhon at tarantool.org
Thu Jun 18 08:36:48 MSK 2020
Needed to port fixes from master for #4090 and commit the following:
1. Before the patch unicode characters encoded with 4 bytes
were always treated as non-printable and displayed as byte
sequences (with 'binary' tag).
With the patch, range of printable characters is extended and
include characters encoded with 4 bytes.
Currently it is: (old printable range) U (icu printable range).
Corresponding changes are also made in tarantool/libyaml.
Closes: #4090
(cherry picked from commit cdf37876189fa005a350d6b69397348354bb2073)
2. Bump libyaml
Bumped libyaml with the following commits:
7f41d551a44a9b947dc341dd5b6c8779b5d83f00 'Make sure libyaml is C89 compliant'
f1d1e5e0a5f6e6adeebe0e2c5e95a6ee729426e4 'cmake: make sure yaml is built statically when used in tarantool'
74a00faf5c4c8720149f7a726a5eda5e8fff86df 'Extend range of printable unicode characters'
Needed for #4090
3. build: remove libyaml from rpm/deb dependencies
After we started using bundled version of libyaml by default (see commit
47b91e90f2a4e23e70a1a6735af3de203ffd59f4), we can remove it from
building dependencies for RPM and DEB packages.
Closes #4442
Reviewed-by: Alexander Turenko <alexander.turenko at tarantool.org>
(cherry picked from commit 1d4e584a7b5a5570486d7dfa6df82e664f8e0f95)
4. build: enable bundled libyaml for all systems.
After we fixed bundled libyaml to correctly print 4-byte Unicode
characters, it is no longer compatible with the upstream version, so
enable building with bundled libyaml for every platform.
This way the tests will pass.
Follow-up #4090
(cherry picked from commit 47b91e90f2a4e23e70a1a6735af3de203ffd59f4)
Github: https://github.com/tarantool/tarantool/tree/avtikhon/gh-5007-pack-yaml
Issue: https://github.com/tarantool/tarantool/issues/5007
Alexander V. Tikhonov (1):
Bump libyaml
Ivan Koptelov (1):
Extend range of printable unicode characters
Serge Petrenko (2):
build: enable bundled libyaml for all systems.
build: remove libyaml from rpm/deb dependencies
apk/APKBUILD | 2 --
debian/control | 3 +--
debian/rules | 1 -
rpm/tarantool.spec | 2 --
snapcraft.yaml | 1 -
src/CMakeLists.txt | 1 +
src/util.c | 5 ++++-
test/app-tap/yaml.test.lua | 10 +++++++++-
third_party/libyaml | 2 +-
9 files changed, 16 insertions(+), 11 deletions(-)
--
2.17.1
More information about the Tarantool-patches
mailing list