- Sponsor
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
texinfo: simplify reference emission. #9578
texinfo: simplify reference emission. #9578
Conversation
1186c21
to
07c93ee
Compare
Would it make sense to update that FAQ entry to mention this new confval? As for the core part of this, I unfortunately don't know enough about texinfo to review it. |
640a08e
to
fc545ee
Compare
Yes, it makes fully sense and I've just done that. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with nits.
6dec70e
to
f7d2eed
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
I'll merge this into 4.3.0 release if @jakobandersen approves until X-Day. |
Approved :-), my change request was just to get the |
Thanks for the approval. |
The commit adds a new config value 'texinfo_emit_document_references' that blocks the emission of inline references and make it better readable with legacy stand-alone reader ``info``. Before the change we emit: Default option value for @ref{e,,-Wshift-overflow3}. while with texinfo_emit_document_references == True: Default option value for -Wshift-overflow3. It addresses limitations mentioned in Sphinx' FAQ: https://www.sphinx-doc.org/en/master/faq.html#texinfo-info
f7d2eed
to
9ec829f
Compare
Rebased for |
Merged. Thank you for your contribution! |
The commit adds a new config value 'texinfo_emit_document_references'
that blocks emission of inline references and make it better readable
with legacy stand-alone reader
info
.Before the change we emit:
Default option value for @ref{e,,-Wshift-overflow3}.
while with texinfo_emit_document_references == True:
Default option value for -Wshift-overflow3.
It addresses limitations mentioned in Sphinx' FAQ:
https://www.sphinx-doc.org/en/master/faq.html#texinfo-info