* [PATCH 2/2] [pr32193] libgomp: add a --enable-werror configure flag
2012-12-23 23:30 [PATCH 1/2] [pr53679] libgo: add a --enable-werror configure flag Mike Frysinger
@ 2012-12-23 23:30 ` Mike Frysinger
2013-01-15 14:56 ` [PATCH 1/2] [pr53679] libgo: " Ian Lance Taylor
1 sibling, 0 replies; 11+ messages in thread
From: Mike Frysinger @ 2012-12-23 23:30 UTC (permalink / raw)
To: gcc-patches
URL: http://gcc.gnu.org/PR32193
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
2012-12-23 Mike Frysinger <vapier@gentoo.org>
PR libgomp/32193
* configure.ac: Call AC_ARG_ENABLE(werror).
(XCFLAGS): Add -Werror when enable_werror is not no.
* configure: Regenerated.
---
libgomp/configure | 16 +++++++++++++---
libgomp/configure.ac | 7 ++++++-
2 files changed, 19 insertions(+), 4 deletions(-)
diff --git a/libgomp/configure b/libgomp/configure
index 238b1af..3add57f 100755
--- a/libgomp/configure
+++ b/libgomp/configure
@@ -760,6 +760,7 @@ enable_version_specific_runtime_libs
enable_generated_files_in_srcdir
enable_multilib
enable_dependency_tracking
+enable_werror
enable_shared
enable_static
with_pic
@@ -1410,6 +1411,7 @@ Optional Features:
--enable-multilib build many library versions (default)
--disable-dependency-tracking speeds up one-time build
--enable-dependency-tracking do not reject slow dependency extractors
+ --enable-werror turns on -Werror [default=yes]
--enable-shared[=PKGS] build shared libraries [default=yes]
--enable-static[=PKGS] build static libraries [default=yes]
--enable-fast-install[=PKGS]
@@ -4280,9 +4282,17 @@ fi
# in both places for now and restore CFLAGS at the end of config.
save_CFLAGS="$CFLAGS"
+# Check whether --enable-werror was given.
+if test "${enable_werror+set}" = set; then :
+ enableval=$enable_werror;
+fi
+
# Add -Wall -Werror if we are using GCC.
if test "x$GCC" = "xyes"; then
- XCFLAGS="$XCFLAGS -Wall -Werror"
+ XCFLAGS="$XCFLAGS -Wall"
+ if test "x$enable_werror" != "xno"; then
+ XCFLAGS="$XCFLAGS -Werror"
+ fi
fi
# Find other programs we need.
@@ -11088,7 +11098,7 @@ else
lt_dlunknown=0; lt_dlno_uscore=1; lt_dlneed_uscore=2
lt_status=$lt_dlunknown
cat > conftest.$ac_ext <<_LT_EOF
-#line 11091 "configure"
+#line 11101 "configure"
#include "confdefs.h"
#if HAVE_DLFCN_H
@@ -11194,7 +11204,7 @@ else
lt_dlunknown=0; lt_dlno_uscore=1; lt_dlneed_uscore=2
lt_status=$lt_dlunknown
cat > conftest.$ac_ext <<_LT_EOF
-#line 11197 "configure"
+#line 11207 "configure"
#include "confdefs.h"
#if HAVE_DLFCN_H
diff --git a/libgomp/configure.ac b/libgomp/configure.ac
index d87ed29..3b9cd4b 100644
--- a/libgomp/configure.ac
+++ b/libgomp/configure.ac
@@ -113,9 +113,14 @@ AC_SUBST(CFLAGS)
# in both places for now and restore CFLAGS at the end of config.
save_CFLAGS="$CFLAGS"
+AC_ARG_ENABLE(werror, [AS_HELP_STRING([--enable-werror],
+ [turns on -Werror @<:@default=yes@:>@])])
# Add -Wall -Werror if we are using GCC.
if test "x$GCC" = "xyes"; then
- XCFLAGS="$XCFLAGS -Wall -Werror"
+ XCFLAGS="$XCFLAGS -Wall"
+ if test "x$enable_werror" != "xno"; then
+ XCFLAGS="$XCFLAGS -Werror"
+ fi
fi
# Find other programs we need.
--
1.8.0
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 1/2] [pr53679] libgo: add a --enable-werror configure flag
@ 2012-12-23 23:30 Mike Frysinger
2012-12-23 23:30 ` [PATCH 2/2] [pr32193] libgomp: " Mike Frysinger
2013-01-15 14:56 ` [PATCH 1/2] [pr53679] libgo: " Ian Lance Taylor
0 siblings, 2 replies; 11+ messages in thread
From: Mike Frysinger @ 2012-12-23 23:30 UTC (permalink / raw)
To: gcc-patches
URL: http://gcc.gnu.org/PR53679
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
---
libgo/configure | 15 ++++++++++++---
libgo/configure.ac | 7 +++++--
2 files changed, 17 insertions(+), 5 deletions(-)
diff --git a/libgo/configure b/libgo/configure
index 04fa89d..49cc4a9 100755
--- a/libgo/configure
+++ b/libgo/configure
@@ -809,6 +809,7 @@ enable_static
with_pic
enable_fast_install
enable_libtool_lock
+enable_werror
enable_version_specific_runtime_libs
with_libffi
with_system_libunwind
@@ -1449,6 +1450,7 @@ Optional Features:
--enable-fast-install[=PKGS]
optimize for fast installation [default=yes]
--disable-libtool-lock avoid locking (might break parallel builds)
+ --enable-werror turns on -Werror [default=yes]
--enable-version-specific-runtime-libs
Specify that runtime libraries should be installed
in a compiler-specific directory
@@ -11102,7 +11104,7 @@ else
lt_dlunknown=0; lt_dlno_uscore=1; lt_dlneed_uscore=2
lt_status=$lt_dlunknown
cat > conftest.$ac_ext <<_LT_EOF
-#line 11105 "configure"
+#line 11107 "configure"
#include "confdefs.h"
#if HAVE_DLFCN_H
@@ -11208,7 +11210,7 @@ else
lt_dlunknown=0; lt_dlno_uscore=1; lt_dlneed_uscore=2
lt_status=$lt_dlunknown
cat > conftest.$ac_ext <<_LT_EOF
-#line 11211 "configure"
+#line 11213 "configure"
#include "confdefs.h"
#if HAVE_DLFCN_H
@@ -13386,7 +13388,14 @@ done
WARN_FLAGS='-Wall -Wextra -Wwrite-strings -Wcast-qual'
-WERROR="-Werror"
+# Check whether --enable-werror was given.
+if test "${enable_werror+set}" = set; then :
+ enableval=$enable_werror;
+fi
+
+if test "x$enable_werror" != "xno"; then
+ WERROR="-Werror"
+fi
glibgo_toolexecdir=no
diff --git a/libgo/configure.ac b/libgo/configure.ac
index 8cde50b..63d8cbc 100644
--- a/libgo/configure.ac
+++ b/libgo/configure.ac
@@ -50,8 +50,11 @@ AC_PROG_AWK
WARN_FLAGS='-Wall -Wextra -Wwrite-strings -Wcast-qual'
AC_SUBST(WARN_FLAGS)
-dnl FIXME: This should be controlled by --enable-maintainer-mode.
-WERROR="-Werror"
+AC_ARG_ENABLE(werror, [AS_HELP_STRING([--enable-werror],
+ [turns on -Werror @<:@default=yes@:>@])])
+if test "x$enable_werror" != "xno"; then
+ WERROR="-Werror"
+fi
AC_SUBST(WERROR)
glibgo_toolexecdir=no
--
1.8.0
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] [pr53679] libgo: add a --enable-werror configure flag
2012-12-23 23:30 [PATCH 1/2] [pr53679] libgo: add a --enable-werror configure flag Mike Frysinger
2012-12-23 23:30 ` [PATCH 2/2] [pr32193] libgomp: " Mike Frysinger
@ 2013-01-15 14:56 ` Ian Lance Taylor
2013-01-15 17:42 ` Mike Frysinger
1 sibling, 1 reply; 11+ messages in thread
From: Ian Lance Taylor @ 2013-01-15 14:56 UTC (permalink / raw)
To: Mike Frysinger; +Cc: gcc-patches
On Sun, Dec 23, 2012 at 3:30 PM, Mike Frysinger <vapier@gentoo.org> wrote:
> diff --git a/libgo/configure.ac b/libgo/configure.ac
> index 8cde50b..63d8cbc 100644
> --- a/libgo/configure.ac
> +++ b/libgo/configure.ac
> @@ -50,8 +50,11 @@ AC_PROG_AWK
> WARN_FLAGS='-Wall -Wextra -Wwrite-strings -Wcast-qual'
> AC_SUBST(WARN_FLAGS)
>
> -dnl FIXME: This should be controlled by --enable-maintainer-mode.
> -WERROR="-Werror"
> +AC_ARG_ENABLE(werror, [AS_HELP_STRING([--enable-werror],
> + [turns on -Werror @<:@default=yes@:>@])])
> +if test "x$enable_werror" != "xno"; then
> + WERROR="-Werror"
> +fi
> AC_SUBST(WERROR)
>
> glibgo_toolexecdir=no
Can you say something about when you needed this? What errors were you seeing?
Ian
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] [pr53679] libgo: add a --enable-werror configure flag
2013-01-15 14:56 ` [PATCH 1/2] [pr53679] libgo: " Ian Lance Taylor
@ 2013-01-15 17:42 ` Mike Frysinger
2013-01-26 0:14 ` Ian Lance Taylor
0 siblings, 1 reply; 11+ messages in thread
From: Mike Frysinger @ 2013-01-15 17:42 UTC (permalink / raw)
To: Ian Lance Taylor; +Cc: gcc-patches
[-- Attachment #1: Type: Text/Plain, Size: 1778 bytes --]
On Tuesday 15 January 2013 09:56:06 Ian Lance Taylor wrote:
> On Sun, Dec 23, 2012 at 3:30 PM, Mike Frysinger <vapier@gentoo.org> wrote:
> > diff --git a/libgo/configure.ac b/libgo/configure.ac
> > index 8cde50b..63d8cbc 100644
> > --- a/libgo/configure.ac
> > +++ b/libgo/configure.ac
> > @@ -50,8 +50,11 @@ AC_PROG_AWK
> >
> > WARN_FLAGS='-Wall -Wextra -Wwrite-strings -Wcast-qual'
> > AC_SUBST(WARN_FLAGS)
> >
> > -dnl FIXME: This should be controlled by --enable-maintainer-mode.
> > -WERROR="-Werror"
> > +AC_ARG_ENABLE(werror, [AS_HELP_STRING([--enable-werror],
> > + [turns on -Werror
> > @<:@default=yes@:>@])]) +if test "x$enable_werror" != "xno"; then
> > + WERROR="-Werror"
> > +fi
> >
> > AC_SUBST(WERROR)
> >
> > glibgo_toolexecdir=no
>
> Can you say something about when you needed this? What errors were you
> seeing?
the referenced PR describes one:
/build/src/gcc-4.7.1/libgo/runtime/print.c: In function 'gwrite':
/build/src/gcc-4.7.1/libgo/runtime/print.c:20:3: error: ignoring return value
of 'write', declared with attribute warn_unused_result [-Werror=unused-result]
cc1: all warnings being treated as errors
this bites distros that enable security settings by default (such as fortify
and ssp). but ignoring even that, i don't believe releases should build all
the time with -Werror -- i'm fine with defaulting to on as long as there is a
configure flag to turn it off which is what this does like is already handled in
much of the sourceware tree. -Werror is great for development, but sucks when
deployed on actual systems. the assumptions made at time of checkin rarely
stay constant forever (in this case, a changing lib C can easily break it).
-mike
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] [pr53679] libgo: add a --enable-werror configure flag
2013-01-15 17:42 ` Mike Frysinger
@ 2013-01-26 0:14 ` Ian Lance Taylor
2013-01-26 3:21 ` Mike Frysinger
0 siblings, 1 reply; 11+ messages in thread
From: Ian Lance Taylor @ 2013-01-26 0:14 UTC (permalink / raw)
To: Mike Frysinger; +Cc: gcc-patches
On Tue, Jan 15, 2013 at 9:45 AM, Mike Frysinger <vapier@gentoo.org> wrote:
> On Tuesday 15 January 2013 09:56:06 Ian Lance Taylor wrote:
>> On Sun, Dec 23, 2012 at 3:30 PM, Mike Frysinger <vapier@gentoo.org> wrote:
>> > diff --git a/libgo/configure.ac b/libgo/configure.ac
>> > index 8cde50b..63d8cbc 100644
>> > --- a/libgo/configure.ac
>> > +++ b/libgo/configure.ac
>> > @@ -50,8 +50,11 @@ AC_PROG_AWK
>> >
>> > WARN_FLAGS='-Wall -Wextra -Wwrite-strings -Wcast-qual'
>> > AC_SUBST(WARN_FLAGS)
>> >
>> > -dnl FIXME: This should be controlled by --enable-maintainer-mode.
>> > -WERROR="-Werror"
>> > +AC_ARG_ENABLE(werror, [AS_HELP_STRING([--enable-werror],
>> > + [turns on -Werror
>> > @<:@default=yes@:>@])]) +if test "x$enable_werror" != "xno"; then
>> > + WERROR="-Werror"
>> > +fi
>> >
>> > AC_SUBST(WERROR)
>> >
>> > glibgo_toolexecdir=no
>>
>> Can you say something about when you needed this? What errors were you
>> seeing?
>
> the referenced PR describes one:
> /build/src/gcc-4.7.1/libgo/runtime/print.c: In function 'gwrite':
> /build/src/gcc-4.7.1/libgo/runtime/print.c:20:3: error: ignoring return value
> of 'write', declared with attribute warn_unused_result [-Werror=unused-result]
> cc1: all warnings being treated as errors
>
> this bites distros that enable security settings by default (such as fortify
> and ssp). but ignoring even that, i don't believe releases should build all
> the time with -Werror -- i'm fine with defaulting to on as long as there is a
> configure flag to turn it off which is what this does like is already handled in
> much of the sourceware tree. -Werror is great for development, but sucks when
> deployed on actual systems. the assumptions made at time of checkin rarely
> stay constant forever (in this case, a changing lib C can easily break it).
> -mike
Thanks for the explanation.
Committed to mainline.
Ian
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] [pr53679] libgo: add a --enable-werror configure flag
2013-01-26 0:14 ` Ian Lance Taylor
@ 2013-01-26 3:21 ` Mike Frysinger
2013-01-27 2:40 ` Ian Lance Taylor
0 siblings, 1 reply; 11+ messages in thread
From: Mike Frysinger @ 2013-01-26 3:21 UTC (permalink / raw)
To: Ian Lance Taylor; +Cc: gcc-patches
[-- Attachment #1: Type: Text/Plain, Size: 2139 bytes --]
On Friday 25 January 2013 19:13:55 Ian Lance Taylor wrote:
> On Tue, Jan 15, 2013 at 9:45 AM, Mike Frysinger wrote:
> > On Tuesday 15 January 2013 09:56:06 Ian Lance Taylor wrote:
> >> On Sun, Dec 23, 2012 at 3:30 PM, Mike Frysinger wrote:
> >> > diff --git a/libgo/configure.ac b/libgo/configure.ac
> >> > index 8cde50b..63d8cbc 100644
> >> > --- a/libgo/configure.ac
> >> > +++ b/libgo/configure.ac
> >> > @@ -50,8 +50,11 @@ AC_PROG_AWK
> >> >
> >> > WARN_FLAGS='-Wall -Wextra -Wwrite-strings -Wcast-qual'
> >> > AC_SUBST(WARN_FLAGS)
> >> >
> >> > -dnl FIXME: This should be controlled by --enable-maintainer-mode.
> >> > -WERROR="-Werror"
> >> > +AC_ARG_ENABLE(werror, [AS_HELP_STRING([--enable-werror],
> >> > + [turns on -Werror
> >> > @<:@default=yes@:>@])]) +if test "x$enable_werror" != "xno"; then
> >> > + WERROR="-Werror"
> >> > +fi
> >> >
> >> > AC_SUBST(WERROR)
> >> >
> >> > glibgo_toolexecdir=no
> >>
> >> Can you say something about when you needed this? What errors were you
> >> seeing?
> >
> > the referenced PR describes one:
> > /build/src/gcc-4.7.1/libgo/runtime/print.c: In function 'gwrite':
> > /build/src/gcc-4.7.1/libgo/runtime/print.c:20:3: error: ignoring return
> > value of 'write', declared with attribute warn_unused_result
> > [-Werror=unused-result] cc1: all warnings being treated as errors
> >
> > this bites distros that enable security settings by default (such as
> > fortify and ssp). but ignoring even that, i don't believe releases
> > should build all the time with -Werror -- i'm fine with defaulting to on
> > as long as there is a configure flag to turn it off which is what this
> > does like is already handled in much of the sourceware tree. -Werror is
> > great for development, but sucks when deployed on actual systems. the
> > assumptions made at time of checkin rarely stay constant forever (in
> > this case, a changing lib C can easily break it). -mike
>
> Thanks for the explanation.
>
> Committed to mainline.
thanks! mind if i commit it to gcc-4.6 and gcc-4.7 too ?
-mike
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] [pr53679] libgo: add a --enable-werror configure flag
2013-01-26 3:21 ` Mike Frysinger
@ 2013-01-27 2:40 ` Ian Lance Taylor
2013-03-05 0:10 ` Mike Frysinger
0 siblings, 1 reply; 11+ messages in thread
From: Ian Lance Taylor @ 2013-01-27 2:40 UTC (permalink / raw)
To: Mike Frysinger; +Cc: gcc-patches
On Fri, Jan 25, 2013 at 7:20 PM, Mike Frysinger <vapier@gentoo.org> wrote:
> On Friday 25 January 2013 19:13:55 Ian Lance Taylor wrote:
>> On Tue, Jan 15, 2013 at 9:45 AM, Mike Frysinger wrote:
>> > On Tuesday 15 January 2013 09:56:06 Ian Lance Taylor wrote:
>> >> On Sun, Dec 23, 2012 at 3:30 PM, Mike Frysinger wrote:
>> >> > diff --git a/libgo/configure.ac b/libgo/configure.ac
>> >> > index 8cde50b..63d8cbc 100644
>> >> > --- a/libgo/configure.ac
>> >> > +++ b/libgo/configure.ac
>> >> > @@ -50,8 +50,11 @@ AC_PROG_AWK
>> >> >
>> >> > WARN_FLAGS='-Wall -Wextra -Wwrite-strings -Wcast-qual'
>> >> > AC_SUBST(WARN_FLAGS)
>> >> >
>> >> > -dnl FIXME: This should be controlled by --enable-maintainer-mode.
>> >> > -WERROR="-Werror"
>> >> > +AC_ARG_ENABLE(werror, [AS_HELP_STRING([--enable-werror],
>> >> > + [turns on -Werror
>> >> > @<:@default=yes@:>@])]) +if test "x$enable_werror" != "xno"; then
>> >> > + WERROR="-Werror"
>> >> > +fi
>> >> >
>> >> > AC_SUBST(WERROR)
>> >> >
>> >> > glibgo_toolexecdir=no
>> >>
>> >> Can you say something about when you needed this? What errors were you
>> >> seeing?
>> >
>> > the referenced PR describes one:
>> > /build/src/gcc-4.7.1/libgo/runtime/print.c: In function 'gwrite':
>> > /build/src/gcc-4.7.1/libgo/runtime/print.c:20:3: error: ignoring return
>> > value of 'write', declared with attribute warn_unused_result
>> > [-Werror=unused-result] cc1: all warnings being treated as errors
>> >
>> > this bites distros that enable security settings by default (such as
>> > fortify and ssp). but ignoring even that, i don't believe releases
>> > should build all the time with -Werror -- i'm fine with defaulting to on
>> > as long as there is a configure flag to turn it off which is what this
>> > does like is already handled in much of the sourceware tree. -Werror is
>> > great for development, but sucks when deployed on actual systems. the
>> > assumptions made at time of checkin rarely stay constant forever (in
>> > this case, a changing lib C can easily break it). -mike
>>
>> Thanks for the explanation.
>>
>> Committed to mainline.
>
> thanks! mind if i commit it to gcc-4.6 and gcc-4.7 too ?
I certainly don't mind. You should probably get agreement from the
release managers although this seems safe enough.
Ian
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] [pr53679] libgo: add a --enable-werror configure flag
2013-01-27 2:40 ` Ian Lance Taylor
@ 2013-03-05 0:10 ` Mike Frysinger
2013-03-05 5:31 ` Ian Lance Taylor
0 siblings, 1 reply; 11+ messages in thread
From: Mike Frysinger @ 2013-03-05 0:10 UTC (permalink / raw)
To: Ian Lance Taylor; +Cc: gcc-patches
[-- Attachment #1: Type: Text/Plain, Size: 2632 bytes --]
On Saturday 26 January 2013 21:40:44 Ian Lance Taylor wrote:
> On Fri, Jan 25, 2013 at 7:20 PM, Mike Frysinger wrote:
> > On Friday 25 January 2013 19:13:55 Ian Lance Taylor wrote:
> >> On Tue, Jan 15, 2013 at 9:45 AM, Mike Frysinger wrote:
> >> > On Tuesday 15 January 2013 09:56:06 Ian Lance Taylor wrote:
> >> >> On Sun, Dec 23, 2012 at 3:30 PM, Mike Frysinger wrote:
> >> >> > diff --git a/libgo/configure.ac b/libgo/configure.ac
> >> >> > index 8cde50b..63d8cbc 100644
> >> >> > --- a/libgo/configure.ac
> >> >> > +++ b/libgo/configure.ac
> >> >> > @@ -50,8 +50,11 @@ AC_PROG_AWK
> >> >> >
> >> >> > WARN_FLAGS='-Wall -Wextra -Wwrite-strings -Wcast-qual'
> >> >> > AC_SUBST(WARN_FLAGS)
> >> >> >
> >> >> > -dnl FIXME: This should be controlled by --enable-maintainer-mode.
> >> >> > -WERROR="-Werror"
> >> >> > +AC_ARG_ENABLE(werror, [AS_HELP_STRING([--enable-werror],
> >> >> > + [turns on -Werror
> >> >> > @<:@default=yes@:>@])]) +if test "x$enable_werror" != "xno"; then
> >> >> > + WERROR="-Werror"
> >> >> > +fi
> >> >> >
> >> >> > AC_SUBST(WERROR)
> >> >> >
> >> >> > glibgo_toolexecdir=no
> >> >>
> >> >> Can you say something about when you needed this? What errors were
> >> >> you seeing?
> >> >
> >> > the referenced PR describes one:
> >> > /build/src/gcc-4.7.1/libgo/runtime/print.c: In function 'gwrite':
> >> > /build/src/gcc-4.7.1/libgo/runtime/print.c:20:3: error: ignoring
> >> > return value of 'write', declared with attribute warn_unused_result
> >> > [-Werror=unused-result] cc1: all warnings being treated as errors
> >> >
> >> > this bites distros that enable security settings by default (such as
> >> > fortify and ssp). but ignoring even that, i don't believe releases
> >> > should build all the time with -Werror -- i'm fine with defaulting to
> >> > on as long as there is a configure flag to turn it off which is what
> >> > this does like is already handled in much of the sourceware tree.
> >> > -Werror is great for development, but sucks when deployed on actual
> >> > systems. the assumptions made at time of checkin rarely stay
> >> > constant forever (in this case, a changing lib C can easily break
> >> > it). -mike
> >>
> >> Thanks for the explanation.
> >>
> >> Committed to mainline.
> >
> > thanks! mind if i commit it to gcc-4.6 and gcc-4.7 too ?
>
> I certainly don't mind. You should probably get agreement from the
> release managers although this seems safe enough.
can you approve merges to the google branches ? that's really where i want
this :).
-mike
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] [pr53679] libgo: add a --enable-werror configure flag
2013-03-05 0:10 ` Mike Frysinger
@ 2013-03-05 5:31 ` Ian Lance Taylor
2013-03-06 15:49 ` Diego Novillo
0 siblings, 1 reply; 11+ messages in thread
From: Ian Lance Taylor @ 2013-03-05 5:31 UTC (permalink / raw)
To: Mike Frysinger; +Cc: gcc-patches, Diego Novillo
On Mon, Mar 4, 2013 at 4:11 PM, Mike Frysinger <vapier@gentoo.org> wrote:
> On Saturday 26 January 2013 21:40:44 Ian Lance Taylor wrote:
>> On Fri, Jan 25, 2013 at 7:20 PM, Mike Frysinger wrote:
>> > On Friday 25 January 2013 19:13:55 Ian Lance Taylor wrote:
>> >> On Tue, Jan 15, 2013 at 9:45 AM, Mike Frysinger wrote:
>> >> > On Tuesday 15 January 2013 09:56:06 Ian Lance Taylor wrote:
>> >> >> On Sun, Dec 23, 2012 at 3:30 PM, Mike Frysinger wrote:
>> >> >> > diff --git a/libgo/configure.ac b/libgo/configure.ac
>> >> >> > index 8cde50b..63d8cbc 100644
>> >> >> > --- a/libgo/configure.ac
>> >> >> > +++ b/libgo/configure.ac
>> >> >> > @@ -50,8 +50,11 @@ AC_PROG_AWK
>> >> >> >
>> >> >> > WARN_FLAGS='-Wall -Wextra -Wwrite-strings -Wcast-qual'
>> >> >> > AC_SUBST(WARN_FLAGS)
>> >> >> >
>> >> >> > -dnl FIXME: This should be controlled by --enable-maintainer-mode.
>> >> >> > -WERROR="-Werror"
>> >> >> > +AC_ARG_ENABLE(werror, [AS_HELP_STRING([--enable-werror],
>> >> >> > + [turns on -Werror
>> >> >> > @<:@default=yes@:>@])]) +if test "x$enable_werror" != "xno"; then
>> >> >> > + WERROR="-Werror"
>> >> >> > +fi
>> >> >> >
>> >> >> > AC_SUBST(WERROR)
>> >> >> >
>> >> >> > glibgo_toolexecdir=no
>> >> >>
>> >> >> Can you say something about when you needed this? What errors were
>> >> >> you seeing?
>> >> >
>> >> > the referenced PR describes one:
>> >> > /build/src/gcc-4.7.1/libgo/runtime/print.c: In function 'gwrite':
>> >> > /build/src/gcc-4.7.1/libgo/runtime/print.c:20:3: error: ignoring
>> >> > return value of 'write', declared with attribute warn_unused_result
>> >> > [-Werror=unused-result] cc1: all warnings being treated as errors
>> >> >
>> >> > this bites distros that enable security settings by default (such as
>> >> > fortify and ssp). but ignoring even that, i don't believe releases
>> >> > should build all the time with -Werror -- i'm fine with defaulting to
>> >> > on as long as there is a configure flag to turn it off which is what
>> >> > this does like is already handled in much of the sourceware tree.
>> >> > -Werror is great for development, but sucks when deployed on actual
>> >> > systems. the assumptions made at time of checkin rarely stay
>> >> > constant forever (in this case, a changing lib C can easily break
>> >> > it). -mike
>> >>
>> >> Thanks for the explanation.
>> >>
>> >> Committed to mainline.
>> >
>> > thanks! mind if i commit it to gcc-4.6 and gcc-4.7 too ?
>>
>> I certainly don't mind. You should probably get agreement from the
>> release managers although this seems safe enough.
>
> can you approve merges to the google branches ? that's really where i want
> this :).
No, I don't work on those branches. Sorry.
CC'ed dnovillo@google.com.
Ian
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] [pr53679] libgo: add a --enable-werror configure flag
2013-03-05 5:31 ` Ian Lance Taylor
@ 2013-03-06 15:49 ` Diego Novillo
2013-03-07 6:59 ` Mike Frysinger
0 siblings, 1 reply; 11+ messages in thread
From: Diego Novillo @ 2013-03-06 15:49 UTC (permalink / raw)
To: Ian Lance Taylor; +Cc: Mike Frysinger, gcc-patches
On Tue, Mar 5, 2013 at 12:31 AM, Ian Lance Taylor <iant@google.com> wrote:
>
> On Mon, Mar 4, 2013 at 4:11 PM, Mike Frysinger <vapier@gentoo.org> wrote:
> > On Saturday 26 January 2013 21:40:44 Ian Lance Taylor wrote:
> >> On Fri, Jan 25, 2013 at 7:20 PM, Mike Frysinger wrote:
> >> > On Friday 25 January 2013 19:13:55 Ian Lance Taylor wrote:
> >> >> On Tue, Jan 15, 2013 at 9:45 AM, Mike Frysinger wrote:
> >> >> > On Tuesday 15 January 2013 09:56:06 Ian Lance Taylor wrote:
> >> >> >> On Sun, Dec 23, 2012 at 3:30 PM, Mike Frysinger wrote:
> >> >> >> > diff --git a/libgo/configure.ac b/libgo/configure.ac
> >> >> >> > index 8cde50b..63d8cbc 100644
> >> >> >> > --- a/libgo/configure.ac
> >> >> >> > +++ b/libgo/configure.ac
> >> >> >> > @@ -50,8 +50,11 @@ AC_PROG_AWK
> >> >> >> >
> >> >> >> > WARN_FLAGS='-Wall -Wextra -Wwrite-strings -Wcast-qual'
> >> >> >> > AC_SUBST(WARN_FLAGS)
> >> >> >> >
> >> >> >> > -dnl FIXME: This should be controlled by
> >> >> >> > --enable-maintainer-mode.
> >> >> >> > -WERROR="-Werror"
> >> >> >> > +AC_ARG_ENABLE(werror, [AS_HELP_STRING([--enable-werror],
> >> >> >> > + [turns on -Werror
> >> >> >> > @<:@default=yes@:>@])]) +if test "x$enable_werror" != "xno";
> >> >> >> > then
> >> >> >> > + WERROR="-Werror"
> >> >> >> > +fi
> >> >> >> >
> >> >> >> > AC_SUBST(WERROR)
> >> >> >> >
> >> >> >> > glibgo_toolexecdir=no
> >> >> >>
> >> >> >> Can you say something about when you needed this? What errors
> >> >> >> were
> >> >> >> you seeing?
> >> >> >
> >> >> > the referenced PR describes one:
> >> >> > /build/src/gcc-4.7.1/libgo/runtime/print.c: In function 'gwrite':
> >> >> > /build/src/gcc-4.7.1/libgo/runtime/print.c:20:3: error: ignoring
> >> >> > return value of 'write', declared with attribute
> >> >> > warn_unused_result
> >> >> > [-Werror=unused-result] cc1: all warnings being treated as errors
> >> >> >
> >> >> > this bites distros that enable security settings by default (such
> >> >> > as
> >> >> > fortify and ssp). but ignoring even that, i don't believe
> >> >> > releases
> >> >> > should build all the time with -Werror -- i'm fine with defaulting
> >> >> > to
> >> >> > on as long as there is a configure flag to turn it off which is
> >> >> > what
> >> >> > this does like is already handled in much of the sourceware tree.
> >> >> > -Werror is great for development, but sucks when deployed on
> >> >> > actual
> >> >> > systems. the assumptions made at time of checkin rarely stay
> >> >> > constant forever (in this case, a changing lib C can easily break
> >> >> > it). -mike
> >> >>
> >> >> Thanks for the explanation.
> >> >>
> >> >> Committed to mainline.
> >> >
> >> > thanks! mind if i commit it to gcc-4.6 and gcc-4.7 too ?
> >>
> >> I certainly don't mind. You should probably get agreement from the
> >> release managers although this seems safe enough.
> >
> > can you approve merges to the google branches ? that's really where i
> > want
> > this :).
>
> No, I don't work on those branches. Sorry.
I committed this patch to google/gcc-4_7 rev 196494. Feel free to
commit to google/gcc-4_6 if you need to.
Diego.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] [pr53679] libgo: add a --enable-werror configure flag
2013-03-06 15:49 ` Diego Novillo
@ 2013-03-07 6:59 ` Mike Frysinger
0 siblings, 0 replies; 11+ messages in thread
From: Mike Frysinger @ 2013-03-07 6:59 UTC (permalink / raw)
To: Diego Novillo; +Cc: Ian Lance Taylor, gcc-patches
[-- Attachment #1: Type: Text/Plain, Size: 3479 bytes --]
On Wednesday 06 March 2013 10:49:23 Diego Novillo wrote:
> On Tue, Mar 5, 2013 at 12:31 AM, Ian Lance Taylor <iant@google.com> wrote:
> > On Mon, Mar 4, 2013 at 4:11 PM, Mike Frysinger <vapier@gentoo.org> wrote:
> > > On Saturday 26 January 2013 21:40:44 Ian Lance Taylor wrote:
> > >> On Fri, Jan 25, 2013 at 7:20 PM, Mike Frysinger wrote:
> > >> > On Friday 25 January 2013 19:13:55 Ian Lance Taylor wrote:
> > >> >> On Tue, Jan 15, 2013 at 9:45 AM, Mike Frysinger wrote:
> > >> >> > On Tuesday 15 January 2013 09:56:06 Ian Lance Taylor wrote:
> > >> >> >> On Sun, Dec 23, 2012 at 3:30 PM, Mike Frysinger wrote:
> > >> >> >> > diff --git a/libgo/configure.ac b/libgo/configure.ac
> > >> >> >> > index 8cde50b..63d8cbc 100644
> > >> >> >> > --- a/libgo/configure.ac
> > >> >> >> > +++ b/libgo/configure.ac
> > >> >> >> > @@ -50,8 +50,11 @@ AC_PROG_AWK
> > >> >> >> >
> > >> >> >> > WARN_FLAGS='-Wall -Wextra -Wwrite-strings -Wcast-qual'
> > >> >> >> > AC_SUBST(WARN_FLAGS)
> > >> >> >> >
> > >> >> >> > -dnl FIXME: This should be controlled by
> > >> >> >> > --enable-maintainer-mode.
> > >> >> >> > -WERROR="-Werror"
> > >> >> >> > +AC_ARG_ENABLE(werror, [AS_HELP_STRING([--enable-werror],
> > >> >> >> > + [turns on -Werror
> > >> >> >> > @<:@default=yes@:>@])]) +if test "x$enable_werror" != "xno";
> > >> >> >> > then
> > >> >> >> > + WERROR="-Werror"
> > >> >> >> > +fi
> > >> >> >> >
> > >> >> >> > AC_SUBST(WERROR)
> > >> >> >> >
> > >> >> >> > glibgo_toolexecdir=no
> > >> >> >>
> > >> >> >> Can you say something about when you needed this? What errors
> > >> >> >> were
> > >> >> >> you seeing?
> > >> >> >
> > >> >> > the referenced PR describes one:
> > >> >> > /build/src/gcc-4.7.1/libgo/runtime/print.c: In function 'gwrite':
> > >> >> > /build/src/gcc-4.7.1/libgo/runtime/print.c:20:3: error: ignoring
> > >> >> > return value of 'write', declared with attribute
> > >> >> > warn_unused_result
> > >> >> > [-Werror=unused-result] cc1: all warnings being treated as errors
> > >> >> >
> > >> >> > this bites distros that enable security settings by default (such
> > >> >> > as
> > >> >> > fortify and ssp). but ignoring even that, i don't believe
> > >> >> > releases
> > >> >> > should build all the time with -Werror -- i'm fine with
> > >> >> > defaulting to
> > >> >> > on as long as there is a configure flag to turn it off which is
> > >> >> > what
> > >> >> > this does like is already handled in much of the sourceware tree.
> > >> >> > -Werror is great for development, but sucks when deployed on
> > >> >> > actual
> > >> >> > systems. the assumptions made at time of checkin rarely stay
> > >> >> > constant forever (in this case, a changing lib C can easily break
> > >> >> > it). -mike
> > >> >>
> > >> >> Thanks for the explanation.
> > >> >>
> > >> >> Committed to mainline.
> > >> >
> > >> > thanks! mind if i commit it to gcc-4.6 and gcc-4.7 too ?
> > >>
> > >> I certainly don't mind. You should probably get agreement from the
> > >> release managers although this seems safe enough.
> > >
> > > can you approve merges to the google branches ? that's really where i
> > > want
> > > this :).
> >
> > No, I don't work on those branches. Sorry.
>
> I committed this patch to google/gcc-4_7 rev 196494. Feel free to
> commit to google/gcc-4_6 if you need to.
nope, we just need it in 4.7, thanks!
-mike
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2013-03-07 6:59 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-12-23 23:30 [PATCH 1/2] [pr53679] libgo: add a --enable-werror configure flag Mike Frysinger
2012-12-23 23:30 ` [PATCH 2/2] [pr32193] libgomp: " Mike Frysinger
2013-01-15 14:56 ` [PATCH 1/2] [pr53679] libgo: " Ian Lance Taylor
2013-01-15 17:42 ` Mike Frysinger
2013-01-26 0:14 ` Ian Lance Taylor
2013-01-26 3:21 ` Mike Frysinger
2013-01-27 2:40 ` Ian Lance Taylor
2013-03-05 0:10 ` Mike Frysinger
2013-03-05 5:31 ` Ian Lance Taylor
2013-03-06 15:49 ` Diego Novillo
2013-03-07 6:59 ` Mike Frysinger
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).