From 0ac8317ddceee6d705c175e69c0825078246c7b9 Mon Sep 17 00:00:00 2001 From: Christopher Allan Webber Date: Tue, 22 Jan 2013 14:32:00 -0600 Subject: [PATCH] Removing docstring bit about delete_file possibly deleting directories in the future I agree that delete_dir as a separate operation is a better way to do things, especially since there is a non-recursive deletion option that will politely fail if the directory is not empty. --- mediagoblin/storage/__init__.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/mediagoblin/storage/__init__.py b/mediagoblin/storage/__init__.py index 77e4212e..57218eb5 100644 --- a/mediagoblin/storage/__init__.py +++ b/mediagoblin/storage/__init__.py @@ -102,10 +102,6 @@ class StorageInterface(object): def delete_file(self, filepath): """ Delete or dereference the file (not directory) at filepath. - - TODO: is the below comment correct? AFAIK, we won't clean up empty directories... - This might need to delete directories, buckets, whatever, for - cleanliness. (Be sure to avoid race conditions on that though) """ # Subclasses should override this method. self.__raise_not_implemented() -- 2.25.1