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

Issue 881893003: NaCl: Update revision in DEPS, 945d33419 -> 939a13aa3 (Closed)

Created:
5 years, 10 months ago by bradn
Modified:
5 years, 10 months ago
Reviewers:
Mark Seaborn, JF
CC:
chromium-reviews, bradn, Derek Schuff, JF, jvoung (off chromium), native-client-reviews_googlegroups.com, Sam Clegg, teravest
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

NaCl: Update revision in DEPS, 945d33419 -> 939a13aa3 This pulls in the following Native Client changes: 939a13a (teravest@chromium.org) Provide an irt_core_lib GN target. dbf2af4 (sbc@chromium.org) Remove include of build_config.h from nacl_exception.h e205317 (jfb@chromium.org) PNaCl: Update LLVM revision in pnacl/COMPONENT_REVISIONS 7c8a96b (jfb@chromium.org) Update revision for PNaCl 3d00a54 (jfb@chromium.org) PNaCl: Update LLVM revision in pnacl/COMPONENT_REVISIONS 1de8ba8 (mseaborn@chromium.org) Update codereview.settings following the switch from SVN to Git 7a6cf46 (jfb@chromium.org) Update revision for PNaCl bc28c0b (jvoung@chromium.org) Fix the native_client URL in the PNaCl REV file. f8aa9dc (bradnelson@google.com) Adding le32-nacl build of pnacl tools. cfda521 (teravest@chromium.org) Explicitly include build_config.h. 61b7ba9 (teravest@chromium.org) Output depfile from build_nexe. 2d65c23 (jfb@chromium.org) PNaCl: Update LLVM and Subzero revision in pnacl/COMPONENT_REVISIONS 5f5cd88 (sbc@chromium.org) Fix PNaCl redirectors for i686-nacl-xx -> x86_64-nacl-xx 1baa204 (jvoung@chromium.org) Remove some more dead code and variables from pnacl/build.sh. f116471 (dschuff@chromium.org) Update revision for PNaCl 01f3d97 (teravest@chromium.org) Always define product_dir in build_nexe. 8a5a846 (mseaborn@chromium.org) Remove the IRT's now-unneeded call to NaClSrpcModuleInit() c14b58a (mseaborn@chromium.org) Remove NaClPluginLowLevelInitializationComplete() (caller of "init_done" SRPC) c5d8394 (bradnelson@google.com) Change driver bash scripts to work in devenv. 490e554 (dschuff@chromium.org) PNaCl: Update Clang revision in pnacl/COMPONENT_REVISIONS 2a7391a (sbc@chromium.org) Fix several issues with update_pnacl_tool_revisions.py fbc3b5c (dschuff@chromium.org) Support filenames with spaces in the PNaCl driver scripts. 4f726e8 (sbc@chromium.org) PNaCl: Update Clang revision in pnacl/COMPONENT_REVISIONS 28a1341 (dschuff@chromium.org) Move pnacl driver test runs from buildbot_pnacl.sh to buildbot_pnacl_toolchain.py 9f9d773 (sbc@chromium.org) Update GCC revision in tools/REVISIONS 29ae54a (jfb@chromium.org) Update revision for PNaCl BUG=none TEST=browser_tests and nacl_integration CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_rel_precise32,linux_arm_compile,linux_nacl_sdk_build Committed: https://crrev.com/4a8dee9ac9fd757f776f76a651d921f347305e82 Cr-Commit-Position: refs/heads/master@{#313447}

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: 9 (3 generated)
bradn
5 years, 10 months ago (2015-01-28 02:42:03 UTC) #2
Mark Seaborn
LGTM
5 years, 10 months ago (2015-01-28 02:52:50 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/881893003/1
5 years, 10 months ago (2015-01-28 02:54:11 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 10 months ago (2015-01-28 04:26:20 UTC) #7
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/4a8dee9ac9fd757f776f76a651d921f347305e82 Cr-Commit-Position: refs/heads/master@{#313447}
5 years, 10 months ago (2015-01-28 04:27:46 UTC) #8
Nico
5 years, 10 months ago (2015-01-28 15:46:00 UTC) #9
Message was sent while issue was closed.
A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/886513002/ by thakis@chromium.org.

The reason for reverting is: Caused http://crbug.com/452846.

Powered by Google App Engine
This is Rietveld 408576698