build_do_not_depend_on_packed_resource_integrity.patch 3.2 KB

1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586878889
  1. From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
  2. From: Samuel Attard <[email protected]>
  3. Date: Thu, 27 May 2021 17:21:07 -0700
  4. Subject: build: do not depend on packed_resource_integrity
  5. This ensures we do not depend on a target that does not exist when
  6. building Electron, electron generates its own .pak files via
  7. electron_repack and therefore this integrity target which is generated
  8. by the chrome_paks target does not exist. This can not be upstreamed,
  9. if we ever align our .pak file generation with Chrome we can remove this
  10. patch.
  11. diff --git a/chrome/BUILD.gn b/chrome/BUILD.gn
  12. index 3611378c052813bd9c9ab9ce7d1840d6535576bf..b148e0acaf28197ee1da634b5e4c449ffcc30ddc 100644
  13. --- a/chrome/BUILD.gn
  14. +++ b/chrome/BUILD.gn
  15. @@ -200,11 +200,16 @@ if (!is_android && !is_mac && !is_fuchsia) {
  16. "common/crash_keys.h",
  17. ]
  18. + if (!is_electron_build) {
  19. + deps = [
  20. + ":packed_resources_integrity_header",
  21. + ]
  22. + }
  23. +
  24. deps += [
  25. ":chrome_dll",
  26. ":chrome_exe_version",
  27. ":copy_first_run",
  28. - ":packed_resources_integrity_header",
  29. ":visual_elements_resources",
  30. "//base",
  31. "//build:branding_buildflags",
  32. diff --git a/chrome/browser/BUILD.gn b/chrome/browser/BUILD.gn
  33. index 9e7f057a1c35d3e6ba3f144cfaf7dfe0ff1148a8..b2b2c49e745530b35b6ee7df080426afa4c64989 100644
  34. --- a/chrome/browser/BUILD.gn
  35. +++ b/chrome/browser/BUILD.gn
  36. @@ -4746,7 +4746,7 @@ static_library("browser") {
  37. # On Windows, the hashes are embedded in //chrome:chrome_initial rather
  38. # than here in :chrome_dll.
  39. - if (!is_win) {
  40. + if (!is_win && !is_electron_build) {
  41. deps += [ "//chrome:packed_resources_integrity_header" ]
  42. sources += [ "certificate_viewer_stub.cc" ]
  43. }
  44. diff --git a/chrome/test/BUILD.gn b/chrome/test/BUILD.gn
  45. index f56b2c37e6176e3d796db6a0d75b9388691a1b1c..4f5b716570e8e235699c90b44ce14b367469e994 100644
  46. --- a/chrome/test/BUILD.gn
  47. +++ b/chrome/test/BUILD.gn
  48. @@ -7190,7 +7190,6 @@ if (!is_fuchsia) {
  49. deps += [
  50. "//chrome:other_version",
  51. - "//chrome:packed_resources_integrity_header",
  52. "//chrome//services/util_win:unit_tests",
  53. "//chrome/app:chrome_dll_resources",
  54. "//chrome/app:win_unit_tests",
  55. @@ -7211,6 +7210,10 @@ if (!is_fuchsia) {
  56. "//ui/resources",
  57. ]
  58. + if (!is_electron_build) {
  59. + deps += [ "//chrome:packed_resources_integrity_header" ]
  60. + }
  61. +
  62. ldflags = [
  63. "/DELAYLOAD:api-ms-win-core-winrt-error-l1-1-0.dll",
  64. "/DELAYLOAD:api-ms-win-core-winrt-l1-1-0.dll",
  65. @@ -8231,7 +8234,6 @@ if (!is_fuchsia) {
  66. }
  67. deps += [
  68. - "//chrome:packed_resources_integrity_header",
  69. "//chrome/browser/apps:icon_standardizer",
  70. "//chrome/browser/apps/app_service",
  71. "//chrome/browser/apps/app_service:app_registry_cache_waiter",
  72. @@ -8324,6 +8326,10 @@ if (!is_fuchsia) {
  73. "//ui/webui/resources/js/browser_command:mojo_bindings",
  74. ]
  75. + if (!is_electron_build) {
  76. + deps += [ "//chrome:packed_resources_integrity_header" ]
  77. + }
  78. +
  79. data += [ "//ash/components/arc/test/data/icons/" ]
  80. if (include_js2gtest_tests && is_chromeos_ash) {