From: Amos Jeffries <squid3@treenet.co.nz>
Date: Wed, 24 Jun 2009 06:49:28 +0000 (+1200)
Subject: Bug 2679: strsep and strtoll should be bundled
X-Git-Tag: SQUID_3_0_STABLE17~23
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=2d1f1847834fe2e289659d35f3a9046c79cdd3f7;p=thirdparty%2Fsquid.git

Bug 2679: strsep and strtoll should be bundled

Incorrect autoconf macro to test for these was leading to a large hack.
Using the right macro fixes this issue and links as needed.
---

diff --git a/configure.in b/configure.in
index c4196b5963..5055896278 100644
--- a/configure.in
+++ b/configure.in
@@ -2651,8 +2651,6 @@ AC_CHECK_FUNCS(\
 	srand48 \
 	srandom \
 	statfs \
-	strsep \
-	strtoll \
 	sysconf \
 	syslog \
 	timegm \
@@ -2663,6 +2661,8 @@ AC_REPLACE_FUNCS(\
 	drand48 \
 	initgroups \
 	strerror \
+	strsep \
+	strtoll \
 	tempnam \
 )
 
@@ -2809,17 +2809,6 @@ else
   AC_DEFINE(HAVE_STRNSTR,0,[Yay! A MacOS X brokenness. Its not good enough to know that strnstr() exists, because MacOSX 10.4 and earlier may have a buffer overrun.])
 fi
 
-
-AM_CONDITIONAL(NEED_OWN_STRSEP, false)
-if test "$ac_cv_func_strsep" = "no" ; then
-  AM_CONDITIONAL(NEED_OWN_STRSEP, true)
-fi
-
-AM_CONDITIONAL(NEED_OWN_STRTOLL, false)
-if test "$ac_cv_func_strtoll" = "no" ; then
-  AM_CONDITIONAL(NEED_OWN_STRTOLL, true)
-fi
-
 dnl
 dnl Test for va_copy
 dnl
diff --git a/lib/Makefile.am b/lib/Makefile.am
index 7d9edc92a2..1fcd645ac4 100644
--- a/lib/Makefile.am
+++ b/lib/Makefile.am
@@ -18,17 +18,6 @@ else
 XPROF_STATS_SOURCE = 
 endif
 
-if NEED_OWN_STRSEP
-STRSEPSOURCE=strsep.c
-else
-STRSEPSOURCE=
-endif
-if NEED_OWN_STRTOLL
-STRTOLLSOURCE=strtoll.c
-else
-STRTOLLSOURCE=
-endif
-
 if ENABLE_WIN32SPECIFIC
 LIBSSPWIN32=libsspwin32.a
 WIN32SRC = win32lib.c
@@ -54,8 +43,6 @@ EXTRA_libmiscutil_a_SOURCES = \
 	getopt.c \
 	md5.c \
 	Profiler.c \
-	strsep.c \
-	strtoll.c \
 	win32lib.c
 libmiscutil_a_SOURCES = \
 	MemPool.cc \
@@ -75,8 +62,6 @@ libmiscutil_a_SOURCES = \
 	$(SNPRINTFSOURCE) \
 	Splay.cc \
 	strnstr.cc \
-	$(STRSEPSOURCE) \
-	$(STRTOLLSOURCE) \
 	stub_memaccount.c \
 	util.c \
 	uudecode.c \