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

Issue 475243002: NaCl: Update revision in DEPS, r13613 -> r13623 (Closed)

Created:
6 years, 4 months ago by Mark Seaborn
Modified:
6 years, 4 months ago
Reviewers:
Derek Schuff
CC:
chromium-reviews, bradn, Derek Schuff, hidehiko, Mark Seaborn, native-client-reviews_googlegroups.com
Project:
chromium
Visibility:
Public.

Description

NaCl: Update revision in DEPS, r13613 -> r13623 This pulls in the following Native Client changes: r13614: (bradnelson) Vary build settings on fastbuild + configuration + buildtype. r13615: (bradnelson) Only pass -O2 as a link option for pnacl test pexes. r13616: (dschuff) Build sandboxed translator with libcxx r13617: (hidehiko) Non-SFI Mode: Fix linux_termios size. r13618: (mseaborn) Disable the imc_socketpair() syscall by default r13619: (mseaborn) Cleanup: Remove Python bindings for NaClDescs and IMC r13620: (dschuff) Update revision for PNaCl r13609->r13616 r13621: (mseaborn) Disable the imc_makeboundsock() syscall by default r13622: (dschuff) Build PNaCl debug/fastbuild IRT with O2 instead of O0 r13623: (mseaborn) Revert r13622, r13615 and r13614 (changes to the untrusted Gyp build) BUG=none TEST=browser_tests and nacl_integration CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_rel_precise32,linux_nacl_sdk_build Committing with NOTRY=true otherwise the CQ is blocked waiting for android_aosp, which isn't affected by NaCl changes. The other trybots are green. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=289864

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 (0 generated)
Mark Seaborn
Nth time lucky...
6 years, 4 months ago (2014-08-14 23:40:54 UTC) #1
Mark Seaborn
The CQ bit was checked by mseaborn@chromium.org
6 years, 4 months ago (2014-08-15 00:24:47 UTC) #2
Derek Schuff
lgtm
6 years, 4 months ago (2014-08-15 00:25:30 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mseaborn@chromium.org/475243002/1
6 years, 4 months ago (2014-08-15 00:28:21 UTC) #4
Mark Seaborn
The CQ bit was unchecked by mseaborn@chromium.org
6 years, 4 months ago (2014-08-15 05:59:20 UTC) #5
Mark Seaborn
The CQ bit was checked by mseaborn@chromium.org
6 years, 4 months ago (2014-08-15 05:59:29 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mseaborn@chromium.org/475243002/1
6 years, 4 months ago (2014-08-15 06:00:05 UTC) #7
commit-bot: I haz the power
6 years, 4 months ago (2014-08-15 15:21:58 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (1) as 289864

Powered by Google App Engine
This is Rietveld 408576698