Skip to content

Use temporary directories for mounting #17

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Use temporary directories for mounting #17

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
f8c120c
Do not hardcode /mnt/clone in rpi-clone-setup
matthijskooijman Jul 14, 2020
8b3210e
Create mount directories earlier
matthijskooijman Jul 14, 2020
43884d9
Use a random temp directory for mounting
matthijskooijman Jul 14, 2020
0ecf02a
fixup! Use a random temp directory for mounting
matthijskooijman Apr 8, 2024
2bf258a
fixup! Use a random temp directory for mounting
matthijskooijman Apr 8, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 21 additions & 13 deletions rpi-clone
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,9 +84,7 @@ then
apt-get install -y --no-install-recommends $need_packages
fi

clone=/mnt/clone
clone_src=/mnt/clone-src
clone_log=/var/log/$PGM.log
export clone_log=/var/log/$PGM.log

HOSTNAME=`hostname`

Expand Down Expand Up @@ -997,6 +995,26 @@ then
exit 0
fi

export clone=$(mktemp --tmpdir --directory rpi-clone-dst.XXXXXX)
export clone_src=$(mktemp --tmpdir --directory rpi-clone-src.XXXXXX)

if [[ -z "$clone" || ! -d "$clone" || -z "$clone_src" || ! -d "$clone_src" ]]
then
echo "Failed to create temporary mount directories"
echo "Aborting!"
exit 1
fi

# Make sure we cleanup on exit, regardless of how we exit
cleanup() {
echo "Cleaning up mounts, this might produce errors if they were already unmounted" >> "$clone_log"
umount "$clone" &>> "$clone_log"
umount "$clone_src" &>> "$clone_log"
rmdir "$clone" &>> "$clone_log"
rmdir "$clone_src" &>> "$clone_log"
}
trap cleanup EXIT

# dst_mount_flag enumerations:
live=1
temp=2
Expand Down Expand Up @@ -1252,16 +1270,6 @@ unmount_or_abort "$mounted_dev" \
mounted_dev=$(findmnt /mnt -o source -n)
unmount_or_abort "$mounted_dev" "$mounted_dev is currently mounted on /mnt."


if [ ! -d $clone ]
then
mkdir $clone
fi
if [ ! -d $clone_src ]
then
mkdir $clone_src
fi

# Do not include a dhpys swapfile in rsync. It regenerates at boot.
#
if [ -f /etc/dphys-swapfile ]
Expand Down
20 changes: 14 additions & 6 deletions rpi-clone-setup
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,13 @@
# eg: sudo rpi-clone-setup bozo
#
# This script is automatically run by rpi-clone (when it is given -s options)
# to setup an alternate hostname. A cloned file system mounted on /mnt/clone
# is expected unless testing with the -t option.
# to setup an alternate hostname. It expects $clone to be set in the
# environment containing the path to a mounted cloned file system unless
# testing with the -t option.
#
# Or, this script can be run by hand at the end of a clone when rpi-clone
# pauses with the cloned file systems still mounted on /mnt/clone.
# pauses with the cloned file systems still mounted (make sure to set
# the $clone env variable properly, then).
#
# Or, run this script by hand with -t to process files in test directories
# under /tmp/clone-test. Run -t and look at the files to see if the files
Expand All @@ -27,7 +29,7 @@

file_list="etc/hostname etc/hosts"

clone=/mnt/clone
# $clone is set by caller
clone_test=/tmp/clone-test

PGM=`basename $0`
Expand Down Expand Up @@ -81,9 +83,15 @@ fi

echo -e "\t$newhost\t- target hostname"

if ((!testing)) && [ ! -d /mnt/clone/etc ]
if ((!testing)) && [ -z "$clone" ]
then
echo "A destination clone file system is not mounted on /mnt/clone"
echo "No clone mountpoint was passed in the \$clone variable"
echo "Aborting!"
exit 0
fi
if ((!testing)) && [ ! -d "$clone/etc" ]
then
echo "A destination clone file system is not mounted on $clone"
echo "Aborting!"
exit 0
fi
Expand Down