Fix 'make install' overly broad 'rm'

When running 'make install' without DESTDIR set the module install
rules would mistakenly destroy the 'modules.*' files for ALL of
your installed kernels.  This could lead to a non-functional system
for the alternate kernels because 'depmod -a' will only be run for
the kernel which was compiled against.  This issue would not impact
anyone using the 'make <deb|rpm|pkg>' build targets to build and
install packages.

The fix for this issue is to only remove extraneous build products
when DESTDIR is set.  This almost exclusively indicates we are
building packages and installed the build products in to a temporary
staging location.  Additionally, limit the removal the unneeded
build products to the target kernel version.

Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #328

diff --git a/module/Makefile.in b/module/Makefile.in
index 70a33ec1438be663325fc4d464cf83b5b1856de8..355089aeb78733366ee1068cd7e26080d57ca79c 100644 (file)
--- a/module/Makefile.in
+++ b/module/Makefile.in
@@ -16,7 +16,11 @@ modules_install:
        $(MAKE) -C @LINUX_OBJ@ SUBDIRS=`pwd` \
                INSTALL_MOD_PATH=$(DESTDIR) \
                INSTALL_MOD_DIR=addon/spl $@
-       find $(DESTDIR)/lib/modules/ -name 'modules.*' | xargs $(RM)
+       @# Remove extraneous build products when packaging
+       if [ -n "$(DESTDIR)" ]; then \
+               find $(DESTDIR)/lib/modules/@LINUX_VERSION@ \
+                       -name 'modules.*' | xargs $(RM); \
+       fi
        sysmap=$(DESTDIR)/boot/System.map-@LINUX_VERSION@; \
        if [ -f $$sysmap ]; then \
                depmod -ae -F $$sysmap @LINUX_VERSION@; \