Skip to content

Conversation

@radarhere
Copy link
Member

brotli 1.2.0 has been released - https://github.com/google/brotli/releases/tag/v1.2.0

@radarhere radarhere changed the title Updated brotli to 1.2.0, except on iOS Updated brotli to 1.2.0 Oct 31, 2025
local out_dir=$(fetch_unpack https://github.com/google/brotli/archive/v$BROTLI_VERSION.tar.gz brotli-$BROTLI_VERSION.tar.gz)
(cd $out_dir \
&& cmake -DCMAKE_INSTALL_PREFIX=$BUILD_PREFIX -DCMAKE_INSTALL_LIBDIR=$BUILD_PREFIX/lib -DCMAKE_INSTALL_NAME_DIR=$BUILD_PREFIX/lib $HOST_CMAKE_FLAGS . \
&& cmake -DCMAKE_INSTALL_PREFIX=$BUILD_PREFIX -DCMAKE_INSTALL_LIBDIR=$BUILD_PREFIX/lib -DCMAKE_INSTALL_NAME_DIR=$BUILD_PREFIX/lib -DCMAKE_MACOSX_BUNDLE=OFF $HOST_CMAKE_FLAGS . \
Copy link
Member Author

@radarhere radarhere Oct 31, 2025

Choose a reason for hiding this comment

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

I was experiencing an error on iOS - https://github.com/radarhere/Pillow/actions/runs/18856549681/job/53805755475#step:5:7941

CMake Error at CMakeLists.txt:188 (install):
install TARGETS given no BUNDLE DESTINATION for MACOSX_BUNDLE executable
target "brotli".

This workaround came from https://gitlab.com/libtiff/libtiff/-/issues/282. I've created google/brotli#1370 to see if there might be a better solution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant