Re: COSS bypass breaks "make check"

Alex Rousskov wrote:
> I am getting the following error when running "make check":
>
> make[4]: Entering directory `squid3/DynamicSslCert/src/fs'
> /bin/bash ../../test-suite/testheaders.sh "g++ -DHAVE_CONFIG_H -I. -I. -I../../include -I. -I../../include -I../../include -I../../src -Werror -Wall -Wpointer-arith -Wwrite-strings -Wcomments -DDEFAULT_SQUID_CONFIG_DIR=\"/usr/local/squid3-ssl/etc\" -DDEFAULT_SQUID_DATA_DIR=\"/usr/local/squid3-ssl/share\" -g " "ufs" || exit 1
> Testing ufs/StoreFSufs.h ...OK.
> Testing ufs/ufscommon.h ...OK.
> /bin/bash ../../test-suite/testheaders.sh "g++ -DHAVE_CONFIG_H -I. -I. -I../../include -I. -I../../include -I../../include -I../../src -Werror -Wall -Wpointer-arith -Wwrite-strings -Wcomments -DDEFAULT_SQUID_CONFIG_DIR=\"/usr/local/squid3-ssl/etc\" -DDEFAULT_SQUID_DATA_DIR=\"/usr/local/squid3-ssl/share\" -g " "coss" || exit 1
> Testing coss/CossSwapDir.h ...In file included from ./testHeaderDeps_CossSwapDir.cc:2:
> ./coss/CossSwapDir.h:1:2: error: #error COSS Support is not stable yet in Squid-3. Please do not use.
> make[4]: *** [testHeaders] Error 1
>
> The code is configured with --enable-ssl and --disable-optimizations.
>
> It looks like either the COSS headers should not be tested by "make
> check" or the COSS #error should be removed from the headers (and
> left/placed in a .cc file).

Agreed. and fixed.

Amos

-- 
Please be using
   Current Stable Squid 2.7.STABLE5 or 3.0.STABLE10
   Current Beta Squid 3.1.0.2