Subject: bug#54572: closed (Re: bug#54572: [PATCH] gnu: solfege: Remove trailing booleans.)

From: [email-masked] (GNU bug Tracking System)
Date: Tue, 29 Mar 2022 08:58:01 +0000

Your bug report

#54572: [PATCH] gnu: solfege: Remove trailing booleans.

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].

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

Subject: Re: bug#54572: [PATCH] gnu: solfege: Remove trailing booleans.
From: Mathieu Othacehe <[email-masked]>
To: Liliana Marie Prikler <[email-masked]>
Cc: [email-masked], jgart <[email-masked]>
Date: Tue, 29 Mar 2022 10:57:29 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

Hey,

> Questionable uses of which.
>
> It'd also be nice if you could update arguments to be a list of G-
> expressions and do all the other modernizations.

That can come in a follow-up patch, pushed that one in the meantime.

Thanks,

Mathieu

Subject: [PATCH] gnu: solfege: Remove trailing booleans.
From: jgart <[email-masked]>
To: [email-masked]
Cc: jgart <[email-masked]>
Date: Fri, 25 Mar 2022 21:15:38 -0400
List-Id: <debbugs-submit.debbugs.gnu.org>

* gnu/packages/music.scm (solfege): Remove trailing booleans.
---
 gnu/packages/music.scm | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm
index 2c6b530aac..d7f0c0d39d 100644
--- a/gnu/packages/music.scm
+++ b/gnu/packages/music.scm
@@ -2058,16 +2058,14 @@ (define-public solfege
                (("/usr/bin/aplay") "aplay")
                (("/usr/bin/timidity") "timidity")
                (("/usr/bin/mpg123") "mpg123")
-               (("/usr/bin/ogg123") "ogg123"))
-             #t))
+               (("/usr/bin/ogg123") "ogg123"))))
          (add-before 'build 'patch-python-shebangs
            (lambda _
              ;; Two python scripts begin with a Unicode BOM, so patch-shebang
              ;; has no effect.
              (substitute* '("solfege/parsetree.py"
                             "solfege/presetup.py")
-               (("#!/usr/bin/python") (string-append "#!" (which "python"))))
-             #t))
+               (("#!/usr/bin/python") (string-append "#!" (which "python"))))))
          (add-before 'build 'add-sitedirs
            ;; .pth files are not automatically interpreted unless the
            ;; directories containing them are added as "sites".  The directories
-- 
2.34.0