Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make coverage tests use development versions of lwt and lwt_ppx #590

Merged
merged 1 commit into from
May 19, 2018

Conversation

smithjessk
Copy link
Contributor

@smithjessk smithjessk commented May 16, 2018

See the comments starting here.

This PR fixes the Travis builds and gets the coverage tests to run. They currently don't because the tests are looking for the installed version of lwt; see e.g. here.

To verify, run make coverage.

@smithjessk
Copy link
Contributor Author

Hmm.. The Linux builds work now, but the OSX ones fail with a different reason. Before, the build failed because lwt wasn't found. But now, it fails when installing and linking Python 2 for Opam, which wasn't installed for Opam in the old Travis build.

Not sure why this happens, but since the error occurs when installing ocaml it shouldn't be related to this PR.

@smithjessk
Copy link
Contributor Author

smithjessk commented May 16, 2018

Hmm.. The Linux builds work now, but the OSX ones fail with a different reason. Before, the build failed because lwt wasn't found. But now, it fails when installing and linking Python 2 for opam, which wasn't installed for opam in the old Travis build.

Not sure why this happens, but since the error occurs when installing ocaml as part of travis.sh it shouldn't be caused by this PR.

@aantron
Copy link
Collaborator

aantron commented May 16, 2018

Thanks! I'll give it a proper try in some hours.

The macOS builds are probably failing because of the Homebrew Python issue. We worked around that in this repo, in cd63b76, however that commit is not in the ppx-expect branch. So, don't worry about macOS :)

@aantron aantron merged commit 08015a2 into ocsigen:ppx-expect May 19, 2018
@aantron
Copy link
Collaborator

aantron commented May 19, 2018

Looks good, thanks again! And sorry for the delay.

I'll get this over into master shortly.

@smithjessk
Copy link
Contributor Author

Thanks, Anton!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants