Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6)

Unified Diff: third_party/harfbuzz-ng/harfbuzz.gyp

Issue 214413003: Use the system harfbuzz library on chrome os (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/common.gypi ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/harfbuzz-ng/harfbuzz.gyp
diff --git a/third_party/harfbuzz-ng/harfbuzz.gyp b/third_party/harfbuzz-ng/harfbuzz.gyp
index 304fcfd0cdaf7a0e8f23d65698af06159fd22052..7bd7d1d8b360872a8ce316fcf090a7817d2a831e 100644
--- a/third_party/harfbuzz-ng/harfbuzz.gyp
+++ b/third_party/harfbuzz-ng/harfbuzz.gyp
@@ -20,10 +20,12 @@
'pkg-config': '<(pkg-config)',
'conditions': [
- ['OS=="linux" and buildtype!="Official" and chromeos==0', {
+ ['OS=="linux" and (buildtype!="Official" or 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.
+ # 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.
'use_system_harfbuzz': '<!(python ../../build/check_return_value.py <(pkg-config) --atleast-version=1.31.0 pangoft2)',
}, {
'use_system_harfbuzz': 0,
@@ -171,34 +173,24 @@
},
],
}, { # use_system_harfbuzz==1
- 'variables': {
- # Check for presence of harfbuzz-icu library, use it if present.
- 'harfbuzz_libraries':
- '<!(python <(DEPTH)/tools/compile_test/compile_test.py '
- '--code "int main() { return 0; }" '
- '--run-linker '
- '--on-success "harfbuzz harfbuzz-icu" '
- '--on-failure "harfbuzz" '
- '-- -lharfbuzz-icu)',
- },
'targets': [
{
'target_name': 'harfbuzz-ng',
'type': 'none',
'cflags': [
- '<!@(<(pkg-config) --cflags <(harfbuzz_libraries))',
+ '<!@(<(pkg-config) --cflags harfbuzz)',
],
'direct_dependent_settings': {
'cflags': [
- '<!@(<(pkg-config) --cflags <(harfbuzz_libraries))',
+ '<!@(<(pkg-config) --cflags harfbuzz)',
],
},
'link_settings': {
'ldflags': [
- '<!@(<(pkg-config) --libs-only-L --libs-only-other <(harfbuzz_libraries))',
+ '<!@(<(pkg-config) --libs-only-L --libs-only-other harfbuzz)',
],
'libraries': [
- '<!@(<(pkg-config) --libs-only-l <(harfbuzz_libraries))',
+ '<!@(<(pkg-config) --libs-only-l harfbuzz)',
],
},
},
« no previous file with comments | « build/common.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698