yesoce / serf

Automatically exported from code.google.com/p/serf
Apache License 2.0
0 stars 0 forks source link

Default CFLAGS='-std=c89' fails on Linux with 'scons check' #164

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. scons
2. scons check
3.

What is the expected output? What do you see instead?

Expect success, get error (and lots of warnings):

(see output in attached log files)

What version of the product are you using? On what operating system?

serf 1.3.8, Debian 7

Please provide any additional information below.

I eliminated the error by editing the SConstruct file and changing '-std=c89' 
to '-std=gnu89' which is required by the use of '//' comments in file 
'./text/test_buckets.c', line 1237.

Output from executing 'scons' and 'scons check' BEFORE my mod to SConstruct is 
attached.

Original issue reported on code.google.com by tom.brow...@gmail.com on 6 Mar 2015 at 4:43

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks for the report. We want to fix that comment.

Original comment by lieven.govaerts@gmail.com on 6 Mar 2015 at 4:46

GoogleCodeExporter commented 9 years ago
This had already been fixed in /trunk r2443, and how merged to /branches/1.3.x 
in r2483

Original comment by andreas.stieger@gmx.de on 7 Mar 2015 at 9:53

GoogleCodeExporter commented 9 years ago
This compiler error was fixed in r2443 on trunk already, so I've merged it to 
the 1.3.x branch.
If we'll make a 1.3.9 release it will be in there, more likely it'll be in the 
upcoming 1.4.0 release.
Thanks!

Original comment by lieven.govaerts@gmail.com on 7 Mar 2015 at 9:53