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

Corinna Vinschen <[email protected]>
Newsgroups gmane.comp.lib.newlib
Message-ID <[email protected]>
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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.