gbranden pushed a commit to branch master
in repository groff.
commit b46e21be5d88ff1508ec01305a0772c43d2c5cb1
Author: G. Branden Robinson <[email protected]>
AuthorDate: Fri Sep 19 19:59:39 2025 -0500
[groff]: Improve test and mark it as XFAIL.
* src/roff/groff/tests/html-device-works-with-grn-and-eqn.sh: Make test
more sensitive. While the infinite loop introduced in commit
c71b4ef4aa and later fixed has not regressed, the input document does
not produce usable output. Look for that usable output. That makes
the test fail. Unfortunately this is not evidence of a recent defect;
groff 1.23.0, 1.22.4, and 1.22.3 fail the same way with this input.
Therefore...
* src/roff/groff/groff.am (XFAIL_TESTS): Mark it as an expected failure.
---
ChangeLog | 12 ++++++++++++
src/roff/groff/groff.am | 1 +
src/roff/groff/tests/html-device-works-with-grn-and-eqn.sh | 13 +++++++++----
3 files changed, 22 insertions(+), 4 deletions(-)
diff --git a/ChangeLog b/ChangeLog
index 7cf6d8aa7..a9deb0f96 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+2025-09-19 G. Branden Robinson <[email protected]>
+
+ * src/roff/groff/tests/html-device-works-with-grn-and-eqn.sh:
+ Make test more sensitive. While the infinite loop introduced in
+ commit c71b4ef4aa and later fixed has not regressed, the input
+ document does not produce usable output. Look for that usable
+ output. That makes the test fail. Unfortunately this is not
+ evidence of a recent defect; groff 1.23.0, 1.22.4, and 1.22.3
+ fail the same way with this input. Therefore...
+ * src/roff/groff/groff.am (XFAIL_TESTS): Mark it as an expected
+ failure.
+
2025-09-19 G. Branden Robinson <[email protected]>
* src/roff/troff/hvunits.h (hunits operator -): Fix misleading
diff --git a/src/roff/groff/groff.am b/src/roff/groff/groff.am
index f99e85558..b132cfe42 100644
--- a/src/roff/groff/groff.am
+++ b/src/roff/groff/groff.am
@@ -129,6 +129,7 @@ EXTRA_DIST += \
groff_XFAIL_TESTS = \
src/roff/groff/tests/current-language-and-environment-in-sync.sh \
+ src/roff/groff/tests/html-device-works-with-grn-and-eqn.sh \
src/roff/groff/tests/stringup-request-transforms-non-basic-latin.sh
XFAIL_TESTS += $(groff_XFAIL_TESTS)
diff --git a/src/roff/groff/tests/html-device-works-with-grn-and-eqn.sh
b/src/roff/groff/tests/html-device-works-with-grn-and-eqn.sh
index c58caba78..33f8a2fee 100755
--- a/src/roff/groff/tests/html-device-works-with-grn-and-eqn.sh
+++ b/src/roff/groff/tests/html-device-works-with-grn-and-eqn.sh
@@ -34,15 +34,20 @@ done
# Commit c71b4ef4aa provoked an infinite loop in post-grohtml with these
# preprocessors.
-input='.EQ
+input='.
+.pp
+Hello, world!
+.EQ
gsize 12
delim $$
.EN
.pp
.pp
-The faster clocks are $ PN $'
+The faster clocks are $ PN $
+.'
-output=$("$groff" -b -ww -Thtml -eg -me "$input")
-test -n "$output"
+output=$(printf "%s\n" | "$groff" -b -ww -Thtml -eg -me)
+echo "$output"
+echo "$output" | grep -q 'faster clocks'
# vim:set autoindent expandtab shiftwidth=4 tabstop=4 textwidth=72:
_______________________________________________
groff-commit mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/groff-commit
