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

Issue 10827070: NaCl: Update revision in DEPS, r9262 -> r9277 (Closed)

Created:
8 years, 4 months ago by Mark Seaborn
Modified:
8 years, 4 months ago
Reviewers:
bradn
CC:
chromium-reviews, native-client-reviews_googlegroups.com, (google.com) Derek Schuff, eaeltsin, halyavin, jvoung - send to chromium..., Mark Seaborn, pasko-google - do not use, robertm
Visibility:
Public.

Description

NaCl: Update revision in DEPS, r9262 -> r9277 This pulls in the following Native Client changes: r9263: (eaeltsin) Debug stub: make gdb tests out file names consistent r9264: (halyavin) Update gdb revision. r9265: (jvoung) Bump chrome revision to 148535 in NaCl. r9267: (jvoung) Properly guard the known-bug check introduced in 9254. Skip it if there r9268: (robertm) Remove LLVM tblgen binaries from the PNaCl toolchain r9269: (robertm) Do not export header files we decided should not be part of the untrusted toolchains. r9270: (mseaborn) ARM: Rename the "new_eip" field in NaClThreadContext to "new_prog_ctr" r9271: (jvoung) Revert "Bump chrome revision to 148535 in NaCl." r9272: (dschuff) Bitcode stability test with archived frontend and current backend r9274: (mseaborn) Implement a sigreturn-like NaClSwitchAllRegs() operation r9275: (dschuff) Remove archived pexe scons tests r9276: (dschuff) Change displayed CommandTest name for pnacl r9277: (pasko) validator_ragel: allow multiple errors in tests BUG=none TEST=nacl_integration Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=148762

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: 3 (0 generated)
Mark Seaborn
8 years, 4 months ago (2012-07-27 16:33:05 UTC) #1
bradn
LGTM On Fri, Jul 27, 2012 at 9:33 AM, <mseaborn@chromium.org> wrote: > Reviewers: bradn, > ...
8 years, 4 months ago (2012-07-27 17:00:23 UTC) #2
bradn
8 years, 4 months ago (2012-07-27 17:00:23 UTC) #3
LGTM


On Fri, Jul 27, 2012 at 9:33 AM, <mseaborn@chromium.org> wrote:

> Reviewers: bradn,
>
> Description:
> NaCl: Update revision in DEPS, r9262 -> r9277
>
> This pulls in the following Native Client changes:
>
> r9263: (eaeltsin) Debug stub: make gdb tests out file names consistent
> r9264: (halyavin) Update gdb revision.
> r9265: (jvoung) Bump chrome revision to 148535 in NaCl.
> r9267: (jvoung) Properly guard the known-bug check introduced in 9254.
> Skip it
> if there
> r9268: (robertm) Remove LLVM tblgen binaries from the PNaCl toolchain
> r9269: (robertm) Do not export header files we decided should not be part
> of the
> untrusted toolchains.
> r9270: (mseaborn) ARM: Rename the "new_eip" field in NaClThreadContext to
> "new_prog_ctr"
> r9271: (jvoung) Revert "Bump chrome revision to 148535 in NaCl."
> r9272: (dschuff) Bitcode stability test with archived frontend and current
> backend
> r9274: (mseaborn) Implement a sigreturn-like NaClSwitchAllRegs() operation
> r9275: (dschuff) Remove archived pexe scons tests
> r9276: (dschuff) Change displayed CommandTest name for pnacl
> r9277: (pasko) validator_ragel: allow multiple errors in tests
>
> BUG=none
> TEST=nacl_integration
>
>
> Please review this at
http://codereview.chromium.**org/10827070/<http://codereview.chromium.org/108...
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/src<http://svn.chromium.org/chrome/trunk/src>
>
> Affected files:
>   M DEPS
>
>
> Index: DEPS
> diff --git a/DEPS b/DEPS
> index 8dea1486c3050b278ffa8c7c98ad18**ff9cf1977f..**
> 2e094c770acd233be464b35a8f1e3c**75640d1d5d 100644
> --- a/DEPS
> +++ b/DEPS
> @@ -12,7 +12,7 @@ vars = {
>    "chromium_git": "http://git.chromium.org/git",
>    "chromiumos_git":
"http://git.chromium.org/**chromiumos<http://git.chromium.org/chromiumos>
> ",
>    "swig_revision": "69281",
> -  "nacl_revision": "9262",
> +  "nacl_revision": "9277",
>    # After changing nacl_revision, run 'glient sync' and check
> native_client/DEPS
>    # to update other nacl_*_revision's.
>    "nacl_tools_revision": "9016",  # native_client/DEPS: tools_rev
>
>
>

-- 
You received this message because you are subscribed to the Google Groups
"Native-Client-Reviews" group.
To post to this group, send email to native-client-reviews@googlegroups.com.
To unsubscribe from this group, send email to
native-client-reviews+unsubscribe@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/native-client-reviews?hl=en.

Powered by Google App Engine
This is Rietveld 408576698