Re: [PATCH 1/7] newlib: libm: merge machine/ trampoline up a level

On 27/01/2022 02:46, Mike Frysinger wrote:
> On 26 Jan 2022 15:58, Jon Turney wrote:
>> On 23/01/2022 06:04, Mike Frysinger wrote:
>>> The machine/{configure,Makefile} files exist only to fan out to the
>>> specific machine/$arch/ subdir.  We already have all that same info
>>> in the libm/ dir itself, so by moving the recursive configure and
>>> make calls into it, we can cut off this logic entirely and save the
>>> overhead.
>>>
>>> For arches that don't have a machine subdir, it means they can skip
>>> the logic entirely.
>>
>> It looks like this (and the following commint) breaks 'man info' and
>> 'make man'?
>>
>>
>> (e.g. https://github.com/cygwin/cygwin/runs/4949106511)
> 
> blah, sorry about that.  i have another series pending related to man page
> unification that i've been testing with and doesn't have this issue.  but
> that's still under review, so i pushed this quick hack fix since my series
> deletes this entire chunk of code.
> -mike

Thanks.
I hadn't noticed before, but there also seems to be something in this 
series which seems to break building for x86.

(see https://github.com/cygwin/cygwin/actions/runs/1754176435)

>   CC       lib_a-f_exp.o
> In file included from ../../../../../../newlib/libm/machine/i386/f_exp.c:29:
> ../../../../../../newlib/libm/machine/i386/f_math.h:5:10: fatal error: fdlibm.h: No such file or directory
>     5 | #include "fdlibm.h"
>       |          ^~~~~~~~~~