conda-forge / cdt-builds

conda-forge cdt builds
BSD 3-Clause "New" or "Revised" License
4 stars 22 forks source link

Add `libmlx*` CDTs #33

Open jakirkham opened 4 years ago

jakirkham commented 4 years ago

Includes libmlx4 and libmlx5 CDTs.

Please see the repo readme for directions on how to make PRs on this repo.

Checklist:

NOTE: If you make any changes to cd_slugs.yaml, you need to reun the generator code via python gen_cdt_recipes.py. Add the --fast option to make this go more quickly.

jakirkham commented 4 years ago

This wound up touching and adding way more CDTs than I would have naively expected. Maybe I'm doing something wrong? 😬

beckermr commented 4 years ago

Your machine is still not generating the CDTs properly. Can you send me the full output of the command you ran? Are there any errors?

jakirkham commented 4 years ago

Yeah I'm not sure I understand why though (or how to fix it). My best guess is requirements might be missing ( https://github.com/conda-forge/cdt-builds/issues/32 ), but I honestly don't know πŸ€·β€β™‚οΈ Happy to make changes πŸ™‚

Just running python gen_cdt_recipes.py. Am using condaforge/linux-anvil-comp7 with the directory passed through (as this doesn't run on macOS).

There's a ton of output. Have attached as a text file.

jakirkham commented 4 years ago

Installed the requirements noted here and reran. It seems to have done a bit better. Not sure if we are still missing things though. Please let me know πŸ™‚

beckermr commented 4 years ago

This will run on OS X. Install the requirements I sent.

I think I might have missed a key detail. Run the script over and over again until all of the warnings or errors go away. It very often hits a race condition.

beckermr commented 4 years ago

Do you need these on cos7? As far as I can tell, there are no RPMs for them on cos7.