search

m-ab-s / media-autobuild_suite

This Windows Batchscript helps setup a Mingw-w64 compiler environment for building ffmpeg and other media tools under Windows.
GNU General Public License v3.0
1.49k stars 257 forks source link

media-suite_deps.sh: set GIT tag to v0.8.2 for libjxl - Fixes #2521 #2565

Closed hnsteyding closed 5 months ago

hnsteyding commented 7 months ago

Fixes https://github.com/m-ab-s/media-autobuild_suite/issues/2521 Fixes https://github.com/m-ab-s/media-autobuild_suite/issues/2563

Description: Current git tag for libjxl library creates following compile error:

Likely error (tail of the failed operation logfile): O:/ffmpeg/build/libjxl-git/lib/jxl/jxl_cms.cc:1003:9: error: 'struct jxl::ColorEncoding' has no member named 'rendering_intent'; did you mean 'GetRenderingIntent'? 1003 | c_enc.rendering_intent = static_cast(rendering_intent32); | ^~~~ | GetRenderingIntent O:/ffmpeg/build/libjxl-git/lib/jxl/jxl_cms.cc: In function 'void jxl::{anonymous}::JxlCmsInit(void, size_t, size_t, const JxlColorProfile, const JxlColorProfile, float)': O:/ffmpeg/build/libjxl-git/lib/jxl/jxl_cms.cc:1234:55: error: 'struct jxl::ColorEncoding' has no member named 'rendering_intent'; did you mean 'GetRenderingIntent'? 1234 | const uint32_t intent = static_cast(c_dst.rendering_intent); | ^~~~ | GetRenderingIntent ninja: build stopped: subcommand failed. build failed. Check O:/ffmpeg/build/libjxl-git/build-64bit/ab-suite.build.log

1480c1 commented 5 months ago

I don't think this is needed anymore, if you believe it is, you can comment and I can re-review