diff options
author | Luke Shumaker <lukeshu@parabola.nu> | 2017-05-05 18:40:58 -0400 |
---|---|---|
committer | Jan Alexander Steffens (heftig) <jan.steffens@gmail.com> | 2017-07-05 18:21:54 +0200 |
commit | 2a9b30ed3594c2dc8c5012fc845572b8d452a9b0 (patch) | |
tree | 73a055572e57ea678a8b326f00bc7c037c6257f4 /archrm.in | |
parent | c9b1fc08b5f63c9f6bb1a992431c661129ab2db1 (diff) | |
download | devtools-2a9b30ed3594c2dc8c5012fc845572b8d452a9b0.tar.xz |
makechrootpkg: delete_chroot: Fix the is-btrfs-subvolume check.
First of all, it ran `is_btrfs "$chrootdir"` to decide if it was on
btrfs, but $chrootdir wasn't defined locally; it just happens to work
because $chrootdir was defined in main(). (I noticed this because in
Parabola, it is called differently, so $chrootdir was empty).
So I was tempted to just change it to `is_btrfs "$copydir"`, but if
$copydir is just a regular directory on a btrfs filesystem, then it
It would leave much of $copydir intact. What we really care about is
if $copydir is a btrfs subvolume; which we can check by combining the
is_btrfs check with inspecting the inum of the directory.
I put this combined check in lib/archroot.sh:is_subvolume.
https://lists.archlinux.org/pipermail/arch-projects/2013-September/003901.html
Diffstat (limited to 'archrm.in')
0 files changed, 0 insertions, 0 deletions