Re: cvs commit: squid3/src HttpMsg.cc HttpReply.cc HttpReply.h

Hi,

On 13/09/2005 11:28 a.m., wessels@squid-cache.org wrote:
> wessels 2005/09/12 17:28:57 MDT
>
> Modified files:
> src HttpMsg.cc HttpReply.cc HttpReply.h
> HttpRequest.cc HttpRequest.h
> HttpStatusLine.cc HttpStatusLine.h
> Makefile.am Makefile.in client_side.h
> protos.h
> Log:
> Changes to HTTP request and reply parsing classes, started by Alex,
> further hacked by DW. These are forward-ported from ICAP development
> on sourceforge.
>
> Alex's original commit message:

This commit I believe, is the one that has broken squid3 -CVS:

make[3]: Leaving directory `/usr/src/squid/squid-3.0/src/repl'
Making all in auth
make[3]: Entering directory `/usr/src/squid/squid-3.0/src/auth'
g++ -DHAVE_CONFIG_H -I. -I. -I../../include -I. -I../../include
-I../../include -I../../src -I/usr/kerberos/include -Werror -Wall
-Wpointer-arith -Wwrite-strings -Wcomments -D_REENTRANT -g -O2 -c -o
basic/auth_basic.o basic/auth_basic.cc
In file included from basic/auth_basic.cc:44:
../../src/HttpReply.h:37:21: error: HttpMsg.h: No such file or directory
../../src/HttpReply.h:86: error: expected class-name before '{' token
cc1plus: warnings being treated as errors
../../src/HttpReply.h:86: warning: 'class HttpReply' has virtual functions but
non-virtual destructor
basic/auth_basic.cc: In member function 'virtual void
AuthBasicConfig::fixHeader(auth_user_request_t*, HttpReply*, http_hdr_type,
HttpRequest*)':
basic/auth_basic.cc:216: error: 'class HttpReply' has no member named 'header'
make[3]: *** [basic/auth_basic.o] Error 1

This is with gcc-4.0.1.

Much as more development on squid3 is great to see, please compile test -CVS
after you commit big chunks of code like this........

reuben
Received on Tue Sep 13 2005 - 02:42:12 MDT