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

Update travis config (dependency cache) #64

Merged
merged 1 commit into from
May 30, 2016

Conversation

phadej
Copy link
Contributor

@phadej phadej commented May 29, 2016

No description provided.

@codedmart
Copy link
Collaborator

@phadej thanks for the PR. Looks like from the failures this could be related to PR #62. I will take a look tomorrow.

@codedmart
Copy link
Collaborator

I just merged #62 if you want to rebase. Thanks!

@phadej
Copy link
Contributor Author

phadej commented May 29, 2016

Rebased

@phadej
Copy link
Contributor Author

phadej commented May 29, 2016

Seems that doctests are broken (require some service to exist?), also 7.6 build doesn't work, as ultimately doctest depens on ghc which depends on bytestring, and newer one than bundled is required somewhere :(

@codedmart
Copy link
Collaborator

@phadej sorry didn't look at the changes in detail before. Not sure about 7.6, but you removed the before_script install/setup of rethinkdb.

@phadej
Copy link
Contributor Author

phadej commented May 29, 2016

@codedmart I see, it would make life easier if you try to get rethinkdb whitelisted on travis: https://github.com/travis-ci/apt-source-whitelist

@phadej phadej force-pushed the travis-update branch 3 times, most recently from b6b8d30 to d55967b Compare May 30, 2016 06:28
@phadej
Copy link
Contributor Author

phadej commented May 30, 2016

Disabled benchmark building on GHC-7.6 which made things green.

@codedmart
Copy link
Collaborator

@phadej Looks good, thanks! I am not a travis expert so the whitelist is a great suggestion. I appreciate it.

@codedmart codedmart merged commit c68efac into AtnNn:master May 30, 2016
@phadej phadej deleted the travis-update branch May 30, 2016 14:24
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