Scons 3.0.0 is out with Visual Studio 2017 support and other enhancements: https://raw.githubusercontent.com/SConsProject/scons/rel_3.0.0/src/CHANGES.txt I jumped to update considering it would reduce my build environment size by > 2GB (the size occupied by Visual Studio 2015 Update 3 toolset which is required to make Scons 2.5.1 aware of Visual Studio 2017 - per bug 100202 comment 2 1st sentence). Unfortunately this didn't work smoothly, Mesa Sconstruct and possibly some sconscripts are not compatible: ********************************************************************** ** Visual Studio 2017 Developer Command Prompt v15.0.26730.15 ** Copyright (c) 2017 Microsoft Corporation ********************************************************************** [vcvarsall.bat] Environment initialized for: 'x64_x86' scons: Reading SConscript files ... File "C:\Software\DEVELO~1\projects\mesa\mesa\SConstruct", line 53 print 'scons: warning: targets option is deprecated; pass the targets on their own such as' ^ SyntaxError: invalid syntax
This feels like a python2 vs python3 compatibility issue (`print 'foo'` instead of `printf('foo')`). Can you set your environment to use python2? I expect that will solve the issue.
I always used Python 2.7. I always knew Mesa3D Scons build isn't Python 3 compatible. So there is a push towards Python 3 by Scons 3.0 apparently.
Created attachment 134347 [details] [review] Scons 3.0.0 compatibility patch Fixing this turned out to be very straight forward. Scons 3.0.0 wants print called like functions, so replacing print '...' and print "..." everywhere with print ('...') and print ("...") respectively did the trick. I checked with Scons 2.5.1 and it recognizes this new format, so there is no breakage with previous Scons release. I used the Everything advanced search utility to make sure I don't miss any print '...' and print "..." unconverted. I know, this patch is large, exceeding the recommended size. It could be split in 6 parts, one for each touched file.
(In reply to Alex Granni from comment #3) > Created attachment 134347 [details] [review] [review] > Scons 3.0.0 compatibility patch I sent the same patch (and more) on the list yesterday afternoon, a few hours before your message: https://lists.freedesktop.org/archives/mesa-dev/2017-September/170124.html I cc'ed you on the email you registered on bugzilla, did you not get it?
Eric Engestrom py2/3 compatibility patch series landed in Mesa master yesterday. The first patch in the series address this issue so I am closing this.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.