GNU bug report logs - #62467
[PATCH gnome-team 0/5] Update fundamental packages

Previous Next

Package: guix-patches;

Reported by: Liliana Marie Prikler <liliana.prikler <at> gmail.com>

Date: Sun, 26 Mar 2023 19:22:02 UTC

Severity: normal

Tags: patch

Done: Liliana Marie Prikler <liliana.prikler <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


Message #182 received at 62467 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Kaelyn Takata <liliana.prikler <at> gmail.com>
Cc: rg <at> raghavgururajan.name, 62467 <at> debbugs.gnu.org
Subject: Re: [PATCH gnome-team v3 01/15] gnu: librsvg-2.40: Fix test failure
 with Pango 1.50.
Date: Tue, 11 Apr 2023 16:52:17 -0400
Hi,

Kaelyn Takata <liliana.prikler <at> gmail.com> writes:

> * gnu/packages/gnome.scm (librsvg-2.40): Fix test failure with Pango 1.50.
>
> Signed-off-by: Andreas Enge <andreas <at> enge.fr>
> Signed-off-by: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
> ---
>  gnu/packages/gnome.scm | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
> index 49ff1464f5..6db6004356 100644
> --- a/gnu/packages/gnome.scm
> +++ b/gnu/packages/gnome.scm
> @@ -73,6 +73,7 @@
>  ;;; Copyright © 2022 Rene Saavedra <nanuui <at> protonmail.com>
>  ;;; Copyright © 2022 Alexandros Theodotou <alex <at> zrythm.org>
>  ;;; Copyright © 2022 Arjan Adriaanse <arjan <at> adriaan.se>
> +;;; Copyright © 2023 Kaelyn Takata <kaelyn.alexi <at> protonmail.com>
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -3780,6 +3781,12 @@ (define-public librsvg-2.40
>                 (("gdk_pixbuf_cache_file = .*$")
>                  "gdk_pixbuf_cache_file = $(TMPDIR)/loaders.cache\n"))
>               #t))
> +         (add-before 'check 'fix-test-with-pango-1.50
> +           (lambda _
> +	     ;; Changes between pango 1.48 and 1.50 caused the text to be one
> +	     ;; pixel lower in the output image compared to the reference.
> +             (substitute* "tests/fixtures/reftests/bugs/587721-text-transform.svg"
> +	       (("660\\.9") "659.9"))))
>           (add-before 'check 'remove-failing-tests
>             (lambda _
>               (with-directory-excursion "tests/fixtures/reftests"

LGTM, although such fragil tests perhaps could be simply disabled to
spare someone the hassle to investigate them when they break again in
the future.

-- 
Thanks,
Maxim




This bug report was last modified 2 years and 39 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.