Skip to content

Conversation

lgray
Copy link
Contributor

@lgray lgray commented Jun 12, 2025

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.

This applies cms thread-safety/c++ modernization patches to fastjet-contrib and enables a future windows build.

Add some strictness to package_contents checks.

c.f. https://github.com/cms-externals/fastjet-contrib/tree/cms/v1.101 and cms-externals/fastjet-contrib#19

Fixes #13

@lgray
Copy link
Contributor Author

lgray commented Jun 12, 2025

@conda-forge-admin, please rerender

@conda-forge-admin
Copy link
Contributor

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

@conda-forge-admin
Copy link
Contributor

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

I tried to rerender for you, but it looks like there was nothing to do.

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

@lgray
Copy link
Contributor Author

lgray commented Jun 12, 2025

@matthewfeickert Just some final cruft clean up and dealing with chores. Also prepares this feedstock for the eventuality of windows builds. Review and merge at your leisure.

@lgray lgray changed the title build: full patchset from cms-externals build: full patchset from cms-externals; docs: improve description Jun 12, 2025
@lgray
Copy link
Contributor Author

lgray commented Jun 12, 2025

ok - added some strictness to the package content checks to make sure we're always producing cmake info from now on.

@matthewfeickert
Copy link
Member

@conda-forge-admin, please rerender

@lgray lgray merged commit bcd56d1 into conda-forge:main Jun 13, 2025
8 checks passed
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.

chore: update about section concerning -split output
3 participants