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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


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(-)
> 



--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel


[DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]     [Fedora Docs]

Add to Google Powered by Linux