[PATCH] Use automake (v5)

Jon Turney jon.turney@dronecode.org.uk
Tue Apr 27 15:54:08 GMT 2021


On 26/04/2021 16:03, Corinna Vinschen wrote:
> On Apr 22 13:57, Corinna Vinschen wrote:
>> On Apr 21 18:40, Corinna Vinschen wrote:
>>> On Apr 20 21:15, Jon Turney wrote:
>>>> On 20/04/2021 21:13, Jon Turney wrote:
>>>> * some object files are in a unexpected places in the build file hierarchy
>>>> (compared to naive expectations and/or the non-automake build)
>>>
>>> This is the only minor qualm I have with this patch.  It would be nice
>>> to have the mingw sources and .o files in the mingw subdir.  It would
>>> simply be a bit cleaner.  The files shared between cygwin and mingw
>>> (that's only path.cc, I think) could be handled by an include, i. e.
>>>
>>>    utils/
>>>
>>>      path.cc (full implementation)
>>>
>>>    utils/mingw/
>>>
>>>      path.cc:
>>>
>>>        #include "../path.cc"
>>
>> I wonder if it wouldn't make sense to split out the mingw-only parts
>> of path.cc entirely.  I had a quick view into the file and it turns
>> out that of the almost 1000 lines in this file, only about 100 lines
>> are used by mount.  All the rest is only used by mingw code, i. e.,
>> cygcheck and strace.
>>
>> That's obviously not part of this patch, but something we should keep
>> in mind for a later cleanup.
> 
> I tried this as a POC and it's not much of a problem.  See the below
> patch.  Cleaning up the includes is still to do.
> 

Thanks, this seems workable.  I'll take a look.


More information about the Cygwin-patches mailing list