Skip to content

Conversation

mfosterw
Copy link
Contributor

@mfosterw mfosterw commented Sep 9, 2025

Description

Added the help subcommand back into Makefile and make.bat. Also change make.bat to use -M instead of -b

Checklist:

  • I've made sure that tests are updated accordingly (especially if adding or updating a template option)
  • I've updated the documentation or confirm that my change doesn't require any updates

Rationale

The Makefile for sphinx previously had "help" as its default command using the (undocumented, see sphinx-doc/sphinx#13881) -M help make-mode option. This was removed by #5347, which was trying to fix #5346 (it didn't, but the problem was later fixed by sphinx-doc/sphinx-autobuild#175). -b help is not valid.


# Put it first so that "make" without argument is like "make html".
html:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We still need the html target don't we? Also the comment above it out of date

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There wasn't one before #5347, probably because make livehtml does the same and more. I fixed the comment.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually make html is provided by the catch-all at the bottom of the Makefile since it's built-in

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.

Docs (sphinx-autobuild) fails with Docker
2 participants