From d7518ad12b716999c1b05a910a95807a7aea5768 Mon Sep 17 00:00:00 2001 From: Saverio Miroddi Date: Sun, 13 Jun 2021 10:36:35 +0200 Subject: [PATCH] Don't set the permissions on rpool directories See the comment. This was also a bug, if the user was not creating `/root` or `/tmp` filesystems. --- install-zfs.sh | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/install-zfs.sh b/install-zfs.sh index 53bc0c1..a32752c 100755 --- a/install-zfs.sh +++ b/install-zfs.sh @@ -1205,9 +1205,10 @@ function create_pools_and_datasets { fi done < <(echo "$interpolated_dataset_create_options") - chmod 700 /mnt/root - # This is fine independently of the user creating a dataset for /tmp or not. - chmod 1777 /mnt/tmp + # Here, the original procedure sets the permissions for /root (700) and /tmp (1777), however, in this + # script we don't need to do it, since we sync the O/S installer result, which is already configured. + # In case of changes, don't forget that the destination layout may be empty, at this point, due to + # the user's ZFS filesystems configuration! # BOOT POOL CREATION #################