Ken Takata wrote:
> 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
Well, I don't know what your setup is, but for the main Vim repository
the builds pass now.
> I wrote another patch. This is simpler than before, but I don't understand
> why this patch solves the problem:
Apparently the line you moved down must be after the rule that copies
the input files to dostmp.
I wonder if we can get rid of this anyway, the tests should be able to
run with Unix file format. I wonder what files if we skip this step and
if we can fix that.
--
There are 2 kinds of people in my world: those who know Unix, Perl, Vim, GNU,
Linux, etc, and those who know COBOL. It gets very difficult for me at
parties, not knowing which group to socialise with :-)
Sitaram Chamarty
/// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ an exciting new programming language -- http://www.Zimbu.org ///
\\\ 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 [email protected].
For more options, visit https://groups.google.com/d/optout.
