GNU bug report logs -
#66052
[PATCH gnome-team 0/1] Update tracker, and ignore i686 missing tests
Previous Next
Reported by: Vivien Kraus <vivien <at> planete-kraus.eu>
Date: Sun, 17 Sep 2023 13:02:01 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
View this message in rfc822 format
Am Sonntag, dem 17.09.2023 um 13:29 +0200 schrieb Vivien Kraus:
> * gnu/packages/gnome.scm (tracker): Update to 3.6.0.
> [#:phases]: On 32-bit systems, disable 2 failing tests.
Should be [#:phases]: Add ‘the-added-phase’ otherwise the ChangeLog
LGTM.
> ---
> gnu/packages/gnome.scm | 20 ++++++++++++++++++--
> 1 file changed, 18 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
> index 2077478ff2..29aa57133d 100644
> --- a/gnu/packages/gnome.scm
> +++ b/gnu/packages/gnome.scm
> @@ -9387,7 +9387,7 @@ (define-public gnome-autoar
> (define-public tracker
> (package
> (name "tracker")
> - (version "3.5.3")
> + (version "3.6.0")
> (source (origin
> (method url-fetch)
> (uri (string-append "mirror://gnome/sources/tracker/"
> @@ -9395,7 +9395,7 @@ (define-public tracker
> "tracker-" version ".tar.xz"))
> (sha256
> (base32
> -
> "1vi878f95a2nlvqz46ph6f05hywjb2ni0znqavhdkrbvi6qchrhl"))))
> +
> "1whdqidxmagsc35pmz9kcc5vs3bmvbkmnis7prnx3zxs37z2qnaj"))))
> (build-system meson-build-system)
> (arguments
> (list
> @@ -9417,6 +9417,22 @@ (define-public tracker
> (substitute* "utils/trackertestutils/__main__.py"
> (("/bin/bash")
> (search-input-file inputs "bin/bash")))))
> + #$@(if (target-x86-32?)
> + ;; On 32-bit systems, the far away dates are
> incorrect, and
> + ;; the floats are not parsed exactly.
> + (list
> + #~(add-after 'unpack 'relax-float-equality-tests
> + (lambda _
> + (substitute* "tests/libtracker-
> sparql/tracker-statement-test.c"
> + (("g_assert_cmpfloat *\\((.*), ==, ([0-
> 9.e-]+)\\);" total actual expected)
> + (string-append
> "g_assert_cmpfloat_with_epsilon ("
> + actual ", " expected ",
> 1e-12);")))))
> + #~(add-after 'unpack 'disable-datetime-test
> + (lambda _
> + (substitute* "tests/core/tracker-sparql-
> test.c"
> + (("\\{ \"datetime/direct-1\", .* \\},")
> + "/* datetime test disabled */")))))
> + '())
You can (and probably should) make this into a ‘disable-failing-tests’
or inline it into an existing one. Otherwise LGTM but waiting for CI
to test.
Cheers
This bug report was last modified 1 year and 239 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.