Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: BUILD.gn

Issue 29733646: Issue 6396 - Error creating non debug build config (Closed)
Patch Set: Created March 26, 2018, 10:47 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | gni/v8.gni » ('j') | gni/v8.gni » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: BUILD.gn
diff --git a/BUILD.gn b/BUILD.gn
index 4e40ed031b07ab1dd05268ffe930a70897b56c6f..90665347596f33f5b522f4954a284342c462de8c 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -170,7 +170,7 @@ config("internal_config") {
include_dirs = [ "." ]
- if (is_component_build) {
+ if (is_component_build || v8_component_build) {
defines = [ "BUILDING_V8_SHARED" ]
}
}
@@ -191,7 +191,7 @@ config("libplatform_config") {
# This config should be applied to code using the libbase.
config("libbase_config") {
- if (is_component_build) {
+ if (is_component_build || v8_component_build) {
defines = [ "USING_V8_BASE_SHARED" ]
}
libs = []
@@ -209,7 +209,7 @@ config("libsampler_config") {
# itself.
config("external_config") {
defines = []
- if (is_component_build) {
+ if (is_component_build || v8_component_build) {
defines += [ "USING_V8_SHARED" ]
}
if (v8_enable_v8_checks) {
@@ -783,9 +783,8 @@ action("run_mksnapshot") {
]
args = [
- "./" + rebase_path(get_label_info(":mksnapshot($v8_snapshot_toolchain)",
- "root_out_dir") + "/mksnapshot",
- root_build_dir),
+ rebase_path(get_label_info(":mksnapshot($v8_snapshot_toolchain)",
+ "root_out_dir") + "/mksnapshot"),
"--startup_src",
rebase_path("$target_gen_dir/snapshot.cc", root_build_dir),
]
@@ -851,6 +850,7 @@ action("v8_dump_build_config") {
"v8_enable_verify_predictable=$v8_enable_verify_predictable",
"v8_target_cpu=\"$v8_target_cpu\"",
"v8_use_snapshot=$v8_use_snapshot",
+ "v8_component_build=$v8_component_build"
]
}
@@ -2463,6 +2463,7 @@ v8_source_set("v8_base") {
}
v8_component("v8_libbase") {
+ force_shared_library = true
sources = [
"src/base/adapters.h",
"src/base/atomic-utils.h",
@@ -2531,7 +2532,7 @@ v8_component("v8_libbase") {
defines = []
- if (is_component_build) {
+ if (is_component_build || v8_component_build) {
defines = [ "BUILDING_V8_BASE_SHARED" ]
}
@@ -2701,6 +2702,12 @@ if (current_toolchain == v8_snapshot_toolchain) {
configs = [ ":internal_config" ]
+ if (v8_component_build) {
+ configs += [
+ "//build/config/gcc:rpath_for_built_shared_libraries"
+ ]
+ }
+
deps = [
":v8_base",
":v8_init",
@@ -2787,8 +2794,11 @@ group("v8_fuzzers") {
]
}
-if (is_component_build) {
+# we need to have v8 shared libs as we have to link libadblockplus-jni.so with it.
+# see https://issues.adblockplus.org/ticket/6369
+if (true) {
v8_component("v8") {
+ force_shared_library = true
sources = [
"src/v8dll-main.cc",
]
« no previous file with comments | « no previous file | gni/v8.gni » ('j') | gni/v8.gni » ('J')

Powered by Google App Engine
This is Rietveld