Skip to content

[15.x] Migrate ucx116#1382

Merged
h-vetinari merged 4 commits into
conda-forge:15.xfrom
xhochy:ucx116
May 8, 2024
Merged

[15.x] Migrate ucx116#1382
h-vetinari merged 4 commits into
conda-forge:15.xfrom
xhochy:ucx116

Conversation

@xhochy

@xhochy xhochy commented May 7, 2024

Copy link
Copy Markdown
Member

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

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) and found it was in an excellent condition.

@xhochy

xhochy commented May 7, 2024

Copy link
Copy Markdown
Member Author

@h-vetinari Rerendering fails with

ValueError: variant config in /Users/uwe/Development/conda-forge/arrow-cpp-feedstock/recipe/conda_build_config.yaml is ambiguous because it
does not fully implement all zipped keys (To be clear: missing ['fortran_compiler_version', 'c_stdlib_version', 'cdt_name', 'cuda_compiler', 'cuda_compiler_version', 'docker_image'])
or specifies a subspace that is not fully implemented (To be clear:
.. we did not find ['13', '10'] from {'c_compiler_version': ['13', '10', '11'], 'cxx_compiler_version': ['13', '10', '11']} in c_compiler_version:['12', '11']).

Do you have an idea what changed and what we do need to adjust?

@xhochy

xhochy commented May 7, 2024

Copy link
Copy Markdown
Member Author

Ah! This is probably related to CUDA?

@h-vetinari

Copy link
Copy Markdown
Member

Ah! This is probably related to CUDA?

It's related to this. I argued not to do this in the core call, but was in the minority. You need 0fc822b to fix it. However, unless there's a big hurry for ucx, I'd like to get #1376 in first and then deal with the various migrations.

@xhochy

xhochy commented May 7, 2024

Copy link
Copy Markdown
Member Author

OK, then let's wait a bit.

@xhochy xhochy closed this May 7, 2024
@h-vetinari h-vetinari reopened this May 7, 2024
@conda-forge-webservices

Copy link
Copy Markdown

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

I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.

Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

@h-vetinari h-vetinari changed the base branch from main to 15.x May 7, 2024 21:49
@h-vetinari h-vetinari changed the title Migrate ucx116 [15.x] Migrate ucx116 May 7, 2024
@conda-forge-webservices

Copy link
Copy Markdown

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) and found it was in an excellent condition.

@h-vetinari h-vetinari mentioned this pull request May 8, 2024
5 tasks
@h-vetinari h-vetinari merged commit dfc6b44 into conda-forge:15.x May 8, 2024
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