Subject: bug#51126: closed (Re: [bug#51126] [PATCH] gnu: kdenlive: Update to 21.08.2.)

From: [email-masked] (GNU bug Tracking System)
Date: Tue, 12 Oct 2021 15:16:02 +0000

Your bug report

#51126: [PATCH] gnu: kdenlive: Update to 21.08.2.

which was filed against the guix-patches package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
51126: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=51126
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: [bug#51126] [PATCH] gnu: kdenlive: Update to 21.08.2.
From: Leo Famulari <[email-masked]>
To: Vinicius Monego <[email-masked]>
Cc: [email-masked]
Date: Tue, 12 Oct 2021 11:15:07 -0400
List-Id: <debbugs-submit.debbugs.gnu.org>

On Mon, Oct 11, 2021 at 12:04:46AM +0000, Vinicius Monego wrote:
> * gnu/packages/kde.scm (kdenlive): Update to 21.08.2.

Pushed as 385c9a4e30e952207df3246acaf13e6519b01af2

>        (arguments
> -       `(#:tests? #f                    ;TODO needs X
> +       ;; XXX: there is a single test that spawns other tests and
> +       ;; 1/3 tests failed and 1/327 assertions failed.  It seems
> +       ;; that individual tests can't be skipped.
> +       `(#:tests? #f

Thank you for taking the time to investigate and write this comment.
It's important work for the long-term maintainability of the packages.

Subject: [PATCH] gnu: kdenlive: Update to 21.08.2.
From: Vinicius Monego <[email-masked]>
To: [email-masked]
Cc: Vinicius Monego <[email-masked]>
Date: Mon, 11 Oct 2021 00:04:46 +0000
List-Id: <debbugs-submit.debbugs.gnu.org>

* gnu/packages/kde.scm (kdenlive): Update to 21.08.2.
---
 gnu/packages/kde.scm | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/kde.scm b/gnu/packages/kde.scm
index 66a112d8b5..93ffa0f43a 100644
--- a/gnu/packages/kde.scm
+++ b/gnu/packages/kde.scm
@@ -219,7 +219,7 @@ browser for easy news reading.")
     (license license:gpl2+)))
 
 (define-public kdenlive
-  (let ((version "21.08.1"))
+  (let ((version "21.08.2"))
     (package
       (name "kdenlive")
       (version version)
@@ -231,7 +231,7 @@ browser for easy news reading.")
                (commit (string-append "v" version))))
          (file-name (string-append name "-" version "-checkout"))
          (sha256
-          (base32 "1fvy2aa86pn4crk8lgxjh1kdn2lxzi66krnrr9m91mp89mmc760k"))))
+          (base32 "1l78xjdf1bmj3s8kysaqqgh67mb3vrc96rsdnp0i4awlyfsh89d7"))))
       (build-system qt-build-system)
       (native-inputs
        `(("extra-cmake-modules" ,extra-cmake-modules)
@@ -269,7 +269,10 @@ browser for easy news reading.")
          ("qtwebkit" ,qtwebkit)
          ("shared-mime-info" ,shared-mime-info)))
       (arguments
-       `(#:tests? #f                    ;TODO needs X
+       ;; XXX: there is a single test that spawns other tests and
+       ;; 1/3 tests failed and 1/327 assertions failed.  It seems
+       ;; that individual tests can't be skipped.
+       `(#:tests? #f
          #:phases
          (modify-phases %standard-phases
            (add-after 'install 'wrap-executable
-- 
2.30.2