* Christian Brabandt <cbli...@256bit.org> [2021-07-05 16:32 +0200]: > > On Mo, 05 Jul 2021, Elimar Riesebieter wrote: > > > Hi Bram, > > > > * Bram Moolenaar <b...@moolenaar.net> [2021-07-05 14:10 +0200]: > > > > > > > > Patch 8.2.3108 > > > Problem: Test for remote_foreground() fails. (Elimar Riesebieter) > > > Solution: Check that $DISPLAY is set. (Christian Brabandt) > > > Files: src/testdir/check.vim, src/testdir/test_clientserver.vim, > > > src/testdir/test_vim9_builtin.vim > > > > > > > your patch differs from Christian's version. Compiling 8.2.3108 gives > > me: > > Yes, I also wondered about some of the changes. The problem is, when > using empty and quoting the argument, the result cannot be true anymore > > So I think it needs this change: > > diff --git a/src/testdir/check.vim b/src/testdir/check.vim > index 504a1928e..b2f457074 100644 > --- a/src/testdir/check.vim > +++ b/src/testdir/check.vim > @@ -139,7 +139,7 @@ endfunc > " Command to Check for an environment variable > command -nargs=1 CheckEnv call CheckEnv(<f-args>) > func CheckEnv(name) > - if empty('$' .. a:name) > + if empty($ .. a:name) > throw 'Skipped: Environment variable ' .. a:name .. ' is not set' > endif > endfunc
This Patch fails with: Failures: From test_clientserver.vim: Found errors in Test_client_server(): Run 1: Caught exception in Test_client_server(): Vim(if):E116: Invalid arguments for function empty($ .. a:name) @ command line..script /source/vim/vim-8.2.3108/src/vim-gtk3/testdir/runtest.vim[473]..function RunTheTest[44]..Test_client_server[5]..Check_X11_Connection[2]..CheckEnv, line 1 Run 2: Caught exception in Test_client_server(): Vim(if):E116: Invalid arguments for function empty($ .. a:name) @ command line..script /source/vim/vim-8.2.3108/src/vim-gtk3/testdir/runtest.vim[507]..function RunTheTest[44]..Test_client_server[5]..Check_X11_Connection[2]..CheckEnv, line 1 Run 3: Caught exception in Test_client_server(): Vim(if):E116: Invalid arguments for function empty($ .. a:name) @ command line..script /source/vim/vim-8.2.3108/src/vim-gtk3/testdir/runtest.vim[507]..function RunTheTest[44]..Test_client_server[5]..Check_X11_Connection[2]..CheckEnv, line 1 Flaky test failed too often, giving up From test_vim9_builtin.vim: Found errors in Test_remote_foreground(): Caught exception in Test_remote_foreground(): Vim(if):E116: Invalid arguments for function empty($ .. a:name) @ command line..script /source/vim/vim-8.2.3108/src/vim-gtk3/testdir/runtest.vim[473]..function RunTheTest[44]..Test_remote_foreground[4]..CheckEnv, line 1 Elimar -- >what IMHO then? IMHO - Inhalation of a Multi-leafed Herbal Opiate ;) --posting from alex in debian-user-- -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For more information, visit http://www.vim.org/maillist.php --- You received this message because you are subscribed to the Google Groups "vim_dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/vim_dev/20210705152843.tkr72jtrhlaqtfgw%40toy.home.lxtec.de.