Route rdkitpython cmake config files into the rdkit package (closes #227) by skearnes · Pull Request #228 · conda-forge/rdkit-feedstock (original) (raw)

@skearnes @claude

Backports rdkit/rdkit#9288 by tagging the two install() calls for rdkitpython-config.cmake / rdkitpython-targets.cmake with COMPONENT python instead of COMPONENT dev. These files hardcode a single python version at build configure time (via configure_file substitution), so they belong with the python wrappers — which are themselves python-version-specific — rather than in librdkit-dev (python-agnostic).

Add tests in librdkit-dev (assert absence) and rdkit (assert presence) that lock the relocation in. Bump build number to rebuild.

See conda-forge#227 for the full diagnosis.

Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com

@skearnes marked this pull request as ready for review

May 18, 2026 00:59

skearnes added a commit to regro-cf-autotick-bot/rdkit-feedstock that referenced this pull request

May 30, 2026

@skearnes @claude

…g tests

v2026_03_3 includes the three component-tagging/linker fixes these patches backported, so they are dropped:

rdkitpython-component-python.patch (rdkit/rdkit#9288) was never on this branch and is also upstream now, so nothing to drop there.

Also adds the test assertions from the closed conda-forge#228 that never merged: librdkit-dev must NOT ship rdkitpython-config.cmake (it is python-version coupled), and the rdkit (python) output MUST ship it.

Verified the four remaining patches apply cleanly in sequence against the v2026_03_3 source tarball.

Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com

This was referenced

May 30, 2026

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})