Skip to content

Commit

Permalink
[meson] disable icu for amalgam build on CI for now
Browse files Browse the repository at this point in the history
Causes linking errors

test-unicode.c:960: undefined reference to `hb_icu_get_unicode_funcs'
test-unicode.c:961: undefined reference to `hb_icu_get_unicode_funcs'

Need to have a separate harfbuzz-icu module and link against that,
and/or generate harfbuzz.cc.
  • Loading branch information
tp-m committed Mar 14, 2020
1 parent 7c24cc3 commit efba55a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -232,8 +232,8 @@ jobs:
- run: apt update && apt install -y ninja-build binutils meson gcc g++ pkg-config ragel gtk-doc-tools libfreetype6-dev libglib2.0-dev libcairo2-dev libicu-dev libgraphite2-dev python python-pip
- run: pip install fonttools
- run: meson build && ninja -Cbuild test && rm -rf build
# test amalgam build
- run: meson build -Damalgam=true && ninja -Cbuild && rm -rf build
# test amalgam build (FIXME: enable icu again)
- run: meson build -Dicu=disabled -Damalgam=true && ninja -Cbuild && rm -rf build
# test meson's own unity feature also
- run: meson build --unity on && ninja -Cbuild && rm -rf build

Expand Down

0 comments on commit efba55a

Please sign in to comment.