# HG changeset patch # User Christian Brabandt # Date 1483880404 -3600 # Node ID 06724e21d8c194a2300a65753cffa3773feaba1a # Parent 80f969a6cae2b3581b618ec45bc8ec07c48c0a7d commit https://github.com/vim/vim/commit/9d9c35651712b88c81f1ae11091de1fd0bbbd35c Author: Bram Moolenaar Date: Sun Jan 8 13:55:06 2017 +0100 patch 8.0.0153: system() test fails on MS-Windows Problem: system() test fails on MS-Windows. Solution: Deal when extra space and CR. diff --git a/src/testdir/test_system.vim b/src/testdir/test_system.vim --- a/src/testdir/test_system.vim +++ b/src/testdir/test_system.vim @@ -4,16 +4,36 @@ function! Test_System() if !executable('echo') || !executable('cat') || !executable('wc') return endif - call assert_equal("123\n", system('echo 123')) - call assert_equal(['123'], systemlist('echo 123')) + let out = system('echo 123') + " On Windows we may get a trailing space. + if out != "123 \n" + call assert_equal("123\n", out) + endif + + let out = systemlist('echo 123') + " On Windows we may get a trailing space and CR. + if out != ["123 \r"] + call assert_equal(['123'], out) + endif + call assert_equal('123', system('cat', '123')) call assert_equal(['123'], systemlist('cat', '123')) call assert_equal(["as\df"], systemlist('cat', ["as\df"])) new Xdummy call setline(1, ['asdf', "pw\er", 'xxxx']) call assert_equal("3\n", system('wc -l', bufnr('%'))) - call assert_equal(['3'], systemlist('wc -l', bufnr('%'))) - call assert_equal(['asdf', "pw\er", 'xxxx'], systemlist('cat', bufnr('%'))) + + let out = systemlist('wc -l', bufnr('%')) + " On Windows we may get a trailing CR. + if out != ["3\r"] + call assert_equal(['3'], out) + endif + + let out = systemlist('cat', bufnr('%')) + " On Windows we may get a trailing CR. + if out != ["asdf\r", "pw\er\r", "xxxx\r"] + call assert_equal(['asdf', "pw\er", 'xxxx'], out) + endif bwipe! call assert_fails('call system("wc -l", 99999)', 'E86:') diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -765,6 +765,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 153, +/**/ 152, /**/ 151,