fix c_stdlib_version zip with cdt_name by h-vetinari · Pull Request #5607 · conda-forge/conda-forge-pinning-feedstock (original) (raw)

Do we need to do this with c_stdlib too as it is also zipped?

So I looked at this again in more detail, and I had wrongly remembered / imagined that c_compiler needed to be zipped with its version. That's why it doesn't have config for CF_CUDA_ENABLED, and why it's not updated as part of the CUDA migrator. I think the same situation applies to c_stdlib, so I've removed that from the zip_keys.

Think we may also need to add both of these to the CUDA 12 migrator as well

Consequently, I've added only c_stdlib_version to cuda120.yaml.

I was kinda expecting the CI here would check matching zip length also for CUDA builds? But in any case, I think this should be good now. I'd propose to merge it and see if it solves the ucx situation.