[build] add help documentation for --opam-switch and --only-install-opam, and try opam update in case of failure

Summary:
Opam can give cryptic errors when all you need to do is `opam update`, so try
that for the user.

Reviewed By: mbouaziz

Differential Revision: D4867878

fbshipit-source-id: f740b13
master
Jules Villard 8 years ago committed by Facebook Github Bot
parent 833295df15
commit 3096f02356

@ -17,6 +17,7 @@ INFER_DEPS_DIR="$INFER_ROOT/dependencies/infer-deps"
PLATFORM="$(uname)"
NCPU="$(getconf _NPROCESSORS_ONLN 2>/dev/null || echo 1)"
OCAML_VERSION="4.02.3"
INFER_OPAM_SWITCH_DEFAULT=infer-"$OCAML_VERSION"
function usage() {
echo "Usage: $0 [-y] [targets]"
@ -28,6 +29,8 @@ function usage() {
echo
echo " options:"
echo " -h,--help show this message"
echo " --only-install-opam install the opam dependencies of infer and exists"
echo " --opam-switch specify the opam switch where to install infer (default: $INFER_OPAM_SWITCH_DEFAULT)"
echo " -y,--yes automatically agree to everything"
echo
echo " examples:"
@ -151,12 +154,18 @@ echo "initializing opam... "
check_installed opam
if [ -z $INFER_OPAM_SWITCH ]; then
# the user didn't pass an opam switch explicitly, set up a custom switch for infer
INFER_OPAM_SWITCH=infer-"$OCAML_VERSION"
INFER_OPAM_SWITCH=$INFER_OPAM_SWITCH_DEFAULT
setup_opam
fi
eval $(SHELL=bash opam config env --switch=$INFER_OPAM_SWITCH)
echo "installing infer dependencies... "
install_opam_deps
install_opam_deps || ( \
echo; \
echo '*** Failed to install opam dependencies'; \
echo '*** Updating opam then retrying'; \
opam update && \
install_opam_deps \
)
if [ "$ONLY_SETUP_OPAM" = "yes" ]; then
exit 0

Loading…
Cancel
Save