[Pkg-ofed-commits] [mstflint] 19/22: Disable -Werror compiler flag

[Benjamin Drung]

This is an automated email from the git hooks/post-receive script.

bdrung pushed a commit to branch old
in repository mstflint.

commit 8058184fbcd8986e999af3392e73e8b47a8e7788
Author: Benjamin Drung <benjamin.drung at profitbricks.com>
Date:   Fri Aug 19 15:35:54 2016 +0200

    Disable -Werror compiler flag
    
    The -Werror compiler flag converts warnings into errors, so that warnings
    will cause the package build to fail. The -Werror option is useful for the
    upstream maintainer to keep the code clean, but is not recommended for
    packaging, because it might break the package if we upgrade the compiler
    to a new release.
    
    The configure script should gain an option to enable or disable -Werror.
    In the meantime just remove -Werror everywhere.
---
 debian/patches/disable-werror.patch | 68 +++++++++++++++++++++++++++++++++++++
 debian/patches/series               |  1 +
 2 files changed, 69 insertions(+)

diff --git a/debian/patches/disable-werror.patch b/debian/patches/disable-werror.patch
new file mode 100644
index 0000000..1c59eb3
--- /dev/null
+++ b/debian/patches/disable-werror.patch
@@ -0,0 +1,68 @@
+Description: Disable -Werror compiler flag
+ The -Werror compiler flag converts warnings into errors, so that warnings
+ will cause the package build to fail. The -Werror option is useful for the
+ upstream maintainer to keep the code clean, but is not recommended for
+ packaging, because it might break the package if we upgrade the compiler
+ to a new release.
+ .
+ The configure script should gain an option to enable or disable -Werror.
+ In the meantime just remove -Werror everywhere.
+Author: Benjamin Drung <benjamin.drung at profitbricks.com>
+
+--- a/cmdif/Makefile.am
++++ b/cmdif/Makefile.am
+@@ -36,7 +36,7 @@
+ TOOLS_LAYOUTS_DIR = $(USER_DIR)/tools_layouts
+ AM_CPPFLAGS = -I. -I../common -I../tools_layouts  -I$(MTCR_DIR) -I.. -I$(USER_DIR)/mtcr_ul
+ 
+-AM_CFLAGS = -W -Wall -Werror -g -MP -MD $(COMPILER_FPIC) -DCMDIF_EXPORTS
++AM_CFLAGS = -W -Wall -g -MP -MD $(COMPILER_FPIC) -DCMDIF_EXPORTS
+ CMDIF_VERSION = 1
+ noinst_LIBRARIES = libcmdif.a
+ 
+--- a/configure.ac
++++ b/configure.ac
+@@ -77,8 +77,8 @@
+   LDFLAGS="$LDFLAGS -Wl,--dynamic-linker=/lib64/ld64.so.2"
+ ])
+ 
+-CFLAGS="$CFLAGS -Werror -DMST_UL"
+-CXXFLAGS="$CXXFLAGS -Werror -DMST_UL"
++CFLAGS="$CFLAGS -DMST_UL"
++CXXFLAGS="$CXXFLAGS -DMST_UL"
+ 
+ AC_CONFIG_FILES( mstflint.spec )
+ 
+--- a/mft_utils/Makefile.am
++++ b/mft_utils/Makefile.am
+@@ -34,7 +34,7 @@
+ USER_DIR = $(top_srcdir)
+ AM_CPPFLAGS = -I. -I$(USER_DIR)/common
+ 
+-AM_CFLAGS = -MD -pipe -Wall -W -Werror
++AM_CFLAGS = -MD -pipe -Wall -W
+ 
+ noinst_HEADERS = mft_sig_handler.h errmsg.h
+ 
+--- a/reg_access/Makefile.am
++++ b/reg_access/Makefile.am
+@@ -38,7 +38,7 @@
+ 
+ AM_CPPFLAGS = -I. -I$(USER_DIR) -I$(srcdir) -I$(LAYOUTS_DIR) -I$(USER_DIR)/include/mtcr_ul
+ 
+-AM_CFLAGS = -W -Wall -Werror -g -MP -MD $(COMPILER_FPIC)
++AM_CFLAGS = -W -Wall -g -MP -MD $(COMPILER_FPIC)
+ 
+ noinst_LTLIBRARIES = libreg_access.a
+ 
+--- a/tools_res_mgmt/Makefile.am
++++ b/tools_res_mgmt/Makefile.am
+@@ -35,7 +35,7 @@
+ MTCR_DIR = $(USER_DIR)/include/mtcr_ul
+ AM_CPPFLAGS = -I. -I../common -I$(MTCR_DIR) -I.. -I$(USER_DIR)/mtcr_ul
+ 
+-AM_CFLAGS = -W -Wall -Werror -g -MP -MD $(COMPILER_FPIC) 
++AM_CFLAGS = -W -Wall -g -MP -MD $(COMPILER_FPIC)
+ noinst_LIBRARIES = libtools_res_mgmt.a
+ 
+ libtools_res_mgmt_a_SOURCES = tools_res_mgmt.c tools_time.c
diff --git a/debian/patches/series b/debian/patches/series
index 70ee3bf..5c80cec 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -6,3 +6,4 @@
 0006-Fix-man-page-errors.patch
 0007-Pass-foreign-to-automake.patch
 Fix-compilation-on-unknown-architectures.patch
+disable-werror.patch

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-ofed/mstflint.git