gbranden pushed a commit to branch master
in repository groff.
commit 644a0f0ee9bf767880f6ea0a48ba118a40218cfb
Author: G. Branden Robinson <[email protected]>
AuthorDate: Mon Jan 26 03:25:58 2026 -0600
[doc]: Quiten build when embeddable fonts absent.
* doc/doc.am: Abstract "-P-e" option to gropdf into new macro,
`groffmanpagesoption`. We can't use it if the build is configured
without URW fonts to embed, so define the macro only if
`HAVE_URW_FONTS` _and_ `USE_GROPDF`.
(doc/groff-man-pages.pdf): Interpolate macro instead of hard-coding
"-P-e".
Fixes noise from build when producing groff-man-pages.pdf without
gropdf's "full service" support level (see gropdf(1)).
---
ChangeLog | 9 +++++++++
doc/doc.am | 12 +++++++-----
2 files changed, 16 insertions(+), 5 deletions(-)
diff --git a/ChangeLog b/ChangeLog
index 16335a31e..860470d39 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2026-01-26 G. Branden Robinson <[email protected]>
+
+ * doc/doc.am: Abstract "-P-e" option to gropdf into new macro,
+ `groffmanpagesoption`. We can't use it if the build is
+ configured without URW fonts to embed, so define the macro only
+ if `HAVE_URW_FONTS` _and_ `USE_GROPDF`.
+ (doc/groff-man-pages.pdf): Interpolate macro instead of
+ hard-coding "-P-e".
+
2026-01-26 G. Branden Robinson <[email protected]>
* font/devpdf/tests/check-default-foundry.sh.in: Die with "hard
diff --git a/doc/doc.am b/doc/doc.am
index 117e1ede0..1dc663b2e 100644
--- a/doc/doc.am
+++ b/doc/doc.am
@@ -44,9 +44,14 @@ DOC_GROFF = \
# There's no point crashing the build due to fonts being unavailable for
# embedding (or selection, even, when their metrics are missing) if we
# knew at configuration time that the venture wouldn't come off.
+#
+# We embed all fonts (-P-e) (if we can) to (1) honor the current PDF
+# standard, (2) ensure consistent rendering of the document, and (3)
+# exercise this feature of gropdf.
if HAVE_URW_FONTS
if USE_GROPDF
pdfmomoption = -P-W
+groffmanpagesoption = -P-e
endif
endif
@@ -284,16 +289,13 @@ $(DOC_GMP_COVER_PAGE):
$(doc_srcdir)/groff-man-pages-cover.groff.in
# Many pages use tbl, a few use eqn, and soelim(1) uses pic. We also
# need groff's FreeEuro font so we can embed it.
-#
-# We embed the fonts (-P-e) to (1) honor the current PDF standard, (2)
-# ensure consistent rendering of the document, and (3) exercise this
-# feature of gropdf.
+
doc/groff-man-pages.pdf: $(GROFF_MAN_PAGES_ALL) \
$(DOC_GMP_COVER_PAGE) \
eqn pic tbl pdfmom \
$(TMAC_PACKAGE_MAN) $(TMAC_PACKAGE_MDOC) font/devps/freeeuro.pfa
$(GROFF_V)$(DOC_PDFMOM) -K utf-8 -pet -mandoc -dHF=HB -rC1 \
- -rCHECKSTYLE=3 -Tpdf -P-e \
+ -rCHECKSTYLE=3 -Tpdf $(groffmanpagesoption) \
$(DOC_GMP_COVER_PAGE) \
$(GROFF_MAN_PAGES1) \
$(tmac_srcdir)/sv.tmac $(GROFF_MAN_PAGES2) \
_______________________________________________
groff-commit mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/groff-commit
