Skip to content

Add cross-compilation for arm64 and aarch64#6

Merged
shermansiu merged 5 commits into
conda-forge:mainfrom
shermansiu:feat/add_add_cross_compilation
Jul 30, 2025
Merged

Add cross-compilation for arm64 and aarch64#6
shermansiu merged 5 commits into
conda-forge:mainfrom
shermansiu:feat/add_add_cross_compilation

Conversation

@shermansiu
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.

@shermansiu shermansiu requested a review from maresb as a code owner July 29, 2025 23:04
@shermansiu
Copy link
Copy Markdown
Contributor Author

@conda-forge-admin, please rerender

@conda-forge-admin
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/recipe.yaml) and found it was in an excellent condition.

conda-forge-webservices[bot] and others added 2 commits July 29, 2025 23:07
@shermansiu
Copy link
Copy Markdown
Contributor Author

@conda-forge-admin, please re-render

@shermansiu
Copy link
Copy Markdown
Contributor Author

@conda-forge-admin, please rerender

@shermansiu shermansiu merged commit 26b644e into conda-forge:main Jul 30, 2025
6 checks passed
@h-vetinari
Copy link
Copy Markdown
Member

Don't know if you noticed, but you haven't gotten any Linux jobs here yet

Comment thread conda-forge.yml
- pull_request
provider:
linux_64: github_actions
linux_aarch64: azure
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Provider is irrelevant for platforms that are being cross compiled (build_platform: has higher precedence, and so this ends up using the providers for linux_64)

@h-vetinari
Copy link
Copy Markdown
Member

Don't know if you noticed, but you haven't gotten any Linux jobs here yet

Hm, you actually do have a job running on main now. Seems like the setup for running on PRs is not working correctly yet. Can you try rebasing #5? I'm on a phone, so can't help this time

@h-vetinari
Copy link
Copy Markdown
Member

Note that you'll have to wait a bit to get some runners. There's a pytorch PR running currently which is using all the GPUs. That's one more reason why you should switch to CPU runners unless actual GPUs are absolutely necessary. It will allow your jobs to be scheduled more quickly.

In general, please be aware that you're using a very rare resource that's needed by other feedstocks as well. In particular, while developing a PR for some change, you should add

build:
  skip: match(python, "!=312")

(or however this is spelled in v1; still don't know it by heart). Once everything is green you can remove the skip to build out all version only once before merging.

@h-vetinari h-vetinari mentioned this pull request Jul 30, 2025
shermansiu added a commit to shermansiu/.cirun that referenced this pull request Jul 30, 2025
h-vetinari pushed a commit to conda-forge/.cirun that referenced this pull request Jul 30, 2025
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