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

Add {{ stdlib('c') }} jinja function for recipes #328

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

danpetry
Copy link
Contributor

@danpetry danpetry commented Feb 25, 2025

Destination channel: defaults

Explanation of changes:

  • This enables you to put {{ stdlib('c') }} in recipes
  • currently it won't do anything, as the sysroot packages are pulled in by the compilers anyway, and the versions are (I assume?) tied together
  • It does allow us to reduce a class of diffs when pulling across conda recipes (that is, we don't have to comment out {{ stdlib('c') }})

See https://docs.conda.io/projects/conda-build/en/stable/resources/compiler-tools.html#expressing-the-relation-between-compiler-and-its-standard-library for more details.

@JeanChristopheMorinPerso
Copy link
Contributor

@danpetry I added a link to the docs in the description. (I still have to review the PR).

Copy link
Contributor

@Jrice1317 Jrice1317 left a comment

Choose a reason for hiding this comment

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

According the link JC added in the description, there should be the following for osx. Does this apply to us?

- macosx_deployment_target # [osx]

@danpetry
Copy link
Contributor Author

@Jrice1317 later, yes, but not needed for now.

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.

3 participants