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

Unified Diff: sdk_build/data/cpp/cpp.sdk

Issue 1709643003: I guess we'll also need the bindings generator in a hypothetical C++ SDK. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: add jinja2 and markupsafe; fix target path for tools Created 4 years, 10 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 | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sdk_build/data/cpp/cpp.sdk
diff --git a/sdk_build/data/cpp/cpp.sdk b/sdk_build/data/cpp/cpp.sdk
index f6f4cec313da44ea61857cdb1088a7127ae83e4f..3e061acd2d154c62682e0482d25e0dec26aafee9 100644
--- a/sdk_build/data/cpp/cpp.sdk
+++ b/sdk_build/data/cpp/cpp.sdk
@@ -7,14 +7,36 @@
# TODO(vtl): This isn't done yet.
EXCLUDE_FILES=[".*", "*.gn", "*.gni", "PRESUBMIT.py", "*_win.*"]
-EXCLUDE_PATHS=["*/tests/"]
+EXCLUDE_PATHS=["*/tests/*"]
-CopyFiles("mojo/public/c", "third_party/mojo/public/c", recursive=True,
+CopyFiles("mojo/public",
+ "third_party/mojo/public",
+ recursive=False,
+ exclude_file_patterns=EXCLUDE_FILES)
+
+CopyFiles("mojo/public/c",
+ "third_party/mojo/public/c",
+ recursive=True,
exclude_file_patterns=EXCLUDE_FILES,
exclude_path_patterns=EXCLUDE_PATHS)
-CopyFiles("mojo/public/cpp", "third_party/mojo/public/cpp", recursive=True,
+CopyFiles("mojo/public/cpp",
+ "third_party/mojo/public/cpp",
+ recursive=True,
exclude_file_patterns=EXCLUDE_FILES,
exclude_path_patterns=EXCLUDE_PATHS+
["mojo/public/cpp/test_support/*"])
-CopyFiles("mojo/public", "third_party/mojo/public", recursive=False,
- exclude_file_patterns=EXCLUDE_FILES)
+
+# For simplicity, copy all of the bindings generators, even though we really
+# only need/want C++.
+CopyFiles("mojo/public/tools/bindings",
+ "third_party/mojo/public/tools/bindings",
+ recursive=True,
+ exclude_file_patterns=EXCLUDE_FILES,
+ exclude_path_patterns=EXCLUDE_PATHS+["*/mojom_tests/*"])
+# The generators need jinja2, which needs markupsafe. Sigh.
+CopyFiles("mojo/public/third_party/jinja2",
+ "third_party/mojo/public/third_party/jinja2",
+ recursive=True)
+CopyFiles("mojo/public/third_party/markupsafe",
+ "third_party/mojo/public/third_party/markupsafe",
+ recursive=True)
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698