summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorrfkelly0 <rfkelly0@67cdc799-7952-0410-af00-57a81ceafa0f>2009-06-07 05:37:33 +0000
committerrfkelly0 <rfkelly0@67cdc799-7952-0410-af00-57a81ceafa0f>2009-06-07 05:37:33 +0000
commit95fce0c619627c1501a071a6640b8bc3f7e6880f (patch)
treefb537498bad81acb21438d828c42f02461762e2f
parent832c918b5575013e8a5b0a584654646815e43a48 (diff)
downloadpyfilesystem-95fce0c619627c1501a071a6640b8bc3f7e6880f.tar.gz
more notes on my API changes
git-svn-id: http://pyfilesystem.googlecode.com/svn/branches/rfk-ideas@156 67cdc799-7952-0410-af00-57a81ceafa0f
-rw-r--r--NOTES.txt15
1 files changed, 9 insertions, 6 deletions
diff --git a/NOTES.txt b/NOTES.txt
index 7e1df55..317b7d2 100644
--- a/NOTES.txt
+++ b/NOTES.txt
@@ -7,16 +7,19 @@ Minify the contained functions:
- remove isabsolutepath(), it wasn't used
Put error class definitions in separate submodule 'errors'
- - deeper exception heirachy and less redundancy
- (e.g. no more `raise UnsupportedError("UNSUPPORTED")`)
+ - less redundancy (e.g. no more `raise UnsupportedError("UNSUPPORTED")`)
+ - deeper exception heirachy (e.g. `ParentDirectoryMissingError`)
+This is designed to allow me to be more optimistic; rather than checking all
+preconditions before trying an action, I can just let it fail and branch on
+the exception. Important for reducing the number of accesses to a remote FS.
Remove the notion of hidden files from the base FS class.
- - it's not the filesystem's business to decide what files I want to see
- has lead to several bugs with copying/moving directories
- - moved the logic into a separate wrapper class, currently
- in fs.wrapfs.HideDotFiles
+ - it's not the filesystem's business to decide what files I want to see
+ - moved the logic into a separate wrapper class "HideDotFiles"
Remove xattr support from base FS class, making it a separate interface.
+ - has lead to several bugs with copying/moving files and directories
- now defined in fs.xattrs module
- SimulateXAttr wrapper class contains the same logic
- removexattr() -> delxattr() in line with python's get/set/del tradition
@@ -31,5 +34,5 @@ Split up the test definitions a little:
- use a separate FSTestCases mixin rather than subclassing TestOSFS
- path helpers testcases in their own module
- zipfs in its own module since it's different to all the others
- - s3fs in its own module since it's very slow
+ - s3fs in its own module since it's very slow and costs money to test