Skip to content
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

Inconsistent compiler versions in staged-recipes vs. package-feedstock #22620

Closed
jan-janssen opened this issue Apr 19, 2023 · 4 comments
Closed
Labels

Comments

@jan-janssen
Copy link
Member

Bug:

I build a conda-forge package for a module which is a bit picky about compiler settings. It worked fine on the staged-recipes:
#22502
But it failed once the feedstock was rerendered:
https://github.com/conda-forge/flux-core-feedstock
So I have the feeling the compiler version are inconsistent. I am not sure how many packages are effected by this, but it would be great to have consistent versions.

@isuruf
Copy link
Member

isuruf commented Apr 20, 2023

A PR was just merged to upgrade to gcc 12. cc @h-vetinari, @conda-forge/core

@beckermr
Copy link
Member

Let's watch it. If this is a one-off, we can help make the change in the affected feedstocks. If a bunch of stuff starts to fail, we can back it out.

@h-vetinari
Copy link
Member

h-vetinari commented Apr 20, 2023

Hey @jan-janssen, sorry about the breakage.

You managed to fall into a very short window where this could happen 😅
where we merged but haven't updated staged-recipes yet (yes of course the versions should be consistent).

To get back to what was the state of affairs until a few hours ago, you can add the following to recipe/conda_build_config.yaml of your new feedstock and then rerender:

c_compiler_version:            # [unix]
  - 11                         # [linux]
  - 14                         # [osx]
cxx_compiler_version:          # [unix]
  - 11                         # [linux]
  - 14                         # [osx]

Edit: Actually, there's nothing to change on this repo, it was really just the window between the last CI of your PR here, merging conda-forge/conda-forge-pinning-feedstock#4215, and the new feedstock being rendered.

@jan-janssen
Copy link
Member Author

That's fine with me - I just thought I bring it up as it might have effected other users as well. Thanks for the quick reply.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

4 participants