Changeset 160

Timestamp:

Aug 19, 2006, 11:19:06 PM (17 years ago)

Author:

Jari Häkkinen

Message:

Fixes #55. Added --enable-debug option to configure. This will turn on
debug stuff (off by default, developers note this).

Location:

trunk

Files:

• ChangeLog (modified) (1 diff)
• INSTALL (modified) (2 diffs)
• configure.ac (modified) (3 diffs)

trunk/ChangeLog

@@ -24 +24 @@
 
 version pre0.5:
+  - Added a --enable-debug option to autoconf. The project is
+    compiled without debug code by default. Use './configure
+    --enable-debug' to turn on debug options and code.
   - Improved permission checks on directories.
 

trunk/INSTALL

@@ -17 +17 @@
 svndigest is distributed in the hope that it will be useful, but
 WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
 General Public License for more details.
 
@@ -39 +39 @@
 # make check
 
-The 'configure' script accepts two options of interest for
-svndigest. You can provide 'configure' with APR and subversion API
+The './configure' script accepts a few options of interest for
+svndigest. You can provide './configure' with APR and subversion API
 location information with --with-apr=DIR and --with-svn=DIR,
-respectively.
+respectively. --enable-debug turns on debug options and code, which
+may be useful for developers.
 
 If you grabbed the source from the subversion repository you need to
-run 'bootstrap' to setup autoconf files.
+run './bootstrap' to setup autoconf files.
 
 

trunk/configure.ac

@@ -46 +46 @@
 # using extension beyond the C++ standard (usage of non C++ standard
 # 'long long' types).
-CXXFLAGS="-g -O -Wall -pedantic -Wno-long-long"
+CXXFLAGS="-Wall -pedantic -Wno-long-long"
+# The have HAVE_INLINE should probably be checked for.
 CPPFLAGS="-DHAVE_INLINE=1"
-# Use these flags when compiling a production library
-#CXXFLAGS="-O3 -Wall -pedantic -Wno-long-long"
-#CPPFLAGS="-DNDEBUG -DGSL_RANGE_CHECK_OFF -DHAVE_INLINE=1"
+
+AC_ARG_ENABLE(debug,[  --enable-debug          turn on debug options and code])
+if test "${enable_debug}" = "yes" ; then
+  CXXFLAGS="$CXXFLAGS -g -O"
+else
+  CPPFLAGS="$CPPFLAGS -DNDEBUG"
+  CXXFLAGS="$CXXFLAGS -O3"
+fi
 
 AC_ARG_WITH(apr,
@@ -72 +78 @@
 AC_OUTPUT
 
+echo ""
+echo "   Ready to compile the executables of svndiget"
+echo "   The following compilers and flags will be used:"
+echo "   +++++++++++++++++++++++++++++++++++++++++++++++"
+echo "    Preprocessor flags: CPPFLAGS=\"$CPPFLAGS\""
+echo "    C++ flags:          CXXFLAGS=\"$CXXFLAGS\""
+echo "    Linker flags:       LDFLAGS=\"$LDFLAGS\""
+echo "   +++++++++++++++++++++++++++++++++++++++++++++++"
+echo ""
+
 test "$GNUPLOT" = "ok" || \
      AC_MSG_WARN([Gnuplot was not found. svndigest will compile
@@ -77 +93 @@
      install gnuplot (available for a wide range of operating systems
      at http://www.gnuplot.info).])
+
+echo ""
+echo "      Now type 'make ; make check'."
+echo ""