gbranden pushed a commit to branch master
in repository groff.

commit d9751814dcba7d54c9c5292a8efd2ba04c375045
Author: G. Branden Robinson <[email protected]>
AuthorDate: Tue Feb 3 10:27:30 2026 -0600

    [devpdf]: Clarify messages BuildFoundries emits.
    
    * font/devpdf/util/BuildFoundries.pl (LoadFoundry): Clarify diagnostic
      messages.  A digital font, like "URWGothic-DemiOblique.t1", is not the
      same thing as font description file for groff, like "ABI".  We do our
      users no favors by conflating these two concepts.
---
 ChangeLog                          |  8 ++++++++
 font/devpdf/util/BuildFoundries.pl | 18 ++++++++++++------
 2 files changed, 20 insertions(+), 6 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index b24e55b0c..8d7d6c058 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2026-02-03  G. Branden Robinson <[email protected]>
+
+       * font/devpdf/util/BuildFoundries.pl (LoadFoundry): Clarify
+       diagnostic messages.  A digital font, like
+       "URWGothic-DemiOblique.t1", is not the same thing as font
+       description file for groff, like "ABI".  We do our users no
+       favors by conflating these two concepts.
+
 2026-02-03  G. Branden Robinson <[email protected]>
 
        * font/devpdf/util/BuildFoundries.pl (LoadFoundry): Tweak
diff --git a/font/devpdf/util/BuildFoundries.pl 
b/font/devpdf/util/BuildFoundries.pl
index b0dcea058..8c3bed87d 100644
--- a/font/devpdf/util/BuildFoundries.pl
+++ b/font/devpdf/util/BuildFoundries.pl
@@ -148,13 +148,14 @@ sub LoadFoundry
                            unlink $gfont;
                        }
                    }
-                   Notice("copied grops font $gfont") if $gotf;
+                   Notice("copied grops font description $gfont for"
+                          . " gropdf") if $gotf;
 
                }
                else
                {
-                   Warn("cannot read grops font '$r[0]' for Foundry"
-                        . " '$foundry'");
+                   Warn("cannot read grops font description '$r[0]'"
+                        . " for Foundry '$foundry'");
                }
            }
            else
@@ -162,7 +163,8 @@ sub LoadFoundry
                # Use afmtodit to create a groff font description file.
                my $afmfile=LocateAF($foundrypath,$r[5]);
                if (!$afmfile) {
-                   Warn("cannot locate AFM file for font '$gfont'");
+                   Warn("cannot locate AFM file for font description"
+                        . " '$gfont'");
                    next;
                }
                my $psfont=RunAfmtodit($gfont,$afmfile,$r[2],$r[3],$r[4]);
@@ -171,11 +173,15 @@ sub LoadFoundry
                {
                    if 
(!PutDownload($psfont,LocatePF($foundrypath,$r[5]),uc($r[1])))
                    {
-                       unlink $gfont;  # Unable to find the postscript file 
for the font just created by afmtodit
+                       # We were unable to find the PostScript file
+                       # corresponding to the font description just
+                       # created by afmtodit.
+                       unlink $gfont;
                    }
                    else
                    {
-                       Notice("generated $gfont");
+                       Notice("generated font description $gfont"
+                              . " for gropdf");
                    }
                }
                else

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

Reply via email to