public inbox for git-commits@fedoraproject.org
help / color / mirror / Atom feed
From: Simone Caronni <negativo17@gmail.com>
To: git-commits@fedoraproject.org
Subject: [rpms/perl-Prima] rawhide: Fix tests with libheif 1.23.1
Date: Mon, 10 Aug 2026 09:02:28 GMT	[thread overview]
Message-ID: <178635254844.1.9065454776370672518.rpms-perl-Prima-22b89ccf109e@fedoraproject.org> (raw)

A new commit has been pushed.

Repo   : rpms/perl-Prima
Branch : rawhide
Commit : 22b89ccf109e6d4f0b9f500e203a701d2e2380f1
Author : Simone Caronni <negativo17@gmail.com>
Date   : 2026-08-10T11:02:14+02:00
Stats  : +34/-1 in 2 file(s)
URL    : https://src.fedoraproject.org/rpms/perl-Prima/c/22b89ccf109e6d4f0b9f500e203a701d2e2380f1?branch=rawhide

Log:
Fix tests with libheif 1.23.1

---
diff --git a/perl-Prima-1.77-libheif-1.23.1.patch b/perl-Prima-1.77-libheif-1.23.1.patch
new file mode 100644
index 0000000..0771b4d
--- /dev/null
+++ b/perl-Prima-1.77-libheif-1.23.1.patch
@@ -0,0 +1,29 @@
+From: Simone Caronni <negativo17@gmail.com>
+Date: Mon, 10 Aug 2026 10:52:00 +0200
+Subject: [PATCH] Test HEIF lossless encoding with AOM SSIM tuning
+
+libheif 1.23.1 defaults the AOM encoder to IQ tuning.  libaom rejects that
+tuning mode together with lossless encoding because IQ tuning enables chroma
+delta-Q.  The test needs lossless encoding for exact pixel comparisons, so
+explicitly retain the previous SSIM tuning when AOM is the selected encoder.
+---
+ t/Image/Loader.t | 4 ++++
+ 1 file changed, 4 insertions(+)
+
+diff --git a/t/Image/Loader.t b/t/Image/Loader.t
+index 6f5ca231..00e2ffe7 100644
+--- a/t/Image/Loader.t
++++ b/t/Image/Loader.t
+@@ -107,6 +107,10 @@ sub test_multiframe_codec
+ 		my $ok = 0;
+ 		my $info = Prima::Image->has_codec('HEIF') or goto FAIL;
+ 		goto FAIL unless length $info->{saveInput}->{encoder}; # codec cannot suggest a good plugin that can both load and save
++		# libheif 1.23.1 defaults AOM to IQ tuning, which libaom does not
++		# permit in lossless mode. Keep the old AOM tuning for this lossless test.
++		$saveopts{'aom.tune'} = 'ssim'
++			if $info->{saveInput}->{encoder} eq 'aom';
+ 		$ok = 1;
+ 	FAIL:
+ 		unless ($ok) {
+-- 
+2.51.0

diff --git a/perl-Prima.spec b/perl-Prima.spec
index 4cf261c..52de10e 100644
--- a/perl-Prima.spec
+++ b/perl-Prima.spec
@@ -36,7 +36,7 @@
 
 Name:           perl-Prima
 Version:        1.77
-Release:        5%{?dist}
+Release:        6%{?dist}
 Summary:        Perl graphic toolkit
 # Copying:              BSD-2-Clause text
 # examples/tiger.eps:   AGPL-3.0-or-later (bundled from GhostScript? CPAN RT#122271)
@@ -57,6 +57,7 @@ Summary:        Perl graphic toolkit
 License:        BSD-2-Clause AND BSD-3-Clause AND BSD-4-Clause AND MIT-open-group AND HPND AND HPND-sell-variant AND TCL AND ImageMagick AND LGPL-2.0-or-later AND AGPL-3.0-or-later
 URL:            https://metacpan.org/dist/Prima
 Source0:        https://cpan.metacpan.org/authors/id/K/KA/KARASIK/Prima-%{version}.tar.gz
+Patch0:         perl-Prima-1.77-libheif-1.23.1.patch
 BuildRequires:  coreutils
 BuildRequires:  findutils
 %if %{with perl_Prima_enables_gtk3}
@@ -333,6 +334,9 @@ unset DISPLAY XDG_SESSION_TYPE
 %{_libexecdir}/%{name}
 
 %changelog
+* Mon Aug 10 2026 Simone Caronni <negativo17@gmail.com> - 1.77-6
+- Fix tests with libheif 1.23.1.
+
 * Mon Aug 10 2026 Simone Caronni <negativo17@gmail.com> - 1.77-5
 - Rebuilt for updated dependencies.
 

                 reply	other threads:[~2026-08-10  9:02 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=178635254844.1.9065454776370672518.rpms-perl-Prima-22b89ccf109e@fedoraproject.org \
    --to=negativo17@gmail.com \
    --cc=git-commits@fedoraproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox