libstdc++: Reduce output of 'make doc-pdf-doxygen'
authorJonathan Wakely <jwakely@redhat.com>
Wed, 28 Apr 2021 13:49:28 +0000 (14:49 +0100)
committerJonathan Wakely <jwakely@redhat.com>
Wed, 28 Apr 2021 13:49:28 +0000 (14:49 +0100)
Use '@' to prevent Make from echoing the recipe, so that users don't see
this every time:

  if [ -f ${doxygen_pdf} ]; then
    mv ${doxygen_pdf} ${api_pdf} ;
    echo ":: PDF file is ${api_pdf}";
  else
    echo "... error";
    grep -F 'LaTeX Error' ${doxygen_outdir}/latex/refman.log;
    grep -F 'TeX capacity exceeded, sorry' ${doxygen_outdir}/latex/refman.log;
    exit 12;
  fi

The presence of the "error" strings in the output makes it look like an
error happened. By suppressing the echoing user's will only see "error"
if the 'else' branch is taken.

libstdc++-v3/ChangeLog:

* doc/Makefile.am (stamp-pdf-doxygen): Improve comment about
dealing with errors. Use '@' to prevent shell command being
echoed.
* doc/Makefile.in: Regenerate.

libstdc++-v3/doc/Makefile.am
libstdc++-v3/doc/Makefile.in

index cb9b68f..2f8bb07 100644 (file)
@@ -256,12 +256,13 @@ stamp-latex-doxygen: ${doxygen_outdir}/latex
 # look at refman.log and see if TeX's memory is exhausted. Symptoms
 # include asking a wizard to enlarge capacity. If this is the case,
 # find texmf.cnf and add a zero for pool_size, string_vacancies,
-# max_strings, and pool_free values.
+# max_strings, and pool_free values. A much simpler workaround is to install
+# lualatex and set LATEX_CMD_NAME = lualatex in the doxygen user.cfg file.
 # Errors like "File `foo.sty' not found" mean a TeX package is missing.
 stamp-pdf-doxygen: stamp-latex-doxygen ${doxygen_outdir}/pdf
        -(cd ${doxygen_outdir}/latex && $(MAKE) -i pdf;)
-       echo "Generating doxygen pdf file...";
-       if [ -f ${doxygen_pdf} ]; then \
+       @echo "Generating doxygen pdf file...";
+       @if [ -f ${doxygen_pdf} ]; then \
          mv ${doxygen_pdf} ${api_pdf} ; \
          echo ":: PDF file is ${api_pdf}"; \
        else \
index 956b641..18a6a99 100644 (file)
@@ -947,12 +947,13 @@ stamp-latex-doxygen: ${doxygen_outdir}/latex
 # look at refman.log and see if TeX's memory is exhausted. Symptoms
 # include asking a wizard to enlarge capacity. If this is the case,
 # find texmf.cnf and add a zero for pool_size, string_vacancies,
-# max_strings, and pool_free values.
+# max_strings, and pool_free values. A much simpler workaround is to install
+# lualatex and set LATEX_CMD_NAME = lualatex in the doxygen user.cfg file.
 # Errors like "File `foo.sty' not found" mean a TeX package is missing.
 stamp-pdf-doxygen: stamp-latex-doxygen ${doxygen_outdir}/pdf
        -(cd ${doxygen_outdir}/latex && $(MAKE) -i pdf;)
-       echo "Generating doxygen pdf file...";
-       if [ -f ${doxygen_pdf} ]; then \
+       @echo "Generating doxygen pdf file...";
+       @if [ -f ${doxygen_pdf} ]; then \
          mv ${doxygen_pdf} ${api_pdf} ; \
          echo ":: PDF file is ${api_pdf}"; \
        else \