This repository was archived by the owner on Jan 6, 2023. It is now read-only.
Conversation
856be70 to
453b17e
Compare
70047d0 to
e231a77
Compare
42a88e0 to
46b7d69
Compare
5a3168e to
d466b16
Compare
f1958c9 to
23739c2
Compare
f8d80db to
fe30b3f
Compare
23739c2 to
5d55b4e
Compare
ac3f3dd to
8f525de
Compare
423732e to
5fe3812
Compare
5fe3812 to
5d784eb
Compare
1d5f62f to
86a8670
Compare
Contributor
|
I'm rebasing this branch on master. |
16ee275 to
5964d8f
Compare
5964d8f to
dfa5994
Compare
The path to built libyuv lib has changed after the "win: Rename clang_x64 to win_clang_x64 in build/toolchain/win" has landed. See https://crbug.com/748501.
These changes are just backported from the newer Chromium revision. Some of the patches (maybe even all) should be removed during the next Chromium upgrade. List of backported commits: 97f5f9df6988 Switch to Debian Stretch sysroot 1ed08ea9dbb0 Remove duplicated code in sysroot-creator.sh 99b48b47f11f Roll sysroots to include dbus changes 81e7f1791035 Unversion libdbus symbols in Linux sysroot 4dfcf263240a CrOs sysroot: update to Debian stretch and add libva* packages 140d82d6f8e2 Use xz level 9 compression for sysroots b4ac34edef3a Make linux sysroot script print nothing in the happy case where it needs to do nothing. 95ecb913508e Add dconf to the sysroot a1d30dcfbaa3 MIPS64: Add support for Stretch to sysroot scripts
Original commit:
commit 8a62aea7a676b02ef09717aabd7b4213420893fc
Author: Dominic Mazzoni <dmazzoni@chromium.org>
Date: Fri Sep 1 22:03:03 2017 +0000
Fix win clang build
Update clang to fix https://bugs.llvm.org/show_bug.cgi?id=34159 Original commit: commit 01f121e242233f5fbbe2669d5d10df6fc0682949 Author: Hans Wennborg <hans@chromium.org> Date: Fri Sep 8 16:45:43 2017 +0000 Roll Clang 310694-2:312679 And remove the local patch
It seems like sysroot is expected to have it but it doesn't. We should probably upstream this change or update the sysroot.
We've got a new sysroot from the upstream, but not the code fixes. This patch should be removed after the upgrade to Chromium 63 or 64.
Node.js was updated in the master, we have to remove V8's patches added during that update. Those patches are either not needed for the Node.js 9.2.1, or were already added in the Ch62 upgrade branch.
This is needed for service worker scripts, whose behavior was changed with script streaming https://chromium-review.googlesource.com/c/chromium/src/+/597033
dfa5994 to
5cb70dc
Compare
5cb70dc to
322eb12
Compare
zcbenz
approved these changes
Feb 26, 2018
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR to Electron: electron/electron#11230
Project board: https://github.com/electron/electron/projects/6