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

Side by Side Diff: remoting/host/win/BUILD.gn

Issue 2621223003: Fix building ASAN instrumented executables with custom entrypoints (Closed)
Patch Set: nits Created 3 years, 11 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 unified diff | Download patch
« no previous file with comments | « chrome/installer/mini_installer/mini_installer.cc ('k') | remoting/host/win/entry_point.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2016 The Chromium Authors. All rights reserved. 1 # Copyright 2016 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import("//build/toolchain/win/midl.gni") 5 import("//build/toolchain/win/midl.gni")
6 import("//build/win/message_compiler.gni") 6 import("//build/win/message_compiler.gni")
7 import("//media/media_options.gni") 7 import("//media/media_options.gni")
8 import("//remoting/build/config/remoting_build.gni") 8 import("//remoting/build/config/remoting_build.gni")
9 import("//remoting/host/installer/win/generate_clsids.gni") 9 import("//remoting/host/installer/win/generate_clsids.gni")
10 10
(...skipping 258 matching lines...) Expand 10 before | Expand all | Expand 10 after
269 ":dpi_aware_exe_manifest", 269 ":dpi_aware_exe_manifest",
270 ":remoting_core", 270 ":remoting_core",
271 ":remoting_windows_resources", 271 ":remoting_windows_resources",
272 ] 272 ]
273 273
274 sources = [ 274 sources = [
275 "$root_gen_dir/remoting/version.rc", 275 "$root_gen_dir/remoting/version.rc",
276 "entry_point.cc", 276 "entry_point.cc",
277 ] 277 ]
278 278
279 ldflags = [ 279 if (!is_asan) {
280 "/ENTRY:HostEntryPoint", 280 ldflags = [
281 # "/NODEFAULTLIB", 281 "/ENTRY:HostEntryPoint",
282 ] 282 # "/NODEFAULTLIB",
283 ]
284 }
283 } 285 }
284 286
285 executable("remoting_me2me_host") { 287 executable("remoting_me2me_host") {
286 configs += [ 288 configs += [
287 "//build/config/compiler:wexit_time_destructors", 289 "//build/config/compiler:wexit_time_destructors",
288 "//build/config/win:windowed", 290 "//build/config/win:windowed",
289 "//remoting/build/config:remoting_me2me_host", 291 "//remoting/build/config:remoting_me2me_host",
290 ] 292 ]
291 293
292 defines = host_predefines + [ "REMOTING_HOST_BINARY=BINARY_HOST_ME2ME" ] 294 defines = host_predefines + [ "REMOTING_HOST_BINARY=BINARY_HOST_ME2ME" ]
293 295
294 deps = [ 296 deps = [
295 ":dpi_aware_exe_manifest", 297 ":dpi_aware_exe_manifest",
296 ":remoting_core", 298 ":remoting_core",
297 ":remoting_windows_resources", 299 ":remoting_windows_resources",
298 ] 300 ]
299 301
300 sources = [ 302 sources = [
301 "$root_gen_dir/remoting/version.rc", 303 "$root_gen_dir/remoting/version.rc",
302 "entry_point.cc", 304 "entry_point.cc",
303 ] 305 ]
304 306
305 output_name = "remoting_host" 307 output_name = "remoting_host"
306 308
307 ldflags = [ "/ENTRY:HostEntryPoint" ] 309 if (!is_asan) {
310 ldflags = [
311 "/ENTRY:HostEntryPoint",
312 # "/NODEFAULTLIB",
313 ]
314 }
308 } 315 }
309 316
310 shared_library("remoting_core") { 317 shared_library("remoting_core") {
311 configs += [ "//build/config/compiler:wexit_time_destructors" ] 318 configs += [ "//build/config/compiler:wexit_time_destructors" ]
312 319
313 defines = host_predefines + [ 320 defines = host_predefines + [
314 "_ATL_APARTMENT_THREADED", 321 "_ATL_APARTMENT_THREADED",
315 "_ATL_CSTRING_EXPLICIT_CONSTRUCTORS", 322 "_ATL_CSTRING_EXPLICIT_CONSTRUCTORS",
316 "_ATL_NO_AUTOMATIC_NAMESPACE", 323 "_ATL_NO_AUTOMATIC_NAMESPACE",
317 "_ATL_NO_EXCEPTIONS", 324 "_ATL_NO_EXCEPTIONS",
(...skipping 115 matching lines...) Expand 10 before | Expand all | Expand 10 after
433 deps += [ ":dpi_aware_uiaccess_require_admin_exe_manifest" ] 440 deps += [ ":dpi_aware_uiaccess_require_admin_exe_manifest" ]
434 } else { 441 } else {
435 deps += [ ":dpi_aware_exe_manifest" ] 442 deps += [ ":dpi_aware_exe_manifest" ]
436 } 443 }
437 444
438 sources = [ 445 sources = [
439 "$root_gen_dir/remoting/version.rc", 446 "$root_gen_dir/remoting/version.rc",
440 "entry_point.cc", 447 "entry_point.cc",
441 ] 448 ]
442 449
443 ldflags = [ 450 if (!is_asan) {
444 "/ENTRY:HostEntryPoint", 451 ldflags = [
445 # "/NODEFAULTLIB", 452 "/ENTRY:HostEntryPoint",
446 ] 453 # "/NODEFAULTLIB",
454 ]
455 }
447 } 456 }
448 457
449 executable("remoting_native_messaging_host") { 458 executable("remoting_native_messaging_host") {
450 configs += [ "//build/config/compiler:wexit_time_destructors" ] 459 configs += [ "//build/config/compiler:wexit_time_destructors" ]
451 460
452 defines = 461 defines =
453 host_predefines + [ "REMOTING_HOST_BINARY=BINARY_NATIVE_MESSAGING_HOST" ] 462 host_predefines + [ "REMOTING_HOST_BINARY=BINARY_NATIVE_MESSAGING_HOST" ]
454 463
455 deps = [ 464 deps = [
456 ":remoting_core", 465 ":remoting_core",
(...skipping 25 matching lines...) Expand all
482 491
483 output = "$root_gen_dir/remoting/{{source_name_part}}" 492 output = "$root_gen_dir/remoting/{{source_name_part}}"
484 493
485 locale_dir = webapp_locale_dir 494 locale_dir = webapp_locale_dir
486 495
487 encoding = "utf-16" 496 encoding = "utf-16"
488 497
489 locales = remoting_locales 498 locales = remoting_locales
490 } 499 }
491 # TODO(GYP) More Windows remoting targets from remoting_host_win.gypi 500 # TODO(GYP) More Windows remoting targets from remoting_host_win.gypi
OLDNEW
« no previous file with comments | « chrome/installer/mini_installer/mini_installer.cc ('k') | remoting/host/win/entry_point.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698