From 63d03edbd51a2b4e87b08bc2e4c41df09e6c1d57 Mon Sep 17 00:00:00 2001 From: hno <> Date: Mon, 16 Apr 2007 20:12:06 +0000 Subject: [PATCH] Author: Amos Jeffries Remove cppunit from the Squid distribution Packaging of cppunit within the squid sources has caused more trouble than was solved. Ease of install independently of squid combined with the apparently widespread use of cppunit as a standard tool we feel there is no need for it to be included. Maintainers and source testers will need to have cppunit installed on their systems or not use "make check". --- bootstrap.sh | 3 +-- configure.in | 21 +++++++-------------- lib/Makefile.am | 6 +++--- 3 files changed, 11 insertions(+), 19 deletions(-) diff --git a/bootstrap.sh b/bootstrap.sh index cc350d1625..ca20a20593 100755 --- a/bootstrap.sh +++ b/bootstrap.sh @@ -66,8 +66,7 @@ echo "libtool : $ltver" for dir in \ "" \ - lib/libTrie \ - lib/cppunit-1.10.0 + lib/libTrie do if [ -z "$dir" ] || [ -d $dir ]; then if ( diff --git a/configure.in b/configure.in index 7169b59789..3fdb54b0ad 100644 --- a/configure.in +++ b/configure.in @@ -1,7 +1,7 @@ dnl Configuration input file for Squid dnl -dnl $Id: configure.in,v 1.450 2007/04/15 14:46:11 serassio Exp $ +dnl $Id: configure.in,v 1.451 2007/04/16 14:12:06 hno Exp $ dnl dnl dnl @@ -11,7 +11,7 @@ AM_CONFIG_HEADER(include/autoconf.h) AC_CONFIG_AUX_DIR(cfgaux) AC_CONFIG_SRCDIR([src/main.cc]) AM_INIT_AUTOMAKE([tar-ustar]) -AC_REVISION($Revision: 1.450 $)dnl +AC_REVISION($Revision: 1.451 $)dnl AC_PREFIX_DEFAULT(/usr/local/squid) AM_MAINTAINER_MODE @@ -1784,22 +1784,20 @@ AC_ARG_WITH(filedescriptors, AC_PATH_PROG(CPPUNITCONFIG, cppunit-config, false) if $CPPUNITCONFIG --help >/dev/null; then echo "using system installed cppunit" - SQUID_CPPUNIT_DIR='' SQUID_CPPUNIT_LIBS="`$CPPUNITCONFIG --libs`" SQUID_CPPUNIT_LA='' SQUID_CPPUNIT_INC="`$CPPUNITCONFIG --cflags`" else - SQUID_CPPUNIT_DIR='cppunit-1.10.0' - SQUID_CPPUNIT_LA='$(top_builddir)/lib/cppunit-1.10.0/src/cppunit/libcppunit.la' - SQUID_CPPUNIT_LIBS='$(SQUID_CPPUNIT_LA)' - SQUID_CPPUNIT_INC='-I$(top_builddir)/lib/cppunit-1.10.0/include -I $(top_srcdir)/lib/cppunit-1.10.0/include' + echo "cppunit does not apear to be installed. squid does not require this, but code testing with 'make check' will fail." + SQUID_CPPUNIT_LA='' + SQUID_CPPUNIT_LIBS='' + SQUID_CPPUNIT_INC='' fi AC_ARG_WITH(cppunit-basedir, [ --with-cppunit-basedir=/path/to/cppunit-base Path where the cppunit headers are libraries are found - for unit testing. (defaults to use internal copies - - version 1.10.0) + for unit testing. ], [ if test -f $withval/include/cppunit/TestCase.h; then echo "Using cppunit includes from $withval" @@ -1816,12 +1814,10 @@ AC_ARG_WITH(cppunit-basedir, echo "ERROR: Cannot find cppunit at $withval" exit 1 fi - SQUID_CPPUNIT_DIR='' ]) AC_SUBST(SQUID_CPPUNIT_LIBS) AC_SUBST(SQUID_CPPUNIT_LA) AC_SUBST(SQUID_CPPUNIT_INC) -AC_SUBST(SQUID_CPPUNIT_DIR) # Force some compilers to use ANSI features # @@ -3304,7 +3300,4 @@ AC_CONFIG_FILES([\ tools/Makefile ]) -AC_CONFIG_SUBDIRS(lib/libTrie) -AC_CONFIG_SUBDIRS(lib/cppunit-1.10.0) - AC_OUTPUT diff --git a/lib/Makefile.am b/lib/Makefile.am index 5dcadd9706..7523ddf132 100644 --- a/lib/Makefile.am +++ b/lib/Makefile.am @@ -1,10 +1,10 @@ ## Process this file with automake to produce Makefile.in # -# $Id: Makefile.am,v 1.23 2006/09/02 13:48:53 serassio Exp $ +# $Id: Makefile.am,v 1.24 2007/04/16 14:12:06 hno Exp $ # -DIST_SUBDIRS = libTrie cppunit-1.10.0 -SUBDIRS= libTrie @SQUID_CPPUNIT_DIR@ +DIST_SUBDIRS = libTrie +SUBDIRS= libTrie install: all install-strip: all -- 2.47.2