-
Notifications
You must be signed in to change notification settings - Fork 46
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
Clean schema #329
Clean schema #329
Conversation
So it can be validated properly Adjust the XSLT transform to match the elements with a non-empty namespace
Ultimately ebml_matroska.xml will go away or will be replaced with this one.
xsltproc transforms/schema_clean.xsl ebml_matroska.xml > $@ | ||
|
||
check: matroska_xsd.xml | ||
xmllint --noout --schema ../ebml-specification/EBMLSchema.xsd matroska_xsd.xml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a new thing and requires the EBML specification repo to be downloaded in a certain place.
This could be used to validate changes to the Matroska Schema on Github before allowing a push.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems reasonable to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK with this, and with it superseding #325
Goes on top of #328
Collides with #325 by removing divx, webm and cppname, until we find a proper way to handle them.