lucy-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Nick Wellnhofer <wellnho...@aevum.de>
Subject Re: [lucy-dev] [GitHub] lucy pull request: Remove clownfish
Date Wed, 09 Apr 2014 16:07:51 GMT
On 22/03/2014 20:17, Nick Wellnhofer wrote:
> On Mar 22, 2014, at 01:28 , timwilkens <git@git.apache.org> wrote:
>
>> GitHub user timwilkens opened a pull request:
>>
>>     https://github.com/apache/lucy/pull/5
>>
>>     Remove clownfish
>>
>>     Remove Clownfish from the Lucy repository. This branch eliminates the Perl bindings'
dependency on the bundled Clownfish directory.
>>
>>     After applying this patch sequence both the Perl bindings and the C bindings
build without error.
>>
>>     Marvin Humphrey and I collaborated on this patch sequence.
>
> I get the following failure:
>
> Parcel 'Clownfish' required by 'Lucy' not found at /Library/Perl/5.16/darwin-thread-multi-2level/Clownfish/CFC/Perl/Build.pm
line 233.
>
> It seems that the system-wide Clownfish include dir for Perl doesn’t get added to the
search path. I do this for my sample extension in Build.PL:
>
> https://github.com/nwellnhof/LucyX-Analysis-WhitespaceTokenizer/blob/master/perl/Build.PL
>
> But this should probably be done in Clownfish::CFC::Perl::Build.

I'm really blocking on this. If I don't hear any objections, I'll finish the 
Clownfish removal myself.

Nick

Mime
View raw message