[Bug 1095] New: [PATCH] Add configure option for -Werror

[bugzilla-daemon at icedtea.classpath.org]

http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=1095

          Priority: P3
            Bug ID: 1095
          Assignee: unassigned at icedtea.classpath.org
           Summary: [PATCH] Add configure option for -Werror
          Severity: enhancement
    Classification: Unclassified
                OS: All
          Reporter: sera at gentoo.org
          Hardware: unspecified
            Status: NEW
           Version: 2.2.1
         Component: IcedTea
           Product: IcedTea

Created attachment 734
  --> http://icedtea.classpath.org/bugzilla/attachment.cgi?id=734&action=edit
add configure option warnings-are-errors

New gcc releases tend to add new warnings and -Werror will let the build fail
for no other reasen than the compiler beeing to new. This might well be desired
in some cases but might cause headaches another time. [1]

Currently the build fails for IcedTea 2.2.1 for some users with rather sane
FLAGS and gcc-4.7. [2]

The jdk build system already uses 2 variables to enable/diasble -Werror, there
are only few places which hardcode it's use.

The attached patch introduces a new configure option warnings-are-errors. If
unset nothing changes, ie. -Werror will be used in the same places as it would
have been before. If disabled -Werror won't be used anywhere and if enabled
IcedTea would set the variables COMPILER_WARNINGS_FATAL and
JAVAC_WARNINGS_FATAL.


[1] http://blog.flameeyes.eu/2009/02/25/future-proof-your-code-dont-use-werror
[2] https://bugs.gentoo.org/show_bug.cgi?id=418359

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20120725/96f33c91/attachment.html