deri pushed a commit to branch master
in repository groff.
commit a8b3f68bffecb8931be9c042aaf82f269ea36f40
Author: Deri James <d...@chuzzlewit.myzen.co.uk>
AuthorDate: Sun Aug 3 12:56:20 2025 +0100
[BuildFoundries]: Improve diagnostics.
* font/devpdf/util/BuildFoundries.pl [RunAfmtodit]: Diagnose
return code from running afmtodit.
Fixes <https://savannah.gnu.org/bugs/index.php?67390> thanks
to G. Branden Robinson for report and patch.
---
ChangeLog | 10 ++++++++++
font/devpdf/util/BuildFoundries.pl | 11 ++++++++++-
2 files changed, 20 insertions(+), 1 deletion(-)
diff --git a/ChangeLog b/ChangeLog
index 6cd545dba..dc6add925 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -67,6 +67,16 @@
with proper white space and correct grammar when only one
duplicate mapping is encountered.
+ 2025-08-03 Deri James <d...@chuzzlewit.myzen.co.uk>
+
+ [BuildFoundries]: Improve diagnostics.
+
+ * font/devpdf/util/BuildFoundries.pl [RunAfmtodit]: Diagnose
+ return code from running afmtodit.
+
+ Fixes <https://savannah.gnu.org/bugs/index.php?67390> thanks
+ to G. Branden Robinson for report and patch.
+
2025-08-02 G. Branden Robinson <g.branden.robin...@gmail.com>
* tmac/groff_man.7.man.in: Un-deprecate `HP` macro. Retain
diff --git a/font/devpdf/util/BuildFoundries.pl
b/font/devpdf/util/BuildFoundries.pl
index 336ecd9fd..c63ca09a6 100644
--- a/font/devpdf/util/BuildFoundries.pl
+++ b/font/devpdf/util/BuildFoundries.pl
@@ -211,9 +211,18 @@ sub RunAfmtodit
system("$cmd $enc '$afmfile' $map $gfont");
+ if ($? == -1) {
+ Die("unable to run afmtodit: $!\n");
+ }
+ elsif ($? & 127) {
+ Die("afmtodit terminated by signal " . ($? & 127) . ", "
+ . (($? & 128) ? "with" : "without") . " core dump");
+ }
+
if ($?)
{
- Warn("failed running \"$cmd $enc '$afmfile' $map $gfont\"");
+ Warn("command \"$cmd $enc '$afmfile' $map $gfont\" exited"
+ . " with status $?");
unlink $gfont;
return('');
}
_______________________________________________
groff-commit mailing list
groff-commit@gnu.org
https://lists.gnu.org/mailman/listinfo/groff-commit
