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

Issue 10854022: NaCl: Update revision in DEPS, r9370 -> r9391 (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, Austin Benson, Brad Chen, bradn, Derek Schuff, (google.com) Derek Schuff, jvoung - send to chromium..., khimg, Karl, Mark Seaborn, Petr Hosek, robertm, sehr (please use chromium), Vlad Shcherbina
Visibility:
Public.

Description

NaCl: Update revision in DEPS, r9370 -> r9391 This pulls in the following Native Client changes: r9371: (khim) Add missing REXW case r9372: (khim) Add validator_internals.html documentation r9373: (khim) Add branch hints to j[er]cxz and loop{,e,ne} instructions r9374: (robertm) PNaCl toolchain: Remove unused binutils static libs and headers r9375: (kschimpf) Remove unnecessary inlines in methods defined within the class definition. r9376: (sehr) Get negative displacement change. r9377: (dschuff) Get gold RPC error update r9378: (arbenson) Move low page protection to nacl_helper_bootstrap r9379: (bradnelson) Revert 9378 - Move low page protection to nacl_helper_bootstrap r9380: (shcherbina) Validator_ragel tests: make --nthreads optional r9381: (robertm) Add step for packaging tranlators and use it on one buildbot. r9382: (jvoung) Allow slower AllocAddrSpace step for xp-bare-metal bot. r9383: (mseaborn) Testing: Reuse RegsAssertEqual() in the trusted-code part of tests r9384: (robertm) Fix missing env variable renames. r9385: (arbenson) Move low page protection to nacl_helper_bootstrap r9386: (dschuff) Update Pnacl toolchain DEPS to 9380 r9387: (khim) Improve compatibility with existing validator r9388: (phosek) Removed all NaClAppVmmap* functions since they are no longer being used. r9389: (bradchen) Make ragel_tester more strict, to align better with nacl_tester r9390: (khim) Fix OWNERS for validator_ragel r9391: (shcherbina) validator_ragel: simplify scons definition of dfagen BUG=none TEST=nacl_integration Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=150377

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-08-07 18:08:08 UTC) #1
bradn
LGTM On Tue, Aug 7, 2012 at 11:08 AM, <mseaborn@chromium.org> wrote: > Reviewers: bradn, > ...
8 years, 4 months ago (2012-08-07 18:10:54 UTC) #2
bradn
8 years, 4 months ago (2012-08-07 18:10:55 UTC) #3
LGTM


On Tue, Aug 7, 2012 at 11:08 AM, <mseaborn@chromium.org> wrote:

> Reviewers: bradn,
>
> Description:
> NaCl: Update revision in DEPS, r9370 -> r9391
>
> This pulls in the following Native Client changes:
>
> r9371: (khim) Add missing REXW case
> r9372: (khim) Add validator_internals.html documentation
> r9373: (khim) Add branch hints to j[er]cxz and loop{,e,ne} instructions
> r9374: (robertm) PNaCl toolchain: Remove unused binutils static libs and
> headers
> r9375: (kschimpf) Remove unnecessary inlines in methods defined within the
> class
> definition.
> r9376: (sehr) Get negative displacement change.
> r9377: (dschuff) Get gold RPC error update
> r9378: (arbenson) Move low page protection to nacl_helper_bootstrap
> r9379: (bradnelson) Revert 9378 - Move low page protection to
> nacl_helper_bootstrap
> r9380: (shcherbina) Validator_ragel tests: make --nthreads optional
> r9381: (robertm) Add step for packaging tranlators and use it on one
> buildbot.
> r9382: (jvoung) Allow slower AllocAddrSpace step for xp-bare-metal bot.
> r9383: (mseaborn) Testing: Reuse RegsAssertEqual() in the trusted-code
> part of
> tests
> r9384: (robertm) Fix missing env variable renames.
> r9385: (arbenson) Move low page protection to nacl_helper_bootstrap
> r9386: (dschuff) Update Pnacl toolchain DEPS to 9380
> r9387: (khim) Improve compatibility with existing validator
> r9388: (phosek) Removed all NaClAppVmmap* functions since they are no
> longer
> being used.
> r9389: (bradchen) Make ragel_tester more strict, to align better with
> nacl_tester
> r9390: (khim) Fix OWNERS for validator_ragel
> r9391: (shcherbina) validator_ragel: simplify scons definition of dfagen
>
> BUG=none
> TEST=nacl_integration
>
>
> Please review this at
http://codereview.chromium.**org/10854022/<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 80688b5e8f92bcc2222abce78fbca8**0630457e15..**
> 102636b9e264d222685db526f99540**ad45082155 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": "9370",
> +  "nacl_revision": "9391",
>    # After changing nacl_revision, run 'glient sync' and check
> native_client/DEPS
>    # to update other nacl_*_revision's.
>    "nacl_tools_revision": "9330",  # native_client/DEPS: tools_rev
>
>
>

Powered by Google App Engine
This is Rietveld 408576698