scons: Move python path management out of the SConstruct.
authorGabe Black <gabeblack@google.com>
Thu, 9 Nov 2017 03:25:42 +0000 (19:25 -0800)
committerGabe Black <gabeblack@google.com>
Fri, 10 Nov 2017 13:17:15 +0000 (13:17 +0000)
Make site_init.py manage sys.path, and the "default" tool set
PYTHONPATH on any environment that's created. The paths to add are
tracked in a common gem5_python_paths.py.

Change-Id: I3387d4394d47a2f9c83322644cfd05909c6890fa
Reviewed-on: https://gem5-review.googlesource.com/5564
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>

SConstruct
site_scons/gem5_python_paths.py [new file with mode: 0644]
site_scons/site_init.py
site_scons/site_tools/default.py

index f0a3b655d525896ead191da333198f6a18b74201..9816c86be60a094c58a4b0a59252cb7baccbb952 100755 (executable)
@@ -95,13 +95,6 @@ from os.path import join as joinpath, split as splitpath
 import SCons
 import SCons.Node
 
-extra_python_paths = [
-    Dir('src/python').srcnode().abspath, # gem5 includes
-    Dir('ext/ply').srcnode().abspath, # ply is used by several files
-    ]
-
-sys.path[1:1] = extra_python_paths
-
 from m5.util import compareVersions, readCommand
 from m5.util.terminal import get_termcap
 
@@ -192,10 +185,6 @@ if not ('CC' in main_dict_keys and 'CXX' in main_dict_keys):
     print "No C++ compiler installed (package g++ on Ubuntu and RedHat)"
     Exit(1)
 
-# add useful python code PYTHONPATH so it can be used by subprocesses
-# as well
-main.AppendENVPath('PYTHONPATH', extra_python_paths)
-
 ###################################################
 #
 # Figure out which configurations to set up based on the path(s) of
diff --git a/site_scons/gem5_python_paths.py b/site_scons/gem5_python_paths.py
new file mode 100644 (file)
index 0000000..2833a4c
--- /dev/null
@@ -0,0 +1,52 @@
+# Copyright (c) 2013, 2015-2017 ARM Limited
+# All rights reserved.
+#
+# The license below extends only to copyright in the software and shall
+# not be construed as granting a license to any other intellectual
+# property including but not limited to intellectual property relating
+# to a hardware implementation of the functionality of the software
+# licensed hereunder.  You may use the software subject to the license
+# terms below provided that you ensure that this notice is replicated
+# unmodified and in its entirety in all distributions of the software,
+# modified or unmodified, in source code or in binary form.
+#
+# Copyright (c) 2011 Advanced Micro Devices, Inc.
+# Copyright (c) 2009 The Hewlett-Packard Development Company
+# Copyright (c) 2004-2005 The Regents of The University of Michigan
+# All rights reserved.
+#
+# Redistribution and use in source and binary forms, with or without
+# modification, are permitted provided that the following conditions are
+# met: redistributions of source code must retain the above copyright
+# notice, this list of conditions and the following disclaimer;
+# redistributions in binary form must reproduce the above copyright
+# notice, this list of conditions and the following disclaimer in the
+# documentation and/or other materials provided with the distribution;
+# neither the name of the copyright holders nor the names of its
+# contributors may be used to endorse or promote products derived from
+# this software without specific prior written permission.
+#
+# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+import SCons.Node.FS
+
+fs = SCons.Node.FS.get_default_fs()
+root = fs.Dir('#')
+extra_python_nodes = [
+    root.Dir('src').Dir('python').srcnode(), # gem5 includes
+    root.Dir('ext').Dir('ply').srcnode(), # ply is used by several files
+]
+
+extra_python_paths = [ node.abspath for node in extra_python_nodes ]
+
+__all__ = ['extra_python_paths']
index 505fe6acb9aaac303be5d76bb5572c730f81f400..6d9d24d3049269c1be83146343b98cd07f6a565e 100644 (file)
@@ -38,6 +38,8 @@
 # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
+from gem5_python_paths import extra_python_paths
+
 # Check for recent-enough Python and SCons versions.
 try:
     # Really old versions of scons only take two options for the
@@ -66,3 +68,5 @@ For more details, see:
     http://gem5.org/wiki/index.php/Using_a_non-default_Python_installation
 """
     raise
+
+sys.path[1:1] = extra_python_paths
index 2d32b56bd9353d8333edc7a8e0f63c962a937f39..73c4a9eaf581a58e9e93fb3cebeebe02e90f2692 100644 (file)
 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 import os
+import sys
 
 import SCons.Tool
 import SCons.Tool.default
 
+from gem5_python_paths import extra_python_paths
+
 def common_config(env):
     # export TERM so that clang reports errors in color
     use_vars = set([ 'AS', 'AR', 'CC', 'CXX', 'HOME', 'LD_LIBRARY_PATH',
@@ -71,6 +74,10 @@ def common_config(env):
     env.root = env.Dir('#')
     env.srcdir = env.root.Dir('src')
 
+    # add useful python code PYTHONPATH so it can be used by subprocesses
+    # as well
+    env.AppendENVPath('PYTHONPATH', extra_python_paths)
+
 gem5_tool_list = [
     'git',
     'mercurial',