olokelo / jxlpy

Cython bindings and Pillow plugin for JPEG XL
MIT License
49 stars 12 forks source link

Support older pillows #26

Closed una-dinosauria closed 5 months ago

una-dinosauria commented 5 months ago

From https://github.com/olokelo/jxlpy/issues/18, https://github.com/olokelo/jxlpy/issues/21, https://github.com/olokelo/jxlpy/issues/22, it seems like a recent pillow is needed.

As indicated by https://github.com/olokelo/jxlpy/pull/19, there was a breaking change to the .mode property in PIL 10.1.0.

This patch ensures that we don't have to drop support for older Pillow versions, following the pattern in https://github.com/fdintino/pillow-avif-plugin/blob/aa7ac7f74c84ac409ad8668cd25f466334fd3753/src/pillow_avif/AvifImagePlugin.py#L72-L75.

olokelo commented 5 months ago

Thank you :)