Check for gcov and lcov only when coverage testing is enabled.

configure

@@ -2484,24 +2484,6 @@ if test "${enable_coverage+set}" = set; then
   enableval=$enable_coverage;
   case $enableval in
     yes)
-      :
-      ;;
-    no)
-      :
-      ;;
-    *)
-      { { echo "$as_me:$LINENO: error: no argument expected for --enable-coverage option" >&5
-echo "$as_me: error: no argument expected for --enable-coverage option" >&2;}
-   { (exit 1); exit 1; }; }
-      ;;
-  esac
-
-else
-  enable_coverage=no
-
-fi
-
-
       for ac_prog in gcov
 do
   # Extract the first word of "$ac_prog", so it can be a program name with args.
@@ -2643,6 +2625,23 @@ if test -z "$GENHTML"; then
 echo "$as_me: error: genhtml not found" >&2;}
    { (exit 1); exit 1; }; }
 fi
+      ;;
+    no)
+      :
+      ;;
+    *)
+      { { echo "$as_me:$LINENO: error: no argument expected for --enable-coverage option" >&5
+echo "$as_me: error: no argument expected for --enable-coverage option" >&2;}
+   { (exit 1); exit 1; }; }
+      ;;
+  esac
+
+else
+  enable_coverage=no
+
+fi
+
+
 
 
 #

configure.in

@@ -1,5 +1,5 @@
 dnl Process this file with autoconf to produce a configure script.
-dnl $PostgreSQL: pgsql/configure.in,v 1.566 2008/09/05 12:11:18 petere Exp $
+dnl $PostgreSQL: pgsql/configure.in,v 1.567 2008/09/05 18:54:58 petere Exp $
 dnl
 dnl Developers, please strive to achieve this order:
 dnl
@@ -207,8 +207,8 @@ AC_SUBST(enable_profiling)
 # --enable-coverage enables generation of code coverage metrics with gcov
 #
 PGAC_ARG_BOOL(enable, coverage, no,
-              [  --enable-coverage       build with coverage testing instrumentation])
-AC_CHECK_PROGS(GCOV, gcov)
+              [  --enable-coverage       build with coverage testing instrumentation],
+[AC_CHECK_PROGS(GCOV, gcov)
 if test -z "$GCOV"; then
   AC_MSG_ERROR([gcov not found])
 fi
@@ -219,7 +219,7 @@ fi
 AC_CHECK_PROGS(GENHTML, genhtml)
 if test -z "$GENHTML"; then
   AC_MSG_ERROR([genhtml not found])
-fi
+fi])
 AC_SUBST(enable_coverage)
 
 #