Skip to content

Commit

Permalink
adapt to apfs.util flag diff in catalina/big sur
Browse files Browse the repository at this point in the history
Fixes #3957. Just runs both forms to minimize moving parts.
  • Loading branch information
abathur committed Oct 19, 2020
1 parent e736f8f commit fe80790
Showing 1 changed file with 15 additions and 1 deletion.
16 changes: 15 additions & 1 deletion scripts/create-darwin-volume.sh
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,20 @@ test_synthetic_conf() {
grep -q "^nix$" /etc/synthetic.conf 2>/dev/null
}

# Create the paths defined in synthetic.conf, saving us a reboot.
create_synthetic_objects(){
# Big Sur takes away the -B flag we were using and replaces it
# with a -t flag that appears to do the same thing (but they
# don't behave exactly the same way in terms of return values).
# This feels a little dirty, but as far as I can tell the
# simplest way to get the right one is to just throw away stderr
# and call both... :]
{
/System/Library/Filesystems/apfs.fs/Contents/Resources/apfs.util -t || true # Big Sur
/System/Library/Filesystems/apfs.fs/Contents/Resources/apfs.util -B || true # Catalina
} >/dev/null 2>&1
}

test_nix() {
test -d "/nix"
}
Expand Down Expand Up @@ -101,7 +115,7 @@ main() {

if ! test_nix; then
echo "Creating mountpoint for /nix..." >&2
/System/Library/Filesystems/apfs.fs/Contents/Resources/apfs.util -B || true
create_synthetic_objects # the ones we defined in synthetic.conf
if ! test_nix; then
sudo mkdir -p /nix 2>/dev/null || true
fi
Expand Down

0 comments on commit fe80790

Please sign in to comment.