google-code-export / v8cgi

Automatically exported from code.google.com/p/v8cgi
BSD 3-Clause "New" or "Revised" License
2 stars 1 forks source link

verbose=1 fcgi=1 don't compile #74

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
If you use the option fcgi=1 at the same time as verbose=1 you get a printf 
error in path.cc.

to reproduce:

scons mysql=1 pgsql=1 sqlite=1 gd=1 dom=1 module=0 cgi=1 fcgi=1 arch=x86

Original issue reported on code.google.com by dean.calver@gmail.com on 23 Oct 2010 at 4:45

GoogleCodeExporter commented 9 years ago
I am unable to reproduce with latest SVN trunk. Can you please past the 
complete build output, as well as OS name/version and compiler name/version?

Original comment by ondrej.zara on 24 Oct 2010 at 4:02

GoogleCodeExporter commented 9 years ago
OS Debian: Squeeze 
Compiler: GCC 4.4.5
Scons: scons mysql=1 pgsql=1 sqlite=1 gd=1 dom=1 module=0 verbose=1 cgi=1 
fcgi=1 arch=x86
--------------
g++ -o src/path.os -c -Wall -O3 -fPIC -DHAVE_MMAN_H -DHAVE_SLEEP 
-DCONFIG_PATH=/etc/v8cgi.conf -DVERSION=0.8.3dev -Dposix -DDSO_EXT=so -DVERBOSE 
-DFASTCGI -Isrc -I/home/deano/v8cgi-0.8.2-src/v8/include -Isrc/fcgi/include 
src/path.cc
src/path.cc: In function 'int path_chdir(std::string)':
src/path.cc:134: error: 'printf' was not declared in this scope

-----
I believe its because FastCGI changes the way stdio.h is included, hence printf 
isn't valid without FastCGI includes.

Thanks,
Deano

Original comment by dean.calver@gmail.com on 25 Oct 2010 at 8:18

GoogleCodeExporter commented 9 years ago
Okay, will try to reproduce and fix this. I agree that the problem is probably 
caused by fcgi-modified headers - something rather nasty to debug :/

Original comment by ondrej.zara on 25 Oct 2010 at 9:00

GoogleCodeExporter commented 9 years ago
Dean,

can your problem be solved by adding "#include <stdio.h>" at the beginning of 
path.cc ?

Original comment by ondrej.zara on 25 Oct 2010 at 9:08

GoogleCodeExporter commented 9 years ago
Fixed in revision r813.

Original comment by ondrej.zara on 25 Oct 2010 at 1:12