Skip to content
Please note that GitHub no longer supports your web browser.

We recommend upgrading to the latest Google Chrome or Firefox.

Learn more
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

Conversation

@stain
Copy link
Contributor

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

This comment has been minimized.

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

This comment has been minimized.

Copy link
Member

ruebot commented Aug 6, 2015

agreed

stain added 3 commits Aug 6, 2015
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.
@stain stain force-pushed the stain:patch-2 branch from bb9195d to 6f803af Aug 6, 2015
@ruebot

This comment has been minimized.

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

This comment has been minimized.

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

This comment has been minimized.

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
Projects
None yet
3 participants
You can’t perform that action at this time.