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

Issue 7701017: Switching NaCl IRT to be built inside the chrome build. (Closed)

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

Description

Switching NaCl IRT to be built inside the chrome build. Fourth attempt: Switching IRT to be built inside the chrome build. Dropping the IRT download step from the DEPS. Step3 of a many step plan to switch where ppapi + irt are built. Dropping download_nacl_irt because we no longer rely on a prebuilt copy. Dropping irt download drop source tarball (assume people using it will have to download / build their own nacl toolchain). Old Review URL: http://codereview.chromium.org/7669058 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=97987

Patch Set 1 #

Patch Set 2 : '' #

Total comments: 1

Patch Set 3 : '' #

Patch Set 4 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+81 lines, -257 lines) Patch
M DEPS View 4 chunks +9 lines, -22 lines 0 comments Download
D build/download_nacl_irt.py View 1 chunk +0 lines, -205 lines 0 comments Download
A + chrome/build_nacl_irt.py View 1 2 3 2 chunks +18 lines, -1 line 0 comments Download
M chrome/nacl.gypi View 2 chunks +54 lines, -18 lines 0 comments Download
M tools/export_tarball/export_tarball.py View 1 chunk +0 lines, -11 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
bradn
9 years, 4 months ago (2011-08-24 00:12:11 UTC) #1
noelallen_use_chromium
LGTM (/w TODO) http://codereview.chromium.org/7701017/diff/2001/chrome/build_nacl_irt.py File chrome/build_nacl_irt.py (right): http://codereview.chromium.org/7701017/diff/2001/chrome/build_nacl_irt.py#newcode63 chrome/build_nacl_irt.py:63: # If things fail on windows, ...
9 years, 4 months ago (2011-08-24 00:21:38 UTC) #2
Dmitry Polukhin
It looks like this CL broke Chrome build inside chroot. chromeos-chrome-15.0.859.0_rc-r1: /home/dpolukhin/chrome_root/src/native_client/toolchain/linux_x86_newlib/bin/x86_64-nacl-gcc -m32 --v -c ...
9 years, 4 months ago (2011-08-24 11:48:32 UTC) #3
noelallen_use_chromium
another set of machines to skip building on like win/devenv? -Noel On Wed, Aug 24, ...
9 years, 4 months ago (2011-08-24 17:22:44 UTC) #4
bradn
Hi Dmitry, I'm concerned that this failed for you (locally?) without making a bot red ...
9 years, 4 months ago (2011-08-24 17:27:48 UTC) #5
Dmitry Polukhin
9 years, 4 months ago (2011-08-24 17:45:27 UTC) #6
It did make bot red but that the bot was on Chrome OS waterfall. Build just
before I reverted your cl:
http://build.chromium.org/p/chromiumos/builders/x86%20generic%20TOT%20chrome%...

Chrome OS bots on Chromium waterfall build Chrome outside chroot.

On Wed, Aug 24, 2011 at 9:27 PM, Bradley Nelson <bradnelson@google.com>wrote:

> Hi Dmitry,
>
> I'm concerned that this failed for you (locally?) without making a bot red
> somewhere?
> Can you give me a little more context?
> At a glance it looks like we'll need to enlarge the set of packages in the
> chroot.
>
> -BradN
>
>
>
> On Wed, Aug 24, 2011 at 4:48 AM, <dpolukhin@chromium.org> wrote:
>
>> It looks like this CL broke Chrome build inside chroot.
>>
>>
>> chromeos-chrome-15.0.859.0_rc-**r1:
>> /home/dpolukhin/chrome_root/**src/native_client/toolchain/**
>> linux_x86_newlib/bin/x86_64-**nacl-gcc
>> -m32 --v -c
>> chromeos-chrome-15.0.859.0_rc-**r1:
>> /home/dpolukhin/chrome_root/**src/native_client/toolchain/**
>> linux_x86_newlib/bin/x86_64-**nacl-gcc:
>> /lib32/libc.so.6: version `GLIBC_2.11' not found (required by
>> /home/dpolukhin/chrome_root/**src/native_client/toolchain/**
>> linux_x86_newlib/bin/x86_64-**nacl-gcc)
>> chromeos-chrome-15.0.859.0_rc-**r1: scons: *** Error 1
>> chromeos-chrome-15.0.859.0_rc-**r1:
>> /home/dpolukhin/chrome_root/**src/native_client/toolchain/**
>> linux_x86_newlib/bin/x86_64-**nacl-gcc
>> -m32 -print-search-dirs
>> chromeos-chrome-15.0.859.0_rc-**r1:
>> /home/dpolukhin/chrome_root/**src/native_client/toolchain/**
>> linux_x86_newlib/bin/x86_64-**nacl-gcc:
>> /lib32/libc.so.6: version `GLIBC_2.11' not found (required by
>> /home/dpolukhin/chrome_root/**src/native_client/toolchain/**
>> linux_x86_newlib/bin/x86_64-**nacl-gcc)
>> chromeos-chrome-15.0.859.0_rc-**r1: scons: *** Error 1
>> chromeos-chrome-15.0.859.0_rc-**r1:
>> /home/dpolukhin/chrome_root/**src/native_client/toolchain/**
>> linux_x86_newlib/bin/x86_64-**nacl-gcc
>> -m32 -print-libgcc-file-name
>> chromeos-chrome-15.0.859.0_rc-**r1:
>> /home/dpolukhin/chrome_root/**src/native_client/toolchain/**
>> linux_x86_newlib/bin/x86_64-**nacl-gcc:
>> /lib32/libc.so.6: version `GLIBC_2.11' not found (required by
>> /home/dpolukhin/chrome_root/**src/native_client/toolchain/**
>> linux_x86_newlib/bin/x86_64-**nacl-gcc)
>> chromeos-chrome-15.0.859.0_rc-**r1: scons: *** Error 1
>>
>> ...
>>
>> chromeos-chrome-15.0.859.0_rc-**r1: Install file:
>> "src/untrusted/stubs/crt1.x" as
>> "scons-out/nacl_irt-x86-32/**obj/src/untrusted/stubs/crt1.**o"
>> chromeos-chrome-15.0.859.0_rc-**r1:
>> /home/dpolukhin/chrome_root/**src/native_client/toolchain/**
>> linux_x86_newlib/bin/x86_64-**nacl-ar
>> rc scons-out/nacl_irt-x86-32/obj/**src/untrusted/stubs/libcrt_**
>> platform.a
>> chromeos-chrome-15.0.859.0_rc-**r1:
>> /home/dpolukhin/chrome_root/**src/native_client/toolchain/**
>> linux_x86_newlib/bin/x86_64-**nacl-ar:
>> error while loading shared libraries: libz.so.1: cannot open shared object
>> file:
>> No such file or directory
>> chromeos-chrome-15.0.859.0_rc-**r1: Install file:
>> "src/shared/platform/nacl_log.**h"
>> as "scons-out/nacl_irt-x86-32/**include/nacl/nacl_log.h"
>> chromeos-chrome-15.0.859.0_rc-**r1: Install file:
>> "src/include/nacl/nacl_**inttypes.h" as
>> "scons-out/nacl_irt-x86-32/**include/nacl/nacl_inttypes.h"
>> chromeos-chrome-15.0.859.0_rc-**r1: scons: ***
>> [scons-out/nacl_irt-x86-32/**obj/src/untrusted/stubs/**libcrt_platform.a]
>> Error 127
>> chromeos-chrome-15.0.859.0_rc-**r1: scons: building terminated because of
>> errors.
>>
>>
>>
>>
http://codereview.chromium.**org/7701017/<http://codereview.chromium.org/7701...
>>
>
>

Powered by Google App Engine
This is Rietveld 408576698