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

#3751 document -M command line option #3938

Merged
merged 3 commits into from
Sep 24, 2017

Conversation

TimKam
Copy link
Member

@TimKam TimKam commented Jul 16, 2017

Documents sphinx-build -M command line option

Feature or Bugfix

  • Doc

Purpose

Documents sphinx-build -M command line option:
Build using the Sphinx make_mode module.

Detail

Only updates the docs. At #3196 (comment) are some suggestions to improve the command line (help) interface. I might create another PR for this eventually.

Relates

#3751

Build LaTeX files and run them through :program:`pdflatex`.

**latexpdfja**
Build LaTeX files and run them through :program:`platex/dvipdfmx`.
Copy link
Member

Choose a reason for hiding this comment

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

Now we recommend to use latexpdf instead.

@@ -89,6 +89,26 @@ Options
See :ref:`builders` for a list of all builders shipped with Sphinx.
Extensions can add their own builders.

.. _make_mode:

.. option:: -M buildername
Copy link
Member

Choose a reason for hiding this comment

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

Sphinx only recognize -M option only if it is placed at first.

$ sphinx-build -v -M html . _build/html
Sphinx v1.7+
Usage: sphinx-build [options] sourcedir outdir [filenames...]

Filename arguments:
  without -a and without filenames, write new and changed files.
  with -a, write all files.
  with filenames, write these.

sphinx-build: error: no such option: -M

Copy link
Contributor

Choose a reason for hiding this comment

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

This prevents doing things like this:

make SPHINXBUILD="python -msphinx -W" -C Doc/api/ html

in place of just:

make -C Doc/api/ html

in order to add extra arguments to sphinx-build (here -W for a strict run).

This worked on older versions of Sphinx as the Makefile did not use the -M argument.

Copy link
Contributor

Choose a reason for hiding this comment

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

Solved my own issue, the new Makefile design lets me use SPHINXOPTS = -W which is a cleaner solution.

Copy link
Contributor

Choose a reason for hiding this comment

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

@peterjc you can use O shortcut, make -C Doc/api/ html O="-W"

@TimKam
Copy link
Member Author

TimKam commented Aug 1, 2017

Thanks for the feedback. I worked in your comments.

@@ -120,7 +120,8 @@ Options

.. option:: --use-make-mode, --no-use-make-mode

Makefile/make.bat uses (or not use) make-mode. Default is use.
:file:`Makefile/make.bat` uses (or doesn't use) :ref:`make-mode <make_mode>`.
Default is `use`, which generates a more concise :file:`Makefile/make.bat`.
Copy link
Member

Choose a reason for hiding this comment

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

Please use literal notation (double-backquote) for use.

@tk0miya tk0miya merged commit 3f1f5bb into sphinx-doc:master Sep 24, 2017
@tk0miya
Copy link
Member

tk0miya commented Sep 24, 2017

Thanks!

@jfbu jfbu mentioned this pull request Dec 22, 2017
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants