* Makefile.am, configure.ac: clear up the pmccabe stuff a bit

This commit is contained in:
Dirk-Jan C. Binnema 2010-02-04 22:45:54 +02:00
parent 3b1fbb79e9
commit 826aeb684d
2 changed files with 21 additions and 6 deletions

View File

@ -28,18 +28,18 @@ cleanupnote:
tags:
gtags
# this warns about function that have a cyclomatic complexity of > 10,
# which is a sign that it needs some refactoring. requires the pmccabe
# tool. If all is fine, it outputs nothing
cc10:
@pmccabe `find -name '*.c' -o -name '*.cc'` | sort -nr | awk '($$1 > 10)'
@$(PMCCABE) `find -name '*.c' -o -name '*.cc'` | sort -nr | awk '($$1 > 10)'
# this warns about functions that are over 33 non-comment lines long, which is a
# sign that they need some refactoring. requires the pmccabe tool. if
# all is fine, it outputs nothing
line33:
@pmccabe -c `find -name '*.c' -o -name '*.cc'` | awk '($$5 > 33)'
@$(PMCCABE) -c `find -name '*.c' -o -name '*.cc'` | awk '($$5 > 33)'
EXTRA_DIST= \
TODO \

View File

@ -46,7 +46,18 @@ AS_IF([test "x$PKG_CONFIG" = "xno"],[
*** The pkg-config script could not be found. Make sure it is
*** in your path, or set the PKG_CONFIG environment variable
*** to the full path to pkg-config.])
])
])
# check for pmccabe
AC_PATH_PROG([PMCCABE], [pmccabe], [no])
AS_IF([test "x$PMCCABE" = "xno"],[
AC_MSG_WARN([
*** Developers: you don't seem to have the 'pmccabe' tool installed.
*** Please install it if you want to run the automated code checks])
])
#
# currently, we don' support systems without d_type in their struct
@ -110,11 +121,15 @@ man/Makefile
])
AC_OUTPUT
echo
echo "mu configuration is complete."
echo "-----------------------------"
echo
echo "Note: the Xapian database is no longer stored as <muhome>/xapian-0.6"
echo "but instead simply as <muhome>/xapian. You can remove the older"
echo "<muhome>xapian-0.6 directory to save some disk space"
echo
echo "mu configuration is complete."
echo "type 'make' to build mu, or 'make check' to perform the unit tests."
echo "type 'make' to build mu, or 'make check' to run the unit tests."