build-infer.sh now works with non-traditional shells

Summary:
fixes #267.

By calling opam with the --shell=bash in the script I don't get the errors I had before.

The new line I added in the docker/README.md was so the list would display correctly when rendered.
Closes https://github.com/facebook/infer/pull/272

Reviewed By: jberdine

Differential Revision: D2916859

Pulled By: jvillard

fb-gh-sync-id: 4243f79
shipit-source-id: 4243f79
master
Skyfold 9 years ago committed by facebook-github-bot-5
parent 38cb5b48ec
commit ee49b436af

@ -105,7 +105,7 @@ check_installed opam
opam init --no-setup --yes > /dev/null || \ opam init --no-setup --yes > /dev/null || \
opam init --no-setup --yes --comp=4.02.3 opam init --no-setup --yes --comp=4.02.3
eval $(opam config env) eval $(SHELL=bash opam config env)
echo "preparing build... " echo "preparing build... "
if [ ! -f .release ]; then if [ ! -f .release ]; then

@ -19,6 +19,7 @@ installation. See the instructions for
This docker file will use the latest This docker file will use the latest
[released](https://github.com/facebook/infer/releases) version of [released](https://github.com/facebook/infer/releases) version of
Infer. Infer.
1. Get docker running, e.g. using Docker Quickstart Terminal. 1. Get docker running, e.g. using Docker Quickstart Terminal.
2. cd to the directory `docker/`, 2. cd to the directory `docker/`,
3. Build Infer in docker container and try on an example: 3. Build Infer in docker container and try on an example:

Loading…
Cancel
Save