0.9.30.1 review - svn 24052
Rob Landley
rob at landley.net
Sat Feb 14 23:00:06 UTC 2009
Svn 24052 is a poorly-defined patch mixes at least four seemingly unrelated
things:
1) It adds UCLIBC_HAS_SSP support to various makefile like nsl and libresolv.
2) It uses #if 0/#endif to chop out a largeish portion of
include/sys/sysinfo.h (no idea why), and it adds 8 zillion #ifdef
__UCLIBC_LINUX_SPECIFIC__ things when linux is the only target uClibc has
historically _ever_ supported.
3) It renames a local variable in libc/inet/ifaddrs.c:map_newlink() from
"index" to "idx".
4) It adds and #ifdef __UCLIBC_LINUX_SPECIFIC__ to a lot of places that
already have a __USE_GNU. Apparently, this needed a synonym.
I'm confused why UCLIBC_LINUX_SPECIFIC exists in the first place since Linux
is the only platform uClibc has ever supported and you can't build the sucker
without Linux kernel headers, but I guess it's just one more way of defining a
set of optional functionality to remove for size reasons. But in this case,
it's adding stuff to _headers_ (making them larger, not removing any code from
the binary).
I also note that the help for the config entry for UCLIBC_LINUX_SPECIFIC from
0.9.30 provides an explicit list of functions to be inexplicably removed:
fstatfs(), inotify_*(), ioperm(), iopl(), madvise(), modify_ldt(),
personality(), ppoll(), setresuid(), signalfd(). This presumably extends that
list (one of the few places it's not merely a synonym for __USE_GNU is where
it inexplicably chops out vhangup()), but this patch does not modify that help
entry.
This patch is a bugfix? For _what_?
Rob
More information about the uClibc
mailing list