Cleanup: remove useless makesnmplib variable from build

Squid coding guidelines require automake variables to use naming pattern
ENABLE_* and there is already ENABLE_SNMP defined with intended overlap
of usage. We can drop this unusual variable completely.

diff --git a/configure.ac b/configure.ac
index 4fd8209975b61475b40a3ee0c16b18376d204012..dd18fe3662f0c3a03c7d60b6dc65813fba554b59 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -1127,11 +1127,9 @@ SQUID_DEFINE_BOOL(SQUID_SNMP,${enable_snmp:=yes},
 AM_CONDITIONAL(ENABLE_SNMP, [test "x$enable_snmp" = "xyes"])
 if test "x$enable_snmp" = "xyes"; then
     SNMPLIB='../lib/snmplib/libsnmplib.la'
-    makesnmplib=snmplib
 fi
 AC_MSG_NOTICE([SNMP support enabled: $enable_snmp])
 AC_SUBST(SNMPLIB)
-AC_SUBST(makesnmplib)
 
 AC_ARG_ENABLE(cachemgr-hostname,
   AS_HELP_STRING([--enable-cachemgr-hostname=hostname],

diff --git a/lib/Makefile.am b/lib/Makefile.am
index 3398a5dc3323b48f9bde58430abb4b09fe4a900a..8e3a58a259c4f2b3f39edd8d296b5f4f83244074 100644 (file)
--- a/lib/Makefile.am
+++ b/lib/Makefile.am
@@ -1,12 +1,15 @@
 include $(top_srcdir)/src/Common.am
 
 DIST_SUBDIRS = ntlmauth profiler rfcnb smblib libTrie snmplib
-SUBDIRS=$(makesnmplib)
+SUBDIRS=
 EXTRA_DIST=
 
 if USE_ESI
 SUBDIRS += libTrie
 endif
+if ENABLE_SNMP
+SUBDIRS += snmplib
+endif
 if ENABLE_XPROF_STATS
 SUBDIRS += profiler
 endif