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

Issue 654413002: NaCl: Update revision in DEPS, r13915 -> r13921 (Closed)

Created:
6 years, 2 months ago by Mark Seaborn
Modified:
6 years, 2 months ago
Reviewers:
Derek Schuff, Sam Clegg
CC:
chromium-reviews, Derek Schuff, Roland McGrath, Mark Seaborn, native-client-reviews_googlegroups.com
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

NaCl: Update revision in DEPS, r13915 -> r13921 This pulls in the following Native Client changes: r13916: (mseaborn) Remove some mostly-unused debug logging for printing memory layout r13917: (sbc) Update newlib version r13918: (mcgrathr) toolchain_build: Update gcc revision r13919: (sbc) Report S_IFCHR in windows sel_ldr stat()/fstat() calls. r13920: (dschuff) PNaCl: Update LLVM revision in pnacl/COMPONENT_REVISIONS r13921: (sbc) Update newlib-based toolchain versions. BUG=none TEST=browser_tests and nacl_integration CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_rel_precise32,linux_nacl_sdk,linux_chromium_dbg Committed: https://crrev.com/69cb1fd68924bb0d915e8c630c9439cc3681dcff Cr-Commit-Position: refs/heads/master@{#300008}

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: 10 (4 generated)
Mark Seaborn
6 years, 2 months ago (2014-10-16 17:32:29 UTC) #2
Sam Clegg
On 2014/10/16 17:32:29, Mark Seaborn wrote: lgtm. please wait for nacl sdk trybots.
6 years, 2 months ago (2014-10-16 18:00:19 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/654413002/1
6 years, 2 months ago (2014-10-16 20:29:40 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/654413002/1
6 years, 2 months ago (2014-10-16 23:32:08 UTC) #8
commit-bot: I haz the power
Committed patchset #1 (id:1)
6 years, 2 months ago (2014-10-16 23:33:52 UTC) #9
commit-bot: I haz the power
6 years, 2 months ago (2014-10-16 23:34:31 UTC) #10
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/69cb1fd68924bb0d915e8c630c9439cc3681dcff
Cr-Commit-Position: refs/heads/master@{#300008}

Powered by Google App Engine
This is Rietveld 408576698