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

Use release version of bagit 1.5.4 #10

Closed
wants to merge 1 commit into from

Conversation

stain
Copy link

@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
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
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
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