question about submitting patch

From: Matthew Morgan <atcs.matthew_at_gmail.com>
Date: Wed, 04 Nov 2009 17:23:34 -0500

Ok, I think I've got the kinks worked out regarding setting
range_offset_limit per a pattern. I've done a decent bit of testing, and
it seems to be working as intended. I did added a file to the source
tree, and I'm pretty sure I've updated Makefile.am properly.

I tried to do a ./test-builds, but it fails identically in my test
repository and in trunk, in areas of squid I didn't touch. I guess HEAD
doesn't always pass? It may be that I don't have some headers that it's
looking for. Here's the output of the test-builds script:

TESTING: layer-00-bootstrap
BUILD: .././test-suite/buildtests/layer-00-bootstrap.opts
TESTING: layer-00-default
BUILD: .././test-suite/buildtests/layer-00-default.opts
../../../src/ssl_support.h:55: error: expected constructor, destructor,
or type conversion before ‘*’ token
../../../src/ssl_support.h:58: error: expected constructor, destructor,
or type conversion before ‘*’ token
../../../src/ssl_support.h:71: error: ‘SSL’ was not declared in this scope
../../../src/ssl_support.h:71: error: ‘ssl’ was not declared in this scope
../../../src/ssl_support.h:74: error: typedef ‘SSLGETATTRIBUTE’ is
initialized (use __typeof__ instead)
../../../src/ssl_support.h:74: error: ‘SSL’ was not declared in this scope
../../../src/ssl_support.h:74: error: expected primary-expression before
‘,’ token
../../../src/ssl_support.h:74: error: expected primary-expression before
‘const’
../../../src/ssl_support.h:77: error: ‘SSLGETATTRIBUTE’ does not name a type
../../../src/ssl_support.h:80: error: ‘SSLGETATTRIBUTE’ does not name a type
../../../src/ssl_support.h:83: error: ‘SSL’ was not declared in this scope
../../../src/ssl_support.h:83: error: ‘ssl’ was not declared in this scope
../../../src/ssl_support.h:86: error: ‘SSL’ was not declared in this scope
../../../src/ssl_support.h:86: error: ‘ssl’ was not declared in this scope
./../../../src/acl/CertificateData.h:45: error: ‘SSL’ was not declared
in this scope
./../../../src/acl/CertificateData.h:45: error: template argument 1 is
invalid
./../../../src/acl/CertificateData.h:51: error: expected `)' before ‘*’
token
./../../../src/acl/CertificateData.h:55: error: ‘SSL’ has not been declared
./../../../src/acl/CertificateData.h:59: error: ‘SSL’ was not declared
in this scope
./../../../src/acl/CertificateData.h:59: error: template argument 1 is
invalid
./../../../src/acl/CertificateData.h:65: error: ISO C++ forbids
declaration of ‘SSLGETATTRIBUTE’ with no type
./../../../src/acl/CertificateData.h:65: error: expected ‘;’ before ‘*’
token
make[5]: *** [testHeaders] Error 1
make[4]: *** [check-am] Error 2
make[3]: *** [check-recursive] Error 1
make[2]: *** [check] Error 2
make[1]: *** [check-recursive] Error 1
make: *** [distcheck] Error 2
Build Failed. Last log lines are:
./../../../src/acl/CertificateData.h:45: error: template argument 1 is
invalid
./../../../src/acl/CertificateData.h:51: error: expected `)' before ‘*’
token
./../../../src/acl/CertificateData.h:55: error: ‘SSL’ has not been declared
./../../../src/acl/CertificateData.h:59: error: ‘SSL’ was not declared
in this scope
./../../../src/acl/CertificateData.h:59: error: template argument 1 is
invalid
./../../../src/acl/CertificateData.h:65: error: ISO C++ forbids
declaration of ‘SSLGETATTRIBUTE’ with no type
./../../../src/acl/CertificateData.h:65: error: expected ‘;’ before ‘*’
token
distcc[31643] ERROR: compile ./testHeaderDeps_CertificateData.cc on
localhost failed
make[5]: *** [testHeaders] Error 1
make[5]: Leaving directory
`/home/lytithwyn/source/squid/trunk/btlayer-00-default/squid-3.HEAD-BZR/_build/src/acl'
make[4]: *** [check-am] Error 2
make[4]: Leaving directory
`/home/lytithwyn/source/squid/trunk/btlayer-00-default/squid-3.HEAD-BZR/_build/src/acl'
make[3]: *** [check-recursive] Error 1
make[3]: Leaving directory
`/home/lytithwyn/source/squid/trunk/btlayer-00-default/squid-3.HEAD-BZR/_build/src'
make[2]: *** [check] Error 2
make[2]: Leaving directory
`/home/lytithwyn/source/squid/trunk/btlayer-00-default/squid-3.HEAD-BZR/_build/src'
make[1]: *** [check-recursive] Error 1
make[1]: Leaving directory
`/home/lytithwyn/source/squid/trunk/btlayer-00-default/squid-3.HEAD-BZR/_build'
make: *** [distcheck] Error 2
buildtest.sh result is 2

Should I go ahead and follow the patch submission instructions on
http://wiki.squid-cache.org/Squid3VCS, or is there something I should
check first?
Received on Wed Nov 04 2009 - 22:23:44 MST

This archive was generated by hypermail 2.2.0 : Thu Nov 05 2009 - 12:00:05 MST