Thanks to visit codestin.com
Credit goes to github.com

Skip to content

BLD: Auto-detect PlatformToolset #16401

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

Merged

Conversation

Kojoley
Copy link
Member

@Kojoley Kojoley commented Feb 3, 2020

Instead of using devenv (which may be not installed (it is IDE part) or
the latest compiler were installed without IDE) modify the project file
and change PlatformToolset to pick the default version.

Fixes #16243

@Kojoley Kojoley added the Build label Feb 3, 2020
@Kojoley Kojoley added this to the v3.3.0 milestone Feb 3, 2020
@codecov
Copy link

codecov bot commented Feb 3, 2020

Codecov Report

Merging #16401 into master will decrease coverage by 0.29%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #16401      +/-   ##
==========================================
- Coverage   80.84%   80.54%   -0.30%     
==========================================
  Files         307      307              
  Lines       75749    74407    -1342     
  Branches     9692     9688       -4     
==========================================
- Hits        61241    59933    -1308     
+ Misses      11969    11934      -35     
- Partials     2539     2540       +1     
Impacted Files Coverage Δ
lib/matplotlib/testing/jpl_units/StrConverter.py 15.55% <0.00%> (-5.28%) ⬇️
lib/matplotlib/testing/jpl_units/EpochConverter.py 69.69% <0.00%> (-4.67%) ⬇️
lib/matplotlib/docstring.py 65.00% <0.00%> (-3.19%) ⬇️
...b/matplotlib/testing/jpl_units/UnitDblConverter.py 65.85% <0.00%> (-2.33%) ⬇️
lib/matplotlib/backend_managers.py 20.95% <0.00%> (-2.30%) ⬇️
lib/matplotlib/pyplot.py 71.72% <0.00%> (-1.14%) ⬇️
lib/matplotlib/animation.py 66.71% <0.00%> (-1.13%) ⬇️
lib/matplotlib/units.py 88.57% <0.00%> (-1.04%) ⬇️
lib/matplotlib/backends/backend_webagg.py 59.14% <0.00%> (-0.98%) ⬇️
lib/matplotlib/backend_tools.py 32.57% <0.00%> (-0.94%) ⬇️
... and 152 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 68bdd3e...2aee9f8. Read the comment docs.

Instead of using `devenv` (which may be not installed (it is IDE part) or
the latest compiler were installed without IDE) modify the project file
and change `PlatformToolset` to pick the default version.
@Kojoley Kojoley force-pushed the bld-autodetect-platform-toolset-version branch from 7fe521f to 2aee9f8 Compare February 5, 2020 16:46
@anntzer
Copy link
Contributor

anntzer commented Feb 5, 2020

@eric-wieser Can you confirm this works for you?

Copy link
Member

@tacaswell tacaswell left a comment

Choose a reason for hiding this comment

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

It builds on both azure and appveyor which means this works under both venv and conda (respectively) which is good evidence that it is working correctly.

@anntzer
Copy link
Contributor

anntzer commented Feb 11, 2020

@anntzer anntzer merged commit 36a3a3f into matplotlib:master Feb 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

windows builds: devenv freetype /update appears not to have an effect
3 participants