From: Taylan Kammer <taylan.kammer@gmail.com>
To: Luis Felipe <luis.felipe.la@protonmail.com>
Cc: "guile-user@gnu.org" <guile-user@gnu.org>
Subject: Re: srfi-64 tests passing when they should not
Date: Thu, 6 May 2021 13:16:17 +0200 [thread overview]
Message-ID: <19c4bb0c-4780-c398-a26f-2e2211c00667@gmail.com> (raw)
In-Reply-To: <W37a4gGYVsVa83iC_20D0krdgc-mMMfToVWIyIbTzbpbQcqSqU4QGnIlmST1fzar9cy01uw6_qsV1Y3Xj4IslftmbMtVE0Lk0PikzeB-_9M=@protonmail.com>
On 05.05.2021 15:47, Luis Felipe wrote:
> Hi Taylan,
>
> On Wednesday, May 5, 2021 6:39 AM, Taylan Kammer <taylan.kammer@gmail.com> wrote:
>
>> On 04.05.2021 10:31, Zelphir Kaltstahl wrote:
>>
>>> The first 2 tests are surprisingly passing. This is also the reason, why I used
>>> test-assert and manually wrote the (equal? ...) in the last test, to see,
>>> whether it makes any difference. Indeed it does.
>>
>> The reference implementation of SRFI-64 (which is what Guile ships)
>> doesn't seem to be written very well.
>>
>> I have an alternative implementation here, if you're interested:
>>
>> https://github.com/TaylanUB/scheme-srfis
>>
>> I'm not sure if the newest Guile is able to run it out of the box
>> though. You might have to create some .scm symlinks to the .sld files.
>
> For what it's worth, I know about your implementation for a long time, but I've never tried to use it because I don't know where to start. Is it not possible to package these libraries so that users can simply install them as any other guile library? Say:
>
> $ guix install r7rs-srfi-64
>
> I see that Guile can be run with the "--r7rs" option "to better support R7RS"...
>
Hmm, I had hoped that with the newest Guile, simply adding the repo's
root directory to the load path would work, at least when invoked with
the --r7rs switch, but it seems that Guile still chokes on library name
parts that are integers. That's an incompatibility with r7rs that's not
mentioned in the manual.
I guess the only way to make the modules work is to rename all the files
and change the library names to not use integer parts.
Maybe I'll make a guile-compatible standalone package for the SRFI-64
implementation, since that's the most fancy thing in that repo.
I might do it in the following days since I'm on a vacation, but... the
vacation is supposed to be a vacation. :-) Work has been really burning
me out in the last year.
If someone else feels like trying: all you have to do is rename .sld
files to .scm, change the integer parts of the module names to symbols
(e.g. s64 instead of 64), and rename the directory '64' accordingly.
- Taylan
next prev parent reply other threads:[~2021-05-06 11:16 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-04 8:31 srfi-64 tests passing when they should not Zelphir Kaltstahl
2021-05-04 10:15 ` Jérémy Korwin-Zmijowski
2021-05-04 11:36 ` Zelphir Kaltstahl
2021-05-05 6:39 ` Taylan Kammer
2021-05-05 13:47 ` Luis Felipe
2021-05-06 11:16 ` Taylan Kammer [this message]
2021-05-06 15:33 ` Luis Felipe
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/guile/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=19c4bb0c-4780-c398-a26f-2e2211c00667@gmail.com \
--to=taylan.kammer@gmail.com \
--cc=guile-user@gnu.org \
--cc=luis.felipe.la@protonmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).