summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAngelos Evripiotis <jevripiotis@bloomberg.net>2019-10-29 13:13:41 +0000
committerAngelos Evripiotis <jevripiotis@bloomberg.net>2019-10-29 15:26:45 +0000
commit86ff45758e8a4b5cd7a2885a60121082a221c01d (patch)
treeeda367d9f4af613a26374b505253325e39c0a943
parent918f07cc075db5891d9429c0c54cb70143249375 (diff)
downloadbuildstream-aevri/fuse_mount_private.tar.gz
_fuse/mount: make mount() and unmount() privateaevri/fuse_mount_private
These aren't used by anything else, so make them private. This will make it easier to later reason about some spawn mode fixes.
-rw-r--r--src/buildstream/_fuse/mount.py14
1 files changed, 7 insertions, 7 deletions
diff --git a/src/buildstream/_fuse/mount.py b/src/buildstream/_fuse/mount.py
index 52d3ed65a..defc41de8 100644
--- a/src/buildstream/_fuse/mount.py
+++ b/src/buildstream/_fuse/mount.py
@@ -91,14 +91,14 @@ class Mount():
def __init__(self, fuse_mount_options=None):
self._fuse_mount_options = {} if fuse_mount_options is None else fuse_mount_options
- # mount():
+ # __mount():
#
# User facing API for mounting a fuse subclass implementation
#
# Args:
# (str): Location to mount this fuse fs
#
- def mount(self, mountpoint):
+ def __mount(self, mountpoint):
assert self.__process is None
@@ -119,11 +119,11 @@ class Mount():
time.sleep(1 / 100)
- # unmount():
+ # __unmount():
#
# User facing API for unmounting a fuse subclass implementation
#
- def unmount(self):
+ def __unmount(self):
# Terminate child process and join
if self.__process is not None:
@@ -156,12 +156,12 @@ class Mount():
with utils._tempnamedfile() as logfile:
self.__logfile = logfile
- self.mount(mountpoint)
+ self.__mount(mountpoint)
try:
- with _signals.terminator(self.unmount):
+ with _signals.terminator(self.__unmount):
yield
finally:
- self.unmount()
+ self.__unmount()
self.__logfile = None