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

Issue 569133002: NaCl: Update revision in DEPS, r13739 -> r13746 (Closed)

Created:
6 years, 3 months ago by Mark Seaborn
Modified:
6 years, 3 months ago
Reviewers:
Derek Schuff, bradn
CC:
chromium-reviews, bradn, dbrazdil, Derek Schuff, David Yen, jvoung (off chromium), jvoung - send to chromium..., native-client-reviews_googlegroups.com, Sam Clegg
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

NaCl: Update revision in DEPS, r13739 -> r13746 This pulls in the following Native Client changes: r13740: (dschuff) Update revision for PNaCl r13726->r13739 r13741: (dbrazdil) Implemented sysconf(), sbrk(), malloc() in MinSFI. r13742: (dyen) Added testing for file dup/dup2 functions for irt_ext. r13743: (jvoung) Add a checkdeps stage to the PNaCl bots/trybots as well. r13744: (bradnelson) Only allow fastbuild to affect symbol level. r13745: (jvoung) PNaCl: Update Subzero revision in pnacl/COMPONENT_REVISIONS r13746: (sbc) Add creat() declaration to newlib fcntl.h 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/1f866c6ccb585f7d75fc03f65669787bf08754f1 Cr-Commit-Position: refs/heads/master@{#294844}

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: 8 (3 generated)
Mark Seaborn
It looks like Nico broke the ARM build again...
6 years, 3 months ago (2014-09-15 04:49:33 UTC) #2
bradn
lgtm
6 years, 3 months ago (2014-09-15 05:12:41 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patchset/569133002/1
6 years, 3 months ago (2014-09-15 13:57:21 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1) as 646f826efb47c915c0344497a0737c7ae93cf227
6 years, 3 months ago (2014-09-15 17:28:10 UTC) #7
commit-bot: I haz the power
6 years, 3 months ago (2014-09-15 17:32:02 UTC) #8
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/1f866c6ccb585f7d75fc03f65669787bf08754f1
Cr-Commit-Position: refs/heads/master@{#294844}

Powered by Google App Engine
This is Rietveld 408576698