Hi Jon,

This small series is against docs/build-script branch.

The first patch addresses the lack of a check after running
sphinx-build to see if it returned some error code.

The second patch is a bonus cleanup: it get rids with
load_config.py, replacing it by a single line at conf.py,
simplifying even further docs Makefile. Thanks for
reminding me about that legacy code that we can drop!

-

I didn't implement yet Akira's suggestion of having a way
to use make -i to ignore errors from latexdocs while
doing pdf builds. I have a couple of ideas on my head.

I'm tempting to implement a:

    --no-sphinx

command line argument at sphinx-build-wrapper, which would
make it ignore the sphinx-build step. this way a makefile
rule like:

    pdf-after-latex: latexdocs
        +$(Q)$(PYTHON3) $(BUILD_WRAPPER) $@ $(RUSTDOC) \
                --sphinxdirs="$(SPHINXDIRS)" \
                --builddir="$(BUILDDIR)" --deny-vf=$(FONTS_CONF_DENY_VF) \
                --css=$(DOCS_CSS) --paper=$(PAPER) --no-sphinx

would allow people to play with "-i", "-k", "--output-sync" and
other make arguments.

However, Friday's happy hour is quickly approaching here... Not the
best time to implement new ideas :-D

I'll try to materialize it next week.

Have a nice weekend!
Mauro

Mauro Carvalho Chehab (2):
  tools/docs: sphinx-build-wrapper: handle sphinx-build errors
  docs: conf.py: get rid of load_config.py

 Documentation/Makefile              |  4 --
 Documentation/conf.py               | 15 +++-----
 Documentation/sphinx/load_config.py | 60 -----------------------------
 tools/docs/sphinx-build-wrapper     |  9 ++++-
 4 files changed, 12 insertions(+), 76 deletions(-)
 delete mode 100644 Documentation/sphinx/load_config.py

-- 
2.51.0