X Tutup
Skip to content

CI: try explicitly including msvc_runtime#6116

Merged
jenshnielsen merged 3 commits intomatplotlib:masterfrom
tacaswell:ci_fix_appveyor
Mar 6, 2016
Merged

CI: try explicitly including msvc_runtime#6116
jenshnielsen merged 3 commits intomatplotlib:masterfrom
tacaswell:ci_fix_appveyor

Conversation

@tacaswell
Copy link
Copy Markdown
Member

Might solve #6115


# same things as the requirements in ci/conda_recipe/meta.yaml
- cmd: conda create -y -q -n test-environment python=%PYTHON_VERSION% pip setuptools numpy python-dateutil freetype msinttypes tk pyparsing pytz tornado libpng zlib pyqt cycler nose mock
- cmd: conda create -y -q -n test-environment python=%PYTHON_VERSION% pip setuptools numpy python-dateutil freetype msinttypes tk pyparsing pytz tornado libpng zlib pyqt cycler nose mock msvc_runtime
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

IMO this isn't necessary, the fix is the libpng (without # [unix]) line below?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ok, strike that this seems to be fixing the 3.3 error, I looked only at the 27 one?

@jankatins
Copy link
Copy Markdown
Contributor

IMO this should be merged... Maybe the conda ppl can shed some light why this is only now failing... OR if the msvc dependency it the right thing to do...

@jenshnielsen
Copy link
Copy Markdown
Member

Ok lets merge this. If it can be done in a more clean way we can change it again

jenshnielsen added a commit that referenced this pull request Mar 6, 2016
CI: try explicitly including msvc_runtime
@jenshnielsen jenshnielsen merged commit 1fb465a into matplotlib:master Mar 6, 2016
@tacaswell tacaswell deleted the ci_fix_appveyor branch March 6, 2016 18:06
@QuLogic QuLogic added this to the 2.1 (next point release) milestone Jul 5, 2016
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.

5 participants

X Tutup