Re: [PATCH 0/7] newlib: reduce number of configure scripts

[Corinna Vinschen <vinschen@redhat.com>]

On Jan 23 01:04, Mike Frysinger wrote:
> This flattens the recursive configure logic quite a bit in the newlib
> tree.  I'll let the delta speak for itself.
> 
> Before:
> 	$ find newlib/ -name configure | wc -l
> 	113
> After:
> 	$ find newlib/ -name configure | wc -l
> 	13
> 
> The strategy largely is about killing off most of the stub configure
> trampolines in the libm & libc machine/ & sys/ trees.  I think of those
> 100 scripts, only 5 were doing something unique (checking preprocessor
> defines), while all the rest existed purely to generate a Makefile.  If
> we move that makefile generation logic into the respective top-level
> dirs, we can kill off entire subpaths.
> 
> Considering how messy the remaining libc/ sys/phoenix/ and sys/linux/
> subdirs are, and I don't use or care about those targets, I don't think
> I'll try and fold the libc/ confiugre logic any more.  But in spite of
> those, I think this patch series is a pretty big win without leaving
> the status quo in a bad way.
> 
> Mike Frysinger (7):
>   newlib: libm: merge machine/ trampoline up a level
>   newlib: libm: merge machine/ configure scripts up a level
>   newlib: libc: merge machine/ trampoline up a level
>   newlib: libc: merge sys/ trampoline up a level
>   newlib: libc: install CRT0 straight out of subdir
>   newlib: libc: merge most sys/ configure scripts up a level
>   newlib: libc: merge machine/ configure scripts up a level

I tested this additionally with Cygwin as target and from what I can
tell, this looks good.  Please push.


Thanks,
Corinna