[PATCH][CMake][OCaml] Add OCaml binding documentation to "all" target

Alex Wang via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 18 11:03:07 PST 2015


Might as well keep the ALL version then, if that's alright.

A "flag" file of some sorts could work, I guess, but as you said earlier, that could be more work than its worth since the documentation builds fairly fast.

-Alex

> On Dec 18, 2015, at 1:39 PM, Chris Bieneman <beanz at apple.com> wrote:
> 
> Sorry I didn’t reply when you sent this out earlier.
> 
> Because the OUTPUT in your patch isn’t a real output (as in a file that can be timestamped against inputs) I don’t think this is any better than just adding ALL to the old ocaml_doc patch.
> 
> If you did want to map inputs and outputs to avoid rebuilding when nothing changes I can talk you through what that would take, but I don’t think any of the documentation builds are setup properly to support that.
> 
> -Chris
> 
>> On Dec 18, 2015, at 6:51 AM, Alex Wang <aw1621107 at gmail.com> wrote:
>> 
>> The OCaml binding documentation isn't included in the default "make" target, which led to "make install" failing when the it couldn't find the generated documentation. This adds the "ocaml_doc" target to the "all" target, and creates the documentation in a custom command instead of a custom target.
>> 
>> -Alex
>> 
>> <llvm-ocaml.patch>
> 



More information about the llvm-commits mailing list