66ced07545
* build: remove common/icu/no_inline_default_constructor.patch It was added in https://github.com/electron/libchromiumcontent/pull/207, the issue is no longer valid in the GN build. * build: remove common/icu/build_gn.patch In GN release build all code including node are linked statically in a single binary, the workaround is no longer valid. * build: remove ununsed patches * build: remove is_electron_gn_build arg * build: remove component mode crt config patch * build: remove executable config patch for character_data_generator * build: remove latency_info.patch * build: remove exec_script_whitelist patch * build: update patches/common/v8/build_gn.patch * build: remove common/chromium/fix-arm64-linking-error.patch It was added ina1df18ca36to workaround a linker issue. * build: remove common/chromium/protobuf_build_gn.patch It was added to workaround a VS linker bug, it has been fixed in Update 1 * build: remove common/chromium/disable-recursive-surface-sync.patch It was added in6bc6626e2cto fix dcheck on macOS, it no longer appears to be an issue.
22 lines
559 B
Diff
22 lines
559 B
Diff
diff --git a/BUILD.gn b/BUILD.gn
|
|
index 4b48f7d687..30e9ec34cf 100644
|
|
--- a/BUILD.gn
|
|
+++ b/BUILD.gn
|
|
@@ -192,7 +192,7 @@ config("internal_config") {
|
|
|
|
include_dirs = [ "." ]
|
|
|
|
- if (is_component_build) {
|
|
+ if (is_component_build || is_electron_build) {
|
|
defines = [ "BUILDING_V8_SHARED" ]
|
|
}
|
|
}
|
|
@@ -2912,6 +2912,8 @@ if (v8_use_snapshot && current_toolchain == v8_snapshot_toolchain) {
|
|
|
|
configs = [ ":internal_config" ]
|
|
|
|
+ configs += [ "//electron/build/config:build_time_executable" ]
|
|
+
|
|
deps = [
|
|
":v8_base",
|
|
":v8_init",
|