varnish / varnish-modules

Collection of Varnish Cache modules (vmods) by Varnish Software
Other
182 stars 86 forks source link

Build on el5 #34

Closed ingvarha closed 8 years ago

ingvarha commented 8 years ago

For RHEL5 and clones, I need the following to make varnish-modules build. Is it possible to automate this by autoconf? (I do understand that rl5 does not have high prio.)

Ingvar

fgsch commented 8 years ago

Rather than adding _GNU_SOURCE blindly, can we get the list of files that fail to compile when it's missing?

ingvarha commented 8 years ago

Seems it's only vmod_cookie.c. The rest compiles fine without.

$ gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I.. -I/usr/include/varnish -I../src/foreign -Wall -Werror -Wunused-parameter -Wmissing-prototypes -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -MT vmod_cookie.lo -MD -MP -MF .deps/vmod_cookie.Tpo -c -o vmod_cookie.lo vmod_cookie.c cc1: warnings being treated as errors vmod_cookie.c: In function ‘vmod_parse’: vmod_cookie.c:109: warning: implicit declaration of function ‘strndup’ vmod_cookie.c:109: warning: incompatible implicit declaration of built-in function ‘strndup’ vmod_cookie.c: In function ‘vmod_filter_except’: vmod_cookie.c:284: warning: incompatible implicit declaration of built-in function ‘strndup’

vs

$ gcc -std=gnu99 -D_GNU_SOURCE -DHAVE_CONFIG_H -I. -I.. -I/usr/include/varnish -I../src/foreign -Wall -Werror -Wunused-parameter -Wmissing-prototypes -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -MT vmod_cookie.lo -MD -MP -MF .deps/vmod_cookie.Tpo -c -o vmod_cookie.lo vmod_cookie.c && echo SUCCESS SUCCESS

fgsch commented 8 years ago

I've committed a fix for strndup().

Re TCP_CONGESTION, can you share the netinet/tcp.h from EL5?

ingvarha commented 8 years ago

netinet/tcp.h from el5: https://ingvar.fedorapeople.org/netinet_tcp.h.el5