Bug 1530 - add -Werror flags to C++ compiler
Summary: add -Werror flags to C++ compiler
Status: RESOLVED FIXED
Alias: None
Product: frysk
Classification: Unclassified
Component: general (show other bugs)
Version: unspecified
: P1 normal
Target Milestone: ---
Assignee: Andrew Cagney
URL:
Keywords:
Depends on:
Blocks: 2235
  Show dependency tree / graph
 
Reported: 2005-10-22 22:10 UTC by Andrew Cagney
Modified: 2006-01-31 16:02 UTC (History)
0 users

See Also:
Host:
Target:
Build:
Last reconfirmed:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Cagney 2005-10-22 22:10:39 UTC 
especially -Wformat
Comment 1 Andrew Cagney 2005-10-23 03:44:46 UTC 
Turns out it was a bug in common/Makefile.rules.  Due to a typo ($(top_buildir)
vs $(top_builddir)), AM_CXXFLAGS expanded to -I -Werror.

Index: frysk-imports/tests/ChangeLog
2005-10-22  Andrew Cagney  <cagney@redhat.com>

        * Makefile.am: Add copyright.
        (werror/java.sh, werror/cxx.sh): New targets.
        (werror/FAIL): Delete.
        * werror/Werror.cxx: New file, contains -Wformat error.

Index: frysk-sys/common/ChangeLog
2005-10-22  Andrew Cagney  <cagney@redhat.com>

        * Makefile.rules (AM_CXXFLAGS): Fix -I argument, $(top_builddir),
        not $(top_buildir).

Index: frysk-sys/frysk/sys/ChangeLog
2005-10-22  Andrew Cagney  <cagney@redhat.com>

        * cni/Errno.cxx (throwRuntimeException): New function.
        (vajprintf): New function, use throughout.
        * cni/Errno.hxx (throwRuntimeException): Declare variant with
        message, suffix, and val parameters.
        * cni/Wait.cxx (processStatus): Use.