summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGabe Black <gabeblack@google.com>2017-11-08 19:25:42 -0800
committerGabe Black <gabeblack@google.com>2017-11-10 13:17:15 +0000
commit91d942a9da18b77b8a0652bc5f07f2e129b8504e (patch)
treea882b3c8f2ad7c37aab747429621ff12432d4312
parent902b3e34beaadaa2ac2932a9ff93f08008a4b16f (diff)
downloadgem5-91d942a9da18b77b8a0652bc5f07f2e129b8504e.tar.xz
scons: Move python path management out of the SConstruct.
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>
-rwxr-xr-xSConstruct11
-rw-r--r--site_scons/gem5_python_paths.py52
-rw-r--r--site_scons/site_init.py4
-rw-r--r--site_scons/site_tools/default.py7
4 files changed, 63 insertions, 11 deletions
diff --git a/SConstruct b/SConstruct
index f0a3b655d..9816c86be 100755
--- a/SConstruct
+++ b/SConstruct
@@ -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
index 000000000..2833a4cc0
--- /dev/null
+++ b/site_scons/gem5_python_paths.py
@@ -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']
diff --git a/site_scons/site_init.py b/site_scons/site_init.py
index 505fe6acb..6d9d24d30 100644
--- a/site_scons/site_init.py
+++ b/site_scons/site_init.py
@@ -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
diff --git a/site_scons/site_tools/default.py b/site_scons/site_tools/default.py
index 2d32b56bd..73c4a9eaf 100644
--- a/site_scons/site_tools/default.py
+++ b/site_scons/site_tools/default.py
@@ -39,10 +39,13 @@
# 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',