[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: c-api.test fails on MS-Windows due to non-portable quoting
From: |
Andy Wingo |
Subject: |
Re: c-api.test fails on MS-Windows due to non-portable quoting |
Date: |
Sat, 23 Jul 2016 23:11:08 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
On Sat 23 Jul 2016 13:18, Eli Zaretskii <address@hidden> writes:
> It fails like this:
>
> Running c-api.test
> 'CUR' is not recognized as an internal or external command,
> operable program or batch file.
> egrep: Unmatched ( or \('CUR' is not recognized as an internal or
> external command, operable program or batch file.
>
> This is because it quotes shell commands /bin/sh '..' style:
Of course, because that's how `system' is specified.
> --- test-suite/tests/c-api.test~0 2016-01-02 13:32:40.000000000 +0200
> +++ test-suite/tests/c-api.test 2016-07-23 14:12:57.257375000 +0300
> @@ -22,7 +22,7 @@
> (define srcdir (cdr (assq 'srcdir %guile-build-info)))
>
> (define (egrep string filename)
> - (zero? (system (string-append "egrep '" string "' " filename
> + (zero? (system (string-append "egrep \"" string "\" " filename
> " >" %null-device))))
>
> (define (seek-offset-test dirname)
>
> OK to push such a change?
I think instead to get this to work on MinGW we should switch to use
system* instead of praying that we get quoting right ;) Something like:
(zero? (system* "egrep" "-q" string filename))
Andy