From b3504d3ca4ead49a1750078d18601560d374d6e4 Mon Sep 17 00:00:00 2001 From: Dimitris Saltaferis <99741372+dimitris-flyr@users.noreply.github.com> Date: Thu, 15 Feb 2024 17:36:17 +0100 Subject: [PATCH 1/3] Use `exist_ok=False` for os.makedirs --- src/fcache/cache.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/fcache/cache.py b/src/fcache/cache.py index ebafa01..d108c61 100644 --- a/src/fcache/cache.py +++ b/src/fcache/cache.py @@ -139,8 +139,7 @@ def create(self): """Create the write buffer and cache directory.""" if not self._sync and not hasattr(self, "_buffer"): self._buffer = {} - if not os.path.exists(self.cache_dir): - os.makedirs(self.cache_dir) + os.makedirs(self.cache_dir, exist_ok=False) def clear(self): """Remove all items from the write buffer and cache. From a845a8124d0e675b17648b913dba0c289e9aa71b Mon Sep 17 00:00:00 2001 From: Dimitris Saltaferis <99741372+dimitris-flyr@users.noreply.github.com> Date: Mon, 19 Feb 2024 16:14:20 +0100 Subject: [PATCH 2/3] Update cache.py --- src/fcache/cache.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/fcache/cache.py b/src/fcache/cache.py index d108c61..e65518f 100644 --- a/src/fcache/cache.py +++ b/src/fcache/cache.py @@ -139,7 +139,7 @@ def create(self): """Create the write buffer and cache directory.""" if not self._sync and not hasattr(self, "_buffer"): self._buffer = {} - os.makedirs(self.cache_dir, exist_ok=False) + os.makedirs(self.cache_dir, exist_ok=True) def clear(self): """Remove all items from the write buffer and cache. From 97f555dfdcfd49a377f9ecbabb2f748fd17c44ee Mon Sep 17 00:00:00 2001 From: Dimitris Saltaferis <99741372+dimitris-flyr@users.noreply.github.com> Date: Thu, 22 Feb 2024 16:37:24 +0100 Subject: [PATCH 3/3] Update src/fcache/cache.py Co-authored-by: Thomas Roten --- src/fcache/cache.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/fcache/cache.py b/src/fcache/cache.py index e65518f..53326af 100644 --- a/src/fcache/cache.py +++ b/src/fcache/cache.py @@ -139,7 +139,7 @@ def create(self): """Create the write buffer and cache directory.""" if not self._sync and not hasattr(self, "_buffer"): self._buffer = {} - os.makedirs(self.cache_dir, exist_ok=True) + os.makedirs(self.cache_dir, exist_ok=True) def clear(self): """Remove all items from the write buffer and cache.