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

Unified Diff: runtime/bin/BUILD.gn

Issue 2951133002: Mostly revert "[fuchsia] Prepare for setting the Dart target architecture appropriately in the host… (Closed)
Patch Set: gn format Created 3 years, 6 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 | « runtime/BUILD.gn ('k') | runtime/observatory/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/bin/BUILD.gn
diff --git a/runtime/bin/BUILD.gn b/runtime/bin/BUILD.gn
index 5176077b9da818f3fdbcca1cf9285e30c1ed7fe4..773777798426c39a164e88f0683566c68f56eff9 100644
--- a/runtime/bin/BUILD.gn
+++ b/runtime/bin/BUILD.gn
@@ -277,7 +277,6 @@ executable("gen_snapshot") {
"..:dart_config",
"..:dart_maybe_product_config",
"..:dart_precompiler_config",
- "..:dart_use_target_arch_config",
]
if (is_fuchsia) {
configs -= [ "//build/config:symbol_visibility_hidden" ]
@@ -683,7 +682,6 @@ template("dart_executable") {
}
dart_executable("dart") {
- extra_configs = [ "..:dart_use_target_arch_config" ]
extra_deps = [
"..:libdart_jit",
":dart_snapshot_cc",
@@ -712,72 +710,57 @@ dart_executable("dart_precompiled_runtime") {
]
}
-template("dart_bootstrap_template") {
- invoker_extra_configs = []
- if (defined(invoker.extra_configs)) {
- invoker_extra_configs += invoker.extra_configs
- }
- dart_executable(target_name) {
- extra_configs = [
- "..:dart_precompiler_config",
- "..:dart_no_snapshot_config",
- ] + invoker_extra_configs
- extra_deps = [
- ":generate_builtin_cc_file",
- ":generate_io_cc_file",
- ":generate_io_patch_cc_file",
- ":generate_html_cc_file",
- ":generate_html_common_cc_file",
- ":generate_js_cc_file",
- ":generate_js_util_cc_file",
- ":generate_blink_cc_file",
- ":generate_indexed_db_cc_file",
- ":generate_cached_patches_cc_file",
- ":generate_web_gl_cc_file",
- ":generate_metadata_cc_file",
- ":generate_web_sql_cc_file",
- ":generate_svg_cc_file",
- ":generate_web_audio_cc_file",
- "..:libdart_nosnapshot_with_precompiler",
- ]
- extra_defines = [ "NO_OBSERVATORY" ]
- extra_sources = [
- "builtin.cc",
- "builtin.h",
- "dfe.cc",
- "dfe.h",
- "loader.cc",
- "loader.h",
- "observatory_assets_empty.cc",
- "snapshot_empty.cc",
-
- # Include generated source files.
- "$target_gen_dir/builtin_gen.cc",
- "$target_gen_dir/io_gen.cc",
- "$target_gen_dir/io_patch_gen.cc",
- "$target_gen_dir/html_gen.cc",
- "$target_gen_dir/html_common_gen.cc",
- "$target_gen_dir/js_gen.cc",
- "$target_gen_dir/js_util_gen.cc",
- "$target_gen_dir/blink_gen.cc",
- "$target_gen_dir/indexed_db_gen.cc",
- "$target_gen_dir/cached_patches_gen.cc",
- "$target_gen_dir/web_gl_gen.cc",
- "$target_gen_dir/metadata_gen.cc",
- "$target_gen_dir/web_sql_gen.cc",
- "$target_gen_dir/svg_gen.cc",
- "$target_gen_dir/web_audio_gen.cc",
- ]
- }
-}
-
-dart_bootstrap_template("dart_bootstrap") {
- extra_configs = [ "..:dart_use_target_arch_config" ]
-}
+dart_executable("dart_bootstrap") {
+ extra_configs = [
+ "..:dart_precompiler_config",
+ "..:dart_no_snapshot_config",
+ ]
+ extra_deps = [
+ ":generate_builtin_cc_file",
+ ":generate_io_cc_file",
+ ":generate_io_patch_cc_file",
+ ":generate_html_cc_file",
+ ":generate_html_common_cc_file",
+ ":generate_js_cc_file",
+ ":generate_js_util_cc_file",
+ ":generate_blink_cc_file",
+ ":generate_indexed_db_cc_file",
+ ":generate_cached_patches_cc_file",
+ ":generate_web_gl_cc_file",
+ ":generate_metadata_cc_file",
+ ":generate_web_sql_cc_file",
+ ":generate_svg_cc_file",
+ ":generate_web_audio_cc_file",
+ "..:libdart_nosnapshot_with_precompiler",
+ ]
+ extra_defines = [ "NO_OBSERVATORY" ]
+ extra_sources = [
+ "builtin.cc",
+ "builtin.h",
+ "dfe.cc",
+ "dfe.h",
+ "loader.cc",
+ "loader.h",
+ "observatory_assets_empty.cc",
+ "snapshot_empty.cc",
-# TODO(rmacnak): Remove if Fuchsia adds a checked-in SDK.
-dart_bootstrap_template("dart_bootstrap_host_arch") {
- extra_configs = []
+ # Include generated source files.
+ "$target_gen_dir/builtin_gen.cc",
+ "$target_gen_dir/io_gen.cc",
+ "$target_gen_dir/io_patch_gen.cc",
+ "$target_gen_dir/html_gen.cc",
+ "$target_gen_dir/html_common_gen.cc",
+ "$target_gen_dir/js_gen.cc",
+ "$target_gen_dir/js_util_gen.cc",
+ "$target_gen_dir/blink_gen.cc",
+ "$target_gen_dir/indexed_db_gen.cc",
+ "$target_gen_dir/cached_patches_gen.cc",
+ "$target_gen_dir/web_gl_gen.cc",
+ "$target_gen_dir/metadata_gen.cc",
+ "$target_gen_dir/web_sql_gen.cc",
+ "$target_gen_dir/svg_gen.cc",
+ "$target_gen_dir/web_audio_gen.cc",
+ ]
}
if (is_fuchsia) {
@@ -866,7 +849,6 @@ executable("run_vm_tests") {
configs += [
"..:dart_config",
"..:dart_maybe_product_config",
- "..:dart_use_target_arch_config",
]
if (is_fuchsia) {
configs -= [ "//build/config:symbol_visibility_hidden" ]
« no previous file with comments | « runtime/BUILD.gn ('k') | runtime/observatory/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698