commit: 689090f1ec53b7b958a903f6ed01f38b71e1b819 Author: Mike Gilbert <floppym <AT> gentoo <DOT> org> AuthorDate: Wed Aug 9 14:04:25 2017 +0000 Commit: Mike Gilbert <floppym <AT> gentoo <DOT> org> CommitDate: Wed Aug 9 14:04:25 2017 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=689090f1
www-client/chromium: backport build fix Bug: https://bugs.gentoo.org/626684 Package-Manager: Portage-2.3.6_p23, Repoman-2.3.3_p1 www-client/chromium/chromium-61.0.3163.31.ebuild | 1 + www-client/chromium/files/chromium-mojo-dep.patch | 54 +++++++++++++++++++++++ 2 files changed, 55 insertions(+) diff --git a/www-client/chromium/chromium-61.0.3163.31.ebuild b/www-client/chromium/chromium-61.0.3163.31.ebuild index 600995b990b..c55fd86c15d 100644 --- a/www-client/chromium/chromium-61.0.3163.31.ebuild +++ b/www-client/chromium/chromium-61.0.3163.31.ebuild @@ -146,6 +146,7 @@ PATCHES=( "${FILESDIR}/${PN}-gcc-r1.patch" "${FILESDIR}/${PN}-gn-bootstrap-r14.patch" "${FILESDIR}/${PN}-atk-r1.patch" + "${FILESDIR}/${PN}-mojo-dep.patch" ) pre_build_checks() { diff --git a/www-client/chromium/files/chromium-mojo-dep.patch b/www-client/chromium/files/chromium-mojo-dep.patch new file mode 100644 index 00000000000..bf618540d17 --- /dev/null +++ b/www-client/chromium/files/chromium-mojo-dep.patch @@ -0,0 +1,54 @@ +From f8e2c057ec6123b0a1d08294f41e78c427636b49 Mon Sep 17 00:00:00 2001 +From: Mike Bjorge <mbjo...@chromium.org> +Date: Mon, 7 Aug 2017 18:41:58 +0000 +Subject: [PATCH] Add mojo_blink dep from WebKit loader/. + +Now that loader has a direct dependency on the generated mojom-blink +headers (https://chromium-review.googlesource.com/c/577488), the loader/ +target should have a dep on the mojo target that generates the headers, +otherwise the build may fail depending on build order. + +All the other platform/ targets that depend on the mojom-blink headers +are directly included in the platform target, which has the necessary +dependency already. + +Bug: 746197 +Bug: 750883 +Change-Id: Ife1429ec649605aae3f7920560cec37f3a218154 +Reviewed-on: https://chromium-review.googlesource.com/602869 +Reviewed-by: Yutaka Hirano <yhir...@chromium.org> +Reviewed-by: Matt Falkenhagen <fal...@chromium.org> +Commit-Queue: Mike Bjorge <mbjo...@chromium.org> +Cr-Commit-Position: refs/heads/master@{#492369} +--- + third_party/WebKit/Source/platform/loader/BUILD.gn | 1 + + third_party/WebKit/public/BUILD.gn | 1 + + 2 files changed, 2 insertions(+) + +diff --git a/third_party/WebKit/Source/platform/loader/BUILD.gn b/third_party/WebKit/Source/platform/loader/BUILD.gn +index d483734b21cf..409befe884a8 100644 +--- a/third_party/WebKit/Source/platform/loader/BUILD.gn ++++ b/third_party/WebKit/Source/platform/loader/BUILD.gn +@@ -99,6 +99,7 @@ source_set("loader") { + "//components/link_header_util:link_header_util", + "//mojo/public/cpp/system:system", + "//storage/public/interfaces:interfaces_blink__generator", ++ "//third_party/WebKit/public:mojo_bindings_blink", + ] + + public_deps = [ +diff --git a/third_party/WebKit/public/BUILD.gn b/third_party/WebKit/public/BUILD.gn +index 84ee3ca9136d..cc847663efc6 100644 +--- a/third_party/WebKit/public/BUILD.gn ++++ b/third_party/WebKit/public/BUILD.gn +@@ -755,6 +755,7 @@ mojom("mojo_bindings") { + visibility_blink = [ + "//content/common:mojo_bindings_blink", + "//third_party/WebKit/Source/platform", ++ "//third_party/WebKit/Source/platform/loader", + ] + sources = [ + "platform/display_mode.mojom", +-- +2.14.0 +
