Skip to content
This repository has been archived by the owner on Aug 20, 2022. It is now read-only.

Add prefix support #1

Closed
CrackerJackMack opened this issue May 25, 2012 · 2 comments
Closed

Add prefix support #1

CrackerJackMack opened this issue May 25, 2012 · 2 comments
Milestone

Comments

@CrackerJackMack
Copy link
Contributor

Add the ability to prefix all uploads in a container. This would just add a string to be beginning of the filenames mimicking a directory structure. An empty object of "application/directory" should probably be created if the prefix is using a forward slash "/".

@CrackerJackMack
Copy link
Contributor Author

Hrm, this is proving to be more difficult than I originally planned due to delta_force_one() using set intersections and differences. While this was a great speed improvement, it's created a problem in comparing prefixed files.

The problem lies in the assumption that the remote object names will match the local object names from os.walk(). This is true right until you add prefix support. Then all the local files are missing the prefix. I could add the prefix to each of the files but then you end up local file names that don't exist. I could add it then remove it each time I went to reference the file but I don't know if that's any more elegant than manually performing the intersections and differences myself.

Thoughts?

@CrackerJackMack
Copy link
Contributor Author

Closed with Commit 9c3592d

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant