Skip to content

avoid using cuda_compiler as selector#196

Merged
conda-forge-admin merged 5 commits into
conda-forge:mainfrom
conda-forge-admin:conda_forge_admin_195
May 30, 2025
Merged

avoid using cuda_compiler as selector#196
conda-forge-admin merged 5 commits into
conda-forge:mainfrom
conda-forge-admin:conda_forge_admin_195

Conversation

@conda-forge-admin

Copy link
Copy Markdown
Contributor

Hi! This is the friendly automated conda-forge-webservice.

I've started rerendering the recipe as instructed in #195.

If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!

Here's a checklist to do before merging.

Fixes #195

@conda-forge-admin

conda-forge-admin commented May 30, 2025

Copy link
Copy Markdown
Contributor Author

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.

I do have some suggestions for making it better though...

For recipe/meta.yaml:

  • ℹ️ The recipe is not parsable by parser conda-recipe-manager. The recipe can only be automatically migrated to the new v1 format if it is parseable by conda-recipe-manager.
  • ℹ️ The feedstock is lowering the image versions for one or more platforms: {'linux_64': 'cos7', 'linux_aarch64': 'cos7', 'linux_ppc64le': 'cos7'} (the default is alma9). Unless you are in the very rare case of repackaging binaryartifacts, consider removing these overrides from conda-forge.yml in the top feedstock directory.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/15349655748. Examine the logs at this URL for more detail.

@conda-forge-admin conda-forge-admin marked this pull request as ready for review May 30, 2025 04:22
@h-vetinari h-vetinari changed the title MNT: rerender avoid using cuda_version as selector May 30, 2025
@h-vetinari

Copy link
Copy Markdown
Member

The error here seems unrelated to the changes

+ mpiexec --help
$PREFIX/bin/mpiexec
prterun: symbol lookup error: $PREFIX/bin/../lib/libprrte.so.3: undefined symbol: pmix_show_help_add_dir

It seems to be happening for mpi_type=conda.

@beckermr

Copy link
Copy Markdown
Member

xref: open-mpi/ompi#13273

@beckermr

Copy link
Copy Markdown
Member

@conda-forge-admin rerender

@beckermr beckermr added the automerge Merge the PR when CI passes label May 30, 2025
@conda-forge-admin conda-forge-admin merged commit 0414cb9 into conda-forge:main May 30, 2025
11 checks passed
@conda-forge-admin

Copy link
Copy Markdown
Contributor Author

Hi! This is the friendly conda-forge automerge bot!

I considered the following status checks when analyzing this PR:

  • linter: passed
  • azure: passed

Thus the PR was passing and merged! Have a great day!

@leofang

leofang commented May 30, 2025

Copy link
Copy Markdown
Member

The error here seems unrelated to the changes

xref: open-mpi/ompi#13273

How come we haven't hit this before? Nothing else has changed and this is a trivial PR 🤔

@dalcinl

dalcinl commented May 31, 2025

Copy link
Copy Markdown
Contributor

Nothing else has changed and this is a trivial PR 🤔

No PR is "trivial" in conda-forge. After a recipe rerender or updated/rebuilt dependencies, things can break from one week to the other.

@h-vetinari h-vetinari changed the title avoid using cuda_version as selector avoid using cuda_compiler as selector Jun 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Merge the PR when CI passes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

@conda-forge-admin, please rerender

5 participants