On Mon, Jun 24, 2013 at 11:45 AM, Nick Wellnhofer <[email protected]> wrote:
>
> We should only merge the separate-clownfish-wip5 branch first.Here are some
> things that are still to do:
>
> * Check that the installation of Clownfish works (Perl and C).
> * Make the Lucy build work with an external Clownfish installation
> (Perl and C).
> * Find a way to extract the DocuComments from the .h files in
> clownfish/compiler/src. Check that the important stuff of the
> compiler is documented.
> * Finish the general Clownfish documentation. You might have noticed
> that I already started. I also think there are quite a few corner
> cases where we have to decide how CFC should handle them. In the
> result, CFC will probably have to make some stricter checks. Some
> obvious things are already marked as TODO in the CFC source.
>
> But all of this can be done after merging the branch. If there are no
> objections, I can go ahead with the merge. I don't have anything else
> pending.
+1 to merge. I've reviewed the commits on that branch. I particularly
enjoyed this one:
commit 9e0f1cc8949c79af65c4762deb4ffef48b25499b
Author: Nick Wellnhofer <[email protected]>
Date: Fri Jun 7 00:00:46 2013 +0200
Separate Clownfish from Lucy
Remove Clownfish files from Lucy and build Lucy with external
Clownfish library.
:)
Installation is most assuredly broken for Lucy's Perl bindings. It's been
broken for a long time on master.
With regards to the documentation, I'm going to reply to your commit here on
the dev list. I think it's fine, even desirable, for it to be incomplete or
contain TODOs, because that reflects the current amorphous state of the API.
Marvin
