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

Issue 10170050: NaCl: Update revision in DEPS, r8414 -> r8430 (Closed)

Created:
8 years, 7 months ago by vissi
Modified:
8 years, 7 months ago
CC:
chromium-reviews, native-client-reviews_googlegroups.com, bbudge, Brad Chen, bsy, (google.com) Derek Schuff, jvoung - send to chromium..., khimg, robertm
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

NaCl: Update toolchain revision in DEPS, r8169 -> r8428 BUG=none TEST=nacl_integration Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=134483

Patch Set 1 #

Patch Set 2 : removed nacl bump to make less impact #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -13 lines) Patch
M DEPS View 1 1 chunk +13 lines, -13 lines 0 comments Download

Messages

Total messages: 21 (0 generated)
vissi
8 years, 7 months ago (2012-04-28 14:33:41 UTC) #1
vissi
On 2012/04/28 14:33:41, vissi wrote: Updated to make less impact by not bumping NaCl toolchain ...
8 years, 7 months ago (2012-04-28 14:45:34 UTC) #2
Mark Seaborn
On 2012/04/28 14:45:34, vissi wrote: > Updated to make less impact by not bumping NaCl ...
8 years, 7 months ago (2012-04-28 17:35:24 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/vissi@google.com/10170050/6001
8 years, 7 months ago (2012-04-28 17:35:38 UTC) #4
vissi
Thanks. Btw, I submitted a tiny patch to your bump script at github, and @pasko ...
8 years, 7 months ago (2012-04-28 17:59:27 UTC) #5
vissi
Thanks. Btw, I submitted a tiny patch to your bump script at github, and @pasko ...
8 years, 7 months ago (2012-04-28 17:59:29 UTC) #6
commit-bot: I haz the power
Try job failure for 10170050-6001 (retry) on win_rel for step "browser_tests". It's a second try, ...
8 years, 7 months ago (2012-04-28 20:18:20 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/vissi@google.com/10170050/6001
8 years, 7 months ago (2012-04-28 21:59:18 UTC) #8
commit-bot: I haz the power
Change committed as 134483
8 years, 7 months ago (2012-04-29 00:01:16 UTC) #9
Nico
Hi, this turned check_licenses on the linux clobber builder red. The error message looks like ...
8 years, 7 months ago (2012-04-29 01:06:15 UTC) #10
Mark Seaborn
Evgeny and Andrey added GDB to the NaCl toolchain recently, I think. Mark On 28 ...
8 years, 7 months ago (2012-04-29 01:24:45 UTC) #11
Mark Seaborn
Evgeny and Andrey added GDB to the NaCl toolchain recently, I think. Mark On 28 ...
8 years, 7 months ago (2012-04-29 01:24:46 UTC) #12
khim
On Sun, Apr 29, 2012 at 5:24 AM, Mark Seaborn <mseaborn@chromium.org> wrote: > Evgeny and ...
8 years, 7 months ago (2012-04-29 09:46:50 UTC) #13
khim
On Sun, Apr 29, 2012 at 5:24 AM, Mark Seaborn <mseaborn@chromium.org> wrote: > Evgeny and ...
8 years, 7 months ago (2012-04-29 09:46:51 UTC) #14
Nico
Please check if it's OK to use gpl3 software at Google (see the page linked ...
8 years, 7 months ago (2012-04-29 14:48:30 UTC) #15
Nico
Please check if it's OK to use gpl3 software at Google (see the page linked ...
8 years, 7 months ago (2012-04-29 14:48:30 UTC) #16
khim
On Sun, Apr 29, 2012 at 6:48 PM, Nico Weber <thakis@chromium.org> wrote: > Please check ...
8 years, 7 months ago (2012-04-29 15:46:54 UTC) #17
khim
On Sun, Apr 29, 2012 at 6:48 PM, Nico Weber <thakis@chromium.org> wrote: > Please check ...
8 years, 7 months ago (2012-04-29 15:46:55 UTC) #18
Nico
Please follow http://www.chromium.org/developers/adding-3rd-party-libraries (email open-source-third-party-reviews@google.com to check that the license change is ok, then update ...
8 years, 7 months ago (2012-04-29 16:15:18 UTC) #19
Nico
Please follow http://www.chromium.org/developers/adding-3rd-party-libraries (email open-source-third-party-reviews@google.com to check that the license change is ok, then update ...
8 years, 7 months ago (2012-04-29 16:15:19 UTC) #20
Avi (use Gerrit)
8 years, 7 months ago (2012-04-30 14:37:59 UTC) #21
On Sun, Apr 29, 2012 at 11:46 AM, Victor Khimenko <khim@chromium.org> wrote:

> I don't know why GPLv3-licensed files
> in native_client/toolchain/linux_x86/bin
> and native_client/toolchain/linux_x86_newlib/bin are accepted by script
> while identically-licensed
> files native_client/toolchain/linux_x86/share/gdb/python/gdb
> and native_client/toolchain/linux_x86_newlib/share/gdb/python/gdb are
> rejected.
>

If you don't understand why, then it's your responsibility to figure it
out. It's unacceptable to land a change that turns a major builder red and
leave the builder burning for two days until the build sheriff comes in on
Monday morning to clean up the mess.

The reason, BTW, is that the license is slightly different in GDB than in
the existing files. I'm extending the whitelist to fix the problem.

Avi

Powered by Google App Engine
This is Rietveld 408576698