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

Rewrite description of sidecar file to remove uncheckable MUST #448

Closed
zimeon opened this issue Apr 8, 2020 · 4 comments · Fixed by #452
Closed

Rewrite description of sidecar file to remove uncheckable MUST #448

zimeon opened this issue Apr 8, 2020 · 4 comments · Fixed by #452
Assignees
Labels
Editorial Editorial issues (no changes to intent) OCFL Object
Milestone

Comments

@zimeon
Copy link
Contributor

zimeon commented Apr 8, 2020

Per #444 we end up with 3 RFC2119 MUST for describing inventory sidecar files. I think we need only 2 MUST and propose changing the text to something like:

Every occurrence of an inventory file [E058]MUST have an accompanying sidecar file named inventory.json.ALGORITHM stating its digest, where ALGORITHM is the chosen digest algorithm for the object. The ALGORITHM [E059]MUST match the value given for the digestAlgorithm key in the inventory. An example might be inventory.json.sha512.

which has the effect of removing E092

@zimeon zimeon added Editorial Editorial issues (no changes to intent) OCFL Object Needs Discussion labels Apr 8, 2020
@zimeon zimeon added this to the 1.0 milestone Apr 8, 2020
@julianmorley
Copy link
Contributor

There's also https://ocfl.io/draft/spec/#E065 a bit further down;

Every inventory file must have a corresponding Inventory Digest.

Which is also redundant.

@zimeon
Copy link
Contributor Author

zimeon commented Apr 8, 2020

New issue for E065?

@julianmorley
Copy link
Contributor

Done!
#449

@zimeon zimeon self-assigned this Apr 21, 2020
@zimeon
Copy link
Contributor Author

zimeon commented Apr 21, 2020

Agreement in editors' call to make change as above #448 (comment)

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) OCFL Object
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants