From patchwork Fri Nov 1 09:27:22 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Nicolas Iooss X-Patchwork-Id: 11222823 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id D98E113BD for ; Fri, 1 Nov 2019 09:29:50 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id A1ECE20862 for ; Fri, 1 Nov 2019 09:29:50 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728638AbfKAJ3u (ORCPT ); Fri, 1 Nov 2019 05:29:50 -0400 Received: from mx1.polytechnique.org ([129.104.30.34]:39188 "EHLO mx1.polytechnique.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728454AbfKAJ3u (ORCPT ); Fri, 1 Nov 2019 05:29:50 -0400 Received: from localhost.localdomain (85-168-38-217.rev.numericable.fr [85.168.38.217]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ssl.polytechnique.org (Postfix) with ESMTPSA id 4389C5649AB; Fri, 1 Nov 2019 10:29:45 +0100 (CET) From: Nicolas Iooss To: selinux@vger.kernel.org Cc: Michael Shigorin Subject: [PATCH 1/3] Makefile: fix cleaning files that starts with "-" Date: Fri, 1 Nov 2019 10:27:22 +0100 Message-Id: <20191101092724.7650-1-nicolas.iooss@m4x.org> X-Mailer: git-send-email 2.23.0 MIME-Version: 1.0 X-AV-Checked: ClamAV using ClamSMTP at svoboda.polytechnique.org (Fri Nov 1 10:29:45 2019 +0100 (CET)) X-Spam-Flag: No, tests=bogofilter, spamicity=0.001390, queueID=912235649AD X-Org-Mail: nicolas.iooss.2010@polytechnique.org Sender: selinux-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: selinux@vger.kernel.org libselinux/src/exception.sh creates a file named "-.o". This file is usually removed at the end of this script, but some failures in it could left it on the filesystem. "make clean" is then supposed to remove this file, but fails instead with: rm: invalid option -- '.' Try 'rm ./-.o' to remove the file '-.o'. Try 'rm --help' for more information. This is due to "rm -f *.o" being expanded to "rm -f -.o". Prevent this buggy expansion by using "./*.o" in the command line. While at it, do so for other patterns like "*~". Signed-off-by: Nicolas Iooss --- checkpolicy/test/Makefile | 2 +- libselinux/src/Makefile | 4 ++-- libselinux/utils/Makefile | 2 +- libsemanage/src/Makefile | 2 +- libsepol/utils/Makefile | 2 +- mcstrans/Makefile | 2 +- mcstrans/man/Makefile | 2 +- mcstrans/src/Makefile | 2 +- mcstrans/utils/Makefile | 2 +- policycoreutils/load_policy/Makefile | 2 +- policycoreutils/newrole/Makefile | 2 +- policycoreutils/po/Makefile | 2 +- policycoreutils/run_init/Makefile | 2 +- policycoreutils/secon/Makefile | 2 +- policycoreutils/semodule/Makefile | 2 +- policycoreutils/sestatus/Makefile | 2 +- policycoreutils/setfiles/Makefile | 2 +- policycoreutils/setsebool/Makefile | 2 +- python/audit2allow/Makefile | 2 +- python/sepolgen/Makefile | 2 +- python/sepolgen/src/Makefile | 2 +- python/sepolgen/src/sepolgen/Makefile | 2 +- python/sepolgen/src/share/Makefile | 2 +- python/sepolgen/tests/Makefile | 2 +- python/sepolicy/Makefile | 2 +- restorecond/Makefile | 2 +- sandbox/Makefile | 2 +- semodule-utils/semodule_expand/Makefile | 2 +- semodule-utils/semodule_link/Makefile | 2 +- semodule-utils/semodule_package/Makefile | 2 +- 30 files changed, 31 insertions(+), 31 deletions(-) diff --git a/checkpolicy/test/Makefile b/checkpolicy/test/Makefile index 89e7557c7aa6..ec8cc7299e4b 100644 --- a/checkpolicy/test/Makefile +++ b/checkpolicy/test/Makefile @@ -19,4 +19,4 @@ dismod: dismod.o $(LIBSEPOLA) $(CC) $(LDFLAGS) -o $@ $^ $(LDLIBS_LIBSEPOLA) clean: - -rm -f dispol dismod *.o + -rm -f dispol dismod ./*.o diff --git a/libselinux/src/Makefile b/libselinux/src/Makefile index 2b1696a0c17b..63d6b0eda270 100644 --- a/libselinux/src/Makefile +++ b/libselinux/src/Makefile @@ -187,13 +187,13 @@ relabel: clean-pywrap: -rm -f $(SWIGLOBJ) $(SWIGSO) $(AUDIT2WHYLOBJ) $(AUDIT2WHYSO) $(PYTHON) setup.py clean - -rm -rf build *~ \#* *pyc .#* + -rm -rf build ./*~ \#* ./*pyc .#* clean-rubywrap: -rm -f $(SWIGRUBYLOBJ) $(SWIGRUBYSO) clean: clean-pywrap clean-rubywrap - -rm -f $(LIBPC) $(OBJS) $(LOBJS) $(LIBA) $(LIBSO) $(TARGET) *.o *.lo *~ + -rm -f $(LIBPC) $(OBJS) $(LOBJS) $(LIBA) $(LIBSO) $(TARGET) ./*.o ./*.lo ./*~ distclean: clean rm -f $(GENERATED) $(SWIGFILES) diff --git a/libselinux/utils/Makefile b/libselinux/utils/Makefile index 361506384d08..1ac6f7dfd9f7 100644 --- a/libselinux/utils/Makefile +++ b/libselinux/utils/Makefile @@ -65,7 +65,7 @@ install: all install -m 755 $(TARGETS) $(DESTDIR)$(SBINDIR) clean: - rm -f $(TARGETS) *.o *~ + rm -f $(TARGETS) ./*.o ./*~ distclean: clean diff --git a/libsemanage/src/Makefile b/libsemanage/src/Makefile index e029f0988dd8..7dcf4715878e 100644 --- a/libsemanage/src/Makefile +++ b/libsemanage/src/Makefile @@ -155,7 +155,7 @@ relabel: /sbin/restorecon $(DESTDIR)$(LIBDIR)/$(LIBSO) clean: - -rm -f $(LIBPC) $(OBJS) $(LOBJS) $(LIBA) $(LIBSO) $(SWIGLOBJ) $(SWIGSO) $(SWIGRUBYSO) $(TARGET) conf-parse.c conf-parse.h conf-scan.c *.o *.lo *~ + -rm -f $(LIBPC) $(OBJS) $(LOBJS) $(LIBA) $(LIBSO) $(SWIGLOBJ) $(SWIGSO) $(SWIGRUBYSO) $(TARGET) conf-parse.c conf-parse.h conf-scan.c ./*.o ./*.lo ./*~ distclean: clean rm -f $(GENERATED) $(SWIGFILES) diff --git a/libsepol/utils/Makefile b/libsepol/utils/Makefile index 31932c111026..26f7a02476a6 100644 --- a/libsepol/utils/Makefile +++ b/libsepol/utils/Makefile @@ -16,7 +16,7 @@ install: all install -m 755 $(TARGETS) $(DESTDIR)$(BINDIR) clean: - -rm -f $(TARGETS) *.o + -rm -f $(TARGETS) ./*.o indent: ../../scripts/Lindent $(wildcard *.[ch]) diff --git a/mcstrans/Makefile b/mcstrans/Makefile index c993a9f52713..c3e4a148c66d 100644 --- a/mcstrans/Makefile +++ b/mcstrans/Makefile @@ -8,7 +8,7 @@ install: $(MAKE) -C man install clean: - rm -f *~ \#* + rm -f ./*~ ./\#* $(MAKE) -C src clean $(MAKE) -C utils clean $(MAKE) -C man clean diff --git a/mcstrans/man/Makefile b/mcstrans/man/Makefile index 71713818c621..696430155c9f 100644 --- a/mcstrans/man/Makefile +++ b/mcstrans/man/Makefile @@ -26,6 +26,6 @@ install: all done clean: - -rm -f *~ \#* + -rm -f ./*~ \#* -rm -f man5/*~ man5/\#* -rm -f man8/*~ man8/\#* diff --git a/mcstrans/src/Makefile b/mcstrans/src/Makefile index 76ef055714e9..b7f14654889f 100644 --- a/mcstrans/src/Makefile +++ b/mcstrans/src/Makefile @@ -34,5 +34,5 @@ install: all install -m 644 mcstrans.service $(DESTDIR)$(SYSTEMDDIR)/system/ clean: - -rm -f $(OBJS) $(LOBJS) $(TARGET) $(PROG) $(PROG_OBJS) *~ \#* + -rm -f $(OBJS) $(LOBJS) $(TARGET) $(PROG) $(PROG_OBJS) ./*~ \#* diff --git a/mcstrans/utils/Makefile b/mcstrans/utils/Makefile index 9dfe772332b4..90ae75c29317 100644 --- a/mcstrans/utils/Makefile +++ b/mcstrans/utils/Makefile @@ -30,7 +30,7 @@ test: ./mlstrans-test-runner.py ../test/*.test clean: - rm -f $(TARGETS) *.o *~ \#* + rm -f $(TARGETS) ./*.o ./*~ \#* relabel: diff --git a/policycoreutils/load_policy/Makefile b/policycoreutils/load_policy/Makefile index c1ba805b6a9a..9581eefc27f5 100644 --- a/policycoreutils/load_policy/Makefile +++ b/policycoreutils/load_policy/Makefile @@ -26,7 +26,7 @@ install: all done clean: - -rm -f $(TARGETS) *.o + -rm -f $(TARGETS) ./*.o indent: ../../scripts/Lindent $(wildcard *.[ch]) diff --git a/policycoreutils/newrole/Makefile b/policycoreutils/newrole/Makefile index 73ebd413da85..11b2a40b3d72 100644 --- a/policycoreutils/newrole/Makefile +++ b/policycoreutils/newrole/Makefile @@ -83,7 +83,7 @@ endif endif clean: - rm -f newrole *.o + rm -f newrole ./*.o indent: ../../scripts/Lindent $(wildcard *.[ch]) diff --git a/policycoreutils/po/Makefile b/policycoreutils/po/Makefile index 575e143122e6..b201b90b2392 100644 --- a/policycoreutils/po/Makefile +++ b/policycoreutils/po/Makefile @@ -123,7 +123,7 @@ refresh-po: Makefile done clean: - @rm -fv *mo *~ .depend + @rm -fv ./*mo ./*~ .depend @rm -rf tmp install: $(MOFILES) diff --git a/policycoreutils/run_init/Makefile b/policycoreutils/run_init/Makefile index e86364a496e6..b19540662eba 100644 --- a/policycoreutils/run_init/Makefile +++ b/policycoreutils/run_init/Makefile @@ -50,7 +50,7 @@ ifeq ($(PAMH), y) endif clean: - -rm -f $(TARGETS) *.o + -rm -f $(TARGETS) ./*.o indent: ../../scripts/Lindent $(wildcard *.[ch]) diff --git a/policycoreutils/secon/Makefile b/policycoreutils/secon/Makefile index 576a6203dfa3..b17997914d69 100644 --- a/policycoreutils/secon/Makefile +++ b/policycoreutils/secon/Makefile @@ -32,7 +32,7 @@ relabel: /sbin/restorecon $(DESTDIR)$(BINDIR)/secon clean: - rm -f *.o core* secon *~ *.bak + rm -f ./*.o core* secon ./*~ ./*.bak indent: ../../scripts/Lindent $(wildcard *.[ch]) diff --git a/policycoreutils/semodule/Makefile b/policycoreutils/semodule/Makefile index 73801e487a76..e6520e877252 100644 --- a/policycoreutils/semodule/Makefile +++ b/policycoreutils/semodule/Makefile @@ -32,7 +32,7 @@ install: all relabel: clean: - -rm -f semodule *.o genhomedircon + -rm -f semodule ./*.o genhomedircon indent: ../../scripts/Lindent $(wildcard *.[ch]) diff --git a/policycoreutils/sestatus/Makefile b/policycoreutils/sestatus/Makefile index 8c4f45f804f2..4849405ec3cb 100644 --- a/policycoreutils/sestatus/Makefile +++ b/policycoreutils/sestatus/Makefile @@ -32,7 +32,7 @@ install: all install -m 644 sestatus.conf $(DESTDIR)$(ETCDIR) clean: - rm -f sestatus *.o + rm -f sestatus ./*.o indent: ../../scripts/Lindent $(wildcard *.[ch]) diff --git a/policycoreutils/setfiles/Makefile b/policycoreutils/setfiles/Makefile index bc5a8db789a5..456f288d649b 100644 --- a/policycoreutils/setfiles/Makefile +++ b/policycoreutils/setfiles/Makefile @@ -45,7 +45,7 @@ install: all done clean: - rm -f setfiles restorecon restorecon_xattr *.o setfiles.8.man + rm -f setfiles restorecon restorecon_xattr ./*.o setfiles.8.man indent: ../../scripts/Lindent $(wildcard *.[ch]) diff --git a/policycoreutils/setsebool/Makefile b/policycoreutils/setsebool/Makefile index 4b55046cecc4..217cf844aaba 100644 --- a/policycoreutils/setsebool/Makefile +++ b/policycoreutils/setsebool/Makefile @@ -32,7 +32,7 @@ install: all relabel: clean: - -rm -f setsebool *.o + -rm -f setsebool ./*.o indent: ../../scripts/Lindent $(wildcard *.[ch]) diff --git a/python/audit2allow/Makefile b/python/audit2allow/Makefile index 15db549081dd..0a8008f63dee 100644 --- a/python/audit2allow/Makefile +++ b/python/audit2allow/Makefile @@ -47,7 +47,7 @@ install: all done clean: - rm -f *~ *.o sepolgen-ifgen-attr-helper test_dummy_policy + rm -f ./*~ ./*.o sepolgen-ifgen-attr-helper test_dummy_policy indent: ../../scripts/Lindent $(wildcard *.[ch]) diff --git a/python/sepolgen/Makefile b/python/sepolgen/Makefile index 3aed330499a4..01d7b46c6918 100644 --- a/python/sepolgen/Makefile +++ b/python/sepolgen/Makefile @@ -8,7 +8,7 @@ relabel: ; clean: $(MAKE) -C src $@ $(MAKE) -C tests $@ - rm -f *~ *.pyc + rm -f ./*~ ./*.pyc rm -f parser.out parsetab.py indent: ; diff --git a/python/sepolgen/src/Makefile b/python/sepolgen/src/Makefile index 2815a13d3499..3e55309f8dcf 100644 --- a/python/sepolgen/src/Makefile +++ b/python/sepolgen/src/Makefile @@ -9,7 +9,7 @@ relabel: ; clean: $(MAKE) -C sepolgen $@ $(MAKE) -C share $@ - rm -f *~ *.pyc + rm -f ./*~ ./*.pyc rm -f parser.out parsetab.py indent: ; diff --git a/python/sepolgen/src/sepolgen/Makefile b/python/sepolgen/src/sepolgen/Makefile index cac8def7204e..83b17bb80227 100644 --- a/python/sepolgen/src/sepolgen/Makefile +++ b/python/sepolgen/src/sepolgen/Makefile @@ -11,5 +11,5 @@ install: all clean: rm -f parser.out parsetab.py - rm -f *~ *.pyc + rm -f ./*~ ./*.pyc rm -rf __pycache__ diff --git a/python/sepolgen/src/share/Makefile b/python/sepolgen/src/share/Makefile index 1a7133cbf5cb..4045be56597b 100644 --- a/python/sepolgen/src/share/Makefile +++ b/python/sepolgen/src/share/Makefile @@ -7,4 +7,4 @@ install: all install -m 644 perm_map $(DESTDIR)$(SHAREDIR) clean: - rm -f *~ + rm -f ./*~ diff --git a/python/sepolgen/tests/Makefile b/python/sepolgen/tests/Makefile index 83d072f4e6aa..a5c23899e8fd 100644 --- a/python/sepolgen/tests/Makefile +++ b/python/sepolgen/tests/Makefile @@ -1,7 +1,7 @@ PYTHON ?= python3 clean: - rm -f *~ *.pyc + rm -f ./*~ ./*.pyc rm -f parser.out parsetab.py rm -f out.txt rm -f module_compile_test.fc diff --git a/python/sepolicy/Makefile b/python/sepolicy/Makefile index 69f29fa962f9..0f55c5f30804 100644 --- a/python/sepolicy/Makefile +++ b/python/sepolicy/Makefile @@ -18,7 +18,7 @@ python-build: clean: $(PYTHON) setup.py clean - -rm -rf build *~ \#* *pyc .#* + -rm -rf build ./*~ \#* ./*pyc .#* sepolgen: ln -sf sepolicy sepolgen diff --git a/restorecond/Makefile b/restorecond/Makefile index 12452cd2f8e1..f200d70e6a8e 100644 --- a/restorecond/Makefile +++ b/restorecond/Makefile @@ -54,7 +54,7 @@ relabel: install /sbin/restorecon $(DESTDIR)$(SBINDIR)/restorecond clean: - -rm -f restorecond *.o *~ + -rm -f restorecond ./*.o ./*~ indent: ../../scripts/Lindent $(wildcard *.[ch]) diff --git a/sandbox/Makefile b/sandbox/Makefile index 9da5e58db9e6..e0e2be5574ce 100644 --- a/sandbox/Makefile +++ b/sandbox/Makefile @@ -44,7 +44,7 @@ test: @$(PYTHON) test_sandbox.py -v clean: - -rm -f seunshare *.o *~ + -rm -f seunshare ./*.o ./*~ indent: ../../scripts/Lindent $(wildcard *.[ch]) diff --git a/semodule-utils/semodule_expand/Makefile b/semodule-utils/semodule_expand/Makefile index e63dcff246d9..e5022fb598e7 100644 --- a/semodule-utils/semodule_expand/Makefile +++ b/semodule-utils/semodule_expand/Makefile @@ -26,7 +26,7 @@ install: all relabel: clean: - -rm -f semodule_expand *.o + -rm -f semodule_expand ./*.o indent: ../../scripts/Lindent $(wildcard *.[ch]) diff --git a/semodule-utils/semodule_link/Makefile b/semodule-utils/semodule_link/Makefile index c5cf69cd9ca3..ba5d97d115ad 100644 --- a/semodule-utils/semodule_link/Makefile +++ b/semodule-utils/semodule_link/Makefile @@ -26,7 +26,7 @@ install: all relabel: clean: - -rm -f semodule_link *.o + -rm -f semodule_link ./*.o indent: ../../scripts/Lindent $(wildcard *.[ch]) diff --git a/semodule-utils/semodule_package/Makefile b/semodule-utils/semodule_package/Makefile index 680ab836cfe6..c53628ae1365 100644 --- a/semodule-utils/semodule_package/Makefile +++ b/semodule-utils/semodule_package/Makefile @@ -28,7 +28,7 @@ install: all relabel: clean: - -rm -f semodule_package semodule_unpackage *.o + -rm -f semodule_package semodule_unpackage ./*.o indent: ../../scripts/Lindent $(wildcard *.[ch])