From: Christine Lemmer-Webber <cwebber@dustycloud.org>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: 56204@debbugs.gnu.org, Maxime Devos <maximedevos@telenet.be>
Subject: [bug#56204] Add guile-goblins
Date: Sun, 26 Jun 2022 15:09:42 -0400 [thread overview]
Message-ID: <87edzbjl2h.fsf@dustycloud.org> (raw)
In-Reply-To: <87zgi0s9f8.fsf_-_@gnu.org>
Ludovic Courtès <ludo@gnu.org> writes:
> Maxime Devos <maximedevos@telenet.be> skribis:
>
>> Christine Lemmer-Webber schreef op vr 24-06-2022 om 17:24 [-0400]:
>>> + (version "0.8")
>>> + (source
>>> + (origin
>>> + (method git-fetch)
>>> + (uri (git-reference
>>> + (url "https://gitlab.com/spritely/guile-goblins/")
>>> + (commit (string-append "v" version))))
>>> + (file-name (string-append name "-" version))
>>> + (sha256
>>> + (base32
>>> + "1mmyykh79jwhrfgnhhw94aw7a8m6qw249kj7k60ynj16mcfm5iyy"))))
>>
>>
>> I'm not seeing "--target" in the invocation of "$(GUILE_TOOLS)
>> compile", so I don't think that currently it will cross-compile
>> correctly. Could you add a --target=... in the Makefile.am?
>
> That can come upstream in the next version of Goblins IMO.
>
> Ludo’.
Cool, I added it to commit 6664daa27e9b3eb3220ba83d9c874b730e792f90 in
guile-goblins. However I was just mimicing guile-gcrypt, so I'm not
sure if it's right:
#+BEGIN_SRC diff
diff --git a/Makefile.am b/Makefile.am
index b8d4944..b056337 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -32,7 +32,7 @@ EXTRA_DIST = $(SOURCES) $(NOCOMP_SOURCES)
GUILE_WARNINGS = -Wunbound-variable -Warity-mismatch -Wformat
SUFFIXES = .scm .go
.scm.go:
- $(AM_V_GEN)$(top_builddir)/pre-inst-env $(GUILE_TOOLS) compile $(GUILE_WARNINGS) -o "$@" "$<"
+ $(AM_V_GEN)$(top_builddir)/pre-inst-env $(GUILE_TOOLS) compile $(GUILE_TARGET) $(GUILE_WARNINGS) -o "$@" "$<"
SOURCES = goblins.scm \
goblins/ocapn/crypto-stubs.scm \
#+END_SRC
Should that be sufficient?
Notably, the Makefile.am here was bootstrapped using guile-hall, so my
suspicion is that this is an update that should be made in guile-hall's
templates also, then everyone can benefit.
The next release of Goblins should be due out very quickly, I think, so
we'll be able to take advantage of it then.
next prev parent reply other threads:[~2022-06-26 19:13 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-24 21:24 [bug#56204] Add guile-goblins Christine Lemmer-Webber
2022-06-24 21:57 ` Maxime Devos
2022-06-25 2:36 ` bug#56204: " Christine Lemmer-Webber
2022-06-25 9:32 ` [bug#56204] " Maxime Devos
2022-06-25 21:46 ` Ludovic Courtès
2022-06-26 19:09 ` Christine Lemmer-Webber [this message]
2022-06-26 20:50 ` Maxime Devos
2022-06-27 10:22 ` Ludovic Courtès
2022-06-25 21:49 ` Ludovic Courtès
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://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87edzbjl2h.fsf@dustycloud.org \
--to=cwebber@dustycloud.org \
--cc=56204@debbugs.gnu.org \
--cc=ludo@gnu.org \
--cc=maximedevos@telenet.be \
/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.
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).