summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJürg Billeter <j@bitron.ch>2018-10-17 17:08:44 +0100
committerJürg Billeter <j@bitron.ch>2018-10-22 17:05:41 +0000
commit12719f0db161358013f3995e2fe83385630c7c4e (patch)
tree0d21fc86013950db7da810ce5aa85d48af167d29
parent43a2eee3f7b5fdb1e7284a934a7090ff0195e71a (diff)
downloadbuildstream-12719f0db161358013f3995e2fe83385630c7c4e.tar.gz
_artifactcache: Rename update_atime() to update_mtime()
os.utime() updates both, however, list_artifacts() sorts refs by mtime, i.e., atime is irrelevant.
-rw-r--r--buildstream/_artifactcache/artifactcache.py12
-rw-r--r--buildstream/_artifactcache/cascache.py2
2 files changed, 7 insertions, 7 deletions
diff --git a/buildstream/_artifactcache/artifactcache.py b/buildstream/_artifactcache/artifactcache.py
index 7977691c0..c6c8afa80 100644
--- a/buildstream/_artifactcache/artifactcache.py
+++ b/buildstream/_artifactcache/artifactcache.py
@@ -228,7 +228,7 @@ class ArtifactCache():
self._required_elements.update(elements)
# For the cache keys which were resolved so far, we bump
- # the atime of them.
+ # the mtime of them.
#
# This is just in case we have concurrent instances of
# BuildStream running with the same artifact cache, it will
@@ -240,7 +240,7 @@ class ArtifactCache():
for key in (strong_key, weak_key):
if key:
try:
- self.update_atime(key)
+ self.update_mtime(key)
except ArtifactError:
pass
@@ -391,15 +391,15 @@ class ArtifactCache():
def preflight(self):
pass
- # update_atime()
+ # update_mtime()
#
- # Update the atime of an artifact.
+ # Update the mtime of an artifact.
#
# Args:
# key (str): The key of the artifact.
#
- def update_atime(self, key):
- raise ImplError("Cache '{kind}' does not implement contains()"
+ def update_mtime(self, key):
+ raise ImplError("Cache '{kind}' does not implement update_mtime()"
.format(kind=type(self).__name__))
# initialize_remotes():
diff --git a/buildstream/_artifactcache/cascache.py b/buildstream/_artifactcache/cascache.py
index ed6d58821..8fb2e8b01 100644
--- a/buildstream/_artifactcache/cascache.py
+++ b/buildstream/_artifactcache/cascache.py
@@ -538,7 +538,7 @@ class CASCache(ArtifactCache):
except FileNotFoundError as e:
raise ArtifactError("Attempt to access unavailable artifact: {}".format(e)) from e
- def update_atime(self, ref):
+ def update_mtime(self, ref):
try:
os.utime(self._refpath(ref))
except FileNotFoundError as e: