Update to 1.6.1 and test on libvips

[conda-forge-admin]

I tested this with conda-forge/pyvips-feedstock#44 by uploading it to my own channel, then trying to build out libvips

without it, I needed a local patch to libvips which I presume would have to be replicated in other feedstocks.

Fixes #48

[conda-forge-admin]

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.

I do have some suggestions for making it better though...

For recipe/recipe.yaml:

• ℹ️ noarch: python packages install on every Python version at or above python_min, but at least one python test only runs against a single python_version. Consider testing against both the minimum and a latest supported Python:

tests:
  - python:
      python_version:
        - ${{ python_min }}.*
        - "*"

• ℹ️ 'Store Build Artifacts' is deprecated.
  Deprecated. Store the conda build_artifacts directory as an Azure pipeline artifact. Use workflow_settings.store_build_artifacts instead.

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

[MementoRC]

Your hint on the issue I opened led me to try this:

          if [ "${CONDA_BUILD_CROSS_COMPILATION:-0}" = "1" ]; then
            for _f in "${PREFIX}"/lib/python*/site-packages/tree_sitter_embedded_template/_binding.abi3-*.so; do
              [ -e "${_f}" ] || continue
              _want="$(dirname "${_f}")/_binding.abi3.so"
              echo "cross-build: renaming $(basename "${_f}") -> $(basename "${_want}")"
              mv -f "${_f}" "${_want}"
            done
          fi

It resolved the issue I had with ppc64le 3.15: conda-forge/tree-sitter-embedded-template-feedstock#7

Hope this helps

[MementoRC]

@hmaarrfk I wish I could help, but this feels like above my brain-grade! 'hack' it'll be for my recipes - unless you think I could help in some way, I don't have much brain, but I have lots of time

[MementoRC]

Just a thought, could a downstream: tree-sitter-embedded-template help confirm that this is the right fix? It is a small and fast build, setuptools >=42 + py_limited_api extension that hits get_abi3_suffix() on the exact path 1.6.1 changes. I'd remove the 'hack', get a red after merge and this PR would still be green (assuming I understand the process correctly, my chickpea is overheating already)

[MementoRC]

Ok, last noise around this from me:

Tested this locally on an x86_64 -> linux-ppc64le cross build with Python 3.15, since that is where I hit the abi3 suffix mismatch downstream.

Build interpreter, x86_64 CPython 3.15.0rc2, _imp.extension_suffixes():

['.cpython-315-x86_64-linux-gnu.so', '.abi3-x86_64-linux-gnu.so', '.abi3.so',
'.abi3t-x86_64-linux-gnu.so', '.abi3t.so', '.so']

Inside a cross-python built from this PR (crossenv 1.6.1 + patches/0004), host linux-ppc64le:

EXTENSION_SUFFIXES : ['.cpython-315-powerpc64le-linux-gnu.so', '.abi3.so', '.so']
get_abi3_suffix() : .abi3.so
EXT_SUFFIX : .cpython-315-powerpc64le-linux-gnu.so
platform : linux-ppc64le

With crossenv 1.5.0 there is no EXTENSION_SUFFIXES line in importlib-machinery-patch.py.tmpl, so get_abi3_suffix() takes the first .abi3 entry from the build interpreter's list above, which is .abi3-x86_64-linux-gnu.so.

So 1.6.1 gives the plain .abi3.so on a cross build. I can drop the rename workaround I am carrying in tree-sitter-embedded-template (and others) once this lands.

One question: the list is [ext_suffix, ".abi3.so", ".so"] with no .abi3t entry. On a free-threaded build the plain .abi3 suffixes are not in _PyImport_DynLoadFiletab at all, since they sit inside the #ifndef Py_GIL_DISABLED
block in Python/dynload_shlib.c. Does that matter for free-threaded cross builds, or is it out of scope here? I have not tested one.

[isuruf]

@MementoRC we should fix https://github.com/robotpy/crossenv/blob/a9434e8d5faaf9149fd23a654dd2a5670aef2c45/crossenv/scripts/importlib-machinery-patch.py.tmpl#L32 for 3.15+

[MementoRC]

Confirmed fixed:

• MNT: drop abi3 rename workaround, fixed by crossenv 1.6.1 tree-sitter-embedded-template-feedstock#9

[jakirkham]

Am not fully caught up here, but just want to mention we ran into the same issue with macOS triples not working in XGBoost that Memento is fixing in PR: robotpy/crossenv#166

For now we are taking the approach of pinning crossenv in the XGBoost recipe

[isuruf]

Fixed in #50 (comment)

[jakirkham]

Thanks Isuru! 🙏

Will give that a try

[jakirkham]

Just to confirm, the XGBoost macOS ARM build was fixed with PR: #50

Thanks again Isuru! 🙏