Skip to content

Commit

Permalink
fixup5
Browse files Browse the repository at this point in the history
  • Loading branch information
fdintino committed Aug 14, 2024
1 parent e632e43 commit 4b1e445
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 9 deletions.
3 changes: 2 additions & 1 deletion .github/workflows/macos-install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,8 @@ brew install \
webp \
dav1d \
aom \
rav1e
rav1e \
ninja
if [[ "$ImageOS" == "macos13" ]]; then
brew install --ignore-dependencies libraqm
else
Expand Down
11 changes: 5 additions & 6 deletions .github/workflows/wheels-dependencies.sh
Original file line number Diff line number Diff line change
Expand Up @@ -82,9 +82,8 @@ function install_rav1e {
fi

# Force libavif to treat system rav1e as if it were local
local cmake=$(get_modern_cmake)
local cmake_root=`$cmake --system-information 2>&1 | grep CMAKE_ROOT | grep -v CMAKE_ROOT:INTERNAL | sed -e s/\"//g -e 's/CMAKE_ROOT //g'`
cat <<EOF > $cmake_root/Modules/Findrav1e.cmake
mkdir -p /tmp/cmake/Modules
cat <<EOF > /tmp/cmake/Modules/Findrav1e.cmake
add_library(rav1e::rav1e STATIC IMPORTED GLOBAL)
set_target_properties(rav1e::rav1e PROPERTIES
IMPORTED_LOCATION "$BUILD_PREFIX/lib/librav1e.a"
Expand Down Expand Up @@ -118,7 +117,7 @@ function build_libavif {
-DAVIF_CODEC_DAV1D=LOCAL \
-DAVIF_CODEC_SVT=LOCAL \
-DENABLE_NASM=ON \
-DCMAKE_MACOSX_RPATH=OFF \
-DCMAKE_MODULE_PATH=/tmp/cmake/Modules \
. \
&& make install)

Expand Down Expand Up @@ -209,11 +208,11 @@ if [[ -n "$IS_MACOS" ]]; then
# remove webp and zstd to avoid inclusion on x86_64
# remove aom and libavif to fix building on arm64
# curl from brew requires zstd, use system curl
brew remove --ignore-dependencies libpng libtiff libxcb libxau libxdmcp curl cairo lcms2 zstd aom libavif
brew remove --ignore-dependencies libpng libtiff libxcb libxau libxdmcp curl cairo lcms2 zstd
if [[ "$CIBW_ARCHS" == "arm64" ]]; then
brew remove --ignore-dependencies jpeg-turbo
else
brew remove --ignore-dependencies webp
brew remove --ignore-dependencies webp aom libavif
fi

brew install pkg-config
Expand Down
2 changes: 1 addition & 1 deletion Tests/check_wheel.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@


def test_wheel_modules() -> None:
expected_modules = {"pil", "tkinter", "freetype2", "littlecms2", "webp"}
expected_modules = {"pil", "tkinter", "freetype2", "littlecms2", "webp", "avif"}

# tkinter is not available in cibuildwheel installed CPython on Windows
try:
Expand Down
2 changes: 1 addition & 1 deletion depends/install_libavif.sh
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,6 @@ cmake -G Ninja -S . -B build \
-DCMAKE_MACOSX_RPATH=OFF \
"${LIBAVIF_CMAKE_FLAGS[@]}"

ninja -C build install
sudo ninja -C build install

popd

0 comments on commit 4b1e445

Please sign in to comment.