-
-
Notifications
You must be signed in to change notification settings - Fork 8
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 windows compilers #65
Comments
Re-opened because of need to re-render: #64 (comment) |
Thanks for the quick response! You can see that it's already rerendered correctly if you look into the generated ci configuration: igraph-feedstock/.ci_support/win_64_.yaml Lines 1 to 2 in 89818fd
|
Ah, great! That saves us some trouble! |
This is still used in the test requirements: igraph-feedstock/recipe/meta.yaml Line 59 in 89818fd
|
Hey all
This feedstock is directly specifying
vs2017_{{ target_platform }}
as a dependency,igraph-feedstock/recipe/meta.yaml
Lines 24 to 29 in c1c6077
but should just rely on the
{{ compiler('c') }}
function to provide the correct compilers on windows. In particular, conda-forge has moved on to vs2019 as the default compiler a while ago already.If you think you need vs2017, please let us know why (generally vs2019 and vs2022 should be fully compatible). If it's really necessary, please use the dedicated mechanism in
conda_build_config.yaml
for that, i.e.as opposed to specifying it in
meta.yaml
directlyThe text was updated successfully, but these errors were encountered: