`gzip` supports (deprecated) `GZIP` environment variable. If it's
already present Makefile would override it and pass it through causing
thre breakage:

    $ dev>GZIP=-n make
    gzip -9 -c mpath_persistent_reserve_in.3 > mpath_persistent_reserve_in.3.gz
    gzip: -c: option not valid in GZIP environment variable
    Try `gzip --help' for more information.

Fix build by renaming GZIP variable to GZIP_PROG to avoid collision.

CC: Martin Wilck <mwi...@suse.com>
CC: Benjamin Marzinski <bmarz...@redhat.com>
Signed-off-by: Sergei Trofimovich <sly...@gmail.com>
---
 Makefile.inc             | 2 +-
 kpartx/Makefile          | 2 +-
 libmpathpersist/Makefile | 4 ++--
 mpathpersist/Makefile    | 2 +-
 multipath/Makefile       | 4 ++--
 multipathd/Makefile      | 2 +-
 6 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/Makefile.inc b/Makefile.inc
index b340f2ae..59856f24 100644
--- a/Makefile.inc
+++ b/Makefile.inc
@@ -75,7 +75,7 @@ nvmedir               = $(TOPDIR)/libmultipath/nvme
 includedir     = $(prefix)/usr/include
 pkgconfdir     = $(usrlibdir)/pkgconfig
 
-GZIP           = gzip -9 -c
+GZIP_PROG      = gzip -9 -c
 RM             = rm -f
 LN             = ln -sf
 INSTALL_PROGRAM        = install
diff --git a/kpartx/Makefile b/kpartx/Makefile
index 2906a984..9be115d6 100644
--- a/kpartx/Makefile
+++ b/kpartx/Makefile
@@ -21,7 +21,7 @@ all: $(EXEC)
 
 $(EXEC): $(OBJS)
        $(CC) $(CFLAGS) $(OBJS) -o $(EXEC) $(LDFLAGS) $(LIBDEPS)
-       $(GZIP) $(EXEC).8 > $(EXEC).8.gz
+       $(GZIP_PROG) $(EXEC).8 > $(EXEC).8.gz
 
 install: $(EXEC) $(EXEC).8
        $(INSTALL_PROGRAM) -d $(DESTDIR)$(bindir)
diff --git a/libmpathpersist/Makefile b/libmpathpersist/Makefile
index 1e6399d6..ccdc3223 100644
--- a/libmpathpersist/Makefile
+++ b/libmpathpersist/Makefile
@@ -33,8 +33,8 @@ $(DEVLIB): $(LIBS)
        $(LN) $(LIBS) $@
 
 man:
-       $(GZIP) mpath_persistent_reserve_in.3 > mpath_persistent_reserve_in.3.gz
-       $(GZIP) mpath_persistent_reserve_out.3 > 
mpath_persistent_reserve_out.3.gz
+       $(GZIP_PROG) mpath_persistent_reserve_in.3 > 
mpath_persistent_reserve_in.3.gz
+       $(GZIP_PROG) mpath_persistent_reserve_out.3 > 
mpath_persistent_reserve_out.3.gz
 
 install: all
        $(INSTALL_PROGRAM) -d $(DESTDIR)$(syslibdir)
diff --git a/mpathpersist/Makefile b/mpathpersist/Makefile
index 51268010..de66b644 100644
--- a/mpathpersist/Makefile
+++ b/mpathpersist/Makefile
@@ -14,7 +14,7 @@ all: $(EXEC)
 
 $(EXEC): $(OBJS)
        $(CC) $(OBJS) -o $(EXEC) $(LDFLAGS) $(CFLAGS) $(LIBDEPS)
-       $(GZIP) $(EXEC).8 > $(EXEC).8.gz
+       $(GZIP_PROG) $(EXEC).8 > $(EXEC).8.gz
 
 install:
        $(INSTALL_PROGRAM) -d $(DESTDIR)$(bindir)
diff --git a/multipath/Makefile b/multipath/Makefile
index 0828a8f7..3f12d75b 100644
--- a/multipath/Makefile
+++ b/multipath/Makefile
@@ -16,8 +16,8 @@ all: $(EXEC)
 
 $(EXEC): $(OBJS) $(multipathdir)/libmultipath.so $(mpathcmddir)/libmpathcmd.so
        $(CC) $(CFLAGS) $(OBJS) -o $(EXEC) $(LDFLAGS) $(LIBDEPS)
-       $(GZIP) $(EXEC).8 > $(EXEC).8.gz
-       $(GZIP) $(EXEC).conf.5 > $(EXEC).conf.5.gz
+       $(GZIP_PROG) $(EXEC).8 > $(EXEC).8.gz
+       $(GZIP_PROG) $(EXEC).conf.5 > $(EXEC).conf.5.gz
 
 install:
        $(INSTALL_PROGRAM) -d $(DESTDIR)$(bindir)
diff --git a/multipathd/Makefile b/multipathd/Makefile
index 393b6cbb..64df2214 100644
--- a/multipathd/Makefile
+++ b/multipathd/Makefile
@@ -40,7 +40,7 @@ all : $(EXEC)
 
 $(EXEC): $(OBJS) $(multipathdir)/libmultipath.so $(mpathcmddir)/libmpathcmd.so
        $(CC) $(CFLAGS) $(OBJS) $(LDFLAGS) -o $(EXEC) $(LIBDEPS)
-       $(GZIP) $(EXEC).8 > $(EXEC).8.gz
+       $(GZIP_PROG) $(EXEC).8 > $(EXEC).8.gz
 
 cli_handlers.o:        cli_handlers.c
        $(CC) $(CFLAGS) -Wno-unused-parameter -c -o $@ $<
-- 
2.34.1

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

Reply via email to