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

Issue 666493004: NaCl: Update revision in DEPS, r13947 -> r13953 (Closed)

Created:
6 years, 2 months ago by Mark Seaborn
Modified:
6 years, 2 months ago
Reviewers:
Derek Schuff
CC:
chromium-reviews, Alexander Potapenko, jvoung - send to chromium..., Mark Seaborn, native-client-reviews_googlegroups.com, nfullagar
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

NaCl: Update revision in DEPS, r13947 -> r13953 This pulls in the following Native Client changes: r13948: (mseaborn) Clean up sys/errno.h's #include guard to match the file's name r13949: (glider) Filter -fsanitize=undefined from cflags when nacl_bootstrap_lib. r13950: (jvoung) Build gold.nexe with a segment gap, and supply -B, to later support IRT use. r13952: (jvoung) Add irt_load_optional flag to chrome_main args. Helps w/ making pnacl ld.nexe use IRT. r13953: (nfullagar) Prep for sdk inclusion - make source more relocatable, so 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/bb964c802787ff8e1a0226fda9ffeb81d7824e3e Cr-Commit-Position: refs/heads/master@{#300592}

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: 7 (2 generated)
Mark Seaborn
6 years, 2 months ago (2014-10-21 20:04:05 UTC) #2
Derek Schuff
lgtm
6 years, 2 months ago (2014-10-21 20:57:21 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/666493004/1
6 years, 2 months ago (2014-10-21 21:28:39 UTC) #5
commit-bot: I haz the power
Committed patchset #1 (id:1)
6 years, 2 months ago (2014-10-22 01:08:41 UTC) #6
commit-bot: I haz the power
6 years, 2 months ago (2014-10-22 04:04:31 UTC) #7
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/bb964c802787ff8e1a0226fda9ffeb81d7824e3e
Cr-Commit-Position: refs/heads/master@{#300592}

Powered by Google App Engine
This is Rietveld 408576698