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

Unified Diff: mojo/public/c/BUILD.gn

Issue 2232833003: Change the canonical way to include the C bindings headers to <mojo/bindings/*.h>. (Closed) Base URL: https://github.com/domokit/mojo.git@work791_mojo_tests
Patch Set: rebased Created 4 years, 4 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 | « examples/c/echo_client/echo_client.c ('k') | mojo/public/c/bindings/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/public/c/BUILD.gn
diff --git a/mojo/public/c/BUILD.gn b/mojo/public/c/BUILD.gn
index 2256d10e9c71d79c5ab44f1a53a03aefffe829aa..2583fa4f7b3d652fe10e1ccec5ee3881a45868bc 100644
--- a/mojo/public/c/BUILD.gn
+++ b/mojo/public/c/BUILD.gn
@@ -16,9 +16,54 @@ group("c") {
]
}
+# bindings ---------------------------------------------------------------------
+
+# Headers in include/mojo/bindings (to be included as <mojo/bindings/HEADER.h>)
+# and library in lib/bindings.
+#
+# Depends on :common and :system (minimally -- just <mojo/system/handle.h>).
+
+mojo_sdk_source_set("bindings") {
+ public_configs = [ ":c_config" ]
+
+ sources = [
+ # Public headers.
+ "include/mojo/bindings/array.h",
+ "include/mojo/bindings/buffer.h",
+ "include/mojo/bindings/interface.h",
+ "include/mojo/bindings/map.h",
+ "include/mojo/bindings/message.h",
+ "include/mojo/bindings/string.h",
+ "include/mojo/bindings/struct.h",
+ "include/mojo/bindings/union.h",
+ "include/mojo/bindings/validation.h",
+
+ # Internal headers.
+ "include/mojo/bindings/internal/type_descriptor.h",
+ "include/mojo/bindings/internal/util.h",
+
+ # Implementation library.
+ # TODO(vtl): Maybe separate this out into a separate source set (or even
+ # static library?).
+ "lib/bindings/array.c",
+ "lib/bindings/buffer.c",
+ "lib/bindings/map.c",
+ "lib/bindings/message.c",
+ "lib/bindings/struct.c",
+ "lib/bindings/type_descriptor.c",
+ "lib/bindings/union.c",
+ ]
+
+ deps = [
+ ":system",
+ ]
+}
+
# common -----------------------------------------------------------------------
# Headers in include/mojo (to be included as <mojo/HEADER.h>).
+#
+# Depends on nothing.
mojo_sdk_source_set("common") {
public_configs = [ ":c_config" ]
@@ -47,6 +92,8 @@ mojo_sdk_source_set("common_unittests") {
# Headers in include/mojo/environment (to be included as
# <mojo//environment/HEADER.h>).
+#
+# Depends on :system (and thus :common).
mojo_sdk_source_set("environment") {
public_configs = [ ":c_config" ]
@@ -64,6 +111,8 @@ mojo_sdk_source_set("environment") {
# system -----------------------------------------------------------------------
# Headers in include/mojo/system (to be included as <mojo/system/HEADER.h>).
+#
+# Depends on :common.
mojo_sdk_source_set("system") {
public_configs = [ ":c_config" ]
« no previous file with comments | « examples/c/echo_client/echo_client.c ('k') | mojo/public/c/bindings/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698