From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
Subject: Re: [PATCH] Add Jellyfish.
Date: Wed, 6 Jan 2016 12:13:59 +0100
Message-ID: <idjh9irgdu0.fsf@bimsb-sys02.mdc-berlin.net>
References: <idjy4cr6748.fsf@bimsb-sys02.mdc-berlin.net>
	<9507d13e837c22beb6e5a79cce045222@openmailbox.org>
	<567495E3.5010803@uq.edu.au> <87oadkrbb8.fsf@gnu.org>
	<idj7fjwhqrg.fsf@bimsb-sys02.mdc-berlin.net>
Mime-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
Return-path: <guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org>
Received: from eggs.gnu.org ([2001:4830:134:3::10]:44129)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <Ricardo.Wurmus@mdc-berlin.de>) id 1aGm2d-00065k-7G
	for guix-devel@gnu.org; Wed, 06 Jan 2016 06:14:15 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <Ricardo.Wurmus@mdc-berlin.de>) id 1aGm2a-00060e-Cb
	for guix-devel@gnu.org; Wed, 06 Jan 2016 06:14:11 -0500
In-Reply-To: <idj7fjwhqrg.fsf@bimsb-sys02.mdc-berlin.net>
List-Id: "Development of GNU Guix and the GNU System distribution."
	<guix-devel.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/guix-devel>
List-Post: <mailto:guix-devel@gnu.org>
List-Help: <mailto:guix-devel-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=subscribe>
Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org
Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org
To: Ludovic =?utf-8?Q?Court=C3=A8s?= <ludo@gnu.org>
Cc: Guix-devel <guix-devel@gnu.org>


Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de> writes:

> Ludovic Court=C3=A8s <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=E2=80=99s the usual space/popularity tradeoff to take into accou=
nt: adding
>> them all makes the package=E2=80=99s closure much larger, so it=E2=80=99=
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 =E2=80=9Ctk=E2=80=9D output of Python), but it=E2=
=80=99s 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: sym=
bol lookup error: /tmp/nix-build-jellyfish-2.2.4.drv-0/jellyfish-2.2.4/.l=
ibs/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=E2=80=99ll play with th=
is later
> and see if I can make it work.  If not I=E2=80=99ll leave the Perl bind=
ings (and
> the =E2=80=9Cperl=E2=80=9D output) away for now.

I pushed a version with Ruby and Python bindings placed in different
outputs.  =E2=80=9Cguix size=E2=80=9D confirmed that there are no referen=
ces to Ruby or
Python in the plain output.

I did not add the Perl bindings.

Thanks for the suggestions!

~~ Ricardo