On Fri, Feb 9, 2018 at 10:52 AM, Ron Yorston <r...@pobox.com> wrote:
> Tweak some tests so they work when the path to the test directory or
> $HOME contains spaces.
>
> Signed-off-by: Ron Yorston <r...@pobox.com>
> ---
>  testsuite/grep.tests                       | 2 +-
>  testsuite/printf.tests                     | 2 +-
>  testsuite/pwd/pwd-prints-working-directory | 2 +-
>  testsuite/sum.tests                        | 6 +++---
>  testsuite/xargs/xargs-works                | 6 ++++--
>  5 files changed, 10 insertions(+), 8 deletions(-)
>
> diff --git a/testsuite/grep.tests b/testsuite/grep.tests
> index ed4ba455e..bed6b451c 100755
> --- a/testsuite/grep.tests
> +++ b/testsuite/grep.tests
> @@ -15,7 +15,7 @@
>
>  testing "grep (exit with error)" "grep nonexistent 2> /dev/null ; echo \$?" \
>         "1\n" "" ""
> -testing "grep (exit success)" "grep grep $0 > /dev/null 2>&1 ; echo \$?" 
> "0\n" \
> +testing "grep (exit success)" "grep grep '$0' > /dev/null 2>&1 ; echo \$?" 
> "0\n" \
>         "" ""
>  # Test various data sources and destinations
>
> diff --git a/testsuite/printf.tests b/testsuite/printf.tests
> index 9a3c87473..963ded94b 100755
> --- a/testsuite/printf.tests
> +++ b/testsuite/printf.tests
> @@ -20,7 +20,7 @@ testing "printf produces no further output 2" \
>         "" ""
>
>  testing "printf repeatedly uses pattern for each argv" \
> -       "${bb}printf '%s\n' foo \$HOME" \
> +       "${bb}printf '%s\n' foo '$HOME'" \

Now it won't work for $HOME containing "'", right?

I think the general consensus is "don't shoot yourself
in the foot by using fancy directory names for your build tree".

Applied, thanks.
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox

Reply via email to