On Mar 22, 2014, at 01:28 , timwilkens <[email protected]> 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. Another thing we have to care about is the build bot. After we remove Clownfish from the Lucy tree, the build bot will have to fetch the Clownfish and Lucy repositories and build both projects. We should also take this opportunity to add the C build. Nick
