gbranden pushed a commit to branch master
in repository groff.
commit 313110f870c3148bcae2f9709623563df8e02fd7
Author: G. Branden Robinson <[email protected]>
AuthorDate: Mon Jul 14 18:40:58 2025 -0500
font/scripts/*: Use a better exit status.
* font/scripts/gendesc.sh:
* font/scripts/genfonts.sh: Exit with status 2, not 255, on usage error.
The latter value is not good practice per POSIX--shells use the eighth
bit of the exit status to indicate that a signal was received.
See <https://pubs.opengroup.org/onlinepubs/9799919799/utilities/\
V3_chap02.html#tag_19_08_02>.
---
ChangeLog | 11 +++++++++++
font/scripts/gendesc.sh | 2 +-
font/scripts/genfonts.sh | 2 +-
3 files changed, 13 insertions(+), 2 deletions(-)
diff --git a/ChangeLog b/ChangeLog
index cc38454fa..16d9e4993 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2025-07-14 G. Branden Robinson <[email protected]>
+
+ * font/scripts/gendesc.sh:
+ * font/scripts/genfonts.sh: Exit with status 2, not 255, on
+ usage error. The latter value is not good practice per
+ POSIX--shells use the eighth bit of the exit status to indicate
+ that a signal was received.
+
+ See <https://pubs.opengroup.org/onlinepubs/9799919799/\
+ utilities/V3_chap02.html#tag_19_08_02>.
+
2025-07-14 G. Branden Robinson <[email protected]>
* font/scripts/gendesc.sh:
diff --git a/font/scripts/gendesc.sh b/font/scripts/gendesc.sh
index db8ec7390..3e633bb20 100755
--- a/font/scripts/gendesc.sh
+++ b/font/scripts/gendesc.sh
@@ -3,7 +3,7 @@
progname=${0##*/}
if test -z "$1" || test -z "$2" || test -z "$3" || test -z "$4" || \
test -z "$5"; then
- echo "$progname: missing parameter"; exit 255;
+ echo "$progname: missing parameter"; exit 2;
fi
INPUT=$1
RES=$2
diff --git a/font/scripts/genfonts.sh b/font/scripts/genfonts.sh
index a5ff3025c..b2ff82188 100755
--- a/font/scripts/genfonts.sh
+++ b/font/scripts/genfonts.sh
@@ -3,7 +3,7 @@
# genfonts.sh <input .proto file> <RES> <CPI> <FONT name>
progname=${0##*/}
if test -z "$1" || test -z "$2" || test -z "$3" || test -z "$4"; then
- echo "$progname: missing parameter"; exit 255;
+ echo "$progname: missing parameter"; exit 2;
fi
INPUT=$1
_______________________________________________
groff-commit mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/groff-commit
