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

Issue 781103002: NaCl: Update revision in DEPS, r14147 -> r14178 (Closed)

Created:
6 years ago by Mark Seaborn
Modified:
6 years ago
Reviewers:
Derek Schuff
CC:
chromium-reviews, bradn, Derek Schuff, David Yen, hamaji, JF, jvoung - send to chromium..., Roland McGrath, Mark Seaborn, native-client-reviews_googlegroups.com, Nick Bray (chromium), petarj, shyamsundarr, teravest
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

NaCl: Update revision in DEPS, r14147 -> r14178 This pulls in the following Native Client changes: r14148: (dschuff) PNaCl: Update LLVM/clang/binutils-x86 revision in pnacl/COMPONENT_REVISIONS r14149: (hamaji) Update revision for PNaCl r14144->r14148 r14150: (petarj) Fix issue in DecodeEscaping() in debug_stub test r14151: (mseaborn) Remove much of the unused parts of pnacl/build.sh r14152: (bradnelson) Nest the pnacl section of untrusted.gypi to the same level. r14153: (dyen) The cached results from toolchain_build should be deterministic. r14154: (jvoung) Add options for base dir and redirecting files to IRT open resource and use it. r14155: (dschuff) nacl-clang: use gas by default instead of the LLVM assembler r14156: (dyen) Toolchain testing within the NaCl tree now use the raw variants. r14157: (teravest) Clean up unused NaClDesc quota code. r14158: (dschuff) Revert SCons portion of r14155 r14159: (teravest) Remove unnecessary PostMessage logic. r14160: (mseaborn) Cleanup: Remove TLS_PRESUMED_ALIGNMENT from tls.c r14161: (mseaborn) Cleanup: Use Bit('build_ARCH') instead of Bit('target_ARCH') r14162: (dschuff) Update revision for PNaCl r14148->r14155 r14163: (ncbray) GN: add a base target for the NaCl build. r14164: (dschuff) Switch nacl-clang back to gas in SCons r14165: (dschuff) PNaCl: Update LLVM/binutils-x86 revision in pnacl/COMPONENT_REVISIONS r14166: (dyen) Made buildbot PNaCl toolchain test extract using packages script. r14167: (dyen) Fixed PNaCl buildbot test by adding extract argument to command. r14168: (dschuff) Update revision for PNaCl r14155->r14165 r14169: (jvoung) Remove __X_LEAST{8,16,32}_TYPE__ hack for appeasing GCC torture tests w/ Clang. r14170: (dschuff) Re-enable exception_tests for nacl_clang on mac r14171: (mcgrathr) Update binutils revision in tools/REVISIONS r14173: (jfb) Use static split module scheduling when using pnacl-translate.py r14174: (shyamsundarr) cleanup scons test rules for validator proofs a bit. r14175: (jfb) Update revision for PNaCl r14165->r14173 r14176: (bradnelson) Revert "Nest the pnacl section of untrusted.gypi to the same level." r14177: (mseaborn) Disable the getdents() syscall by default for security hardening r14178: (dschuff) PNaCl: Update Clang revision in pnacl/COMPONENT_REVISIONS BUG=none TEST=browser_tests and nacl_integration CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_rel_precise32,linux_arm_compile,linux_nacl_sdk Committed: https://crrev.com/40da97400ffbf1a9b508b87338d14a27c5163d7c Cr-Commit-Position: refs/heads/master@{#307089}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 18 (8 generated)
Mark Seaborn
6 years ago (2014-12-05 20:41:05 UTC) #2
Derek Schuff
lgtm
6 years ago (2014-12-05 20:42:03 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/781103002/1
6 years ago (2014-12-05 20:51:02 UTC) #7
commit-bot: I haz the power
Try jobs failed on following builders: linux_chromium_dbg_ng on tryserver.chromium.linux (http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_dbg_ng/builds/28)
6 years ago (2014-12-05 21:00:07 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/781103002/1
6 years ago (2014-12-05 21:02:52 UTC) #11
commit-bot: I haz the power
Try jobs failed on following builders: linux_chromium_dbg_ng on tryserver.chromium.linux (http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_dbg_ng/builds/35)
6 years ago (2014-12-05 21:11:18 UTC) #13
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/781103002/1
6 years ago (2014-12-05 21:15:34 UTC) #15
commit-bot: I haz the power
Committed patchset #1 (id:1)
6 years ago (2014-12-05 21:57:02 UTC) #16
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/40da97400ffbf1a9b508b87338d14a27c5163d7c Cr-Commit-Position: refs/heads/master@{#307089}
6 years ago (2014-12-05 21:57:48 UTC) #17
M-A Ruel
6 years ago (2014-12-06 16:12:57 UTC) #18
Message was sent while issue was closed.
On 2014/12/05 21:57:48, I haz the power (commit-bot) wrote:
> Patchset 1 (id:??) landed as
> https://crrev.com/40da97400ffbf1a9b508b87338d14a27c5163d7c
> Cr-Commit-Position: refs/heads/master@{#307089}

http://build.chromium.org/p/chromium.swarm/builders/Linux%20deterministic%20b...
49 files are equal, 0 are different.

OMG OMG thanks to everyone involved!

Powered by Google App Engine
This is Rietveld 408576698