-
Notifications
You must be signed in to change notification settings - Fork 9
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
Fix Travis-CI integration to use conda #74
base: master
Are you sure you want to change the base?
Conversation
Thanks @daf 😄 |
- pip install -r requirements.txt | ||
# Customise the testing environment | ||
# --------------------------------- | ||
- conda install --channel https://conda.binstar.org/binstar --file requirements.txt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think you need the --channel ...
part.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Without it, it could not find python-dateutil
- i'm no Conda expert but it seems that is not a default channel, despite the misleading name. Adding this got TravisCI further along.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(see the earlier travis builds of this PR on each of the commit for more details)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmmm, you're right, python-dateutil
isn't in the "defaults" channel. The package has been named "dateutil" instead.
But as I understand it, the "defaults" channels is intended to be created from the conda-recipes repo, and there the package is called "python-dateutil" as one would expect. I'll raise an issue with Continuum...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See conda/conda-recipes#193.
h/t to @rhattersley who figured this out for pyugrid/pyugrid#44
Let's see how travis handles this PR!