gbranden pushed a commit to branch master
in repository groff.

commit 198ed13f224a2e11c5fde6664c1519f7e6c3f969
Author: G. Branden Robinson <[email protected]>
AuthorDate: Wed Feb 15 23:05:15 2023 -0600

    font/devpdf/tests/basic-fonts-present.sh: Revise.
    
    * font/devpdf/tests/basic-fonts-present.sh: Revise test again, per
      feedback from Deri James.  Even without Ghostscript or URW fonts
      available, gropdf can generate valid PDF documents; it simply can't
      embed fonts in general (without further arrangements made on the host
      system), which means it is restricted to the PDF base 14 fonts.
      That's enough for many purposes, including generating our compiled man
      pages document (with minor degradations to the typeface lists in
      gropdf(1) and grops(1)).  Drop the 'gs' command check, which also
      didn't check for the name of the Ghostscript interpreter determined by
      the "configure" script.  Replace dynamically generated font list with
      a static one (the base 14 fonts plus groff's EURO).
---
 ChangeLog                                | 15 +++++++++++++
 font/devpdf/tests/basic-fonts-present.sh | 36 ++++++++++++++++++--------------
 2 files changed, 35 insertions(+), 16 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 8fd041872..f121f59af 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,18 @@
+2023-02-15  G. Branden Robinson <[email protected]>
+
+       * font/devpdf/tests/basic-fonts-present.sh: Revise test again,
+       per feedback from Deri James.  Even without Ghostscript or URW
+       fonts available, gropdf can generate valid PDF documents; it
+       simply can't embed fonts in general (without further
+       arrangements made on the host system), which means it is
+       restricted to the PDF base 14 fonts.  That's enough for many
+       purposes, including generating our compiled man pages document
+       {with minor degradations to the typeface lists in gropdf(1) and
+       grops(1)}.  Drop the 'gs' command check, which also didn't check
+       for the name of the Ghostscript interpreter determined by the
+       "configure" script.  Replace dynamically generated font list
+       with a static one (the base 14 fonts plus groff's EURO).
+
 2023-02-13  G. Branden Robinson <[email protected]>
 
        * font/devpdf/tests/basic-fonts-present.sh: Revise test.
diff --git a/font/devpdf/tests/basic-fonts-present.sh 
b/font/devpdf/tests/basic-fonts-present.sh
index f54534697..a8dd05b82 100755
--- a/font/devpdf/tests/basic-fonts-present.sh
+++ b/font/devpdf/tests/basic-fonts-present.sh
@@ -21,20 +21,11 @@
 # Ensure that groff's PDF device has the copies it needs of PostScript
 # device font descriptions.
 #
-# We need all of them except SS and ZDR.
-
-# Our configure script warns that gropdf will not be fully functional if
-# gs (Ghostscript) is not available.  Namely, we will have only
-# descriptions for the PostScript Level 1 base 14 fonts, not the Level 2
-# base 35 fonts.  We're using the presence of the command as a proxy for
-# the availabilty of the fonts because locating the latter is
-# unreliable (package names, directory locations, and file names all
-# vary), but if the command is present, the fonts usually are too.
-if ! command -v gs
-then
-    echo "$0: gs command not available; skipping test" >&2
-    exit 77 # skip
-fi
+# This is for the reduced-functionality configuration that occurs when
+# optional dependencies are not met; we require only font descriptions
+# of the PDF base 14 fonts (plus groff's EURO).
+#
+# Another test script checks the fully armed gropdf configuration.
 
 # Locate directory containing the font descriptions for the PostScript
 # device.
@@ -64,8 +55,21 @@ fi
 
 devpdf_fontbuilddir="${abs_top_builddir:-.}"/font/devpdf
 
-psfonts=$(cd "$devps_fontsrcdir" && ls [A-Z]* \
-    | grep -Evx '(DESC\.in|SS|ZDR)')
+psfonts='CB
+CBI
+CI
+CR
+EURO
+HB
+HBI
+HI
+HR
+S
+TB
+TBI
+TI
+TR
+ZD'
 
 fail=
 

_______________________________________________
Groff-commit mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/groff-commit

Reply via email to