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

Problem: Archivematica core contains old METS.xsd #245

Closed
ablwr opened this issue Oct 3, 2018 · 3 comments
Closed

Problem: Archivematica core contains old METS.xsd #245

ablwr opened this issue Oct 3, 2018 · 3 comments
Assignees
Labels
IISH International Institute of Social History
Milestone

Comments

@ablwr
Copy link
Contributor

ablwr commented Oct 3, 2018

Likely related to artefactual/archivematica#773 and may be resolved through that initiative.

Archivematica contains a METS.xsd version 0.9, but the docs and Mets Reader & Writer both use 0.11. Also 0.12 is the latest. Unsure if there is some legacy code that needs clearing out or if it's still being used.

Your environment (version of Archivematica, OS version, etc)

qa/1.x (1.8)


For Artefactual use:
Please make sure these steps are taken before moving this issue from Review to Verified in Waffle:

  • All PRs related to this issue are properly linked 👍
  • All PRs related to this issue have been merged 👍
  • Test plan for this issue has been implemented and passed 👍
  • Documentation regarding this issue has been written and it has been added to the release notes, if needed 👍
@ross-spencer
Copy link
Contributor

ross-spencer commented Oct 18, 2018

@ablwr I happen to be looking at a few things around this today. It is used here in the code https://github.com/artefactual/archivematica/blob/f6be2ab032aa3b5622a3111c26b04555d719339f/src/MCPClient/lib/clientScripts/archivematicaVerifyMets.sh#L32 which is a shell script called at the various points verify mets_structmap.xml is triggered. I'm mot entirely sure on xmllint's effectiveness but that aside, perhaps we should probably update the xsd for now, and then like you say artefactual/archivematica#773 will likely have a say on its long-term future.

@ross-spencer
Copy link
Contributor

Hi @sromkey I added the 1.10 milestone as this will be resolved by artefactual/archivematica#1404

@ablwr
Copy link
Contributor Author

ablwr commented Apr 17, 2019

yayayayayay, bye old code!

@ross-spencer ross-spencer added Status: review The issue's code has been merged and is ready for testing/review. and removed Status: in progress Issue that is currently being worked on. labels Apr 19, 2019
@sallain sallain closed this as completed Jul 8, 2019
@sallain sallain removed the Status: review The issue's code has been merged and is ready for testing/review. label Jul 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IISH International Institute of Social History
Projects
None yet
Development

No branches or pull requests

3 participants