Skip to content

fix(bibxml): Process abstract to match current BibXML - #11387

Merged
kesara merged 3 commits into
ietf-tools:feat/bibgenfrom
kesara:fix/spaces_to_nl
Aug 11, 2026
Merged

fix(bibxml): Process abstract to match current BibXML#11387
kesara merged 3 commits into
ietf-tools:feat/bibgenfrom
kesara:fix/spaces_to_nl

Conversation

@kesara

@kesara kesara commented Aug 6, 2026

Copy link
Copy Markdown
Member

No description provided.

@kesara
kesara requested a review from rjsparks August 6, 2026 05:56
@jennifer-richards

Copy link
Copy Markdown
Member

Seems like a strange convention?

@rjsparks

rjsparks commented Aug 7, 2026

Copy link
Copy Markdown
Member

yes.
We're matching what bib spits out now though so that we minimally affect users when we switch bib to using this output. I plan on revisiting it with the RPC after we've alleviated the pain on bib.

@jennifer-richards

Copy link
Copy Markdown
Member

We're matching what bib spits out now though so that we minimally affect users when we switch bib to using this output. I plan on revisiting it with the RPC after we've alleviated the pain on bib.

Matching is fine, but I'm not following context. Are you saying that making this change (replacing two spaces with a newline) will make it match?

@rjsparks

Copy link
Copy Markdown
Member

making this change ... will make it match?

Hmm, yes, but no - I don't know where the newline is coming from - I thought this would be converting two spaces after a period to one. : @kesara?

This is the bibxml served by bib.ietf.org at the moment:

% cat STD/reference.STD.0009.xml| grep -A2 "<abstract"
      <abstract>
        <t>This memo is the official specification of the File Transfer Protocol (FTP) for the DARPA Internet community. The primary intent is to clarify and correct the documentation of the FTP specification, not to change the protocol. The following new optional commands are included in this edition of the specification: Change to Parent Directory (CDUP), Structure Mount (SMNT), Store Unique (STOU), Remove Directory (RMD), Make Directory (MKD), Print Directory (PWD), and System (SYST). Note that this specification is compatible with the previous edition.</t>
      </abstract>

@kesara

kesara commented Aug 10, 2026

Copy link
Copy Markdown
Member Author

@rjsparks I'm not 100% about logic behind new lines in the current BibXML output.

Abstracts as stored separate paragraphs with a blank line and sentences
with two spaces. bib.ietf.org emits one <t> per paragraph and collapses
the whitespace within a paragraph to single spaces; do the same.

Replacing the double spaces with a newline, as 384d384 did, matched
neither: measured against the 9112 RFCs whose abstract bib.ietf.org also
publishes, 1299 abstracts matched before this change and 8946 after.
707 RFCs have no abstract. An <abstract> holding an empty <t> is not
valid BibXML -- it requires at least one non-empty <t> -- and is not what
bib.ietf.org publishes for those RFCs, which is no element at all.
@kesara kesara changed the title fix(bibxml): Replace double spaces in abstract with new line fix(bibxml): Process abstract to match current BibXML Aug 11, 2026
@kesara
kesara merged commit 5711d76 into ietf-tools:feat/bibgen Aug 11, 2026
2 checks passed
@kesara
kesara deleted the fix/spaces_to_nl branch August 11, 2026 22:25
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

Successfully merging this pull request may close these issues.

3 participants