Bug#1076067: avifile: armhf test regression with ffmpeg 6.1.1-5 and no neon instructions [PATCH]

[Gianfranco Costamagna]

Package: avifile
Version: 0.7.48~20090503.ds-26
Severity: serious
Tags: patch

Hello,
The attached should work to make sure we don't error out if neon flag is not 
built by ffmpeg.
I don't know how differently can try to make it smarter, or maybe we can just 
drop all that code.

  * Make sure the test doesn't fail when ffmpeg is built without neon support
    on armhf


Thanks for considering the patch.
diff -Nru avifile-0.7.48~20090503.ds/debian/tests/decoding-test-data/test1.cc 
avifile-0.7.48~20090503.ds/debian/tests/decoding-test-data/test1.cc
--- avifile-0.7.48~20090503.ds/debian/tests/decoding-test-data/test1.cc 
2022-04-05 00:15:21.000000000 +0200
+++ avifile-0.7.48~20090503.ds/debian/tests/decoding-test-data/test1.cc 
2024-07-10 10:11:42.000000000 +0200
@@ -166,13 +166,13 @@
     flags = av_get_cpu_flags();
     std::cout << "Old cpu flags: 0x" << std::hex << flags
              << std::dec << std::endl;
-    if (av_parse_cpu_caps(&flags, "-neon") < 0) {
-      std::cerr << "failed to parse cpu caps" << std::endl;
-      return 2;
+    if (av_parse_cpu_caps(&flags, "-neon") >= 0) {
+      av_force_cpu_flags(flags);
+      std::cout << "New cpu flags: 0x" << std::hex << flags
+                << std::dec << std::endl;
+    } else {
+      std::cout << "failed disable neon, ffmpeg built without support for it?" 
<< std::endl;
     }
-    av_force_cpu_flags(flags);
-    std::cout << "New cpu flags: 0x" << std::hex << flags
-             << std::dec << std::endl;
   }



thanks

Gianfranco

/* get filename */

OpenPGP_signature.asc
Description: OpenPGP digital signature