[gem5-dev] Change in public/gem5[master]: scons: Use brackets for print

[Siddhesh Poyarekar (Gerrit)]

Hello Gabe Black, Anthony Gutierrez, Andreas Sandberg, Brandon Potter,

I'd like you to reexamine a change. Please visit

https://gem5-review.googlesource.com/8321

to look at the new patch set (#3).

Change subject: scons: Use brackets for print
..

scons: Use brackets for print

Scons on Fedora 27 imports print_function from the future as a result
of which a gem5 build errors out with a syntax error.  Put braces
around the exception print statements to fix this.

Change-Id: I67b7ef978fd7567f94d3cd9a904f8a0c1af07ffb
Signed-off-by: Siddhesh Poyarekar 
---
M site_scons/site_init.py
1 file changed, 4 insertions(+), 4 deletions(-)


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


Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I67b7ef978fd7567f94d3cd9a904f8a0c1af07ffb
Gerrit-Change-Number: 8321
Gerrit-PatchSet: 3
Gerrit-Owner: Siddhesh Poyarekar 
Gerrit-Reviewer: Andreas Sandberg 
Gerrit-Reviewer: Anthony Gutierrez 
Gerrit-Reviewer: Brandon Potter 
Gerrit-Reviewer: Gabe Black 
Gerrit-Reviewer: Siddhesh Poyarekar 
___
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

[gem5-dev] Change in public/gem5[master]: scons: Use brackets for print

[Siddhesh Poyarekar (Gerrit)]

Hello Brandon Potter,

I'd like you to reexamine a change. Please visit

https://gem5-review.googlesource.com/8321

to look at the new patch set (#2).

Change subject: scons: Use brackets for print
..

scons: Use brackets for print

Scons on Fedora 27 imports print_function from __future__[1] as a result
of which a gem5 build errors out with a syntax error.  Put braces
around the exception print statements to fix this.

[1]  
https://github.com/SCons/scons/commit/34cf3bdb1743de9a5534bfd25998d0a01297f004


Change-Id: I67b7ef978fd7567f94d3cd9a904f8a0c1af07ffb
Signed-off-by: Siddhesh Poyarekar 
---
M site_scons/site_init.py
1 file changed, 4 insertions(+), 4 deletions(-)


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


Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I67b7ef978fd7567f94d3cd9a904f8a0c1af07ffb
Gerrit-Change-Number: 8321
Gerrit-PatchSet: 2
Gerrit-Owner: Siddhesh Poyarekar 
Gerrit-Reviewer: Brandon Potter 
___
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

[gem5-dev] Change in public/gem5[master]: scons: Use brackets for print

[Siddhesh Poyarekar (Gerrit)]

Siddhesh Poyarekar has uploaded this change for review. (  
https://gem5-review.googlesource.com/8321



Change subject: scons: Use brackets for print
..

scons: Use brackets for print

Scons on Fedora 27 imports print_function from the future as a result
of which a gem5 build errors out with a syntax error.  Put braces
around the exception print statements to fix this.

Change-Id: I67b7ef978fd7567f94d3cd9a904f8a0c1af07ffb
Signed-off-by: Siddhesh Poyarekar 
---
M site_scons/site_init.py
1 file changed, 4 insertions(+), 4 deletions(-)



diff --git a/site_scons/site_init.py b/site_scons/site_init.py
index 6d9d24d..869d725 100644
--- a/site_scons/site_init.py
+++ b/site_scons/site_init.py
@@ -49,24 +49,24 @@
 EnsureSConsVersion(0, 98)
 EnsureSConsVersion(0, 98, 1)
 except SystemExit, e:
-print """
+print("""
 For more details, see:
 http://gem5.org/Dependencies
-"""
+""")
 raise

 # pybind11 requires python 2.7
 try:
 EnsurePythonVersion(2, 7)
 except SystemExit, e:
-print """
+print ("""
 You can use a non-default installation of the Python interpreter by
 rearranging your PATH so that scons finds the non-default 'python' and
 'python-config' first.

 For more details, see:
 http://gem5.org/wiki/index.php/Using_a_non-default_Python_installation
-"""
+""")
 raise

 sys.path[1:1] = extra_python_paths

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


Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I67b7ef978fd7567f94d3cd9a904f8a0c1af07ffb
Gerrit-Change-Number: 8321
Gerrit-PatchSet: 1
Gerrit-Owner: Siddhesh Poyarekar 
___
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev