Matteo M. Fusi has submitted this change and it was merged. ( https://gem5-review.googlesource.com/11389 )

Change subject: SConstruct: additional message for the error checking of the python 2.7 headers
......................................................................

SConstruct: additional message for the error checking of the python 2.7 headers

One of the most common errors during the installation of gem5 is the one
related to the failure of the check of the Python2.7 headers in the
SConstruct file. In some cases the headers are correctly installed, but a
wrong selection of the C compiler can make this check fail. This commit
wants to add some useful information this error message.

Change-Id: I0d087ad01c6e4cca3559f23070c37b5c13600962
Reviewed-on: https://gem5-review.googlesource.com/11389
Reviewed-by: Gabe Black <[email protected]>
Maintainer: Gabe Black <[email protected]>
---
M SConstruct
1 file changed, 7 insertions(+), 3 deletions(-)

Approvals:
  Gabe Black: Looks good to me, approved; Looks good to me, approved



diff --git a/SConstruct b/SConstruct
index b382d33..5209aa6 100755
--- a/SConstruct
+++ b/SConstruct
@@ -746,9 +746,13 @@

     # verify that this stuff works
     if not conf.CheckHeader('Python.h', '<>'):
-        print("Error: can't find Python.h header in", py_includes)
-        print("Install Python headers (package python-dev on " +
-              "Ubuntu and RedHat)")
+        print("Error: Check failed for Python.h header in", py_includes)
+        print("Two possible reasons:")
+        print("1. Python headers are not installed (You can install the "
+              "package python-dev on Ubuntu and RedHat)")
+        print("2. SCons is using a wrong C compiler. This can happen if "
+              "CC has the wrong value.")
+        print("CC = %s" % main['CC'])
         Exit(1)

     for lib in py_libs:

--
To view, visit https://gem5-review.googlesource.com/11389
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: I0d087ad01c6e4cca3559f23070c37b5c13600962
Gerrit-Change-Number: 11389
Gerrit-PatchSet: 3
Gerrit-Owner: Matteo M. Fusi <[email protected]>
Gerrit-Reviewer: Gabe Black <[email protected]>
Gerrit-Reviewer: Matteo M. Fusi <[email protected]>
Gerrit-CC: Jason Lowe-Power <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to