On 09/09/21 13:08, Richard W.M. Jones wrote:
On Thu, Sep 09, 2021 at 12:42:45PM +0200, Laszlo Ersek wrote:
> On 09/08/21 17:49, Richard W.M. Jones wrote:
>>
>> Yes the series looks fine, ACK series.
>
> Thanks!
>
>> I think the presence of $(top_builddir)/lib to look for the header
>> file was likely a copy-paste thing from some other project. The
>> generator always writes files into the source directory (or if you
>> build from the tarball, everything is already in the source directory.)
>
> Right, I'd not been familiar with autotools internals, so I first needed
> to understand that the generator ran during autogen, and (therefore) its
> output would be included by "make dist" in the *source* tree.
It's because we want to allow people to build from tarballs without
needing the ocamlc dependency.
Ah, good point! I didn't realize there was a specific incentive.
It may not be necessary any longer, but it's what we're doing
now.
Oh I didn't mean it as criticism at all -- it makes perfect sense to me,
it was just something new I had to learn :)
Thanks!
Laszlo