Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not use removed iteritems from conda_build #62

Merged
merged 2 commits into from
Jan 23, 2023

Conversation

hmaarrfk
Copy link
Contributor

Do I really need to bump the cdt number in https://github.com/conda-forge/cdt-builds#changing-the-cdt-generation-script-rpmpy

it seems that this is just a problem with the generation script.

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

Checklist:

  • if you have added a CDT, it appears in the cdt_slugs.yaml file and
    you have rerun the script python gen_cdt_recipes.py.
  • if you have changed the CDT generator code (rpm.py), you have bumped
    the build number in conda_build_config.yaml and have remade all of the
    recipes via running python gen_cdt_recipes.py --force
  • if you have added a custom CDT recipe, you have added the name of the CDT
    with custom: true in the cdt_slugs.yaml file.
  • all CDT recipes have build number set by {{ cdt_build_number }} for
    old-style/legacy CDTs or {{ cdt_build_number|int + 1000 }} for new-style CDTs
  • if you see a warning about a CDT not having a license, you have added the
    license_file key in the cdt_slugs.yaml file with the path to the appropriate
    license in licenses/

NOTE: If you make any changes to cd_slugs.yaml, you need to reun the generator code
via python gen_cdt_recipes.py.

@hmaarrfk hmaarrfk mentioned this pull request Jan 23, 2023
6 tasks
@beckermr beckermr enabled auto-merge January 23, 2023 14:28
@beckermr beckermr merged commit 792eefd into conda-forge:main Jan 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants