Skip to content

Comments

Deploy only when all tests pass. Use jpeg-turbo#3497

Merged
snowman2 merged 3 commits intorasterio:mainfrom
w8sl:Publish_after_tests
Dec 26, 2025
Merged

Deploy only when all tests pass. Use jpeg-turbo#3497
snowman2 merged 3 commits intorasterio:mainfrom
w8sl:Publish_after_tests

Conversation

@w8sl
Copy link
Contributor

@w8sl w8sl commented Dec 26, 2025

  • Deploy after tests
  • Ensure wheels from build artifacts are tested
  • Use jpeg-turbo on macOS and Linux:
    • open-source
    • actively maintained
    • 2-3 x faster
    • enables compatibility with 12 bit jpegs
    • avoids conflicts when libjpeg is used for macOS x86_64 build
    • already used as a GDAL dependency with vcpkg and Homebrew builds
  • Windows build is unchanged (already uses jpeg-turbo)

w8sl added 2 commits December 26, 2025 14:27
…cts.

Fix for bug when prerelease wheels from PyPI were downloaded and tested instead of build artifacts
@snowman2 snowman2 added enhancement packaging dependencies Pull requests that update a dependency file labels Dec 26, 2025
@snowman2 snowman2 added this to the 1.5.0 milestone Dec 26, 2025
Copy link
Member

@snowman2 snowman2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work @w8sl! Thanks 👍

run: |
python -m pip install numpy aiohttp attrs pytest click mock boto3 packaging hypothesis fsspec requests
python -m pip install --pre --no-deps --find-links dist rasterio
python -m pip install --pre --no-deps --no-index --find-links dist rasterio
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix for a problem, when prerelease wheels from PyPI are tested instead of build artifacts !

@snowman2 snowman2 merged commit 0c9d1d2 into rasterio:main Dec 26, 2025
29 of 33 checks passed
@w8sl w8sl deleted the Publish_after_tests branch December 26, 2025 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file enhancement packaging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants