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

Issue 23576014: NaCl: Update revision in DEPS, r12098 -> r12120 (Closed)

Created:
7 years, 3 months ago by Sam Clegg
Modified:
7 years, 3 months ago
CC:
chromium-reviews, binji, Sam Clegg
Visibility:
Public.

Description

NaCl: Update revision in DEPS, r12098 -> r12120 This change also contains chrome-side changes to handle the renaming of the PNaCl toolchain folder within the NaCl SDK. This pulls in the following Native Client changes: r12099: (shcherbina) Add custom tempfile prefix to old validator targeted tests. r12100: (stichnot) Fix and harden error reporting in the report parser. r12101: (stichnot) Exclude ms_struct_pack_layout. r12102: (ncbray) Enable validation caching on ARM. r12103: (petarj) [MIPS] Enable validator tests r12104: (halyavin) Fix copy paste error in the log message. r12105: (khim) Support precise toolchain trybots/buildbots r12106: (kschimpf) Update pnacl llvm revisions r12107: (stichnot) Fix breakage when running the LLVM regression tests. r12108: (jvoung) Handle unicode as strings instead of lists in the pnacl driver env setmany. r12109: (khim) Fix precise64 toolchain bots r12110: (jvoung) Turn off allow_double again, for callingconv test on arm-nacl-gcc. r12111: (kschimpf) Update pnacl llvm revisions. r12112: (bradnelson) Temporarily remove breakpad from DEPS in order to move it. r12114: (bradnelson) Clobbering the breakpad directory. r12115: (petarj) [MIPS] Fix expectations for mips_test_read_only_regs r12116: (sbc) Roll newlib toolchains in TOOL_REVISIONS. r12117: (petarj) [MIPS] Remove unneeded checks for TLS register r12118: (bradnelson) Pull in all of breakpad for crash dump testing. r12119: (sbc) Rename PNaCl toolchain folder within the SDK. BUG=none TEST=browser_tests and nacl_integration Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221664

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+57 lines, -55 lines) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download
M native_client_sdk/src/build_tools/build_sdk.py View 8 chunks +33 lines, -31 lines 0 comments Download
M native_client_sdk/src/build_tools/nacl-mono-runtime.sh View 1 chunk +1 line, -1 line 0 comments Download
M native_client_sdk/src/build_tools/sdk_files.list View 1 chunk +21 lines, -21 lines 0 comments Download
M native_client_sdk/src/tools/nacl_llvm.mk View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5 (0 generated)
Sam Clegg
7 years, 3 months ago (2013-09-05 18:55:42 UTC) #1
binji
lgtm
7 years, 3 months ago (2013-09-05 21:20:39 UTC) #2
jvoung (off chromium)
LGTM too thanks!
7 years, 3 months ago (2013-09-05 22:53:38 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/23576014/1
7 years, 3 months ago (2013-09-06 00:13:59 UTC) #4
commit-bot: I haz the power
7 years, 3 months ago (2013-09-06 09:48:56 UTC) #5
Message was sent while issue was closed.
Change committed as 221664

Powered by Google App Engine
This is Rietveld 408576698