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

generate a new document for corrections #413

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

robUx4
Copy link
Contributor

@robUx4 robUx4 commented Jun 20, 2021

The format is simpler that the one in RFC 8540. If needed I can update the format.

We may also add ebml.xml as an annex, making it normative. But it's more work than necessary, possibly leading to new fixes...

Fixes #412

@robUx4 robUx4 added the errata-rfc8794 Errors found in RFC 8794 label Jun 20, 2021
@robUx4
Copy link
Contributor Author

robUx4 commented Jun 20, 2021

We need to create a new entry in the CELLAR documents https://datatracker.ietf.org/wg/cellar/documents/

@SpencerDawkins
Copy link

@robUx4, thanks for starting this work. I'm not good at generating the EBML specifications, but I did poke around a bit.

If I could make a couple of suggestions (which you may have already taken care of - my apologies, if so):

  • I'd suggest using draft-ietf-cellar-rfc8794-corrections as the output file name. Unless @mcr has "other thoughts" ("spencer-corrections"?), I don't see that we need to go through the "post as individual and see if the working group adopts it, and then rename" shuffle.
  • It is going to be much easier to take this draft through the approvals process, at some point in time, if you actually cut-and-paste the text being modified, and then the complete text with corrections, for each correction, using "OLD" and "NEW" to prefix them, just as we would for errata (if we were doing the RFC errata process). If you're correcting a sentence in the middle of a paragraph, show the entire paragraph (OLD and NEW), so that it's obvious to the reader where this text is in the document.
  • If you can summarize in a sentence or two about why the original text needs to be corrected, that would be very helpful. I see something like that (I think!), so thank you for that.
  • Just looking at the corrections listed so far - I see that https://www.rfc-editor.org/rfc/rfc8141.html obsoletes RFC 2141, but I'm not seeing that RFC 2141 has been deprecated (unless I'm missing it). Is the goal here to make use of extensions that are allowed in RFC 8141 (https://www.rfc-editor.org/rfc/rfc8141.html#appendix-B),, or just to use the current version of the standards-track URN specification? Either way, we should talk about who deprecated RFC 2141, or change the wording to Obsolete.

@robUx4
Copy link
Contributor Author

robUx4 commented Jun 22, 2021

@robUx4
Copy link
Contributor Author

robUx4 commented Oct 30, 2022

Given how RFC errata work, I opted to keep updating the current document with changes, so it can be diff'ed with the current RFC to produce the errata text.

@robUx4
Copy link
Contributor Author

robUx4 commented Apr 9, 2023

There is now an rfc8794 branch that collects all the errata so we can regenerate a plain with these errata applies.

@robUx4 robUx4 added this to the new-rfc milestone Jul 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
errata-rfc8794 Errors found in RFC 8794
Projects
None yet
Development

Successfully merging this pull request may close these issues.

create a new document containing all errata and additions to RFC 8794
2 participants