unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#46021] [PATCH]: gnu: ecl: Propagate some dependencies used in header files
@ 2021-01-21 15:12 Zhu Zihao
       [not found] ` <handler.46021.B.16112429855183.ack@debbugs.gnu.org>
  0 siblings, 1 reply; 6+ messages in thread
From: Zhu Zihao @ 2021-01-21 15:12 UTC (permalink / raw)
  To: 46021

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


ECL's header includes the header of gmp, libgc, libatomic-ops and
libffi. This help fix some program use ecl as build input failed to link
with it.



-- 
Retrieve my PGP public key:

  gpg --recv-keys D47A9C8B2AE3905B563D9135BE42B352A9F6821F

Zihao

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 255 bytes --]

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

* [bug#46021] Acknowledgement ([PATCH]: gnu: ecl: Propagate some dependencies used in header files)
       [not found] ` <handler.46021.B.16112429855183.ack@debbugs.gnu.org>
@ 2021-01-21 15:40   ` Zhu Zihao
  2021-01-29  1:56     ` Zhu Zihao
  0 siblings, 1 reply; 6+ messages in thread
From: Zhu Zihao @ 2021-01-21 15:40 UTC (permalink / raw)
  To: 46021


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



[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 255 bytes --]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-ecl-Propagate-some-dependencies-used-in-header-f.patch --]
[-- Type: text/x-patch, Size: 843 bytes --]

From 191564b413e7e95e0e246935026392cd152464fe Mon Sep 17 00:00:00 2001
From: Zhu Zihao <all_but_last@163.com>
Date: Thu, 21 Jan 2021 23:06:52 +0800
Subject: [PATCH] gnu: ecl: Propagate some dependencies used in header files.

* gnu/packages/lisp.scm(ecl)[inputs]: Remove.
[propagated-inputs]: Add gmp, libatomic-ops, libgc, libffi.
---
 gnu/packages/lisp.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/lisp.scm b/gnu/packages/lisp.scm
index 3c09b0af8a..fb4f7cb7bd 100644
--- a/gnu/packages/lisp.scm
+++ b/gnu/packages/lisp.scm
@@ -236,7 +236,7 @@ interface to the Tk widget system.")
      `(("cl-asdf" ,cl-asdf)
        ("which" ,which)
        ("texinfo" ,texinfo)))
-    (inputs
+    (propagated-inputs
      `(("gmp" ,gmp)
        ("libatomic-ops" ,libatomic-ops)
        ("libgc" ,libgc)
-- 
2.30.0


[-- Attachment #3: Type: text/plain, Size: 102 bytes --]




-- 
Retrieve my PGP public key:

  gpg --recv-keys D47A9C8B2AE3905B563D9135BE42B352A9F6821F

Zihao

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #4: 0001-gnu-ecl-Propagate-some-dependencies-used-in-header-f.patch --]
[-- Type: text/x-patch, Size: 843 bytes --]

From 191564b413e7e95e0e246935026392cd152464fe Mon Sep 17 00:00:00 2001
From: Zhu Zihao <all_but_last@163.com>
Date: Thu, 21 Jan 2021 23:06:52 +0800
Subject: [PATCH] gnu: ecl: Propagate some dependencies used in header files.

* gnu/packages/lisp.scm(ecl)[inputs]: Remove.
[propagated-inputs]: Add gmp, libatomic-ops, libgc, libffi.
---
 gnu/packages/lisp.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/lisp.scm b/gnu/packages/lisp.scm
index 3c09b0af8a..fb4f7cb7bd 100644
--- a/gnu/packages/lisp.scm
+++ b/gnu/packages/lisp.scm
@@ -236,7 +236,7 @@ interface to the Tk widget system.")
      `(("cl-asdf" ,cl-asdf)
        ("which" ,which)
        ("texinfo" ,texinfo)))
-    (inputs
+    (propagated-inputs
      `(("gmp" ,gmp)
        ("libatomic-ops" ,libatomic-ops)
        ("libgc" ,libgc)
-- 
2.30.0


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

* [bug#46021] Acknowledgement ([PATCH]: gnu: ecl: Propagate some dependencies used in header files)
  2021-01-21 15:40   ` [bug#46021] Acknowledgement ([PATCH]: gnu: ecl: Propagate some dependencies used in header files) Zhu Zihao
@ 2021-01-29  1:56     ` Zhu Zihao
  2021-01-29  8:49       ` Guillaume Le Vaillant
  0 siblings, 1 reply; 6+ messages in thread
From: Zhu Zihao @ 2021-01-29  1:56 UTC (permalink / raw)
  To: 46021

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

ping.
-- 
Retrieve my PGP public key:

  gpg --recv-keys D47A9C8B2AE3905B563D9135BE42B352A9F6821F

Zihao

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 255 bytes --]

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

* [bug#46021] Acknowledgement ([PATCH]: gnu: ecl: Propagate some dependencies used in header files)
  2021-01-29  1:56     ` Zhu Zihao
@ 2021-01-29  8:49       ` Guillaume Le Vaillant
  2021-01-29  9:34         ` Zhu Zihao
  0 siblings, 1 reply; 6+ messages in thread
From: Guillaume Le Vaillant @ 2021-01-29  8:49 UTC (permalink / raw)
  To: Zhu Zihao; +Cc: 46021

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


Zhu Zihao <all_but_last@163.com> skribis:

> ECL's header includes the header of gmp, libgc, libatomic-ops and
> libffi. This help fix some program use ecl as build input failed to link
> with it.

I tried a `guix environment -C --ad-hoc ecl -- ecl --eval '(compile-file "test.lisp")'`
and ECL compiled the file successfully, so it must have found the
headers it needed.

Could you give an example of build failure solved by your patch?

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 247 bytes --]

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

* [bug#46021] Acknowledgement ([PATCH]: gnu: ecl: Propagate some dependencies used in header files)
  2021-01-29  8:49       ` Guillaume Le Vaillant
@ 2021-01-29  9:34         ` Zhu Zihao
  2021-02-02  9:39           ` bug#46021: " Guillaume Le Vaillant
  0 siblings, 1 reply; 6+ messages in thread
From: Zhu Zihao @ 2021-01-29  9:34 UTC (permalink / raw)
  To: Guillaume Le Vaillant; +Cc: 46021

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


Guillaume Le Vaillant writes:

> Zhu Zihao <all_but_last@163.com> skribis:
>
>> ECL's header includes the header of gmp, libgc, libatomic-ops and
>> libffi. This help fix some program use ecl as build input failed to link
>> with it.
>
> I tried a `guix environment -C --ad-hoc ecl -- ecl --eval '(compile-file "test.lisp")'`
> and ECL compiled the file successfully, so it must have found the
> headers it needed.

Because we already wrapped ECL executable with CPATH and LIBRARY_PATH
environment variable. so any compilation done in ECL can find the
library and header.

If some 3-rd party program want to link with ECL(If they use ECL as
thier extension language). Compiler will failed to find gmp, libgc etc.

A simple example here, start with `guix environment --ad-hoc ecl`

```
chino@asus-laptop:~/tmp$ cat test.c
#include <ecl/ecl.h>

int main() {
  cl_boot(0, NULL);
  return 0;
}
chino@asus-laptop:~/tmp$ CFLAGS="$(ecl-config --cflags) $(ecl-config --ldflags) $(ecl-config --libs)"
chino@asus-laptop:~/tmp$ LANG=en_US.utf-8 gcc -c test.c -o test $CFLAGS
In file included from /gnu/store/lzfxjn036h3kis13lcc222rpwcnqazkr-ecl-20.4.24/include/ecl/ecl.h:37,
                 from test.c:1:
/gnu/store/lzfxjn036h3kis13lcc222rpwcnqazkr-ecl-20.4.24/include/ecl/config.h:59:10: fatal error: gmp.h: No such file or directory
   59 | #include "gmp.h"
      |          ^~~~~~~
compilation terminated.
```

-- 
Retrieve my PGP public key:

  gpg --recv-keys D47A9C8B2AE3905B563D9135BE42B352A9F6821F

Zihao

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 255 bytes --]

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

* bug#46021: Acknowledgement ([PATCH]: gnu: ecl: Propagate some dependencies used in header files)
  2021-01-29  9:34         ` Zhu Zihao
@ 2021-02-02  9:39           ` Guillaume Le Vaillant
  0 siblings, 0 replies; 6+ messages in thread
From: Guillaume Le Vaillant @ 2021-02-02  9:39 UTC (permalink / raw)
  To: Zhu Zihao; +Cc: 46021-done

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

Patch pushed as 90ad8bd1a63abbf94f2ebb9f10630773d1f15ece.
Thanks.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 247 bytes --]

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

end of thread, other threads:[~2021-02-02  9:40 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-01-21 15:12 [bug#46021] [PATCH]: gnu: ecl: Propagate some dependencies used in header files Zhu Zihao
     [not found] ` <handler.46021.B.16112429855183.ack@debbugs.gnu.org>
2021-01-21 15:40   ` [bug#46021] Acknowledgement ([PATCH]: gnu: ecl: Propagate some dependencies used in header files) Zhu Zihao
2021-01-29  1:56     ` Zhu Zihao
2021-01-29  8:49       ` Guillaume Le Vaillant
2021-01-29  9:34         ` Zhu Zihao
2021-02-02  9:39           ` bug#46021: " Guillaume Le Vaillant

Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).