GNU bug report logs - #69068
[PATCH] gnu: kitsas: Update to 5.4.1

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 69068 in the body.
You can then email your comments to 69068 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

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

From: Saku Laesvuori <saku <at> laesvuori.fi>
To: guix-patches <at> gnu.org
Cc: Saku Laesvuori <saku <at> laesvuori.fi>
Subject: [PATCH] gnu: kitsas: Update to 5.4.1
Date: Mon, 12 Feb 2024 09:03:57 +0200

* gnu/packages/finance.scm (kitsas): Update to 5.4.1.

Change-Id: I968579d762ebecef9f9e8b602682587a054c41ad
---
 gnu/packages/finance.scm | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/finance.scm b/gnu/packages/finance.scm
index 98a220385b..96e2729dda 100644
--- a/gnu/packages/finance.scm
+++ b/gnu/packages/finance.scm
@@ -1307,7 +1307,7 @@ (define-public keepkey-agent
 (define-public kitsas
   (package
     (name "kitsas")
-    (version "4.0.3")
+    (version "5.4.1")
     (source (origin
               (method git-fetch)
               (uri (git-reference
@@ -1316,17 +1316,24 @@ (define-public kitsas
               (file-name (git-file-name name version))
               (sha256
                (base32
-                "0hrbsqqm6v2pmjq17s7i4akjgclz3d051mg02vcykq80xgxvbkgf"))))
+                "16zkfpl5d9ia202fqg5vrhjqdw0g6wp044ih6n7nz2hbxj9y3m1z"))))
     (build-system qt-build-system)
-    (inputs (list qtbase-5 libzip poppler-qt5 qtsvg-5))
+    (inputs (list libzip qtsvg qtwebengine qt5compat))
     (arguments
      (list #:tests? #f               ;XXX: some tests fail and others segfault
            #:test-target "check"
+           #:qtbase qtbase
            #:phases
            #~(modify-phases %standard-phases
                (replace 'configure
                  (lambda* _
                    (invoke "qmake" "kitsasproject.pro" "CONFIG+=release")))
+               ;; The tests are not maintained and some don't even build
+               (add-before 'configure 'disable-broken-tests
+                 (lambda _
+                   (substitute* "kitsasproject.pro"
+                     ((" *(unittest|testit).*") "")
+                     (("\\\\") ""))))
                (replace 'install
                  (lambda* _
                    (install-file "kitsas/kitsas"

base-commit: f2102cb5b9239be2ed2023b908b726af542e0124
-- 
2.41.0

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

From: Christopher Baines <mail <at> cbaines.net>
To: Saku Laesvuori <saku <at> laesvuori.fi>
Cc: 69068 <at> debbugs.gnu.org, guix-patches <at> gnu.org
Subject: Re: [bug#69068] [PATCH] gnu: kitsas: Update to 5.4.1
Date: Tue, 13 Feb 2024 11:30:50 +0000

[Message part 1 (text/plain, inline)]

Saku Laesvuori via Guix-patches via <guix-patches <at> gnu.org> writes:

> * gnu/packages/finance.scm (kitsas): Update to 5.4.1.
>
> Change-Id: I968579d762ebecef9f9e8b602682587a054c41ad
> ---
>  gnu/packages/finance.scm | 13 ++++++++++---
>  1 file changed, 10 insertions(+), 3 deletions(-)
>
> diff --git a/gnu/packages/finance.scm b/gnu/packages/finance.scm
> index 98a220385b..96e2729dda 100644
> --- a/gnu/packages/finance.scm
> +++ b/gnu/packages/finance.scm
> @@ -1307,7 +1307,7 @@ (define-public keepkey-agent
>  (define-public kitsas
>    (package
>      (name "kitsas")
> -    (version "4.0.3")
> +    (version "5.4.1")
>      (source (origin
>                (method git-fetch)
>                (uri (git-reference
> @@ -1316,17 +1316,24 @@ (define-public kitsas
>                (file-name (git-file-name name version))
>                (sha256
>                 (base32
> -                "0hrbsqqm6v2pmjq17s7i4akjgclz3d051mg02vcykq80xgxvbkgf"))))
> +                "16zkfpl5d9ia202fqg5vrhjqdw0g6wp044ih6n7nz2hbxj9y3m1z"))))
>      (build-system qt-build-system)
> -    (inputs (list qtbase-5 libzip poppler-qt5 qtsvg-5))
> +    (inputs (list libzip qtsvg qtwebengine qt5compat))
>      (arguments
>       (list #:tests? #f               ;XXX: some tests fail and others segfault
>             #:test-target "check"
> +           #:qtbase qtbase
>             #:phases
>             #~(modify-phases %standard-phases
>                 (replace 'configure
>                   (lambda* _
>                     (invoke "qmake" "kitsasproject.pro" "CONFIG+=release")))
> +               ;; The tests are not maintained and some don't even build
> +               (add-before 'configure 'disable-broken-tests
> +                 (lambda _
> +                   (substitute* "kitsasproject.pro"
> +                     ((" *(unittest|testit).*") "")
> +                     (("\\\\") ""))))
>                 (replace 'install
>                   (lambda* _
>                     (install-file "kitsas/kitsas"
>
> base-commit: f2102cb5b9239be2ed2023b908b726af542e0124

The commit message is missing information on the non-version changes to
the package.

In particular, the removal of qtbase from inputs confuses me, is this
not needed?

[signature.asc (application/pgp-signature, inline)]

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

From: Saku Laesvuori <saku <at> laesvuori.fi>
To: Christopher Baines <mail <at> cbaines.net>
Cc: 69068 <at> debbugs.gnu.org, guix-patches <at> gnu.org
Subject: Re: [bug#69068] [PATCH] gnu: kitsas: Update to 5.4.1
Date: Tue, 13 Feb 2024 14:17:56 +0200

[Message part 1 (text/plain, inline)]

On Tue, Feb 13, 2024 at 11:30:50AM +0000, Christopher Baines wrote:
> 
> Saku Laesvuori via Guix-patches via <guix-patches <at> gnu.org> writes:
> 
> > * gnu/packages/finance.scm (kitsas): Update to 5.4.1.
> >
> > Change-Id: I968579d762ebecef9f9e8b602682587a054c41ad
> > ---
> >  gnu/packages/finance.scm | 13 ++++++++++---
> >  1 file changed, 10 insertions(+), 3 deletions(-)
> >
> > diff --git a/gnu/packages/finance.scm b/gnu/packages/finance.scm
> > index 98a220385b..96e2729dda 100644
> > --- a/gnu/packages/finance.scm
> > +++ b/gnu/packages/finance.scm
> > @@ -1307,7 +1307,7 @@ (define-public keepkey-agent
> >  (define-public kitsas
> >    (package
> >      (name "kitsas")
> > -    (version "4.0.3")
> > +    (version "5.4.1")
> >      (source (origin
> >                (method git-fetch)
> >                (uri (git-reference
> > @@ -1316,17 +1316,24 @@ (define-public kitsas
> >                (file-name (git-file-name name version))
> >                (sha256
> >                 (base32
> > -                "0hrbsqqm6v2pmjq17s7i4akjgclz3d051mg02vcykq80xgxvbkgf"))))
> > +                "16zkfpl5d9ia202fqg5vrhjqdw0g6wp044ih6n7nz2hbxj9y3m1z"))))
> >      (build-system qt-build-system)
> > -    (inputs (list qtbase-5 libzip poppler-qt5 qtsvg-5))
> > +    (inputs (list libzip qtsvg qtwebengine qt5compat))
> >      (arguments
> >       (list #:tests? #f               ;XXX: some tests fail and others segfault
> >             #:test-target "check"
> > +           #:qtbase qtbase
> >             #:phases
> >             #~(modify-phases %standard-phases
> >                 (replace 'configure
> >                   (lambda* _
> >                     (invoke "qmake" "kitsasproject.pro" "CONFIG+=release")))
> > +               ;; The tests are not maintained and some don't even build
> > +               (add-before 'configure 'disable-broken-tests
> > +                 (lambda _
> > +                   (substitute* "kitsasproject.pro"
> > +                     ((" *(unittest|testit).*") "")
> > +                     (("\\\\") ""))))
> >                 (replace 'install
> >                   (lambda* _
> >                     (install-file "kitsas/kitsas"
> >
> > base-commit: f2102cb5b9239be2ed2023b908b726af542e0124
> 
> The commit message is missing information on the non-version changes to
> the package.

All the changes are part of updating the package. The new version is
built against Qt6 and requires different inputs, which also makes
compiling the tests fail as they are not maintained to work with those
new inputs. Earlier just passing #:tests? #f was sufficient to bypass
them because they compiled fine and only failed when ran.

I can try to write a better commit message if you tell which parts
require their own lines, or you can clarify it yourself when applying.

> In particular, the removal of qtbase from inputs confuses me, is this
> not needed?

It is added implicitly by qt-build-system based on the #:qtbase
argument. I don't know if qt-build-system worked like that when I
submitted the first version of this package, but it probably did and I
just didn't realize it.

[signature.asc (application/pgp-signature, inline)]

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

From: Andreas Enge <andreas <at> enge.fr>
To: 69068 <at> debbugs.gnu.org
Subject: To test or not to test?
Date: Wed, 28 Feb 2024 11:32:43 +0100

Hello,

I also looked at this just now and am confused by the test situation.
Should one not either disable tests altogether, or just comment out
some of them?

Andreas

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

From: Saku Laesvuori <saku <at> laesvuori.fi>
To: Andreas Enge <andreas <at> enge.fr>
Cc: 69068 <at> debbugs.gnu.org
Subject: Re: [bug#69068] To test or not to test?
Date: Wed, 28 Feb 2024 13:47:47 +0200

[Message part 1 (text/plain, inline)]

On Wed, Feb 28, 2024 at 11:32:43AM +0100, Andreas Enge wrote:
> Hello,
> 
> I also looked at this just now and am confused by the test situation.
> Should one not either disable tests altogether, or just comment out
> some of them?

I don't believe any of them even compile with Qt6. The test suite has
been unmaintained for years, so it doesn't really tell anything about
whether the program works correctly or not.

I would certainly agree that it is better to remove just the failing
ones if the issues were actual test failures due to how Guix packaging
works. Here the problem is that the project effectively no longer has a
test suite.

[signature.asc (application/pgp-signature, inline)]

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

From: Andreas Enge <andreas <at> enge.fr>
To: Saku Laesvuori <saku <at> laesvuori.fi>
Cc: 69068 <at> debbugs.gnu.org
Subject: Re: [bug#69068] To test or not to test?
Date: Wed, 28 Feb 2024 13:36:54 +0100

Am Wed, Feb 28, 2024 at 01:47:47PM +0200 schrieb Saku Laesvuori:
> I don't believe any of them even compile with Qt6. The test suite has
> been unmaintained for years, so it doesn't really tell anything about
> whether the program works correctly or not.

Sorry for not being more precise:

I am wondering if this part of the patch:
+               ;; The tests are not maintained and some don't even build
+               (add-before 'configure 'disable-broken-tests
+                 (lambda _
+                   (substitute* "kitsasproject.pro"
+                     ((" *(unittest|testit).*") "")
+                     (("\\\\") ""))))

is of interest given that there already is
      (list #:tests? #f               ;XXX: some tests fail and others segfault
            #:test-target "check"

?

Andreas

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

From: Andreas Enge <andreas <at> enge.fr>
To: Saku Laesvuori <saku <at> laesvuori.fi>
Cc: 69068 <at> debbugs.gnu.org
Subject: Re: [bug#69068] To test or not to test?
Date: Wed, 28 Feb 2024 13:49:22 +0100

Sorry, I see that you already answered this question before. I will
prepare to push.

Andreas

Message #34 received at 69068-done <at> debbugs.gnu.org (full text, mbox):

From: Andreas Enge <andreas <at> enge.fr>
To: Saku Laesvuori <saku <at> laesvuori.fi>
Cc: 69068-done <at> debbugs.gnu.org
Subject: Re: [bug#69068] To test or not to test?
Date: Wed, 28 Feb 2024 16:36:04 +0100

Am Wed, Feb 28, 2024 at 01:49:22PM +0100 schrieb Andreas Enge:
> Sorry, I see that you already answered this question before. I will
> prepare to push.

Following the rust team merge, I wanted to rebuild the package once more,
which succeeded. Pushed, thanks for your patience!

Andreas

Previous Next

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