From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: Guix-devel <guix-devel@gnu.org>
Subject: Re: [PATCH] Add Jellyfish.
Date: Wed, 6 Jan 2016 12:13:59 +0100 [thread overview]
Message-ID: <idjh9irgdu0.fsf@bimsb-sys02.mdc-berlin.net> (raw)
In-Reply-To: <idj7fjwhqrg.fsf@bimsb-sys02.mdc-berlin.net>
Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de> writes:
> Ludovic Courtès <ludo@gnu.org> writes:
>
>> Ben Woodcroft <b.woodcroft@uq.edu.au> skribis:
>>
>>> But would it be possible to include the scripting language bindings,
>>> something along these lines?
>>>
>>> + (arguments
>>> + `(#:configure-flags '("--enable-ruby-binding"
>>> + "--enable-python-binding"
>>> + "--enable-perl-binding")
>>
>> There’s the usual space/popularity tradeoff to take into account: adding
>> them all makes the package’s closure much larger, so it’s important to
>> add only the useful bindings by default.
>>
>> Ideally, the .so for these bindings could be moved to separate outputs
>> (like we did for the “tk” output of Python), but it’s not always easy to
>> do.
>
> In this case it seems to be very easy to separate the bindings into
> different outputs as the flags take an optional path.
>
> However, the test for the Perl bindings does not pass:
>
> /gnu/store/czs63sm4l0s4a56ab38dqvkx19yzylbq-perl-5.16.1/bin/perl: symbol lookup error: /tmp/nix-build-jellyfish-2.2.4.drv-0/jellyfish-2.2.4/.libs/libjellyfish-2.0.so.2: undefined symbol: pthread_create
> FAIL tests/swig_perl.sh (exit status: 127)
>
> Maybe the library needs another linker flag? I’ll play with this later
> and see if I can make it work. If not I’ll leave the Perl bindings (and
> the “perl” output) away for now.
I pushed a version with Ruby and Python bindings placed in different
outputs. “guix size” confirmed that there are no references to Ruby or
Python in the plain output.
I did not add the Perl bindings.
Thanks for the suggestions!
~~ Ricardo
prev parent reply other threads:[~2016-01-06 11:14 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-18 16:42 [PATCH] Add Jellyfish Ricardo Wurmus
2015-12-18 18:19 ` Eric Bavier
2015-12-18 23:25 ` Ben Woodcroft
2015-12-20 22:43 ` Ludovic Courtès
2015-12-30 15:58 ` Ricardo Wurmus
2016-01-01 22:15 ` Ludovic Courtès
2016-01-06 11:13 ` Ricardo Wurmus [this message]
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=idjh9irgdu0.fsf@bimsb-sys02.mdc-berlin.net \
--to=ricardo.wurmus@mdc-berlin.de \
--cc=guix-devel@gnu.org \
--cc=ludo@gnu.org \
/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).