The ${srcdir} -- usually relative path to notmuch source -- works fine in current ./configure and all makefiles. To have simple access to notmuch source in tests and out of tree builds holding absolute path to the source directory is useful. --- Fixed version of series starting with id:1480769604-11307-1-git-send-email-tomi.ollila@iki.fi diff to that one: : -if [ "$srcdir" != "." ] && [ "$srcdir" != "$NOTMUCH_SRCDIR" ]; then : +if [ "$srcdir" != "." ]; then That one broke OOT build when using absolute path (and is now pretty obvious)... That change is not needed here (and should not have been there in the first place...) configure | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/configure b/configure index e16a9b7..bcfa416 100755 --- a/configure +++ b/configure @@ -19,7 +19,12 @@ To work around this problem you may try to execute: # Store original IFS value so it can be changed (and restored) in many places. readonly DEFAULT_IFS="$IFS" +# The top-level directory for the source. This ./configure and all Makefiles +# are good with ${srcdir} usually being relative. Some components (e.g. tests) +# are executed in subdirectories and for those it is simpler to use +# ${NOTMUCH_SRCDIR} which holds absolute path to the source. srcdir=$(dirname "$0") +NOTMUCH_SRCDIR=$(cd "$srcdir" && pwd) subdirs="util compat lib parse-time-string completion doc emacs" subdirs="${subdirs} performance-test test test/test-databases" @@ -913,6 +918,7 @@ cat > Makefile.config <<EOF # the configure script). This may be different than the build # directory (the current directory at the time configure was run). srcdir = ${srcdir} +NOTMUCH_SRCDIR = ${NOTMUCH_SRCDIR} # subdirectories to build subdirs = ${subdirs} @@ -1164,6 +1170,8 @@ cat > sh.config <<EOF # This sh.config was automatically generated by the ./configure # script of notmuch. +NOTMUCH_SRCDIR='${NOTMUCH_SRCDIR}' + # Whether the Xapian version in use supports compaction NOTMUCH_HAVE_XAPIAN_COMPACT=${have_xapian_compact} -- 2.9.3