Re: [PATCH 0/2] multipath: compile cleanup

[Christophe Varoqui <christophe.varoqui@gmail.com>]

On jeu., 2012-05-24 at 23:57 -0500, Benjamin Marzinski wrote:
> When the multipath code is compiled with the standard redhat rpm cflags,
> there are a number of warnings that get issued, some about real problems.
> 
> These patches switch multipath to use those cflags, and then clean up
> warnings. If people want upstream to continue using the existing cflags,
> I don't particularly mind.  I really included the first patch to make sure
> people can see the warnings that the second patch corrects.
> 
> Also, some of the warnings are for actual errors, and how to fix them is
> obvious.  Others won't actually cause problems, and if people object to
> the changes I made to make them go away, I'm happy to fix them differently. 
> 
Merged.

Thanks.

> Benjamin Marzinski (2):
>   multipath: Build with standard rpm cflags
>   multipath: Fix warnings from stricter compile options.
> 
>  Makefile.inc                     |    6 +++++-
>  libmpathpersist/mpath_persist.c  |   10 +++++++---
>  libmpathpersist/mpath_pr_ioctl.c |    9 +++++++++
>  libmultipath/alias.c             |    8 ++++++--
>  mpathpersist/main.c              |    2 --
>  multipathd/main.c                |   24 +++++++++++++++++++-----
>  6 files changed, 46 insertions(+), 13 deletions(-)
>