[20.x] Add patches for ROOT 6.38.00#401
Conversation
|
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 ( I do have some suggestions for making it better though... For recipe/meta.yaml:
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/19757094887. Examine the logs at this URL for more detail. |
There was a problem hiding this comment.
This LGTM.
It bears repeating though that clang 20 has a bunch of ABI breaks (especially in C++ containers), and we've tried avoid using this at scale, c.f. conda-forge/conda-forge-pinning-feedstock#7839
In other words, it's possible you'll get issues when you combine llvm-20-based root with packages built by llvm 19 or 21 (which fixed the ABI break)
…5.11.27.22.01.22 Other tools: - conda-build 25.5.0 - rattler-build 0.44.0 - rattler-build-conda-compat 1.4.5
9d42d2c to
31cbc6d
Compare
|
@h-vetinari thank you for your comment, if we'll see some issues at least we know where to start looking from. Meanwhile, I removed the leftover comment from |
There was a problem hiding this comment.
I added this file on the main branch in 7613f51, so we don't have to check this in anew every time root adds support for a version
There was a problem hiding this comment.
Thank you so much!
Checklist
0(if the version changed)conda-smithy(Use the phrase@conda-forge-admin, please rerenderin a comment in this PR for automated rerendering)