-
Notifications
You must be signed in to change notification settings - Fork 38
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
Index sort should disregard capitalization #680
Comments
@mahoney@nostrum.com commented Huh, according to https://xml2rfc.tools.ietf.org/xml2rfc-doc.html#name-iref-2, it's supposed to be case sensitive? Please look at the subitems under the "Grammar" entry (capitalized entries span two pages in the PDF, and lowercase entries are on the next two pages): |
@johnl@taugh.com commented I believe it says case-sensitive, but that is clearly wrong. Nobody wants a case-sensitive index. Note for the future: case folding ASCII is easy, but case folding Unicode correctly is close to impossible since the folding and even alphabetic order rules are language specific. Unicode UTR#30 provides some rules that are close enough. |
@johnl@taugh.com commented I checked with the author of 7991 and the intention was that the index entries be displayed in their original case. Nobody thinks it is useful to sort all the A's before the a's. So we should fix it. |
@kesara@staff.ietf.org changed status from |
@kesara@staff.ietf.org set owner to |
@kesara@staff.ietf.org changed status from |
@kesara@staff.ietf.org set resolution to |
@kesara@staff.ietf.org commented Fixed in a894852: Fixes a bug where the index is case sensitive. Fixes #680. Commit ready for merge. |
This issue is still seen in RFC 9110 (was draft-ietf-httpbis-semantics-19): https://www.rfc-editor.org/v3test/rfc9110.html |
Looks like the fix (a894852) hasn't been merged into the main branch. |
owner:kesara@staff.ietf.org
resolution_fixed
type_defect
| by mahoney@nostrum.comEntries in the index should be sorted alphabetically without regard for capitalization. Please see the linked file.
Current output:
Desired output:
https://www.rfc-editor.org/v3test/draft-ietf-httpbis-semantics-19.html
https://www.rfc-editor.org/v3test/draft-ietf-httpbis-semantics-19.xml
This issue was in v2 a while back. See https://trac.ietf.org/trac/xml2rfc/ticket/255
Issue migrated from trac:680 at 2022-02-08 07:17:48 +0000
The text was updated successfully, but these errors were encountered: