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

/abnf Extract - bogons #37

Open
becarpenter opened this issue Aug 8, 2022 · 2 comments
Open

/abnf Extract - bogons #37

becarpenter opened this issue Aug 8, 2022 · 2 comments

Comments

@becarpenter
Copy link

I couldn't leave this alone. More FYI: I noticed that running the extraction on RFC 3986 not only collects duplicates, but also collects a few lines that are not even intended to be ABNF:
result = ""
$1 = http:
$2 = http
$3 = //www.ics.uci.edu
$4 = www.ics.uci.edu
$5 = /pub/ietf/uri/
$6 =
$7 =
$8 = #Related
$9 = Related
Fixing duplicates is trivial to script, but fixing these bogons looks like an AI problem. This really strengthens the argument that actual code should be embedded in and the checker applied to the xml source, going forward.

@becarpenter
Copy link
Author

And there's this microtool for your amusement:
https://github.com/becarpenter/misc/blob/main/dedup.py

@kesara
Copy link
Member

kesara commented Aug 8, 2022

I am transferring this to the bap. This probably needs to be addressed on bap aex or the replacement tool.

But https://github.com/becarpenter/misc/blob/main/dedup.py could be something author tools can adapt. But I'm not sure about the future of bap.

@kesara kesara transferred this issue from ietf-tools/ietf-at-ui Aug 8, 2022
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

No branches or pull requests

2 participants