diff options
| -rw-r--r-- | third_party/harfbuzz-ng/BUILD.gn | 24 | ||||
| -rw-r--r-- | third_party/harfbuzz-ng/harfbuzz.gyp | 10 |
2 files changed, 18 insertions, 16 deletions
diff --git a/third_party/harfbuzz-ng/BUILD.gn b/third_party/harfbuzz-ng/BUILD.gn index 2ce1280..24c6927 100644 --- a/third_party/harfbuzz-ng/BUILD.gn +++ b/third_party/harfbuzz-ng/BUILD.gn @@ -16,18 +16,15 @@ import("//build/config/linux/pkg_config.gni") declare_args() { # Since version 1.31.0, pangoft2 which we depend on pulls in harfbuzz # anyways. However, we want to have control of the version of harfbuzz - # we use, so don't use system harfbuzz for official builds, unless we - # are building for chrome os, where we have the system harfbuzz under - # control as well. We also do not want to use the system harfbuzz - # when cross-compiling for ARM. - use_system_harfbuzz = is_linux && (is_chromeos || (!is_official_build && - current_cpu != "arm")) && - exec_script(pkg_config_script, - pkg_config_args + [ - "--atleast-version=1.31.0", - "pangoft2", - ], - "value") + # we use, so don't use system harfbuzz unless we are building for + # chrome os, where we have the system harfbuzz under control. + use_system_harfbuzz = + is_linux && is_chromeos && exec_script(pkg_config_script, + pkg_config_args + [ + "--atleast-version=1.31.0", + "pangoft2", + ], + "value") } if (use_system_harfbuzz) { @@ -166,5 +163,8 @@ if (use_system_harfbuzz) { ] defines += [ "HAVE_CORETEXT" ] } + if (is_linux && !is_chromeos && !is_official_build && current_cpu != "arm") { + configs -= [ "//build/config/gcc:symbol_visibility_hidden" ] + } } } diff --git a/third_party/harfbuzz-ng/harfbuzz.gyp b/third_party/harfbuzz-ng/harfbuzz.gyp index f381e74..c277e8e 100644 --- a/third_party/harfbuzz-ng/harfbuzz.gyp +++ b/third_party/harfbuzz-ng/harfbuzz.gyp @@ -8,12 +8,11 @@ ], 'variables': { 'conditions': [ - ['OS=="linux" and (buildtype!="Official" or chromeos==1) and embedded==0', { + ['OS=="linux" and chromeos==1', { # Since version 1.31.0, pangoft2 which we depend on pulls in harfbuzz # anyways. However, we want to have control of the version of harfbuzz - # we use, so don't use system harfbuzz for official builds, unless we - # are building for chrome os, where we have the system harfbuzz under - # control as well. + # we use, so don't use system harfbuzz unless we are building for + # chrome os, where we have the system harfbuzz under control. 'use_system_harfbuzz%': '<!(python ../../build/check_return_value.py <(pkg-config) --atleast-version=1.31.0 pangoft2)', }, { 'use_system_harfbuzz': 0, @@ -152,6 +151,9 @@ 'src/hb-coretext.h', ], }], + ['OS=="linux" and chromeos==0 and buildtype!="Official" and target_arch!="arm"', { + 'cflags!': ['-fvisibility=hidden'], + }], ], }, ], |