From mboxrd@z Thu Jan  1 00:00:00 1970
From: Daniel =?utf-8?q?Weism=C3=BCller?= <daniel.weismueller@ipfire.org>
To: development@lists.ipfire.org
Subject: [PATCH] filesystem-cleanup: Add parameter to show changes
Date: Fri, 10 Jan 2020 16:06:00 +0100
Message-ID: <20200110150600.24259-1-daniel.weismueller@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============5199718077524231887=="
List-Id: <development.lists.ipfire.org>

--===============5199718077524231887==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit

From: Daniel Weismüller <whytea(a)ipfire.org>

Use --dry-run to only show files that would be deleted, but do
not actually delete them.

Signed-off-by: Daniel Weismüller <daniel.weismueller(a)ipfire.org>
---
 src/scripts/filesystem-cleanup | 23 ++++++++++++++++++++++-
 1 file changed, 22 insertions(+), 1 deletion(-)

diff --git a/src/scripts/filesystem-cleanup b/src/scripts/filesystem-cleanup
index d4cdfe8d3..7e7accc7a 100644
--- a/src/scripts/filesystem-cleanup
+++ b/src/scripts/filesystem-cleanup
@@ -27,6 +27,22 @@ PATHS=(
 )
 
 main() {
+	local dry_run="false"
+
+	while [ $# -gt 0 ]; do
+		case "${1}" in
+			--dry-run)
+				dry_run="true"
+				;;
+
+			*)
+				echo "${0}: Invalid argument: ${1}" >&2
+				return 2
+				;;
+		esac
+		shift
+	done
+
 	# Update the runtime linker cache (this may fix any broken symlinks)
 	ldconfig
 
@@ -65,7 +81,12 @@ main() {
 						;;
 
 					*)
-						unlink "${file}"
+						echo "Removing ${file}..."
+
+						# Actually remove the file (maybe)
+						if [ "${dry_run}" = "false" ]; then
+							unlink "${file}"
+						fi
 						;;
 				esac
 			done
-- 
2.12.2


--===============5199718077524231887==--