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

Proposing Manifests-Complete #6

Closed
wants to merge 3 commits into from
Closed

Conversation

stain
Copy link
Contributor

@stain stain commented Aug 6, 2015

See jkunze/bagitspec#6 which says that manifests are not necessarily individually complete.

I think a profile should be able to restrict this so that a consumer can rely on a fixed list of manifests to be known to be complete lists of the whole payload.

A similar Tag-Manifests-Complete property could be even more useful as the BagIt spec do not require all tagfiles to be listed at all, but would have to specify which tagfiles are excluded (e.g. the tag manifest itself)

@mjordan
Copy link
Collaborator

mjordan commented Aug 6, 2015

Given the discussion at jkunze/bagitspec#6 this addition makes sense but let's wait until that issue is marked as closed before moving forward with this.

@ruebot
Copy link
Member

ruebot commented Aug 6, 2015

agreed

stain added 3 commits August 6, 2015 15:52
See jkunze/bagitspec#6 - I think a profile should be able to restrict this so that a consumer can rely on a shortlist of manifests as complete manifests.
.. also handling case of no Manifests-Required listing.
.. adding Manifests-Complete means a minor version upgrade.
@ruebot
Copy link
Member

ruebot commented Nov 14, 2018

Since there has been a fair bit of discussion on jkunze/bagitspec#6, should we revisit this one again? ...fully acknowledging it's been a really long time since it was originally submitted 😩

@mjordan
Copy link
Collaborator

mjordan commented Nov 14, 2018

@stain I'm trying to work backward from this issue... you said at jkunze/bagitspec#6 (comment) that the Bagit spec itself will resolve this but I can't find reference on PR 19 to this being resolved. Are you still of that opinion?

That said, I also have no problem with your initial proposal to add a Tag-Manifests-Complete directive to the Profiles spec since its purpose is to tighten up the Bagit spec.

@ruebot
Copy link
Member

ruebot commented Nov 24, 2019

I believe this is resolved via jkunze/bagitspec#6 (comment)

@ruebot ruebot closed this Nov 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants