From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff Layton Subject: Re: [PATCH] automake: revert -Werror by default Date: Thu, 14 Jun 2012 11:00:41 -0400 Message-ID: <20120614110041.1ff4f2db@corrin.poochiereds.net> References: <1339496358-27940-1-git-send-email-jlayton@samba.org> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: sjayaraman-IBi9RG/b67k@public.gmane.org To: linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Return-path: In-Reply-To: <1339496358-27940-1-git-send-email-jlayton-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org> Sender: linux-cifs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: On Tue, 12 Jun 2012 06:19:18 -0400 Jeff Layton wrote: > I think in hindsight, that adding -Werror by default was a mistake. > cifs-utils is built in a wide range of environments and tools, and it's > very difficult to eliminate all of the possible warnings. > > Let's go ahead and remove it and reduce the steady trickle of patches > that are simply to silence obscure warnings. > > Cc: Suresh Jayaraman > Signed-off-by: Jeff Layton > --- > Makefile.am | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/Makefile.am b/Makefile.am > index 05729ca..b412262 100644 > --- a/Makefile.am > +++ b/Makefile.am > @@ -1,4 +1,4 @@ > -AM_CFLAGS = -Wall -Wextra -Werror -D_FORTIFY_SOURCE=2 > +AM_CFLAGS = -Wall -Wextra -D_FORTIFY_SOURCE=2 > ACLOCAL_AMFLAGS = -I aclocal > > root_sbindir = $(ROOTSBINDIR) Merged... -- Jeff Layton