Skip to content
This repository has been archived by the owner on Dec 4, 2018. It is now read-only.

Fix AppVeyor builds #114

Merged
merged 1 commit into from
Oct 12, 2015
Merged

Fix AppVeyor builds #114

merged 1 commit into from
Oct 12, 2015

Conversation

ocefpaf
Copy link
Member

@ocefpaf ocefpaf commented Oct 8, 2015

The first change is just for fun. (I spent the weekend reading AppVeyor's docs.)

The second change should fix the problem we are seeing in #113. One side effect is that the Python 2.7 builds will be slightly slower.

@@ -33,8 +33,8 @@ platform:
- x64

install:
- cmd: python -c "from urllib2 import urlopen; open('bootstrap-obvious-ci-and-miniconda.py', 'w').write(urlopen('https://raw.githubusercontent.com/pelson/Obvious-CI/master/bootstrap-obvious-ci-and-miniconda.py').read())"
- cmd: python bootstrap-obvious-ci-and-miniconda.py %CONDA_INSTALL_LOCN% %TARGET_ARCH% %CONDA_PY:~0,1% --without-obvci
- appveyor DownloadFile "https://raw.githubusercontent.com/pelson/Obvious-CI/master/bootstrap-obvious-ci-and-miniconda.py"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 A definite improvement! 😉

@ocefpaf
Copy link
Member Author

ocefpaf commented Oct 8, 2015

@pelson Now the problem is numpy 1.10.0. How do you want to go about it?

For now I limited IOOS builds to numpy >=1.8,<1.10.

@pelson
Copy link
Member

pelson commented Oct 9, 2015

For now I limited IOOS builds to numpy >=1.8,<1.10.

Makes sense. Incidentally, this is highly likely to result in really slow conda resolutions (see conda/conda#1338 for some info).

@ocefpaf
Copy link
Member Author

ocefpaf commented Oct 9, 2015

Makes sense.

Not sure how to bake the numpy string in that case since we would have to change numpy * to numpy >=1.8,<1.10.

Incidentally, this is highly likely to result in really slow conda resolutions

Ah! Maybe we can just wait it out then. I guess that with my new power I can re-start AppVeyor 😉 I will try that later.

@ocefpaf
Copy link
Member Author

ocefpaf commented Oct 11, 2015

Waiting it out did the trick. AppVeyor is good to go. I will try to address some of the circle-ci failures in another PR.

@ocefpaf ocefpaf force-pushed the miniconda3 branch 4 times, most recently from 2795313 to 5d8c56c Compare October 11, 2015 23:52
@ocefpaf ocefpaf mentioned this pull request Oct 12, 2015
pelson added a commit that referenced this pull request Oct 12, 2015
@pelson pelson merged commit c31f687 into SciTools:master Oct 12, 2015
@ocefpaf ocefpaf deleted the miniconda3 branch October 12, 2015 09:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants