Skip to content

Personal fork update to compile CUDA 12#21

Merged
lvotapka merged 12 commits into
conda-forge:mainfrom
lvotapka:master
Aug 19, 2024
Merged

Personal fork update to compile CUDA 12#21
lvotapka merged 12 commits into
conda-forge:mainfrom
lvotapka:master

Conversation

@lvotapka

Copy link
Copy Markdown
Contributor

Checklist

  • Used a personal fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Reset the build number to 0 (if the version changed)
  • Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
  • Ensured the license file is being packaged.

@conda-forge-webservices

Copy link
Copy Markdown
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

@lvotapka

Copy link
Copy Markdown
Contributor Author

@conda-forge-admin, please rerender

Comment thread recipe/conda_build_config.yaml Outdated
Comment thread recipe/meta.yaml Outdated
Comment thread recipe/meta.yaml
@jakirkham

Copy link
Copy Markdown
Member

Thanks Lane! 🙏

Added a few suggestions above

@lvotapka

Copy link
Copy Markdown
Contributor Author

Thanks for the suggestions, @jakirkham! I'm pushing those suggested changes now - let's see if it makes the tests pass...

@jakirkham

Copy link
Copy Markdown
Member

@conda-forge-admin , please re-render

@jakirkham

Copy link
Copy Markdown
Member

Turns out re-rendering ran into issues with conda_build_config.yaml. So switching to conda-forge.yml for the CentOS 7 setting fixed that as well

@jakirkham

Copy link
Copy Markdown
Member

Thanks Lane! 🙏

Looks like it passes 🥳

@lvotapka

Copy link
Copy Markdown
Contributor Author

Fantastic! Let me try the merge...

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.

2 participants