Added check to ensure the current user has permissions to delete Xcode versions

juanjonol · juanjonol · commit 9a0bb3008f36 · 2022-02-06T18:02:24.000+01:00
The `verify_permissions()` function checks that it has permissions to install Xcode, not to delete old versions.
diff --git a/xcode-update.py b/xcode-update.py
@@ -90,6 +90,8 @@ def delete_xcode(dry_run: bool):
 	xcode_version_to_delete = oldest_xcode_version(include_releases=should_delete_release_version)
 	if xcode_version_to_delete:
 		print(f'- {xcode_version_to_delete} will be deleted.')
+		if not os.access(xcode_version_to_delete, os.R_OK | os.W_OK):
+			raise PermissionError(f"The current user doesn't have permissions to delete {xcode_version_to_delete}")
 	
 	if not dry_run:
 		shutil.rmtree(str(xcode_version_to_delete))
