diff mbox

autopoint should be required for bootstrap.sh

Message ID CAHzfiZ7L88DTgopcjdF6xp4-2JxE+CBfd7U9Kp_5pLKjOaKDag@mail.gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Russell Treleaven Sept. 7, 2017, 5:14 p.m. UTC
I attempted to bootstrap pulseaudio and got a non obvious error.

log is here https://pastebin.com/CctNdLwD

installing autopoint fixed the problem

the following patch makes bootstrap.sh fail if autopoint is not available.
diff mbox

Patch

diff --git a/bootstrap.sh b/bootstrap.sh
index 57494fc..0a05a3e 100755
--- a/bootstrap.sh
+++ b/bootstrap.sh
@@ -32,13 +32,17 @@  fi
 # configure file faulty.
 if ! pkg-config --version &>/dev/null; then
     echo "pkg-config is required to bootstrap this program"
-    DIE=1
+    exit 1
 fi
-
 # Other necessary programs
-intltoolize --version >/dev/null || DIE=1
-test "$DIE" = 1 && exit 1
-
+if ! autopoint --version &>/dev/null ; then
+    echo "autopoint is required to bootstrap this program"
+    exit 1
+fi
+if ! intltoolize --version >/dev/null ; then
+    echo "intltoolize is required to bootstrap this program"
+    exit 1
+fi
 autopoint --force
 AUTOPOINT='intltoolize --automake --copy' autoreconf --force --install
--verbose