[Bug 1742] Warnings in system headers

[bugzilla-daemon at rtems.org]

https://www.rtems.org/bugzilla/show_bug.cgi?id=1742

--- Comment #10 from Ralf Corsepius <ralf.corsepius at rtems.org> 2011-02-24 09:45:23 CST ---
(In reply to comment #9)
> Ralf, thanks for the explanation.  I now understand the problem a bit more.

> This statement is not true, and the key to solve the warnings.
Once again: What is the bug you are trying to solve?

AFAIU, the warnings you see originate from you using silly warnings-level.

>  It seems that I
> was not the only one who did not understand the cdefs.h problem.

> ~/rtems/cpukit > grep -r 'include.*<sys.cdefs.h' .
> ./libcsupport/include/sys/statvfs.h:#include <sys/cdefs.h>
> ./libmisc/shell/vwarnx.c:#include <sys/cdefs.h>
> ./libmisc/shell/pwcache.c:#include <sys/cdefs.h>
> ./libmisc/shell/fts.h:#include <sys/cdefs.h>
> ./libmisc/shell/filemode.c:#include <sys/cdefs.h>
> ./libmisc/shell/dd-misc.c:#include <sys/cdefs.h>
> ./libmisc/shell/hexdump-parse.c:#include <sys/cdefs.h>
> ./libmisc/shell/hexdump-odsyntax.c:#include <sys/cdefs.h>
> ./libmisc/shell/verr.c:#include <sys/cdefs.h>
> ./libmisc/shell/utils-cp.c:#include <sys/cdefs.h>
> ./libmisc/shell/cmp-ls.c:#include <sys/cdefs.h>
> ./libmisc/shell/hexsyntax.c:#include <sys/cdefs.h>
> ./libmisc/shell/main_mknod.c:#include <sys/cdefs.h>
> ./libmisc/shell/dd-conv_tab.c:#include <sys/cdefs.h>
> ./libmisc/shell/fts.c:#include <sys/cdefs.h>
> ./libmisc/shell/main_mv.c:#include <sys/cdefs.h>
> ./libmisc/shell/dd-position.c:#include <sys/cdefs.h>
> ./libmisc/shell/dd-args.c:#include <sys/cdefs.h>
> ./libmisc/shell/vis.h:#include <sys/cdefs.h>
> ./libmisc/shell/extern-ls.h:#include <sys/cdefs.h>
> ./libmisc/shell/utils-ls.c:#include <sys/cdefs.h>
> ./libmisc/shell/main_dd.c:#include <sys/cdefs.h>
> ./libmisc/shell/verrx.c:#include <sys/cdefs.h>
> ./libmisc/shell/print-ls.c:#include <sys/cdefs.h>
> ./libmisc/shell/hexdump-conv.c:#include <sys/cdefs.h>
> ./libmisc/shell/warnx.c:#include <sys/cdefs.h>
> ./libmisc/shell/mknod-pack_dev.c:#include <sys/cdefs.h>
> ./libmisc/shell/vwarn.c:#include <sys/cdefs.h>
> ./libmisc/shell/main_hexdump.c:#include <sys/cdefs.h>
> ./libmisc/shell/warn.c:#include <sys/cdefs.h>
> ./libmisc/shell/main_rm.c:#include <sys/cdefs.h>
> ./libmisc/shell/extern-cp.h:#include <sys/cdefs.h>
> ./libmisc/shell/err.c:#include <sys/cdefs.h>
> ./libmisc/shell/err.h:#include <sys/cdefs.h>
> ./libmisc/shell/dd-conv.c:#include <sys/cdefs.h>
> ./libmisc/shell/vis.c:#include <sys/cdefs.h>
> ./libmisc/shell/main_ls.c:#include <sys/cdefs.h>
> ./libmisc/shell/hexdump-display.c:#include <sys/cdefs.h>
> ./libmisc/shell/main_ln.c:#include <sys/cdefs.h>
> ./libmisc/shell/errx.c:#include <sys/cdefs.h>
> ./telnetd/des.c:#include <sys/cdefs.h>

<sigh/> These all are files which did not support the posix BSP, i.e. defacto
were newlib-only.

> In particular:
> 
> ./libcsupport/include/sys/statvfs.h:#include <sys/cdefs.h>

Except that this file is multiply broken (missing C++ guards, superflous
include <sys/cdefs.h>) I don't see what's would make this file special.

> This causes the warning (e.g. included from <rtems/confdefs.h>).
How to reproduce this warning?

To bring this all to a sudden death, I am gradually leaning to what I mentioned
in my last response: 

To eliminate rtems/bsd/ and to tie RTEMS very tightly to newlib.

Joel will yell, you will yell, but it seems what you want.

-- 
Configure bugmail: https://www.rtems.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.