zeitgeist team mailing list archive

[Markus Korn <thekorn@xxxxxx>]

Markus Korn has proposed merging lp:~thekorn/zeitgeist/fix-595577-check-rapper into lp:zeitgeist.

Requested reviews:
  Zeitgeist Framework Team (zeitgeist)
Related bugs:
  #595577 Check for 'rapper' in configure (raptor-utils)
  https://bugs.launchpad.net/bugs/595577


check in 'configure' if 'rapper' is installed, and quit with an error if
this is not the case (LP: #595577)

It would be great if someone could esp. check the wording of the error message, it is hard to write stuff which is supposed to make sense when you are drunken :)

-- 
https://code.launchpad.net/~thekorn/zeitgeist/fix-595577-check-rapper/+merge/27949
Your team Zeitgeist Framework Team is requested to review the proposed merge of lp:~thekorn/zeitgeist/fix-595577-check-rapper into lp:zeitgeist.

=== modified file 'configure.ac'
--- configure.ac	2010-06-10 22:01:53 +0000
+++ configure.ac	2010-06-18 17:06:34 +0000
@@ -27,4 +27,11 @@
 	_zeitgeist/engine/extensions/Makefile
 	_zeitgeist/engine/upgrades/Makefile
 ])
+
+# check for rapper
+AC_CHECK_PROG(HAVE_RAPPER, rapper, yes, no)
+if test "x$HAVE_RAPPER" = "xno"; then
+  AC_MSG_ERROR(you need to have rapper installed !)
+fi
+
 AC_OUTPUT