[PATCH] gnu: ocaml-ctypes: do not override install headers

  • Done
  • quality assurance status badge
Details
2 participants
  • David Arroyo
  • Julien Lepiller
Owner
unassigned
Submitted by
David Arroyo
Severity
normal
D
D
David Arroyo wrote on 29 May 2022 22:42
(address . guix-patches@gnu.org)(name . David Arroyo)(address . david@aqwari.net)
20220529204234.657407-1-david@aqwari.net
Overriding INSTALL_HEADERS in the build prevents the "ocaml_integers.h"
header from being copied from the ocaml-integers package to ocaml-ctypes.
The header's absence breaks all dune-based projects which build cstubs
using the new (ctypes ...) stanza.

* gnu/packages/ocaml.scm:
[arguments]: remove INSTALL_HEADERS override.
---
gnu/packages/ocaml.scm | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)

Toggle diff (17 lines)
diff --git a/gnu/packages/ocaml.scm b/gnu/packages/ocaml.scm
index e6d6bc7a89..c7da5a2106 100644
--- a/gnu/packages/ocaml.scm
+++ b/gnu/packages/ocaml.scm
@@ -4333,9 +4333,7 @@ (define-public ocaml-ctypes
"0ilzq9qzvwv9rc08cc9wchsx636zp870i7qvqmbigaa2qb812m0z"))))
(build-system ocaml-build-system)
(arguments
- `(#:make-flags
- (list (string-append "INSTALL_HEADERS = $(wildcard $($(PROJECT).dir)/*.h)"))
- #:phases
+ `(#:phases
(modify-phases %standard-phases
(add-after 'unpack 'make-writable
(lambda _
--
2.36.1
J
J
Julien Lepiller wrote on 2 Jun 2022 07:54
(name . David Arroyo)(address . david@aqwari.net)(address . 55715-done@debbugs.gnu.org)
20220602075440.09a9127c@sybil.lepiller.eu
Le Sun, 29 May 2022 16:42:34 -0400,
David Arroyo <david@aqwari.net> a écrit :

Toggle quote (3 lines)
> * gnu/packages/ocaml.scm:
> [arguments]: remove INSTALL_HEADERS override.

Pushed to master as 21800bf8b98871c99c0a91d97690be1df07a2659 with a
small change to the commit message. Thanks for the patch and enjoy!
Closed
?