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

libavif version note #199

Merged
merged 3 commits into from
Jul 27, 2023
Merged

libavif version note #199

merged 3 commits into from
Jul 27, 2023

Conversation

0xb4lint
Copy link
Contributor

@0xb4lint 0xb4lint commented Jul 26, 2023

Currently Debian 11 (bullseye, which is used in php-fpm Docker image until 8.0) only supports v0.8.4 libavif-bin package, however some avifdec options are only introduced in v0.9.3.

Right now v0.8.4's avifdec command produces the following error:
Too many positional arguments: 0

And unfortunately there's another issue with avifenc command's older version:
ERROR: Failed to encode image: No codec available (ref)

Removing the following avifdec parameters would fix the decoding command:

  • --ignore-icc: Set PNG compression level (PNG only; 0-9, 0=none, 9=max). Defaults to libpng's builtin default.
  • --no-strict: Disable strict decoding, which disables strict validation checks and errors
  • --png-compress 0: If the input file contains an embedded ICC profile, ignore it

Unfortunately, there's no easy fix for avifenc's missing codec issue.

There 2 possible solutions for the situation:

  • use Debian 12 (bookworm) which is the default for php:8.1-fpm and php:8.2-fpm Docker images
  • add bullseye-backports as source and install a more recent version libavif-bin:
    echo "deb http://deb.debian.org/debian bullseye-backports main" | sudo tee -a /etc/apt/sources.list
    sudo apt install -t bullseye-backports libavif-bin

@0xb4lint 0xb4lint changed the title Support for older avifdec version WIP: Support for older avifdec version Jul 26, 2023
@0xb4lint 0xb4lint changed the title WIP: Support for older avifdec version libavif version note Jul 26, 2023
@freekmurze freekmurze merged commit af17999 into spatie:main Jul 27, 2023
9 checks passed
@freekmurze
Copy link
Member

thanks!

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