fuchsia: Various build fixes for building in Fuchsia tree

- Use "deprecated_system_image" (merging from downstream)
- Add package for crashpad_handler
- Depend on launchpad target instead of a lib when in tree, as launchpad
  is no longer in the sysroot.
- Don't try to remove the -Wexit_time_destructors unless building
  standalone, when it's added by mini_chromiums BUILDCONFIG.gn

Bug: crashpad:196
Change-Id: I08e0faaa989346b078a41896eb4ace69e7b1bcdc
Reviewed-on: https://chromium-review.googlesource.com/1026514
Reviewed-by: Joshua Peraza <jperaza@chromium.org>
Commit-Queue: Scott Graham <scottmg@chromium.org>
This commit is contained in:
Scott Graham 2018-04-24 13:02:29 -07:00 committed by Commit Bot
parent 0e144fb9ae
commit 7274c9823f
3 changed files with 21 additions and 5 deletions

View File

@ -36,7 +36,7 @@ if (crashpad_is_in_chromium || crashpad_is_in_fuchsia) {
import("//build/package.gni")
package("crashpad_test") {
testonly = true
system_image = true
deprecated_system_image = true
deps = [
":crashpad_tests",
@ -48,6 +48,14 @@ if (crashpad_is_in_chromium || crashpad_is_in_fuchsia) {
},
]
}
package("crashpad_handler") {
deprecated_system_image = true
deps = [
"handler:crashpad_handler",
]
}
}
} else if (crashpad_is_standalone) {
test("crashpad_client_test") {

View File

@ -125,7 +125,13 @@ static_library("test") {
}
if (crashpad_is_fuchsia) {
libs = [ "launchpad" ]
if (crashpad_is_in_fuchsia) {
deps += [
"//zircon/public/lib/launchpad",
]
} else {
libs = [ "launchpad" ]
}
}
}

View File

@ -462,9 +462,11 @@ crashpad_executable("http_transport_test_server") {
"../tools:tool_support",
]
remove_configs = [
"//third_party/mini_chromium/mini_chromium/build:Wexit_time_destructors",
]
if (crashpad_is_standalone) {
remove_configs = [
"//third_party/mini_chromium/mini_chromium/build:Wexit_time_destructors",
]
}
if (crashpad_is_win) {
libs = [ "ws2_32.lib" ]