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

Issue 6331011: Update NaCl revision in DEPS (Closed)

Created:
9 years, 11 months ago by Mark Seaborn
Modified:
9 years, 5 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Update NaCl revision in DEPS BUG=none TEST=trybots (in particular, nacl_ui_tests) Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=72362

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
I thought we should get a DEPS update in early in the day.
9 years, 11 months ago (2011-01-24 18:32:17 UTC) #1
sehr (please use chromium)
On 2011/01/24 18:32:17, Mark Seaborn wrote: > I thought we should get a DEPS update ...
9 years, 11 months ago (2011-01-24 18:44:15 UTC) #2
cstefansen
9 years, 11 months ago (2011-01-24 18:45:55 UTC) #3
LGTM. As Sehr stated we'll need another DEPS bump today, but there is no harm is
doing this now.

On 2011/01/24 18:44:15, sehr wrote:
> On 2011/01/24 18:32:17, Mark Seaborn wrote:
> > I thought we should get a DEPS update in early in the day.
> 
> This will make the surf-away hang obvious to everyone -- but not the fix that
> I'm working on.  LGTM, but we'll need another one today...

Powered by Google App Engine
This is Rietveld 408576698