Re: Test failure building 8.2.3404
Bram Moolenaar wrote: > Elimar Riesebieter wrote: > > > Environment: amd64 > > Shell: zsh > > Term: tmux session > > Compiler: clang-12 > > LANG: C > > > > Hi all, > > > > Building 8.2.3404 gives me > > > > ### > > Failures: > > From test_signals.vim: > > Found errors in Test_deadly_signal_TERM(): > > Run 1: > > command line..script > > /source/vim/vim-8.2.3404/src/vim-nox/testdir/runtest.vim[486]..function > > RunTheTest[44]..Test_deadly_signal_TERM line 35: Expected ['VimLeavePre > > triggered', 'VimLeave triggered'] but got ['VimLeavePre triggered'] > > Run 2: > > command line..script > > /source/vim/vim-8.2.3404/src/vim-nox/testdir/runtest.vim[520]..function > > RunTheTest[44]..Test_deadly_signal_TERM line 35: Expected ['VimLeavePre > > triggered', 'VimLeave triggered'] but got ['VimLeavePre triggered'] > > Run 3: > > command line..script > > /source/vim/vim-8.2.3404/src/vim-nox/testdir/runtest.vim[520]..function > > RunTheTest[44]..Test_deadly_signal_TERM line 35: Expected ['VimLeavePre > > triggered', 'VimLeave triggered'] but got ['VimLeavePre triggered'] > > Flaky test failed too often, giving up > > > > TEST FAILURE > > ### > > It's unrelated to the 8.2.3404 patch, with what patch did it start > failing? > > CI doesn't report this error. Does the failure go away when you change > something in your setup? e.g. not use tmux or use another shell. This failure looks related or maybe similar to https://github.com/vim/vim/issues/7402 There it was found that building Vim with `-D_REENTRANT` fixes it. Can you confirm Elimar? Regards Dominique -- -- 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/CAON-T_gCnW4DHciwmbBiQWqeKY_FBBFz1UG-B63MjgMs2m-Oig%40mail.gmail.com.
Re: Test failure building 8.2.3404
Elimar Riesebieter wrote: > Environment: amd64 > Shell: zsh > Term: tmux session > Compiler: clang-12 > LANG: C > > Hi all, > > Building 8.2.3404 gives me > > ### > Failures: > From test_signals.vim: > Found errors in Test_deadly_signal_TERM(): > Run 1: > command line..script > /source/vim/vim-8.2.3404/src/vim-nox/testdir/runtest.vim[486]..function > RunTheTest[44]..Test_deadly_signal_TERM line 35: Expected ['VimLeavePre > triggered', 'VimLeave triggered'] but got ['VimLeavePre triggered'] > Run 2: > command line..script > /source/vim/vim-8.2.3404/src/vim-nox/testdir/runtest.vim[520]..function > RunTheTest[44]..Test_deadly_signal_TERM line 35: Expected ['VimLeavePre > triggered', 'VimLeave triggered'] but got ['VimLeavePre triggered'] > Run 3: > command line..script > /source/vim/vim-8.2.3404/src/vim-nox/testdir/runtest.vim[520]..function > RunTheTest[44]..Test_deadly_signal_TERM line 35: Expected ['VimLeavePre > triggered', 'VimLeave triggered'] but got ['VimLeavePre triggered'] > Flaky test failed too often, giving up > > TEST FAILURE > ### It's unrelated to the 8.2.3404 patch, with what patch did it start failing? CI doesn't report this error. Does the failure go away when you change something in your setup? e.g. not use tmux or use another shell. -- I used to be indecisive, now I'm not sure. /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net \\\ /// \\\ \\\sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ /// \\\help me help AIDS victims -- http://ICCF-Holland.org/// -- -- 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/202109052022.185KMTkO1169302%40masaka.moolenaar.net.
Test failure building 8.2.3404
Environment: amd64 Shell: zsh Term: tmux session Compiler: clang-12 LANG: C Hi all, Building 8.2.3404 gives me ### Failures: From test_signals.vim: Found errors in Test_deadly_signal_TERM(): Run 1: command line..script /source/vim/vim-8.2.3404/src/vim-nox/testdir/runtest.vim[486]..function RunTheTest[44]..Test_deadly_signal_TERM line 35: Expected ['VimLeavePre triggered', 'VimLeave triggered'] but got ['VimLeavePre triggered'] Run 2: command line..script /source/vim/vim-8.2.3404/src/vim-nox/testdir/runtest.vim[520]..function RunTheTest[44]..Test_deadly_signal_TERM line 35: Expected ['VimLeavePre triggered', 'VimLeave triggered'] but got ['VimLeavePre triggered'] Run 3: command line..script /source/vim/vim-8.2.3404/src/vim-nox/testdir/runtest.vim[520]..function RunTheTest[44]..Test_deadly_signal_TERM line 35: Expected ['VimLeavePre triggered', 'VimLeave triggered'] but got ['VimLeavePre triggered'] Flaky test failed too often, giving up TEST FAILURE ### Thanks for investigation Elimar -- The path to source is always uphill! -unknown- -- -- 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/20210905194032.ndxpyuhnkdcnqw74%40baumbart.home.lxtec.de.
Typo fixes from maintainer Charles E. Campbell
Hey! Here are typo fixes from maintainer Charles E. Campbell. Their email turned out to be invalid so I'm contacting the dev maillist instead. Tried sending a normal email to the vim-dev maillist with the patch attachment but got no response so I assume that also failed. I've uploaded the patch to https://pastebin.com/TcNq4LcX Dundar Göc. -- -- 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/65c9c467-db7e-458e-87f5-c865d223bf92n%40googlegroups.com.
Patch 8.2.3404
Patch 8.2.3404 Problem:Vim9: no error for white space before "(". Solution: Give an error, like in a compiled function. Files: src/userfunc.c, src/testdir/test_vim9_func.vim *** ../vim-8.2.3403/src/userfunc.c 2021-08-28 15:56:03.461179888 +0200 --- src/userfunc.c 2021-09-05 16:29:39.056668483 +0200 *** *** 4921,4933 // Skip white space to allow ":call func ()". Not good, but required for // backward compatibility. startarg = skipwhite(arg); - rettv.v_type = VAR_UNKNOWN; // clear_tv() uses this - if (*startarg != '(') { semsg(_(e_missing_paren), eap->arg); goto end; } /* * When skipping, evaluate the function once, to find the end of the --- 4921,4936 // Skip white space to allow ":call func ()". Not good, but required for // backward compatibility. startarg = skipwhite(arg); if (*startarg != '(') { semsg(_(e_missing_paren), eap->arg); goto end; } + if (in_vim9script() && startarg > arg) + { + semsg(_(e_no_white_space_allowed_before_str_str), "(", eap->arg); + goto end; + } /* * When skipping, evaluate the function once, to find the end of the *** *** 4969,4974 --- 4972,4978 funcexe.partial = partial; funcexe.selfdict = fudi.fd_dict; funcexe.check_type = type; + rettv.v_type = VAR_UNKNOWN; // clear_tv() uses this if (get_func_tv(name, -1, , , , ) == FAIL) { failed = TRUE; *** ../vim-8.2.3403/src/testdir/test_vim9_func.vim 2021-08-19 21:08:26.178565751 +0200 --- src/testdir/test_vim9_func.vim 2021-09-05 16:35:12.603822749 +0200 *** *** 467,472 --- 467,476 MyVarargs('one', 'two', 'three')->assert_equal('one,two,three') enddef + def Test_call_white_space() + CheckDefAndScriptFailure2(["call Test ('text')"], 'E476:', 'E1068:') + enddef + def MyDefaultArgs(name = 'string'): string return name enddef *** ../vim-8.2.3403/src/version.c 2021-09-04 21:20:37.895881447 +0200 --- src/version.c 2021-09-05 16:32:22.820257344 +0200 *** *** 757,758 --- 757,760 { /* Add new patch number below this line */ + /**/ + 3404, /**/ -- No engineer can take a shower without wondering if some sort of Teflon coating would make showering unnecessary. (Scott Adams - The Dilbert principle) /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net \\\ /// \\\ \\\sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ /// \\\help me help AIDS victims -- http://ICCF-Holland.org/// -- -- 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/202109051437.185EbbfL1097705%40masaka.moolenaar.net.