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

Unified Diff: tools/gn/secondary/build/config/linux/BUILD.gn

Issue 24657003: GN: Return array from pkgconfig wrapper (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: empty list Created 7 years, 3 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 | « no previous file | tools/gn/secondary/build/config/linux/pkg-config.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/gn/secondary/build/config/linux/BUILD.gn
diff --git a/tools/gn/secondary/build/config/linux/BUILD.gn b/tools/gn/secondary/build/config/linux/BUILD.gn
index 159eb2fad4770459fc36a61f87e767f2c8ecc567..8d4b6f9e1473d1a5abac6022936bcbdfe55b3b26 100644
--- a/tools/gn/secondary/build/config/linux/BUILD.gn
+++ b/tools/gn/secondary/build/config/linux/BUILD.gn
@@ -13,6 +13,8 @@ config("executable_ldconfig") {
]
}
+# This script returns a list consisting of two nested lists: the first is the
+# list of cflags, the second are the linker flags.
pkg_script = "pkg-config.py"
config("fontconfig") {
@@ -20,38 +22,44 @@ config("fontconfig") {
}
config("freetype2") {
- cflags = exec_script(pkg_script, [ "--cflags", "freetype2" ], "list lines")
- ldflags = exec_script(pkg_script, [ "--libs", "freetype2" ], "list lines")
+ pkgresult = exec_script(pkg_script, [ "freetype2" ], "value")
+ includes = pkgresult[0]
+ cflags = pkgresult[1]
+ ldflags = pkgresult[2]
}
config("glib") {
- glib_packages = "glib-2.0 gmodule-2.0 gobject-2.0 gthread-2.0"
-
- cflags = exec_script(pkg_script, [ "--cflags", glib_packages ], "list lines")
- ldflags = exec_script(pkg_script, [ "--libs", glib_packages ], "list lines")
+ pkgresult = exec_script(pkg_script,
+ [ "glib-2.0", "gmodule-2.0", "gobject-2.0", "gthread-2.0" ], "value" )
+ includes = pkgresult[0]
+ cflags = pkgresult[1]
+ ldflags = pkgresult[2]
}
config("gtk") {
# Gtk requires gmodule, but it does not list it as a dependency in some
# misconfigured systems.
- gtk_packages = "gmodule-2.0 gtk+-2.0 gthread-2.0"
+ pkgresult = exec_script(pkg_script,
+ [ "gmodule-2.0", "gtk+-2.0", "gthread-2.0" ], "value" )
+ includes = pkgresult[0]
+ cflags = pkgresult[1]
+ ldflags = pkgresult[2]
defines = [ "TOOLKIT_GTK" ]
-
- cflags = exec_script(pkg_script, [ "--cflags", gtk_packages ], "list lines")
- ldflags = exec_script(pkg_script, [ "--libs", gtk_packages ], "list lines")
}
config("pangocairo") {
- cflags = exec_script(pkg_script, [ "--cflags", "pangocairo" ], "list lines")
- ldflags = exec_script(pkg_script, [ "--libs", "pangocairo" ], "list lines")
+ pkgresult = exec_script(pkg_script, [ "pangocairo" ], "value" )
+ includes = pkgresult[0]
+ cflags = pkgresult[1]
+ ldflags = pkgresult[2]
}
config("udev") {
- udev_packages = "libudev"
-
- cflags = exec_script(pkg_script, [ "--cflags", udev_packages ], "list lines")
- ldflags = exec_script(pkg_script, [ "--libs", udev_packages ], "list lines")
+ pkgresult = exec_script(pkg_script, [ "libudev" ], "value" )
+ includes = pkgresult[0]
+ cflags = pkgresult[1]
+ ldflags = pkgresult[2]
}
config("x11") {
« no previous file with comments | « no previous file | tools/gn/secondary/build/config/linux/pkg-config.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698