Hi Bram, 2015/12/29 Tue 21:59:54 UTC+9 Bram Moolenaar wrote: > Ken Takata wrote: > > > 2015/12/29 Tue 6:25:06 UTC+9 Bram Moolenaar wrote: > > > Christian Brabandt wrote: > > > > > > > On Mo, 28 Dez 2015, Bram Moolenaar wrote: > > > > > > > > > Patch 7.4.982 > > > > > Problem: Keeping the list of tests updated is a hassle. > > > > > Solution: Move the list to a separate file, so that it only needs > > > > > to be > > > > > udpated in one place. > > > > > Files: src/testdir/Make_amiga.mak, src/testdir/Make_dos.mak, > > > > > src/testdir/Make_ming.mak, src/testdir/Make_os2.mak, > > > > > src/testdir/Make_vms.mms, src/testdir/Makefile, > > > > > src/testdir/Make_all.mak > > > > > > > > Starting from this patch, this breaks appveyor: > > > > This built timed out after 60 minutes when trying to run test49: > > > > https://ci.appveyor.com/project/chrisbra/vim/build/204 > > > > After that patch, it fails when copying test49.in: > > > > https://ci.appveyor.com/project/chrisbra/vim/build/205 > > > > https://ci.appveyor.com/project/chrisbra/vim/build/206 > > > > > > > > Don't know, what the problem is and how to solve it. > > > > > > I noticed. Somehow test49.in wasn't processed. It was not in the list > > > of tests for MS-Windows before, so I removed it in patch 7.4.986. > > > > > > Now it appears it doesn't run any tests... > > > OK, I can reproduce it locally. > > > It's because appveyor doesn't specify a target, and the first dependency > > > is in Make_all.mak: > > > > > > test49.out: test49.vim > > > > > > I'll specify a default target before including Make_all.mak > > > > After 7.4.988, still something strange happen. When running a single test > > as the following: > > > > nmake -f Make_dos.mak clean > > nmake -f Make_dos.mak VIMPROG=..\gvim test86.out > > > > Copy command fails like this: > > > > > > https://ci.appveyor.com/project/chrisbra/vim/build/206 > > > > Maybe this is caused by that testXX.out has multiple dependency rules. > > Attached patch fixes this. > > That problem has already been fixed. We are now back to test 86 > failing. Before the diff was just one line, now it's a big difference. > On my system the test passes without problems. It might depend on the > Python version.
No, it still occurs: https://ci.appveyor.com/project/k-takata/vim/build/37 I changed appveyor.yml to execute only test11: https://github.com/k-takata/vim/commit/d16eba3da82b8e7ba6dd672650eeb5b89225eab5 I wrote another patch. This is simpler than before, but I don't understand why this patch solves the problem: --- a/src/testdir/Make_dos.mak +++ b/src/testdir/Make_dos.mak @@ -27,9 +27,6 @@ DOSTMP_INFILES = $(DOSTMP_OUTFILES:.out=.in) .SUFFIXES: .in .out .res .vim -# Must run test1 first to create small.vim. -$(SCRIPTS) $(SCRIPTS_GUI) $(SCRIPTS_WIN32) $(NEW_TESTS): $(SCRIPTS_FIRST) - nongui: nolog $(SCRIPTS_FIRST) $(SCRIPTS) newtests report small: nolog report @@ -67,6 +64,9 @@ $(TEST_OUTFILES): $(DOSTMP)\$(*B).in & echo $* FAILED >> test.log ) \ else ( move /y test.out $*.out ) +# Must run test1 first to create small.vim. +$(SCRIPTS) $(SCRIPTS_GUI) $(SCRIPTS_WIN32) $(NEW_TESTS): $(SCRIPTS_FIRST) + report: @echo "" @echo Test results: Here is the result: https://ci.appveyor.com/project/k-takata/vim/build/38 Regards, Ken Takata -- -- 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 [email protected]. For more options, visit https://groups.google.com/d/optout.
