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

extra AUTH48 editorial changes #827

Merged
merged 4 commits into from
May 19, 2024
Merged

extra AUTH48 editorial changes #827

merged 4 commits into from
May 19, 2024

Conversation

robUx4
Copy link
Contributor

@robUx4 robUx4 commented May 5, 2024

On top of #826.

These are changes that need to be pushed in the next AUTH48 cycle. Keeping as draft for now as we may add more changes before the new AUTH48 reply.

@robUx4 robUx4 added spec_main Main Matroska spec document target AUTH48 Final IETF review changes - https://www.rfc-editor.org/pubprocess/auth48/ labels May 5, 2024
@robUx4 robUx4 force-pushed the matroska_auth48_extra branch from 12eadc8 to 46c5861 Compare May 5, 2024 06:47
@robUx4
Copy link
Contributor Author

robUx4 commented May 5, 2024

There's also the https://github.com/ietf-wg-cellar/matroska-specification/tree/9559rebased branch which contains all the changes, the original AUTH48 XML, the changes we made to it and the XML reply file. Ultimately generating rfc9559.xml should match the XML from the RFC editors. But mmark is a bit limited in the formatting so some extra manual changes are needed.

@robUx4
Copy link
Contributor Author

robUx4 commented May 5, 2024

Among other things we should also put the correct/consistent backticking of Matroska element citation in the text.

robUx4 and others added 3 commits May 19, 2024 08:04
Co-authored-by: Moritz Bunkus <mo@bunkus.online>
64-bit EBML VINT is uncommon and not within the Matroska requirements.
@robUx4 robUx4 force-pushed the matroska_auth48_extra branch from 46c5861 to d4f3ae0 Compare May 19, 2024 06:04
@robUx4 robUx4 marked this pull request as ready for review May 19, 2024 06:05
@robUx4
Copy link
Contributor Author

robUx4 commented May 19, 2024

These changes have been integrated in the v2 of the AUTH48 review.

@robUx4 robUx4 merged commit b2c6897 into master May 19, 2024
2 checks passed
@robUx4 robUx4 deleted the matroska_auth48_extra branch May 19, 2024 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AUTH48 Final IETF review changes - https://www.rfc-editor.org/pubprocess/auth48/ spec_main Main Matroska spec document target
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant