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

Latex 'oneside' classoption causes build warning #8215

Closed
glikely opened this issue Sep 16, 2020 · 0 comments · Fixed by #8216
Closed

Latex 'oneside' classoption causes build warning #8215

glikely opened this issue Sep 16, 2020 · 0 comments · Fixed by #8216

Comments

@glikely
Copy link
Contributor

glikely commented Sep 16, 2020

Describe the bug
Projects which have latex_elements = { 'classoptions':',oneside', } will output irrelevant warnings about useless options. For example:

Package Fancyhdr Warning: \fancyfoot's `E' option without twoside option is use
less on input line 536

To Reproduce

$ git clone https://github.com/arm-software/ebbr
$ cd ebbr
$ make latexpdf

Expected behavior
Should not get warnings from the Fancyhdr package.

Environment info

  • OS: Linux
  • Python version: 3.7.3
  • Sphinx version: mainline

I've got a fix for this bug. I'll post it shortly.

glikely added a commit to glikely/sphinx that referenced this issue Sep 16, 2020
Sphinx.sty had an FIXME bug where documents with the 'oneside' option
would output lots of the following warnings when building the document:

   Package Fancyhdr Warning: \fancyfoot's `E' option without twoside
   option is useless on input line 309.

Fix the problem by using \if@twoside around the offending tex code that
tries to set format for the even pages. 'Even' pages don't exist on
oneside builds, which causes the warning to be emitted.

Fixes: sphinx-doc#8215
@tk0miya tk0miya added this to the 3.3.0 milestone Oct 25, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants