[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#53039: [PATCH] build-system/python: Change let* to let.
From: |
Maxim Cournoyer |
Subject: |
bug#53039: [PATCH] build-system/python: Change let* to let. |
Date: |
Wed, 18 May 2022 09:06:58 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.1 (gnu/linux) |
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
- bug#53039: [PATCH] build-system/python: Change let* to let.,
Maxim Cournoyer <=