(re)build libheif with X265 encoder instead of kvazaar

[jtomson]

imgproxy is generating larger-than-expected heif images, with the wrong profile

imgproxy builds its own libheif, and uses something called the kvazaar library to encode. This causes heics to be encoded as large-ish videos of one frame with the Main HEVC profile , instead of using the Main Still Picture HEVC profile

The base image builds the libheif:

https://github.com/imgproxy/imgproxy-docker-base/blob/9095a2cbb6128571fd4e047f1175db1881302954/build-deps.sh#L317-L328

Not sure if re-building it like this is more or less maintainable than forking the base image too, but modifying the compile options seems to do the trick - generating proper heic files

[bantonj]

🔥

[jtomson]

actually this whole encoding lib diversion may be unnecessary and using quality=50, which is still required used by x265, is the better fix? I'm not sure that the profile (Main vs Main Still Picture) makes a practical impact on size or performance

[bantonj]

actually this whole encoding lib diversion may be unnecessary and using quality=50, which is still required used x265, is the better fix? I'm not sure that the profile (Main vs Main Still Picture) makes a practical impact on size or performance

x265 is included in the existing build? It just doesn't always use it?

[jtomson]

sorry I meant to say we need to use q=50 regardless of building with x265 (which is not currently used in prod) or kvazaar (currently used in prod)

[viralauraframes]

I think this is still good to merge. I did some testing and output looks good.

[viralauraframes]

Merging this.
I did on frame testing on an carver with quality 50 HEIC with the same images as before and the performance was similar to what was before so this change should be good.