serpilliere
commented
7 months ago Hi @cessenat We also have sanzu on centOS 7 here, and if I remember correctly, I agree with you: I may haved missed a dependency update on clang of clang-devel (I will double check this, and update the README accordingly)
By the way, you need another fix to run sanzu on centos 7: As it distributes an old FFmpeg version, you have to use this patch:
diff --git a/sanzu/src/video_decoder.rs b/sanzu/src/video_decoder.rs
index b1e1838c..c909b620 100644
--- a/sanzu/src/video_decoder.rs
+++ b/sanzu/src/video_decoder.rs
@@ -33,6 +33,10 @@ pub fn init_video_codec<'a>(
ffmpeg::av_log_set_level(ffmpeg::AV_LOG_FATAL);
}
+ unsafe {
+ ffmpeg::av_register_all();
+ }
+
let decoder: Box<dyn Decoder> = match name {
"null" => Box::new(DecoderNull {
data_rgb: None,
diff --git a/sanzu/src/video_encoder.rs b/sanzu/src/video_encoder.rs
index c4d210b0..d08a8553 100644
--- a/sanzu/src/video_encoder.rs
+++ b/sanzu/src/video_encoder.rs
@@ -611,6 +611,11 @@ pub fn init_video_encoder<'a>(
unsafe {
ffmpeg::av_log_set_level(ffmpeg::AV_LOG_FATAL);
}
+
+ unsafe {
+ ffmpeg::av_register_all();
+ }
+
let encoder: Box<dyn Encoder> = match name {
"null" => Box::new(EncoderNull::new()),
name => {which you can find in this branch: https://github.com/cea-sec/sanzu/tree/ffmpeg_old
The av_register_all is mandatory in old FFmpeg versions to prepare codec search (for decoding and encoding), and is now deprecated (so must be removed).
But I may stop maintain it in a near future, as centos7 EOL is soon ( :skull: in june 2024)
cessenat
I try to build sanzu for CentOS7 as a simple user with no sudo privilege. I use a decent gcc compiler (8.4). I obviously have missed a pre-requisite about "libclang": it fails as follows: << $ spack install sanzu %gcc ^cmake@3.27.9
I join the spack package. package.py.gz