gbranden pushed a commit to branch master
in repository groff.
commit 8b28f276cf05c80a1f45667ba04fe46a58164565
Author: G. Branden Robinson <[email protected]>
AuthorDate: Tue Feb 17 15:59:01 2026 -0600
[gropdf]: Clarify BuildFoundries warning message.
* font/devpdf/util/BuildFoundries.pl (LoadFoundry): Clarify warning
diagnostic message when a font description file is unavailable;
"foundry ''" can look like a logic error. Refer to "default foundry"
instead in that scenario.
We expect these diagnostics at the gropdf's "basic" feature support
level, per the "Configuration A" arrangement that Deri implemented. See
commit 6475a5fa8d, 17 February, and
<https://lists.gnu.org/archive/html/groff/2026-02/msg00036.html>.
---
ChangeLog | 7 +++++++
font/devpdf/util/BuildFoundries.pl | 10 +++++++++-
2 files changed, 16 insertions(+), 1 deletion(-)
diff --git a/ChangeLog b/ChangeLog
index 216da5759..0306fdd46 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2026-02-17 G. Branden Robinson <[email protected]>
+
+ * font/devpdf/util/BuildFoundries.pl (LoadFoundry): Clarify
+ warning diagnostic message when a font description file is
+ unavailable; "foundry ''" can look like a logic error. Refer to
+ "default foundry" instead in that scenario.
+
2026-02-17 G. Branden Robinson <[email protected]>
Eliminate new but superfluous Automake variable.
diff --git a/font/devpdf/util/BuildFoundries.pl
b/font/devpdf/util/BuildFoundries.pl
index 8c3bed87d..29cfe3820 100644
--- a/font/devpdf/util/BuildFoundries.pl
+++ b/font/devpdf/util/BuildFoundries.pl
@@ -154,8 +154,16 @@ sub LoadFoundry
}
else
{
+ my $foundry_string;
+ if (!$foundry) {
+ $foundry_string = "default foundry";
+ }
+ else
+ {
+ $foundry_string = "foundry '$foundry'";
+ }
Warn("cannot read grops font description '$r[0]'"
- . " for Foundry '$foundry'");
+ . " for $foundry_string");
}
}
else
_______________________________________________
groff-commit mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/groff-commit
