search

GrumbelsTrashbin / pingus.old-auto-export

Automatically exported from code.google.com/p/pingus
GNU General Public License v3.0
0 stars 0 forks source link

Fails to build with multiple CXXFLAGS #141

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
Hi,

(I am currently updating the Pingus Debian package)

Building pingus with more than one CXXFLAGS passed to scons on the command line 
fails. For instance:
scons CXXFLAGS='-g -O2'

A bit of investigation shows that this is because CXXFLAGS are passed as list 
in SConscript and scons doesn't mix well strings and list. I asked on the scons 
mailing list and the best answer I got was to use SCons.Util.CLVar.

The attached patch fixes this issue accordingly, please include it in the next 
version of Pingus.

Thanks,
Bertrand

Original issue reported on code.google.com by BeberK...@gmail.com on 10 Aug 2014 at 12:25

Attachments: