Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable flexible channel priority to install tiledb::tiledb-py for py37 #33

Merged
merged 1 commit into from
Jul 11, 2023

Conversation

jdblischak
Copy link
Collaborator

There are currently 2 issues that are breaking the linux-64 Python 3.7 nightly builds on the nightly-build branch:

  1. The conda solver is unable to install tiledbsoma-py/libtiledbsoma/tiledb-py/libtiledb in a Python 3.7 environment, so conda is unable to install the run time dependencies for the test section of the recipe. This conda solver error first started back on 2023-06-26. I believe the problem is the strict channel priority, which by default prefers older packages in a higher-priority channel (in this case conda-forge, which only has older tiledb-py available for Python 3.7) over the newer packages available in lower-priority channel (in this case tiledb, because we only build for Python 3.7 in the TileDB-Inc feedstock fork). This PR updates the feedstock to use flexible channel priority, which fixes this problem. Once we no longer need to build for Python 3.7, we should switch back to strict channel priority, since in general this ensures more stable environments

  2. While fixing the above Issue, I discovered that the recently updated zipp-feedstock has broken the build step (it's a dependency of setuptools) because zipp dropped support for Python 3.7. This is currently being fixed (zipp 3.16.0 no longer compatible with python 3.7 conda-forge/zipp-feedstock#42, Bump to Python >= 3.8 conda-forge/zipp-feedstock#43), but this PR will fail until the fix is deployed. However, I tested in my fork by temporarily setting zipp <3.16 to confirm that the flexible channel priority fixes the conda solver error

xref: TileDB-Inc/somacore-feedstock#4

@jdblischak
Copy link
Collaborator Author

The zipp issue with Python 3.7 has been resolved. The recipe was updated in conda-forge/zipp-feedstock#43 and the repodata patched in conda-forge/conda-forge-repodata-patches-feedstock#478

I restarted the CI for this PR, and now it's passing

@jdblischak jdblischak merged commit 2a42fa9 into TileDB-Inc:main Jul 11, 2023
@jdblischak jdblischak deleted the flexible-channel-priority branch July 11, 2023 13:23
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.

2 participants