Skip to content

Commit

Permalink
CI: avoid running the unit tests of cohttp
Browse files Browse the repository at this point in the history
These tests appear to block on appveyor.

As a general rule we don't re-run the unit tests of all our dependencies
here, but we make a special exception for tcpip which we have heavily
modified. This patch installs and runs the tests for tcpip first, before
installing (without tests) all the other dependencies.

Signed-off-by: David Scott <[email protected]>
  • Loading branch information
djs55 committed May 26, 2017
1 parent d05153e commit e6437ea
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions scripts/common.sh
Original file line number Diff line number Diff line change
Expand Up @@ -25,11 +25,12 @@ opam install depext-cygwinports -y || true

OPAMBUILDTEST=1 opam depext -u slirp
# Don't run all the unit tests of all upstream packages in the universe
# for speed
# for speed. As a special exception we will run the tests for tcpip
OPAMVERBOSE=1 opam install --deps-only tcpip -y
OPAMVERBOSE=1 opam install tcpip -t

opam install $(ls -1 ${OPAM_REPO}/packages/upstream) -y
OPAMVERBOSE=1 opam install --deps-only slirp -y
# ... but install tcpip with tests enabled
OPAMVERBOSE=1 opam reinstall tcpip -y -t

OPAMVERBOSE=1 make
OPAMVERBOSE=1 make test
Expand Down

0 comments on commit e6437ea

Please sign in to comment.