GNU bug report logs - #43976
[PATCH] Chicken build system + some example eggs

Previous Next

Package: guix-patches;

Reported by: raingloom <raingloom <at> riseup.net>

Date: Tue, 13 Oct 2020 08:53:02 UTC

Severity: normal

Tags: patch

Done: Ludovic Courtès <ludo <at> gnu.org>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 43976 in the body.
You can then email your comments to 43976 AT debbugs.gnu.org in the normal way.

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

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#43976; Package guix-patches. (Tue, 13 Oct 2020 08:53:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to raingloom <raingloom <at> riseup.net>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Tue, 13 Oct 2020 08:53:02 GMT) Full text and rfc822 format available.

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

From: raingloom <raingloom <at> riseup.net>
To: Guix Patches <guix-patches <at> gnu.org>
Subject: [PATCH] Chicken build system + some example eggs
Date: Tue, 13 Oct 2020 10:52:20 +0200
[Message part 1 (text/plain, inline)]
Here it is, chicken-build-system.

# What works
Building eggs, dependencies, importing them from search path.

# What's broken
Cross-compilation has not been attempted beacuse the Go build system I
based this on does not support it either.

# Necessary improvements
The Go build system removes some references. I was not sure if this is
needed for Chicken, so for now I left it out.

# Eggs
Some were selected because I'll need for the 9p egg, the rest because
the agrep egg was the first one I found that had dependencies, so it
was selected as a perfect test case.
I don't know if all eggs will work. Eggs that bind to native libraries
(like SDL) still need to be tested.

## SRFI-14 licensing
SRFI-14 has a weird license. The metadata says it's "BSD" but it's
clearly not BSD. I'm not sure what it is or if it's compatible with
Guix. If not, we could probably contact the authors and ask them to
relicense it.

# Chicken 4
Not attempted. Nix supports it but it's old and looks mostly abandoned.
I think we can skip it.
[0001-gnu-Added-search-paths-for-Chicken-Scheme.patch (text/x-patch, attachment)]
[0002-guix-Added-chicken-build-system.patch (text/x-patch, attachment)]
[0003-gnu-Use-qualified-license-names-in-chicken.scm-inste.patch (text/x-patch, attachment)]
[0004-gnu-Added-imports-for-chicken-eggs.patch (text/x-patch, attachment)]
[0005-gnu-Added-chicken-srfi-1.patch (text/x-patch, attachment)]
[0006-gnu-Added-chicken-srfi-69.patch (text/x-patch, attachment)]
[0007-gnu-Added-chicken-iset.patch (text/x-patch, attachment)]
[0008-gnu-Added-chicken-datatype.patch (text/x-patch, attachment)]
[0009-gnu-Added-chicken-srfi-14.patch (text/x-patch, attachment)]
[0010-gnu-Added-chicken-agrep.patch (text/x-patch, attachment)]
[0011-gnu-Added-myself-to-chicken.scm-copyright.patch (text/x-patch, attachment)]

Information forwarded to guix-patches <at> gnu.org:
bug#43976; Package guix-patches. (Sun, 18 Oct 2020 16:11:01 GMT) Full text and rfc822 format available.

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: raingloom <raingloom <at> riseup.net>
Cc: 43976 <at> debbugs.gnu.org
Subject: Re: [bug#43976] [PATCH] Chicken build system + some example eggs
Date: Sun, 18 Oct 2020 18:10:19 +0200
Hi!

raingloom <raingloom <at> riseup.net> skribis:

> Here it is, chicken-build-system.

Woohoo, really nice!  Great to welcome another Scheme in our home.  :-)

Overall the series LGTM.  Inline below are a few suggestions for minor
issues.

> # What's broken
> Cross-compilation has not been attempted beacuse the Go build system I
> based this on does not support it either.

That’s fine, it can come later if/when someone feels like it.

> # Necessary improvements
> The Go build system removes some references. I was not sure if this is
> needed for Chicken, so for now I left it out.

You can check the output of ‘guix size chicken-srfi-14’ (say).  If it
contains things that shouldn’t be there, like GCC or whatever, then
we should do something about it.

[...]

>>From 502235505c75446758cc1932bd1ba333644423ca Mon Sep 17 00:00:00 2001
> From: raingloom <raingloom <at> riseup.net>
> Date: Mon, 12 Oct 2020 04:11:59 +0200
> Subject: [PATCH 01/11] gnu: Added search paths for Chicken Scheme.
>
> * gnu/packages/chicken.scm (chicken): Added search paths
>   [native-search-paths]: added CHICKEN_REPOSITORY_PATH and CHICKEN_INCLUDE_PATH

[...]

> +    (native-search-paths
> +     (list (search-path-specification
> +            (variable "CHICKEN_REPOSITORY_PATH")
> +            ;; TODO extract binary version into a module level definition.
> +            (files (list "var/lib/chicken/11")))
> +           (search-path-specification
> +            (variable "CHICKEN_INCLUDE_PATH")
> +            (files '("share")))))

Is it just share/, not share/chicken/ or something?  A Chicken-specific
directory name would be better, but if that’s really what Chicken
expects, then so be it.

Otherwise LGTM!

> From a734e591ad066c20a53f9d0f955716ba8422bac5 Mon Sep 17 00:00:00 2001
> From: raingloom <raingloom <at> riseup.net>
> Date: Tue, 13 Oct 2020 09:26:52 +0200
> Subject: [PATCH 02/11] guix: Added chicken-build-system.
>
> * guix/build-system/chicken.scm: New file.
> * guix/build/chicken-build-system.scm: New file.
> * Makefile.am: Add them.

Please mention it in doc/guix.texi under “Build Systems” with a
paragraph explaining the basics, as is done for the other build systems.

> --- /dev/null
> +++ b/guix/build/chicken-build-system.scm
> @@ -0,0 +1,103 @@
> +(define-module (guix build chicken-build-system)

Please add the GPLv3+ copyright header.

> +;; TODO how do we run tests???
> +
> +;; TODO remove references

You can remove the second TODO unless/until we have reasons to believe
this has to be done.

The first TODO is more problematic though.  Is there a standard way to
run tests?  It would be great if you could skim the packages you added
to see how they handle tests, so that ‘chicken-build-system’ can have a
‘check’ phase that follows common practice.

Otherwise LGTM!

>>From a7e3b91b76625e01eed749c2c83a94862e3ef738 Mon Sep 17 00:00:00 2001
> From: raingloom <raingloom <at> riseup.net>
> Date: Tue, 13 Oct 2020 09:55:42 +0200
> Subject: [PATCH 04/11] gnu: Added imports for chicken eggs.
>
> * gnu/packages/chicken.scm: New module imports.

Usually we’d import modules in the patch where we first make use of
them.  Otherwise one might think this patch has no effect.

> +    (home-page "http://wiki.call-cc.org/eggref/5/srfi-69")
> +    (synopsis "An implementation of SRFI 69 with SRFI 90 extensions")

I think ‘guix lint’ won’t like that…

> +    (description
> +     "Hash table implementation and binary search")

… and a full sentence here would be welcome.  :-)

  https://guix.gnu.org/manual/devel/en/html_node/Synopses-and-Descriptions.html

Same for the other packages.

> +             (url (string-append "https://code.call-cc.org/svn/chicken-eggs/"
> +                                 "release/5/srfi-14/tags/" version))
> +             (revision 39057)
> +             (user-name "anonymous")
> +             (password "")))
> +       (sha256
> +        (base32
> +         "0wjsqfwawh9bx6vvii1gwag166bxkflc0ib374fbws14914g2ac1"))))
> +    (build-system chicken-build-system)
> +    (arguments '(#:egg-name "srfi-14"))
> +    (home-page "http://wiki.call-cc.org/eggref/5/srfi-14")
> +    (synopsis "Character set library")
> +    (description
> +     "Character sets can be created, extended, tested for the membership of
> +a characters and be compared to other character sets")
> +    (license (license:non-copyleft
> +              "file://srfi-14.scm"
> +              "See end of srfi-14.scm in the distribution."))))

You can use <http://wiki.call-cc.org/eggref/5/srfi-14#license> instead
of <file://...>.

The license looks weird indeed, but it looks like a valid free software
license.  The only discussion I found is at:
<https://srfi.schemers.org/srfi-14/mail-archive/msg00029.html>.

> From 52a27d11eb3d4d0ced3deda01fe901bf2f1937fd Mon Sep 17 00:00:00 2001
> From: raingloom <raingloom <at> riseup.net>
> Date: Mon, 12 Oct 2020 04:19:46 +0200
> Subject: [PATCH 11/11] gnu: Added myself to chicken.scm copyright.
>
> ---
>  gnu/packages/chicken.scm | 1 +
>  1 file changed, 1 insertion(+)

Please do that along with your first changes to the file.

That’s it.

Could you send a v2?

Thank you for working on it!

Ludo’.




Information forwarded to guix-patches <at> gnu.org:
bug#43976; Package guix-patches. (Fri, 20 Nov 2020 04:52:02 GMT) Full text and rfc822 format available.

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

From: raingloom <raingloom <at> riseup.net>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 43976 <at> debbugs.gnu.org
Subject: Re: [bug#43976] [PATCH] Chicken build system + some example eggs
Date: Fri, 20 Nov 2020 05:51:17 +0100
Well, CHICKEN_IMPORT_PATH seems to just be some kind of historical
baggage, at least as far as I can tell. It is not mentioned anywhere on
https://wiki.call-cc.org/man/5 and from what I deciphered from the
sources, the only time it's used it gets prepended to
CHICKEN_REPOSITORY_PATH.

I removed the search path and things still seem to work. It was
probably used in earlier version of Chicken and Nix just never removed
it. Didn't find any related issues either.
I think I'll comment it out for now but leave it in? If there are no
issues with it a few months after merging, it can probably be deleted
entirely...

Possibly more important: I noticed some warnings in some builds about
cp(1). So far it hasn't caused any issues. Gonna fix this, then I think
it'll be mergeable.

```
building srfi-18
   /tmp/guix-build-chicken-srfi-18-0.1.5.drv-0/srfi-18/build-srfi-18
-host -D compiling-extension -J -s -regenerate-import-libraries
-setup-mode -I /tmp/guix-build-chicken-srfi-18-0.1.5.drv-0/srfi-18 -C
-I/tmp/guix-build-chicken-srfi-18-0.1.5.drv-0/srfi-18 -O2 -d1
srfi-18.scm -o
/tmp/guix-build-chicken-srfi-18-0.1.5.drv-0/srfi-18/srfi-18.so
/tmp/guix-build-chicken-srfi-18-0.1.5.drv-0/srfi-18/build-srfi-18: line
8: srfi-18.types: Permission denied
/tmp/guix-build-chicken-srfi-18-0.1.5.drv-0/srfi-18/build-srfi-18
-regenerate-import-libraries -M -setup-mode -static -I
/tmp/guix-build-chicken-srfi-18-0.1.5.drv-0/srfi-18 -emit-link-file
/tmp/guix-build-chicken-srfi-18-0.1.5.drv-0/srfi-18/srfi-18.link -host
-D compiling-extension -c -unit srfi-18 -D compiling-static-extension
-C -I/tmp/guix-build-chicken-srfi-18-0.1.5.drv-0/srfi-18 -O2 -d1
srfi-18.scm -o
/tmp/guix-build-chicken-srfi-18-0.1.5.drv-0/srfi-18/srfi-18.static.o
cp: cannot create regular file 'srfi-18.types': Permission denied
/tmp/guix-build-chicken-srfi-18-0.1.5.drv-0/srfi-18/build-srfi-18: line
8: srfi-18.types: Permission denied
/gnu/store/aags0k5s6pnk1askg8k3czyhi34gz4pg-chicken-5.2.0/bin/csc
-setup-mode -s -host -I
/tmp/guix-build-chicken-srfi-18-0.1.5.drv-0/srfi-18 -C
-I/tmp/guix-build-chicken-srfi-18-0.1.5.drv-0/srfi-18 -O2 -d0
srfi-18.import.scm -o
/tmp/guix-build-chicken-srfi-18-0.1.5.drv-0/srfi-18/srfi-18.import.so
```




Information forwarded to guix-patches <at> gnu.org:
bug#43976; Package guix-patches. (Sat, 21 Nov 2020 11:24:02 GMT) Full text and rfc822 format available.

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: raingloom <raingloom <at> riseup.net>
Cc: 43976 <at> debbugs.gnu.org
Subject: Re: [bug#43976] [PATCH] Chicken build system + some example eggs
Date: Sat, 21 Nov 2020 12:23:30 +0100
Hi,

raingloom <raingloom <at> riseup.net> skribis:

> Well, CHICKEN_IMPORT_PATH seems to just be some kind of historical
> baggage, at least as far as I can tell. It is not mentioned anywhere on
> https://wiki.call-cc.org/man/5 and from what I deciphered from the
> sources, the only time it's used it gets prepended to
> CHICKEN_REPOSITORY_PATH.
>
> I removed the search path and things still seem to work. It was
> probably used in earlier version of Chicken and Nix just never removed
> it. Didn't find any related issues either.
> I think I'll comment it out for now but leave it in? If there are no
> issues with it a few months after merging, it can probably be deleted
> entirely...

I’m not familiar with Chicken.  The only advice I could give is to refer
to the Chicken documentation before checking how other distros do it,
it’s probably safer.

> Possibly more important: I noticed some warnings in some builds about
> cp(1). So far it hasn't caused any issues. Gonna fix this, then I think
> it'll be mergeable.

Yup weird.

Anyway, let me know when you have a v2!

Thanks,
Ludo’.




Information forwarded to guix-patches <at> gnu.org:
bug#43976; Package guix-patches. (Sat, 21 Nov 2020 20:46:02 GMT) Full text and rfc822 format available.

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

From: raingloom <raingloom <at> riseup.net>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 43976 <at> debbugs.gnu.org
Subject: Re: [bug#43976] [PATCH] Chicken build system + some example eggs
Date: Sat, 21 Nov 2020 21:45:43 +0100
On Fri, 20 Nov 2020 05:51:17 +0100
raingloom <raingloom <at> riseup.net> wrote:

> Possibly more important: I noticed some warnings in some builds about
> cp(1). So far it hasn't caused any issues. Gonna fix this, then I
> think it'll be mergeable.

Seems to be a srfi-18 specific issue, so I'm calling
chicken-build-system final for now. You can merge it if it stands up to
a review.
The other eggs also seem fine, srfi-18 just has a somewhat shoddily
written auxiliary build script to generate some type information,
probably based on the current Chicken version? I'm not sure what it's
doing to be honest. It doesn't fail to build because the script doesn't
check for errors, which is why it took a while to notice it.

This is why we always `set -e` in our Bash scripts, kids.

Anyways, all other eggs seem to be fine, and none of them depend on
srfi-18.




Information forwarded to guix-patches <at> gnu.org:
bug#43976; Package guix-patches. (Sat, 21 Nov 2020 21:00:01 GMT) Full text and rfc822 format available.

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

From: Efraim Flashner <efraim <at> flashner.co.il>
To: raingloom <raingloom <at> riseup.net>
Cc: 43976 <at> debbugs.gnu.org, Ludovic Courtès <ludo <at> gnu.org>
Subject: Re: [bug#43976] [PATCH] Chicken build system + some example eggs
Date: Sat, 21 Nov 2020 22:58:58 +0200
[Message part 1 (text/plain, inline)]
On Sat, Nov 21, 2020 at 09:45:43PM +0100, raingloom wrote:
> On Fri, 20 Nov 2020 05:51:17 +0100
> raingloom <raingloom <at> riseup.net> wrote:
> 
> > Possibly more important: I noticed some warnings in some builds about
> > cp(1). So far it hasn't caused any issues. Gonna fix this, then I
> > think it'll be mergeable.
> 
> Seems to be a srfi-18 specific issue, so I'm calling
> chicken-build-system final for now. You can merge it if it stands up to
> a review.

Well that's exciting! That's the patches at the beginning of the email
thread?

> The other eggs also seem fine, srfi-18 just has a somewhat shoddily
> written auxiliary build script to generate some type information,
> probably based on the current Chicken version? I'm not sure what it's
> doing to be honest. It doesn't fail to build because the script doesn't
> check for errors, which is why it took a while to notice it.
> 
> This is why we always `set -e` in our Bash scripts, kids.
> 
> Anyways, all other eggs seem to be fine, and none of them depend on
> srfi-18.

You can always rant at upstream or your monitor, your choice :)

-- 
Efraim Flashner   <efraim <at> flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted
[signature.asc (application/pgp-signature, inline)]

Information forwarded to guix-patches <at> gnu.org:
bug#43976; Package guix-patches. (Sat, 21 Nov 2020 22:16:02 GMT) Full text and rfc822 format available.

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

From: raingloom <raingloom <at> riseup.net>
To: Efraim Flashner <efraim <at> flashner.co.il>
Cc: 43976 <at> debbugs.gnu.org, Ludovic Courtès <ludo <at> gnu.org>
Subject: Re: [bug#43976] [PATCH] Chicken build system + some example eggs
Date: Sat, 21 Nov 2020 23:13:38 +0100
On Sat, 21 Nov 2020 22:58:58 +0200
Efraim Flashner <efraim <at> flashner.co.il> wrote:

> > The other eggs also seem fine, srfi-18 just has a somewhat shoddily
> > written auxiliary build script to generate some type information,
> > probably based on the current Chicken version? I'm not sure what
> > it's doing to be honest. It doesn't fail to build because the
> > script doesn't check for errors, which is why it took a while to
> > notice it.
> > 
> > This is why we always `set -e` in our Bash scripts, kids.
> > 
> > Anyways, all other eggs seem to be fine, and none of them depend on
> > srfi-18.  
> 
> You can always rant at upstream or your monitor, your choice :)
> 

Oh I will, once I figure out where to do that. So far I've been more
focused on just getting enough things working to hack on the 9p egg.




Information forwarded to guix-patches <at> gnu.org:
bug#43976; Package guix-patches. (Sun, 22 Nov 2020 23:13:02 GMT) Full text and rfc822 format available.

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

From: raingloom <raingloom <at> riseup.net>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 43976 <at> debbugs.gnu.org
Subject: Re: [bug#43976] [PATCH] Chicken build system + some example eggs
Date: Mon, 23 Nov 2020 00:12:41 +0100
[Message part 1 (text/plain, inline)]
On Sat, 21 Nov 2020 21:45:43 +0100
raingloom <raingloom <at> riseup.net> wrote:

> On Fri, 20 Nov 2020 05:51:17 +0100
> raingloom <raingloom <at> riseup.net> wrote:
> 
> > Possibly more important: I noticed some warnings in some builds
> > about cp(1). So far it hasn't caused any issues. Gonna fix this,
> > then I think it'll be mergeable.  
> 
> Seems to be a srfi-18 specific issue, so I'm calling
> chicken-build-system final for now. You can merge it if it stands up
> to a review.
> The other eggs also seem fine, srfi-18 just has a somewhat shoddily
> written auxiliary build script to generate some type information,
> probably based on the current Chicken version? I'm not sure what it's
> doing to be honest. It doesn't fail to build because the script
> doesn't check for errors, which is why it took a while to notice it.
> 
> This is why we always `set -e` in our Bash scripts, kids.
> 
> Anyways, all other eggs seem to be fine, and none of them depend on
> srfi-18.

Forgot to actually send the modified patch :D
The others should be unchanged.
[0001-gnu-Added-search-paths-for-Chicken-Scheme.patch (text/x-patch, attachment)]

Information forwarded to guix-patches <at> gnu.org:
bug#43976; Package guix-patches. (Wed, 25 Nov 2020 01:47:02 GMT) Full text and rfc822 format available.

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

From: raingloom <raingloom <at> riseup.net>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 43976 <at> debbugs.gnu.org
Subject: Re: [bug#43976] [PATCH] Chicken build system + some example eggs
Date: Tue, 24 Nov 2020 21:22:54 +0100
[Message part 1 (text/plain, inline)]
On Mon, 23 Nov 2020 00:12:41 +0100
raingloom <raingloom <at> riseup.net> wrote:

> On Sat, 21 Nov 2020 21:45:43 +0100
> raingloom <raingloom <at> riseup.net> wrote:
> 
> > On Fri, 20 Nov 2020 05:51:17 +0100
> > raingloom <raingloom <at> riseup.net> wrote:
> >   
> > > Possibly more important: I noticed some warnings in some builds
> > > about cp(1). So far it hasn't caused any issues. Gonna fix this,
> > > then I think it'll be mergeable.    
> > 
> > Seems to be a srfi-18 specific issue, so I'm calling
> > chicken-build-system final for now. You can merge it if it stands up
> > to a review.
> > The other eggs also seem fine, srfi-18 just has a somewhat shoddily
> > written auxiliary build script to generate some type information,
> > probably based on the current Chicken version? I'm not sure what
> > it's doing to be honest. It doesn't fail to build because the script
> > doesn't check for errors, which is why it took a while to notice it.
> > 
> > This is why we always `set -e` in our Bash scripts, kids.
> > 
> > Anyways, all other eggs seem to be fine, and none of them depend on
> > srfi-18.  
> 
> Forgot to actually send the modified patch :D
> The others should be unchanged.

*facepalm* i somehow messed up the parens in that one. maybe i didn't
have Paredit on or something??? here's the fixed version.
but yeah, with this, it should be complete.
[0001-gnu-Added-search-paths-for-Chicken-Scheme.patch (text/x-patch, attachment)]

Information forwarded to guix-patches <at> gnu.org:
bug#43976; Package guix-patches. (Fri, 27 Nov 2020 09:10:02 GMT) Full text and rfc822 format available.

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: raingloom <raingloom <at> riseup.net>
Cc: 43976 <at> debbugs.gnu.org
Subject: Re: [bug#43976] [PATCH] Chicken build system + some example eggs
Date: Fri, 27 Nov 2020 10:09:29 +0100
Hi raingloom!

raingloom <raingloom <at> riseup.net> skribis:

> From 2ba52705cf909590a428ea598dfdf1c61ada6a8c Mon Sep 17 00:00:00 2001
> From: raingloom <raingloom <at> riseup.net>
> Date: Mon, 12 Oct 2020 04:11:59 +0200
> Subject: [PATCH 01/20] gnu: Added search paths for Chicken Scheme.
>
> * gnu/packages/chicken.scm (chicken): Added search paths
>   [native-search-paths]: added CHICKEN_REPOSITORY_PATH and a comment about
>                          CHICKEN_INCLUDE_PATH.

[...]

> +    (native-search-paths
> +     (list (search-path-specification
> +            (variable "CHICKEN_REPOSITORY_PATH")
> +            ;; TODO extract binary version into a module level definition.
> +            (files (list "var/lib/chicken/11")))))
> +    ;; the use of this variable is unclear. the online docs don't even
> +    ;; mention it. i'm leaving it in as a comment for now, in case
> +    ;; something breaks.
> +    ;; (search-path-specification
> +    ;;  (variable "CHICKEN_INCLUDE_PATH")
> +    ;;  (files '("share")))

I think you can remove the comment here; presumably we now know that
CHICKEN_REPOSITORY_PATH is the one that matters.  :-)

Could you please ensure that earlier comments notably at
<https://issues.guix.gnu.org/43976#1> have been taken into account, that
the SRFI packages still build and have a layout consistent with the
var/lib/chicken/11 search path above, and send a v2 of the whole series?

Thanks in advance!

Ludo’.




Information forwarded to guix-patches <at> gnu.org:
bug#43976; Package guix-patches. (Tue, 01 Dec 2020 04:22:01 GMT) Full text and rfc822 format available.

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

From: raingloom <raingloom <at> riseup.net>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 43976 <at> debbugs.gnu.org
Subject: Re: [bug#43976] [PATCH] Chicken build system + some example eggs
Date: Tue, 1 Dec 2020 05:14:28 +0100
[Message part 1 (text/plain, inline)]
On Fri, 27 Nov 2020 10:09:29 +0100
Ludovic Courtès <ludo <at> gnu.org> wrote:

> Hi raingloom!
> 
> raingloom <raingloom <at> riseup.net> skribis:
> 
> > From 2ba52705cf909590a428ea598dfdf1c61ada6a8c Mon Sep 17 00:00:00
> > 2001 From: raingloom <raingloom <at> riseup.net>
> > Date: Mon, 12 Oct 2020 04:11:59 +0200
> > Subject: [PATCH 01/20] gnu: Added search paths for Chicken Scheme.
> >
> > * gnu/packages/chicken.scm (chicken): Added search paths
> >   [native-search-paths]: added CHICKEN_REPOSITORY_PATH and a
> > comment about CHICKEN_INCLUDE_PATH.  
> 
> [...]
> 
> > +    (native-search-paths
> > +     (list (search-path-specification
> > +            (variable "CHICKEN_REPOSITORY_PATH")
> > +            ;; TODO extract binary version into a module level
> > definition.
> > +            (files (list "var/lib/chicken/11")))))
> > +    ;; the use of this variable is unclear. the online docs don't
> > even
> > +    ;; mention it. i'm leaving it in as a comment for now, in case
> > +    ;; something breaks.
> > +    ;; (search-path-specification
> > +    ;;  (variable "CHICKEN_INCLUDE_PATH")
> > +    ;;  (files '("share")))  
> 
> I think you can remove the comment here; presumably we now know that
> CHICKEN_REPOSITORY_PATH is the one that matters.  :-)

I guess I can, since it doesn't looks like it will cause problems. I
left leave the commented path there though.

> Could you please ensure that earlier comments notably at
> <https://issues.guix.gnu.org/43976#1> have been taken into account,
> that the SRFI packages still build and have a layout consistent with
> the var/lib/chicken/11 search path above, and send a v2 of the whole
> series?
> 
> Thanks in advance!
> 
> Ludo’.

Everything builds, tests succeed, and the layouts are the same.
The comments have been taken into account. I listed them in previous
mails so I'll skip them now.
For srfi-14, I hope "non-copyleft" is fine.
Ran guix lint on all packages and it only mentioned archival.

Thanks for the feedback!

ps.: sending the updated original patchset.
i also have these for a later patchset:
0011-gnu-Added-chicken-compile-file.patch
0012-gnu-Added-chicken-srfi-18.patch
0013-gnu-Added-chicken-srfi-13.patch
0014-Added-chicken-check-errors.patch
0015-Added-chicken-defstruct.patch
0016-gnu-Added-chicken-matchable.patch
0017-gnu-Added-chicken-record-variants.patch
0018-gnu-Added-chicken-srfi-41.patch
0019-gnu-Added-chicken-uri-common.patch
0020-gnu-Added-chicken-uri-generic.patch
[0001-gnu-Added-search-paths-for-Chicken-Scheme.patch (text/x-patch, attachment)]
[0002-guix-Added-chicken-build-system.patch (text/x-patch, attachment)]
[0003-gnu-Use-qualified-license-names-in-chicken.scm-inste.patch (text/x-patch, attachment)]
[0004-gnu-Added-chicken-test.patch (text/x-patch, attachment)]
[0005-gnu-Added-chicken-srfi-1.patch (text/x-patch, attachment)]
[0006-gnu-Added-chicken-srfi-69.patch (text/x-patch, attachment)]
[0007-gnu-Added-chicken-iset.patch (text/x-patch, attachment)]
[0008-gnu-Added-chicken-datatype.patch (text/x-patch, attachment)]
[0009-gnu-Added-chicken-srfi-14.patch (text/x-patch, attachment)]
[0010-gnu-Added-chicken-agrep.patch (text/x-patch, attachment)]

Reply sent to Ludovic Courtès <ludo <at> gnu.org>:
You have taken responsibility. (Thu, 03 Dec 2020 16:06:02 GMT) Full text and rfc822 format available.

Notification sent to raingloom <raingloom <at> riseup.net>:
bug acknowledged by developer. (Thu, 03 Dec 2020 16:06:02 GMT) Full text and rfc822 format available.

Message #40 received at 43976-done <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: raingloom <raingloom <at> riseup.net>
Cc: 43976-done <at> debbugs.gnu.org
Subject: Re: [bug#43976] [PATCH] Chicken build system + some example eggs
Date: Thu, 03 Dec 2020 17:04:58 +0100
Hi raingloom,

I’ve now pushed the 11 patches as of
b5a6eddd8e9f2e840f47d3307f436433e74e16f3, tweaking packages in minor
ways: tweaking descriptions (info "(guix) Synopses and Descriptions"),
adding a ‘file-name’ field in origin that contains the full package name
and version, changing ‘native-inputs’ to ‘inputs’ in a couple of cases,
removing the stale search path comment as discussed earlier, adjusting
commit logs.

Thanks!

Ludo’.




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Fri, 01 Jan 2021 12:24:12 GMT) Full text and rfc822 format available.

This bug report was last modified 3 years and 87 days ago.

Previous Next


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