Skip to content

Commit

Permalink
Merge pull request #7696 from ThomasWaldmann/macfuse-volname-master
Browse files Browse the repository at this point in the history
mount: make up volname if not given (macOS), fixes #7690
  • Loading branch information
ThomasWaldmann committed Jul 1, 2023
2 parents c3cc33f + bd9af62 commit cd678a0
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions src/borg/fuse.py
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ def async_wrapper(fn):
from .helpers.lrucache import LRUCache
from .item import Item
from .platform import uid2user, gid2group
from .platformflags import is_darwin
from .remote import RemoteRepository


Expand Down Expand Up @@ -513,6 +514,13 @@ def pop_option(options, key, present, not_present, wanted_type, int_base=0):
options = ["fsname=borgfs", "ro", "default_permissions"]
if mount_options:
options.extend(mount_options.split(","))
if is_darwin:
# macFUSE supports a volname mount option to give what finder displays on desktop / in directory list.
volname = pop_option(options, "volname", "", "", str)
# if the user did not specify it, we make something up,
# because otherwise it would be "macFUSE Volume 0 (Python)", #7690.
volname = volname or f"{os.path.basename(mountpoint)} (borgfs)"
options.append(f"volname={volname}")
ignore_permissions = pop_option(options, "ignore_permissions", True, False, bool)
if ignore_permissions:
# in case users have a use-case that requires NOT giving "default_permissions",
Expand Down

0 comments on commit cd678a0

Please sign in to comment.