all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#53039] [PATCH] build-system/python: Change let* to let.
@ 2022-01-05 22:46 jgart via Guix-patches via
  2022-01-09 21:03 ` Maxime Devos
  2022-05-18 13:06 ` bug#53039: " Maxim Cournoyer
  0 siblings, 2 replies; 4+ messages in thread
From: jgart via Guix-patches via @ 2022-01-05 22:46 UTC (permalink / raw)
  To: 53039; +Cc: jgart

* guix/build/python-build-system.scm (site-packages): Change let* to let for
readability and since let* is unused.
---
 guix/build/python-build-system.scm | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/guix/build/python-build-system.scm b/guix/build/python-build-system.scm
index 08871f60cd..a9b1c74f63 100644
--- a/guix/build/python-build-system.scm
+++ b/guix/build/python-build-system.scm
@@ -11,6 +11,7 @@
 ;;; Copyright © 2020 Efraim Flashner <efraim@flashner.co.il>
 ;;; Copyright © 2021 Lars-Dominik Braun <lars@6xq.net>
 ;;; Copyright © 2021 Maxime Devos <maximedevos@telenet.be>
+;;; Copyright © 2022 jgart <jgart@dismail.de>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -176,8 +177,8 @@ (define (python-output outputs)
 
 (define (site-packages inputs outputs)
   "Return the path of the current output's Python site-package."
-  (let* ((out (python-output outputs))
-         (python (assoc-ref inputs "python")))
+  (let ((out (python-output outputs))
+        (python (assoc-ref inputs "python")))
     (string-append out "/lib/python" (python-version python) "/site-packages")))
 
 (define (add-installed-pythonpath inputs outputs)
-- 
2.34.1





^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [bug#53039] [PATCH] build-system/python: Change let* to let.
  2022-01-05 22:46 [bug#53039] [PATCH] build-system/python: Change let* to let jgart via Guix-patches via
@ 2022-01-09 21:03 ` Maxime Devos
  2022-05-18 13:06 ` bug#53039: " Maxim Cournoyer
  1 sibling, 0 replies; 4+ messages in thread
From: Maxime Devos @ 2022-01-09 21:03 UTC (permalink / raw)
  To: 53039

[-- Attachment #1: Type: text/plain, Size: 460 bytes --]

retitle [PATCH core-updates-frozen] build-system-python: Change let* to let.
thanks

>[python-build-system.scm]
> -  (let* ((out (python-output outputs))
> -         (python (assoc-ref inputs "python")))
> +  (let ((out (python-output outputs))
> +        (python (assoc-ref inputs "python")))

According to "guix refresh -l python-six", this would rebuild a lot
(at least 3122), so this would need to be done on core-updates.  Seems
ok otherwise.

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 260 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#53039: [PATCH] build-system/python: Change let* to let.
  2022-01-05 22:46 [bug#53039] [PATCH] build-system/python: Change let* to let jgart via Guix-patches via
  2022-01-09 21:03 ` Maxime Devos
@ 2022-05-18 13:06 ` Maxim Cournoyer
  2022-05-19  4:50   ` [bug#53039] " jgart via Guix-patches via
  1 sibling, 1 reply; 4+ messages in thread
From: Maxim Cournoyer @ 2022-05-18 13:06 UTC (permalink / raw)
  To: jgart; +Cc: 53039-done

Hi,

jgart <jgart@dismail.de> writes:

> * guix/build/python-build-system.scm (site-packages): Change let* to let for
> readability and since let* is unused.
> ---
>  guix/build/python-build-system.scm | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/guix/build/python-build-system.scm b/guix/build/python-build-system.scm
> index 08871f60cd..a9b1c74f63 100644
> --- a/guix/build/python-build-system.scm
> +++ b/guix/build/python-build-system.scm
> @@ -11,6 +11,7 @@
>  ;;; Copyright © 2020 Efraim Flashner <efraim@flashner.co.il>
>  ;;; Copyright © 2021 Lars-Dominik Braun <lars@6xq.net>
>  ;;; Copyright © 2021 Maxime Devos <maximedevos@telenet.be>
> +;;; Copyright © 2022 jgart <jgart@dismail.de>
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -176,8 +177,8 @@ (define (python-output outputs)
>  
>  (define (site-packages inputs outputs)
>    "Return the path of the current output's Python site-package."
> -  (let* ((out (python-output outputs))
> -         (python (assoc-ref inputs "python")))
> +  (let ((out (python-output outputs))
> +        (python (assoc-ref inputs "python")))
>      (string-append out "/lib/python" (python-version python) "/site-packages")))
>  
>  (define (add-installed-pythonpath inputs outputs)

As Maxime mentioned, that should go to core-updates.  It's nice to use
the --subject-prefix option of 'git format-patch' to change it to
"[PATCH core-updates]", so that we don't apply these mistakenly to
master and cause world rebuilds :-)

Another note; only changes of minimally 10 lines are copyrightable, so
you do not need to add a copyright attribution for trivial changes such
as this one.

I've made the change to core-updates and committed as 47fac5dc59; closing.

Thank you!

Maxim




^ permalink raw reply	[flat|nested] 4+ messages in thread

* [bug#53039] [PATCH] build-system/python: Change let* to let.
  2022-05-18 13:06 ` bug#53039: " Maxim Cournoyer
@ 2022-05-19  4:50   ` jgart via Guix-patches via
  0 siblings, 0 replies; 4+ messages in thread
From: jgart via Guix-patches via @ 2022-05-19  4:50 UTC (permalink / raw)
  To: Maxim Cournoyer; +Cc: 53039-done

On Wed, 18 May 2022 09:06:58 -0400 Maxim Cournoyer <maxim.cournoyer@gmail.com> wrote:
> I've made the change to core-updates and committed as 47fac5dc59; closing.

Hi Maxim,

Thank you for doing that. Sorry, I couldn't get to it. I've been in the
onboarding process of starting a new job and moving to a new city and
have been really busy.

Hope to get some free time soon to continue contributing to Guix :)

all best,

jgart





^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-05-19  4:54 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-01-05 22:46 [bug#53039] [PATCH] build-system/python: Change let* to let jgart via Guix-patches via
2022-01-09 21:03 ` Maxime Devos
2022-05-18 13:06 ` bug#53039: " Maxim Cournoyer
2022-05-19  4:50   ` [bug#53039] " jgart via Guix-patches via

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.