[PATCH] configure: set platform variables also when `uname` is unrecognized

Tomi Ollila tomi.ollila at iki.fi
Thu Sep 29 06:12:49 PDT 2016


Since commit

124a67e96ecab5495c0f17b6875d53dfd67ff137: configure: add set -u

all variables must be set before their expansion are attempted. These
2 variables: "platform" and "linker_resolves_library_dependencies" were
not given value in the final 'else' branch when platform check failed
due to unrecognized kernel name (output of `uname`). Now those two are
given reasonable non-empty values.
---

Unfortunately I did not catch these 2 while doing 124a67e96ecab5; That
else branch just skipped my radar. Now I also (re-)looked all other
if/then/elif/else/fi parts to catch any other similar potential problems,
and did not notice anything.

 configure | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/configure b/configure
index fa4c5b5..cc6746c 100755
--- a/configure
+++ b/configure
@@ -650,6 +650,8 @@ elif [ $uname = "Linux" ] || [ $uname = "GNU" ] ; then
     fi
 else
     printf "Unknown.\n"
+    platform="$uname"
+    linker_resolves_library_dependencies=0
     cat <<EOF
 
 *** Warning: Unknown platform. Notmuch might or might not build correctly.
-- 
2.7.4



More information about the notmuch mailing list