Skip to content

Use release version of bagit 1.5.4#10

Closed
stain wants to merge 1 commit into
fair-research:masterfrom
stain:patch-1
Closed

Use release version of bagit 1.5.4#10
stain wants to merge 1 commit into
fair-research:masterfrom
stain:patch-1

Conversation

@stain
Copy link
Copy Markdown

@stain stain commented Jul 20, 2017

Presumably the official bagit 1.5.4 can replace 1.5.4.dev-4?
(what aboug bagit-profiles-validator?)

Presumably the official bagit 1.5.4 can replace 1.5.4.dev-4?
(what aboug bagit-profiles-validator?)
@stain stain mentioned this pull request Jul 20, 2017
@mikedarcy
Copy link
Copy Markdown
Collaborator

mikedarcy commented Jul 20, 2017

Unfortunately, this is not so simple. BDBag uses a modified version of the bagit 1.5.4 source: https://github.com/ini-bdds/bagit-python and will not work with the current publicly released bagit-python code. Since I suspect that it is very unlikely that they will accept a PR with my changes, the only way to make this easier would be to include the modified source versions of bagit and bagit-profiles-validator with the BDBag code, and the BDBag code itself could be uploaded to PyPi.

@stain
Copy link
Copy Markdown
Author

stain commented Oct 4, 2017

Not sure what changes you have done -- if this is effectively a fork of bagit-python then yes it should be repackaged under a different name.

@mikedarcy
Copy link
Copy Markdown
Collaborator

Closing this PR. Superseded by #13.

@mikedarcy mikedarcy closed this Oct 10, 2017
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.

2 participants