Revert "Use SCons -C to invoke the v8 build from our own build directory,"

This reverts r4041.

Review URL: http://codereview.chromium.org/8822


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4045 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 27 insertions, 26 deletions

diff --git a/build/SConscript.v8 b/build/SConscript.v8
index 8469c63..a3eb983 100644
--- a/build/SConscript.v8
+++ b/build/SConscript.v8
@@ -2,8 +2,6 @@
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
-import os
-
 Import('env')
 
 # Grab the -j flag from the outer environment, if available.
@@ -15,12 +13,11 @@ except AttributeError:
 
 env = env.Clone(
     V8_MODE = 'release',
-    V8_MODE_DIR = '$V8_DIR/obj/$V8_MODE',
-    V8_SCONS_COM = '$PYTHON $SCONS $SCONSFLAGS mode=$V8_MODE',
+    V8_SRC_DIR = '$CHROME_SRC_DIR/v8',
+    V8_MODE_DIR = '$V8_SRC_DIR/obj/$V8_MODE',
+    V8_SCONS_COM = 'cd $CHROME_SRC_DIR/v8 && $PYTHON $SCONS $SCONSFLAGS mode=$V8_MODE',
     SCONS='$CHROME_SRC_DIR/third_party/scons/scons.py',
     SCONSFLAGS = ('-Q '
-                  '-C $OBJ_ROOT/v8 '
-                  '-Y $CHROME_SRC_DIR/v8 '
                   '--warn=no-deprecated '
                   '--warn=no-no-parallel-support' + cpu_flag),
 )
@@ -28,30 +25,31 @@ env = env.Clone(
 # Rather than build v8 with our own commands, we just shell out to v8's
 # own SCons-based build, since their build system is complicated.
 # This SConscript just declares dependencies on the outputs of that build.
-#
-# Arrange to make sure the build directory exists, since the subsidiary
-# SCons we invoke will chdir there of its own accord.
-
-if not os.path.exists(env.Dir('$OBJ_ROOT/v8').abspath):
-  Mkdir(env.Dir('$OBJ_ROOT/v8').abspath)
 
+mksnapshot_exe = env.File('$V8_MODE_DIR/mksnapshot${PROGSUFFIX}')
+libraries_obj = env.File('$V8_MODE_DIR/libraries${OBJSUFFIX}')
+libraries_empty_obj = env.File('$V8_MODE_DIR/libraries-empty${OBJSUFFIX}')
+snapshot_obj = env.File('$V8_MODE_DIR/snapshot${OBJSUFFIX}')
+snapshot_empty_obj = env.File('$V8_MODE_DIR/snapshot-empty${OBJSUFFIX}')
+v8_bin = env.File('$V8_SRC_DIR/shell${PROGSUFFIX}')
+v8_lib = env.File('$V8_SRC_DIR/${LIBPREFIX}v8${LIBSUFFIX}'),
 
 v8_scons_targets_off = [
-    '$V8_DIR/shell${PROGSUFFIX}',
-    '$V8_DIR/${LIBPREFIX}v8${LIBSUFFIX}',
-    '$V8_MODE_DIR/mksnapshot${PROGSUFFIX}',
-    '$V8_MODE_DIR/libraries${OBJSUFFIX}',
-    '$V8_MODE_DIR/snapshot-empty${OBJSUFFIX}',
+    mksnapshot_exe,
+    libraries_obj,
+    snapshot_empty_obj,
+    v8_bin,
+    v8_lib,
 ]
 
 v8_scons_targets_on = [
-    '$V8_MODE_DIR/libraries-empty${OBJSUFFIX}',
-    '$V8_MODE_DIR/snapshot${OBJSUFFIX}',
+    libraries_empty_obj,
+    snapshot_obj,
 ]
 
 if env['PLATFORM'] == 'win32':
   v8_scons_targets_off.extend([
-      env.File('$V8_DIR/vc80.pdb')
+      env.File('$CHROME_SRC_DIR/v8/vc80.pdb')
   ])
 
 v8 = env.Command(v8_scons_targets_off + v8_scons_targets_on,
@@ -66,17 +64,20 @@ env.AlwaysBuild(v8)
 # rebuild them (thereby causing us to always rebuild their dependents).
 env.Precious(v8)
 
+env.Install('$V8_DIR', v8)
+
 
+i = env.Install('$LIBS_DIR', v8_lib)
+env.Alias('webkit', i)
 
-env.Install('$TARGET_ROOT', v8[0])
-env.Install('$LIB_DIR', v8[1])
-env.Install('$V8_DIR', '$V8_MODE_DIR/snapshot-empty${OBJSUFFIX}')
+i = env.Install('$TARGET_ROOT', v8_bin)
+env.Alias('chrome', i)
 
 # To satisfy tests expecting the following .exe name.
 if env['PLATFORM'] == 'win32':
   # TODO(evanm): this may be necessary on other platforms(?)
-  i = env.InstallAs('$TARGET_ROOT/v8_shell_sample${PROGSUFFIX}', v8[0])
+  i = env.InstallAs('$TARGET_ROOT/v8_shell_sample${PROGSUFFIX}', v8_bin)
+  env.Alias('chrome', i)
 
 env.ChromeStaticLibrary('v8_snapshot',
-                        ['$V8_MODE_DIR/libraries-empty${OBJSUFFIX}',
-                         '$V8_MODE_DIR/snapshot${OBJSUFFIX}'])
+                        [libraries_empty_obj, snapshot_obj])