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

get_encoder_for_format test fails when building on Fedora #14

Closed
kalev opened this issue Aug 26, 2023 · 5 comments
Closed

get_encoder_for_format test fails when building on Fedora #14

kalev opened this issue Aug 26, 2023 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@kalev
Copy link

kalev commented Aug 26, 2023

When running libheif-rs tests on Fedora, I get:

---- get_encoder_for_format stdout ----
thread 'get_encoder_for_format' panicked at 'assertion failed: `(left == right)`
  left: `"AOMedia Project AV1 Encoder v3.6.1"`,
 right: `"SVT-AV1 encoder v1.4.1"`', tests/heif_struct_test.rs:35:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Any ideas what's going wrong here?

@sophie-h
Copy link

sophie-h commented Aug 27, 2023

I think what might be happening here is that libheif was built with AOM and SVT decoder for av1 enabled, and the tests of libheif-rs are not prepared for this?

So it might be fixed by replacing the loop with an .any() check or something similar.

@Cykooz Cykooz self-assigned this Aug 28, 2023
@Cykooz Cykooz added the bug Something isn't working label Aug 28, 2023
@Cykooz
Copy link
Owner

Cykooz commented Aug 28, 2023

What if I will fix this test in next "major" release of libheif-rs (v0.21)?
Is it worth fixing the test in v0.20?

@kalev
Copy link
Author

kalev commented Aug 28, 2023

Thanks! Looks like it's indeed the case that libheif is built with both aom and svt enabled: https://src.fedoraproject.org/rpms/libheif/blob/rawhide/f/libheif.spec

For context, I'm interested in libheif-rs because it's a dependency for loupe. Loupe is a new image viewer that @sophie-h is developing and we are planning on shipping in Fedora 39.

This issue here isn't much of a problem for packaging libheif-rs for Fedora; I just noticed it in passing. No need to do a new 0.20 release just for that. There is another issue that I just filed that is a bit of a blocker though: #15

@Cykooz
Copy link
Owner

Cykooz commented Sep 8, 2023

I have released v0.21 with fixed get_encoder_for_format test.

@Cykooz Cykooz closed this as completed Sep 13, 2023
@kalev
Copy link
Author

kalev commented Sep 20, 2023

Thanks a lot! I can confirm that it now builds fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants