-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
1 test fails on WSL Debian #2996
Comments
We run check on CI so it works for us. Looks like an issue of NO_COLOR support, but I have no clue. Please describe more information about how you build and run tests. |
That test is assuming that the login shell of the current user is a POSIX shell; it probably has something to do with that |
Can you please try this patch? diff --git a/tests/shtest b/tests/shtest
index 14aafbf..ca7e41f 100755
--- a/tests/shtest
+++ b/tests/shtest
@@ -507,11 +507,11 @@ test_no_color=true
$msys && test_no_color=false
$mingw && test_no_color=false
if $test_no_color && command -v script >/dev/null 2>&1; then
- unset NO_COLOR
- if script -qc echo /dev/null >/dev/null 2>&1; then
- faketty() { script -qec "$*" /dev/null; }
- else # macOS
- faketty() { script -q /dev/null "$@" /dev/null |
+ unset -v NO_COLOR
+ if script -qc true /dev/null >/dev/null 2>&1; then
+ faketty() { SHELL=sh script -qec "$*" /dev/null; }
+ else # BSD
+ faketty() { SHELL=sh script -q /dev/null -- "$@" /dev/null |
sed 's/^\x5E\x44\x08\x08//'; }
fi
|
Also, I am confused by that In any case, we should probably just use something more portable like |
@emanuele6 macOS $ printf '\x12HELLO\n' | sed -n 's/^\x12//; l'
HELLO$ |
@emanuele6 Sorry, your patch didn't help. I'm using a simple bash.
|
@polluks Can you try the following commands on the terminal?
|
Or
|
|
|
|
Can you try with |
|
@polluks Did you find any other clue? You're the only one that can debug step by step so appreciated if you can find the cause yourself. Otherwise I'm thinking of closing this issue because there was no further context provided and the test implementation looks correct. |
Describe the bug
FAIL: tests/shtest
To Reproduce
make check
Expected behavior
FAIL: 0
Environment:
Additional context
test-suite.log
The text was updated successfully, but these errors were encountered: