Uh oh!
There was an error while loading. Please reload this page.
[main] Split shared Python library to libpython - #894
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/32334869920. Examine the logs at this URL for more detail. |
010b93e to
b74a768Compareh-vetinari
commented
Aug 12, 2026
Rebased after #888. Could you PTAL @conda-forge/python? :) |
h-vetinari
commented
Aug 13, 2026
@conda-forge-admin, please relint |
isuruf
left a comment
There was a problem hiding this comment.
libpython should be a dependency of python for python<3.15 to not break anything downstream
h-vetinari
commented
Aug 13, 2026
Hm. So I guess I need to switch the build order here. Conda can't figure out to build Also, quick question, I just stumbled upon https://github.com/conda-forge/libpython-feedstock/blob/main/recipe/meta.yaml (by looking through the package metadata for existing |
isuruf
commented
Aug 13, 2026
We should move feedstocks to use |
h-vetinari
commented
Aug 13, 2026
And repodata-patch existing users of |
h-vetinari
commented
Aug 14, 2026
This turned into a pretty big operation (and likely still contains bugs). Not sure if this is along the lines of what you had in mind. |
h-vetinari
commented
Aug 15, 2026
Before spending more time on this, can you confirm whether the last commit here is of the shape & kind that you had in mind @isuruf? |
…026.08.17.07.23.37 Other tools: - conda-build 26.7.0 - rattler-build 0.73.0 - rattler-build-conda-compat 1.4.19
h-vetinari
commented
Aug 20, 2026
OK, this is freshly rebased and ready now. I also improved the git history around the build order switch. PTAL @isuruf |
h-vetinari
commented
Aug 21, 2026
I guess we should merge conda-forge/conda-forge-repodata-patches-feedstock#1236 at the same time or before (due to the bare |
h-vetinari
commented
Aug 21, 2026
@isuruf, could you review here please? 🙏 |
Ping @isuruf. It'd be nice to be able to avoid having to keep rebasing this PR for anything that touches the build scripts (e.g. risc-v support) |
Uh oh!
There was an error while loading. Please reload this page.
isuruf
commented
Aug 26, 2026
Thanks |
Partial backport of #842, specifically 6eeda71, as suggested in #885. I also picked up another useful commit from the dev branch while rebasing. Once this PR (and a rebased #885) are merged, I'm happy to backport the pair as combined PRs to the maintenance branches.