[bug#53584] guix: Use 'gexp->derivation' instead of 'build-expression->derivation'.

[bug#53584] guix: Use 'gexp->derivation' instead of 'build-expression->derivation'.

[phodina via Guix-patches via]

[-- Attachment #1.1: Type: text/plain, Size: 118 bytes --]

Hi,

I've noticed in the output of bootstraping the guix package some warnings about deprecated expression.

----
Petr

[-- Attachment #1.2: Type: text/html, Size: 342 bytes --]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-guix-Use-gexp-derivation-instead-of-build-expression.patch --]
[-- Type: text/x-patch; name=0001-guix-Use-gexp-derivation-instead-of-build-expression.patch, Size: 1531 bytes --]

From e128b9d35060187b04ebd36cabd152f0e968dc87 Mon Sep 17 00:00:00 2001
From: Petr Hodina <phodina@protonmail.com>
Date: Thu, 27 Jan 2022 15:32:35 +0100
Subject: [PATCH] guix: Use 'gexp->derivation' instead of
 'build-expression->derivation'.

* guix/derivations.scm: Update deprecated expression.
* guix/grafts.scm: Same as above.

diff --git a/guix/derivations.scm b/guix/derivations.scm
index f77ea179f4..615bb161d1 100644
--- a/guix/derivations.scm
+++ b/guix/derivations.scm
@@ -1218,7 +1218,7 @@ (define builder
                                     #:opts %auto-compilation-options)))))
                 files)))
 
-    (build-expression->derivation store name builder
+    (gexp->derivation store name builder
                                   #:inputs `(("modules" ,module-dir))
                                   #:system system
                                   #:guile-for-build guile
diff --git a/guix/grafts.scm b/guix/grafts.scm
index 0ffda8f9aa..e0a97e895b 100644
--- a/guix/grafts.scm
+++ b/guix/grafts.scm
@@ -134,7 +134,7 @@ (define properties
     ((($ <graft> sources source-outputs targets target-outputs) ...)
      (let ((sources (zip sources source-outputs))
            (targets (zip targets target-outputs)))
-       (build-expression->derivation store name build
+       (gexp->derivation store name build
                                      #:system system
                                      #:guile-for-build guile
                                      #:modules '((guix build graft)
-- 
2.34.0

[bug#53584] guix: Use 'gexp->derivation' instead of 'build-expression->derivation'.

[Ludovic Courtès]

Hi,

phodina <phodina@protonmail.com> skribis:

> From e128b9d35060187b04ebd36cabd152f0e968dc87 Mon Sep 17 00:00:00 2001
> From: Petr Hodina <phodina@protonmail.com>
> Date: Thu, 27 Jan 2022 15:32:35 +0100
> Subject: [PATCH] guix: Use 'gexp->derivation' instead of
>  'build-expression->derivation'.
>
> * guix/derivations.scm: Update deprecated expression.
> * guix/grafts.scm: Same as above.
>
> diff --git a/guix/derivations.scm b/guix/derivations.scm
> index f77ea179f4..615bb161d1 100644
> --- a/guix/derivations.scm
> +++ b/guix/derivations.scm
> @@ -1218,7 +1218,7 @@ (define builder
>                                      #:opts %auto-compilation-options)))))
>                  files)))
>  
> -    (build-expression->derivation store name builder
> +    (gexp->derivation store name builder
>                                    #:inputs `(("modules" ,module-dir))
>                                    #:system system
>                                    #:guile-for-build guile
> diff --git a/guix/grafts.scm b/guix/grafts.scm
> index 0ffda8f9aa..e0a97e895b 100644
> --- a/guix/grafts.scm
> +++ b/guix/grafts.scm
> @@ -134,7 +134,7 @@ (define properties
>      ((($ <graft> sources source-outputs targets target-outputs) ...)
>       (let ((sources (zip sources source-outputs))
>             (targets (zip targets target-outputs)))
> -       (build-expression->derivation store name build
> +       (gexp->derivation store name build
>                                       #:system system
>                                       #:guile-for-build guile
>                                       #:modules '((guix build graft)

It’s unfortunately not that simple: the above code doesn’t work, notably
because ‘gexp->derivation’ is a monadic procedure, unlike
‘build-expression->derivation’.

Also, the first hunk is unnecessary since it’s in an internal procedure
that’s used by ‘build-expression->derivation’ itself.

I’m closing it for now but this is something we should address
eventually!

Thanks,
Ludo’.