Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 4fdb6ad

Browse files
committedMay 31, 2023
tests/functions.sh: Fix regression test (re: a566596)
The addition of 'cd' to the fun directory mysteriously caused one of the regression tests to fail, though that directory wasn't in FPATH. The culprit is the .paths file installed in arch/*/bin. This contains the line FPATH=../fun, which causes ksh to look for functions in the fun directory even if FPATH isn't set. The 'typeset -fu cd' command in that tests was not expecting to find anything, and now does, so the test fails. src/cmd/ksh93/tests/functions.sh: - Minimal fix to the regression test. We don't need the old $PATH there in the subshell. Removing it avoids that .paths file being used. I've verified the regression test continues to fail in the same way on ksh 93u+ 2012-08-01 and now again succeeds on 93u+m.
1 parent a566596 commit 4fdb6ad

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed
 

‎src/cmd/ksh93/tests/functions.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1258,7 +1258,7 @@ actual=$(
12581258
print 'function cd { echo "Func cd called with |$*|"; command cd "$@"; }' >"$prefix/functions/cd"
12591259
typeset -fu cd
12601260
1261-
PATH=$tmp/arglebargle:$PATH:$tmp/usr/bin:$tmp/bin
1261+
PATH=$tmp/arglebargle:$tmp/usr/bin:$tmp/bin
12621262
cd "$tmp/usr"
12631263
pwd
12641264
)

0 commit comments

Comments
 (0)
Please sign in to comment.