Build wit tests fails on Test_incsearch_substitute_03 from patch 8.1.0277

[mimi1vx]

Test_incsearch_substitute_03.dump.failed :

|f+0&#ffff4012|o@1| +0&#ffffff0|1| @64
|f+0&#ffff4012|o@1| +0&#ffffff0|2| @64
|f+0&#ffff4012|o@1| +0&#ffffff0|3| @64
|f+0&#ffff4012|o@1| +0&#ffffff0|4| @64
|f+0&#ffff4012|o@1| +0&#ffffff0|5| @64
|f+0&#ffff4012|o@1| +0&#ffffff0|6| @64
|f+0&#ffff4012|o@1| +0&#ffffff0|7| @64
|f+0&#ffff4012|o@1| +0&#ffffff0|8| @64
|:|.|,|.|+|2|s|/> @43|4|,|1| @10|T|o|p| 

[chrisbra]

can you please run term_dumpdiff() with the expected and failed dump?

[mimi1vx]

foo 1                                                                                                                                                  
foo 2                                                                                                                                                  
foo 3                                                                                                                                                  
foo 4                                                                                                                                                  
foo 5                                                                                                                                                  
foo 6                                                                                                                                                  
foo 7                                                                                                                                                  
foo 8                                                                                                                                                  
:.,.+2s/                                            4,1           Top                                                                                  
============== Test_incsearch_substitute_03.dump.failed ==============   

[chrisbra]

I don't think this is the output of term_dumpdiff(). Please show that.

[mimi1vx]

this is output od term_dumpdiff ..

excactly : :call term_dumpdiff("Test_incsearch_substitute_03.dump.failed", "dumps/Test_incsearch_substitute_03.dump") and failed is in first comment

On Mon, 20 Aug 2018 at 10:15, Christian Brabandt notifications@github.com wrote:

I don't think this is the output of term_dumpdiff(). Please show that.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/vim/vim/issues/3348#issuecomment-414234824, or mute the thread https://github.com/notifications/unsubscribe-auth/AA4035iumUCG7s1jcQBJ0N2B0cIUl8vpks5uSnAzgaJpZM4WDINa .

[brammool]

Test_incsearch_substitute_03.dump.failed :

|f+0&#ffff4012|o@1| +0&#ffffff0|1| @64
|f+0&#ffff4012|o@1| +0&#ffffff0|2| @64
|f+0&#ffff4012|o@1| +0&#ffffff0|3| @64
|f+0&#ffff4012|o@1| +0&#ffffff0|4| @64
|f+0&#ffff4012|o@1| +0&#ffffff0|5| @64
|f+0&#ffff4012|o@1| +0&#ffffff0|6| @64
|f+0&#ffff4012|o@1| +0&#ffffff0|7| @64
|f+0&#ffff4012|o@1| +0&#ffffff0|8| @64
|:|.|,|.|+|2|s|/> @43|4|,|1| @10|T|o|p| 

This means the ruler and "Top" are displayed in the command line. That is weird. As if the terminal wasn't redrawn. But it appears to work find for everybody else.

Also, if the screendump before this succeeded, this can't really happen. Or did you have more failures?

What is the environment you run the tests in? GUI or terminal? What system?

-- "I've been teaching myself to play the piano for about 5 years and now write most of my songs on it, mainly because I can never find any paper." Jeff Lynne, ELO's greatest hits

/// Bram Moolenaar -- Bram@Moolenaar.net -- 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 ///

[mimi1vx]

https://build.opensuse.org/project/show/home:mimi_vx:branches:editors , build is passed on PPC because tests on this arch are completely disabled.

terminal,

env :

HOSTTYPE=x86_64
LESSCLOSE=lessclose.sh %s %s
XKEYSYMDB=/usr/X11R6/lib/X11/XKeysymDB
MALLOC_PERTURB_=69
WINDOWMANAGER=xterm
LESS=-M -I -R
HOSTNAME=penguin
CSHEDIT=emacs
GPG_TTY=not a tty
LESS_ADVANCED_PREPROCESSOR=no
COLORTERM=1
MACHTYPE=x86_64-suse-linux
PERL_HASH_SEED=42
MINICOM=-c on
OSTYPE=linux
USER=abuild
PAGER=less
PYTHONHASHSEED=0
MORE=-sl
SUSE_ASNEEDED=1
PWD=/home/abuild
HOME=/home/abuild
HOST=penguin
CCACHE_DIR=/.ccache
XNLSPATH=/usr/X11R6/lib/X11/nls
XDG_DATA_DIRS=/usr/share
MALLOC_CHECK_=3
PROFILEREAD=true
SUSE_IGNORED_RPATHS=/etc/suse-ignored-rpaths.conf
MAIL=/var/spool/mail/abuild
LESSKEY=/etc/lesskey.bin
SHELL=/bin/bash
TERM=xterm-256color
PYTHONSTARTUP=/etc/pythonstart
SHLVL=1
MANPATH=
LOGNAME=abuild
XDG_CONFIG_DIRS=/etc/xdg
PATH=/var/lib/build/ccache/bin:/usr/local/bin:/usr/bin:/bin:/usr/lib/mit/bin:/usr/lib/mit/sbin
G_BROKEN_FILENAMES=1
HISTSIZE=1000
CPU=x86_64
LESSOPEN=lessopen.sh %s
QT_HASH_SEED=42
BASH_FUNC_mc%%=() {  . /usr/share/mc/mc-wrapper.sh
}
_=/usr/bin/env

and %check section of specfile:

%check
# vim does quite an extensive test relying on a full fledged terminal
# inside OBS, stdio is redirected to a serial console (where the build log
# is being recorded/extracted. Systemd set non-local tty by default to vt220
# in upcoming versions
export TERM=linux
# Reset the terminal scrolling region left behind by the testsuite
trap "printf '\e[r'" EXIT
# Look for "TEST FAILURE" in the build log
%ifarch ppc ppc64 ppc64le
LC_ALL=en_US.UTF-8 make -j1 test || { echo "Ignore transient errors for PowerPC. boo#1072651"; }
%else
LC_ALL=en_US.UTF-8 make -j1 test
%endif