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