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

Issue 7761004: Adding a special toolchain that will work with chromeos. (Closed)

Created:
9 years, 4 months ago by bradn
Modified:
9 years, 4 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Adding a special toolchain that will work with chromeos. BUG=http://code.google.com/p/chromium-os/issues/detail?id=19719 TEST=None R=noelallen@google.com,zelidrag@google.com Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=98492

Patch Set 1 #

Total comments: 1

Patch Set 2 : '' #

Patch Set 3 : '' #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -6 lines) Patch
M DEPS View 1 2 1 chunk +1 line, -1 line 0 comments Download
M build/download_nacl_toolchains.py View 1 2 chunks +13 lines, -3 lines 2 comments Download
M chrome/nacl.gypi View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
bradn
9 years, 4 months ago (2011-08-26 20:28:40 UTC) #1
noelallen_use_chromium
http://codereview.chromium.org/7761004/diff/1/build/download_nacl_toolchains.py File build/download_nacl_toolchains.py (right): http://codereview.chromium.org/7761004/diff/1/build/download_nacl_toolchains.py#newcode13 build/download_nacl_toolchains.py:13: # Handle chromeos=1 specially (until its building its own ...
9 years, 4 months ago (2011-08-26 21:05:35 UTC) #2
bradn
PTAL
9 years, 4 months ago (2011-08-26 21:17:39 UTC) #3
noelallen_use_chromium
LGTM
9 years, 4 months ago (2011-08-26 21:37:29 UTC) #4
Mark Seaborn
http://codereview.chromium.org/7761004/diff/3001/build/download_nacl_toolchains.py File build/download_nacl_toolchains.py (right): http://codereview.chromium.org/7761004/diff/3001/build/download_nacl_toolchains.py#newcode18 build/download_nacl_toolchains.py:18: '8337d5ec327d857a49b500723ec9b792f4973abc', So who is going to update this? How ...
9 years, 4 months ago (2011-08-26 22:26:09 UTC) #5
bradn
9 years, 4 months ago (2011-08-26 22:37:16 UTC) #6
I built in on my desk in the chromeos chroot (zelidrag, noel, and I agreed
to this offline, as it continue to allow people experimenting with nacl in
chromeos to make progress).
This is a stopgap until we have an ebuild for chromeos (which much as Ivan
claimed currently just requires turning off gdb).
Victor has committed to shepherding out IRT impacting changes to the newlib
toolchain to the relevant places (chromeos + distros).
This obviously isn't sufficient for ubuntu, but will unblock chromeos
people.

-BradN

On Fri, Aug 26, 2011 at 3:26 PM, <mseaborn@chromium.org> wrote:

>
> http://codereview.chromium.**org/7761004/diff/3001/build/**
>
download_nacl_toolchains.py<http://codereview.chromium.org/7761004/diff/3001/build/download_nacl_toolchains.py>
>
> File build/download_nacl_**toolchains.py (right):
>
> http://codereview.chromium.**org/7761004/diff/3001/build/**
>
download_nacl_toolchains.py#**newcode18<http://codereview.chromium.org/7761004/diff/3001/build/download_nacl_toolchains.py#newcode18>
> build/download_nacl_**toolchains.py:18:
> '**8337d5ec327d857a49b500723ec9b7**92f4973abc',
> So who is going to update this?  How long do you intend this hack to
> last for?
>
> http://codereview.chromium.**org/7761004/diff/3001/build/**
>
download_nacl_toolchains.py#**newcode20<http://codereview.chromium.org/7761004/diff/3001/build/download_nacl_toolchains.py#newcode20>
> build/download_nacl_**toolchains.py:20:
> 'nativeclient-archive2/**special_chromeos'),
> Who uploaded this, and how was it built?
>
>
>
http://codereview.chromium.**org/7761004/<http://codereview.chromium.org/7761...
>

Powered by Google App Engine
This is Rietveld 408576698