[PATCH 1/3] platform: fix missing strchrnul

Daniel Borca dborca at yahoo.com
Wed Nov 27 11:14:53 UTC 2013



Matthias Andree wrote:
>
> The whole #ifdef approach is massively broken and prone to mis-guess.
>
> It denies one of the most basic considerations, and that is: operating
> systems change over time, and add features missing from earlier
> versions.  Looking at the operating system is insufficient.
>
> [snip]
>
> Note I am not saying you need to use a fully-fledged autoconf approach,
> lighter-weight alternatives have been seen in the wild, and possibly it
> suffices - for FreeBSD - to have a separate header (.h) file that checks
> the major FreeBSD version and #defines a few HAVE_SOMEFUNCTION or leaves
> it #undef'd, so the actual code can then

That just means the cpp dancing is moved elsewhere.  Someone needs to
maintain that header, because OSes <quote> change over time </quote>.

> on his hands, so some concept similar to autoconf might work in more
> places with less human work and more machine work during the build.

I agree, autoconf is the best approach.  But *someone* needs to write
it.

-dborca


More information about the busybox mailing list