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

Issue 7693028: 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. Fifth attempt to land! 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 R=noelallen@google.com BUG=http://code.google.com/p/chromium/issues/detail?id=93520 TEST=None Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=98106

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+62 lines, -255 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 0 chunks +-1 lines, --1 lines 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: 2 (0 generated)
bradn
Ok, turned it off for chromeos.
9 years, 4 months ago (2011-08-24 19:20:49 UTC) #1
bradn
9 years, 4 months ago (2011-08-24 20:41:05 UTC) #2
Submitting TBR (minor change from previous versions).

Powered by Google App Engine
This is Rietveld 408576698