[pkgsrc/trunk]: pkgsrc/regress/tools/files regress/tools: clarify that only t...

[rillig <rillig%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/fa9bde6ef168
branches:  trunk
changeset: 316129:fa9bde6ef168
user:      rillig <rillig%pkgsrc.org@localhost>
date:      Wed Dec 05 19:01:40 2018 +0000

description:
regress/tools: clarify that only the NetBSD 7 shell exhibits this bug

The test has been changed again to fail since making the test pass would
mean this bug could later silently show up inside a large GNU configure
script where it would be impossible to find. Therefore, when using pkgsrc
on NetBSD 7, it is better to switch to another shell for doing pkgsrc
work.

The comment has been updated to be more accurate.

diffstat:

 regress/tools/files/sh-test.sh |  8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diffs (22 lines):

diff -r 2a53a9f0716d -r fa9bde6ef168 regress/tools/files/sh-test.sh
--- a/regress/tools/files/sh-test.sh    Wed Dec 05 18:50:25 2018 +0000
+++ b/regress/tools/files/sh-test.sh    Wed Dec 05 19:01:40 2018 +0000
@@ -1,5 +1,5 @@
 #! /bin/sh
-# $NetBSD: sh-test.sh,v 1.2 2018/11/30 19:55:26 rillig Exp $
+# $NetBSD: sh-test.sh,v 1.3 2018/12/05 19:01:40 rillig Exp $
 #
 
 set -e
@@ -37,8 +37,8 @@
 nl="
 "
 
-# This test fails with /bin/sh from 2018-11-08, see bin/53754.
-if [ "`uname -s`" != "NetBSD" ]; then
+# If this test fails because the result is "#define bindir \"", the
+# tested shell may be from NetBSD 7.0, which is known to have this bug.
+# Since NetBSD 8, this bug is fixed. See https://gnats.netbsd.org/43469.
 line="#define bindir \"/usr/bin\" /* bar */"
 sh_assert_equals "removing C comments" "#define bindir \"/usr/bin\" " "${line%%/\**}"
-fi