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

Issue 6719038: Revert r4644, "Updated NaCl's build system to use SCons 2.0.1." (Closed)

Created:
9 years, 9 months ago by Mark Seaborn
Modified:
9 years, 5 months ago
Reviewers:
Nick Bray, robertm
CC:
native-client-reviews_googlegroups.com
Visibility:
Public.

Description

Revert r4644, "Updated NaCl's build system to use SCons 2.0.1." The change produced two failures: 1) The Windows built_elsewhere=1 bots fail with: scons: *** ('No installed VCs',) 2) The Mac toolchain buildbot fails with the following when running Scons: distutils.errors.DistutilsPlatformError: $MACOSX_DEPLOYMENT_TARGET mismatch: now "10.5" but "10.6" during configure BUG=http://code.google.com/p/nativeclient/issues/detail?id=1250 TEST=none Committed: http://src.chromium.org/viewvc/native_client?view=rev&revision=4647

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -30 lines) Patch
M src/native_client/DEPS View 2 chunks +3 lines, -4 lines 0 comments Download
M src/native_client/SConstruct View 2 chunks +2 lines, -1 line 0 comments Download
M src/native_client/scons View 1 chunk +2 lines, -2 lines 0 comments Download
src/native_client/scons.bat View 2 chunks +2 lines, -2 lines 0 comments Download
M src/native_client/site_scons/site_init.py View 1 chunk +1 line, -1 line 0 comments Download
M src/native_client/site_scons/site_tools/component_builders.py View 1 chunk +0 lines, -11 lines 0 comments Download
M src/native_client/site_scons/site_tools/target_platform_windows.py View 3 chunks +6 lines, -8 lines 0 comments Download
M src/native_client/tools/modular-build/treemappers.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
Mark Seaborn
9 years, 9 months ago (2011-03-28 20:25:51 UTC) #1
robertm
LGTM
9 years, 9 months ago (2011-03-28 20:29:04 UTC) #2
Nick Bray
9 years, 9 months ago (2011-03-28 20:31:16 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698