GNU bug report logs - #69682
Opam should be using a minimal version of extlib

Previous Next

To reply to this bug, email your comments to 69682 AT debbugs.gnu.org.

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

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

From: Vivien Kraus <vivien <at> planete-kraus.eu>
To: bug-guix <bug-guix <at> gnu.org>
Subject: Opam should be using a minimal version of extlib
Date: Sat, 09 Mar 2024 20:45:17 +0100

Dear OCaml-aware Guix,

opam seems to imply that we have to use a minimal variant of ocaml-
extlib to build opam:

https://github.com/ocaml/opam/issues/5694

This is likely related to a build failure we are experiencing on the
gnome-team branch:

https://issues.guix.gnu.org/69677

http://ci.guix.gnu.org/build/3415343/log/raw

Best regards,

Vivien

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

From: Vivien Kraus <vivien <at> planete-kraus.eu>
To: 69682 <at> debbugs.gnu.org
Cc: pukkamustard <at> posteo.net, julien <at> lepiller.eu
Subject: [PATCH 0/1] ocaml-extlib: Build with minimal=1.
Date: Sat, 9 Mar 2024 22:19:01 +0100

Dear Guix, brave ocaml-team members,

According to https://github.com/ocaml/opam/issues/5694

it looks like ocaml-extlib should build with minimal=1.

This also fixes a build failure on gnome-team.

What do you think?

Best regards,

Vivien

Vivien Kraus (1):
  gnu: ocaml-extlib: Build with minimal=1.

 gnu/packages/ocaml.scm | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)


base-commit: 2f441fc738976175d438f7942211b1894e2eb416
-- 
2.41.0

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

From: Vivien Kraus <vivien <at> planete-kraus.eu>
To: 69682 <at> debbugs.gnu.org
Cc: pukkamustard <at> posteo.net, julien <at> lepiller.eu
Subject: [PATCH 1/1] gnu: ocaml-extlib: Build with minimal=1.
Date: Sat, 9 Mar 2024 22:03:15 +0100

* gnu/packages/ocaml.scm (ocaml-extlib) [arguments]: Convert to list of
G-Expressions.  Add #:make-flags.

Change-Id: I42ee3c21a52788f20ddc3381927ef6ef40b2a354
---
 gnu/packages/ocaml.scm | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/ocaml.scm b/gnu/packages/ocaml.scm
index 0f4c351141..b911da8e5b 100644
--- a/gnu/packages/ocaml.scm
+++ b/gnu/packages/ocaml.scm
@@ -646,9 +646,11 @@ (define-public ocaml-extlib
                 "1jydzw2n84cfiz9y6lk4gih4wbr8jybanmiryfs01svd07g4vpjq"))))
     (build-system ocaml-build-system)
     (arguments
-     `(#:phases
-       (modify-phases %standard-phases
-         (delete 'configure))))
+     (list
+      #:make-flags #~'("minimal=1")
+      #:phases
+      #~(modify-phases %standard-phases
+          (delete 'configure))))
     (native-inputs
       (list ocaml-cppo))
     (home-page "https://github.com/ygrek/ocaml-extlib")
-- 
2.41.0

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

From: Julien Lepiller <julien <at> lepiller.eu>
To: Vivien Kraus <vivien <at> planete-kraus.eu>, 69682 <at> debbugs.gnu.org
Cc: pukkamustard <at> posteo.net
Subject: Re: [PATCH 1/1] gnu: ocaml-extlib: Build with minimal=1.
Date: Sat, 09 Mar 2024 22:37:24 +0100

Ah, you sent this while I was writing to the other bug. Would it be possible to convert the recipe to use the dune-build-system instead? It sounds like it would be more future-proof, and it's also prefered by opam people.

Le 9 mars 2024 22:03:15 GMT+01:00, Vivien Kraus <vivien <at> planete-kraus.eu> a écrit :
>* gnu/packages/ocaml.scm (ocaml-extlib) [arguments]: Convert to list of
>G-Expressions.  Add #:make-flags.
>
>Change-Id: I42ee3c21a52788f20ddc3381927ef6ef40b2a354
>---
> gnu/packages/ocaml.scm | 8 +++++---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
>diff --git a/gnu/packages/ocaml.scm b/gnu/packages/ocaml.scm
>index 0f4c351141..b911da8e5b 100644
>--- a/gnu/packages/ocaml.scm
>+++ b/gnu/packages/ocaml.scm
>@@ -646,9 +646,11 @@ (define-public ocaml-extlib
>                 "1jydzw2n84cfiz9y6lk4gih4wbr8jybanmiryfs01svd07g4vpjq"))))
>     (build-system ocaml-build-system)
>     (arguments
>-     `(#:phases
>-       (modify-phases %standard-phases
>-         (delete 'configure))))
>+     (list
>+      #:make-flags #~'("minimal=1")
>+      #:phases
>+      #~(modify-phases %standard-phases
>+          (delete 'configure))))
>     (native-inputs
>       (list ocaml-cppo))
>     (home-page "https://github.com/ygrek/ocaml-extlib")

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

From: Vivien Kraus <vivien <at> planete-kraus.eu>
To: 69682 <at> debbugs.gnu.org
Cc: pukkamustard <at> posteo.net, julien <at> lepiller.eu
Subject: [PATCH v2] gnu: ocaml-extlib: Convert to dune-build-system.
Date: Sat, 9 Mar 2024 22:03:15 +0100

* gnu/packages/ocaml.scm (ocaml-extlib) [build-system]: Convert to
dune-build-system.
[arguments]: Remove '#:phases' and add '#:package'.

Change-Id: Ia50c05423f3062200704fbcbb0680f2b326a7ca4
---
Opam still builds on gnome-team.

 gnu/packages/ocaml.scm | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/gnu/packages/ocaml.scm b/gnu/packages/ocaml.scm
index 0f4c351141..b70e925d51 100644
--- a/gnu/packages/ocaml.scm
+++ b/gnu/packages/ocaml.scm
@@ -644,11 +644,9 @@ (define-public ocaml-extlib
               (sha256
                (base32
                 "1jydzw2n84cfiz9y6lk4gih4wbr8jybanmiryfs01svd07g4vpjq"))))
-    (build-system ocaml-build-system)
+    (build-system dune-build-system)
     (arguments
-     `(#:phases
-       (modify-phases %standard-phases
-         (delete 'configure))))
+     (list #:package "extlib"))
     (native-inputs
       (list ocaml-cppo))
     (home-page "https://github.com/ygrek/ocaml-extlib")

base-commit: 2f441fc738976175d438f7942211b1894e2eb416
-- 
2.41.0

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

From: pukkamustard <pukkamustard <at> posteo.net>
To: Vivien Kraus <vivien <at> planete-kraus.eu>
Cc: 69682 <at> debbugs.gnu.org, julien <at> lepiller.eu
Subject: Re: [PATCH v2] gnu: ocaml-extlib: Convert to dune-build-system.
Date: Sun, 24 Mar 2024 07:12:19 +0000

Sorry for being late to the party. But the conversion to use
dune-build-system looks good to me!

I ran into the same issue when updating opam to 2.1.5 but never managed
to get to the bottom of it. Thanks for figuring this out!

-pukkamustard

Vivien Kraus <vivien <at> planete-kraus.eu> writes:

> * gnu/packages/ocaml.scm (ocaml-extlib) [build-system]: Convert to
> dune-build-system.
> [arguments]: Remove '#:phases' and add '#:package'.
>
> Change-Id: Ia50c05423f3062200704fbcbb0680f2b326a7ca4
> ---
> Opam still builds on gnome-team.
>
>  gnu/packages/ocaml.scm | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/gnu/packages/ocaml.scm b/gnu/packages/ocaml.scm
> index 0f4c351141..b70e925d51 100644
> --- a/gnu/packages/ocaml.scm
> +++ b/gnu/packages/ocaml.scm
> @@ -644,11 +644,9 @@ (define-public ocaml-extlib
>                (sha256
>                 (base32
>                  "1jydzw2n84cfiz9y6lk4gih4wbr8jybanmiryfs01svd07g4vpjq"))))
> -    (build-system ocaml-build-system)
> +    (build-system dune-build-system)
>      (arguments
> -     `(#:phases
> -       (modify-phases %standard-phases
> -         (delete 'configure))))
> +     (list #:package "extlib"))
>      (native-inputs
>        (list ocaml-cppo))
>      (home-page "https://github.com/ygrek/ocaml-extlib")
>
> base-commit: 2f441fc738976175d438f7942211b1894e2eb416

Previous Next

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