regression: two zathura launched after starting compiling

[moetayuko]

Description

After committing 3b4a5a1, two zathura get launched after starting compiling w/ <localleader>lv in vim8, but nvim works fine.

Readding echos in 3b4a5a1#diff-f7d9b3dc2fb7089c1755af5c49217c00982d466ddac80601ab0bfa8f81522dfbL453-R459 fixed the issue.

I guess the regex ^ needs to be applied to

vimtex/autoload/vimtex/compiler/_template.vim

Line 319 in 3c14f69

call s:check_callback(a:msg)

as well?

Steps to reproduce

No response

Expected behavior

No response

Actual behavior

No response

Do you use a latexmkrc file?

No

VimtexInfo

System info:
  OS: Arch Linux
  Vim version: VIM 8.2 (1-4651)
  Has clientserver: true
  Servername: VIM

VimTeX project: t
  base: t.tex
  root: /home/xxx/tmp
  tex: /home/xxx/tmp/t.tex
  main parser: current file
  document class: 
  compiler: latexmk
    engine: -pdf
    options:
      -verbose
      -file-line-error
      -shell-escape
      -synctex=1
      -interaction=nonstopmode
    callback: 1
    continuous: 1
    executable: latexmk
  viewer: Zathura
    xwin id: 0
  qf method: LaTeX logfile

[moetayuko]

#2367 seems to be a quick fix

[lervag]

Strange; this does not happen on my end. I use the regular minimal files

set nocompatible
" These are correct paths on my end
let &rtp = '../..,' . &rtp
let &rtp .= ',../../after'
filetype plugin indent on
syntax enable

let g:vimtex_view_method = 'zathura'

silent edit main.tex

and

\documentclass{minimal}
\begin{document}
Hello world!
\end{document}

I start with vim -u test.vim, then do \ll. A single window appears. Then \lv - forward search works as expected, still only a single window. Similar with nvim --clean -u test.vim.

[lervag]

No, sorry, I'm stupid. I tested after applying your patch. I can confirm it seems to work as expected with your patch and I confirm the bug without it. Thanks!