Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deposit directory #396

Closed
ahankinson opened this issue Oct 8, 2019 · 3 comments · Fixed by #400
Closed

Deposit directory #396

ahankinson opened this issue Oct 8, 2019 · 3 comments · Fixed by #400
Assignees
Labels
Editorial Editorial issues (no changes to intent) Question Further information is requested Storage Root
Milestone

Comments

@ahankinson
Copy link
Contributor

ahankinson commented Oct 8, 2019

Am I right in remembering that this paragraph:

An OCFL Storage Root may contain a directory named deposit, which may be empty. Implementations may use this directory for assembling new or updated content. Clients performing any other operation, or validating a storage root, must ignore the deposit directory if present.

should now be removed from the spec?

@ahankinson ahankinson added Question Further information is requested Editorial Editorial issues (no changes to intent) Storage Root labels Oct 8, 2019
@ahankinson
Copy link
Contributor Author

And this one:

Except for a deposit directory, sub-directories within an OCFL Storage Root must not contain files that are not part of an OCFL Object. Empty directories must not appear within a storage root.

@awoods
Copy link
Member

awoods commented Oct 8, 2019

Yes, my understanding is that we are removing the concept of an internal deposit directory.

@neilsjefferies
Copy link
Member

Yes, mutable stuff should now reside outside the Storage Root and OCFL. Hence discussed in the Imp Notes, and only as an example.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Editorial Editorial issues (no changes to intent) Question Further information is requested Storage Root
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants