Skip to content

Commit

Permalink
Eliminate unneeded use of getconf
Browse files Browse the repository at this point in the history
The `getconf` command hasn't been a builtin for a long time. No known, non
AST implementation, supports `getconf LIBPATH`. So remove that pointless
test code.
  • Loading branch information
krader1961 committed Jan 15, 2020
1 parent b29a24d commit b277665
Showing 1 changed file with 3 additions and 13 deletions.
16 changes: 3 additions & 13 deletions src/cmd/ksh93/tests/builtins.sh
Original file line number Diff line number Diff line change
Expand Up @@ -194,19 +194,9 @@ then
log_error 'set -f not working'
fi
env=
v=$(getconf LIBPATH 2> /dev/null)
for v in ${v//,/ }
do
v=${v#*:}
v=${v%%:*}
eval [[ \$$v ]] && env="$env $v=\"\$$v\""
done
if [[ $(foo=bar; eval foo=\$foo $env exec -c \$SHELL -c \'print \$foo\') != bar ]]
then
log_error '"name=value exec -c ..." not working'
fi
expect=bar
actual=$(foo=bar; eval foo=\$foo exec -c \$SHELL -c \'print \$foo\')
[[ "$actual" == "$expect" ]] || log_error '"name=value exec -c ..." broken' "$expect" "$actual"
#(print -n a;sleep 1; print -n bcde) | { read -N3 a; read -N1 b;}
#[[ $a == $exp ]] || log_error "read -N3 from pipe failed -- expected '$exp', got '$a'"
Expand Down

0 comments on commit b277665

Please sign in to comment.