Translate fails to run on Windows

[vds2212]

I'm running on Windows 10 and bash is available on my system.

When the library is running on my system any call to translate launch a process of the kind:

C:\Windows\system32\cmd.exe /c (^(C:\Windows\system32\cmd.exe C:\Users\vds\AppData\Local\Temp\VWS6C41.tmp^) ^>C:\Users\vds\AppData\Local\Temp\V3F6C42.tmp 2^>^&1)

That doesn't return.
To solve the problem I have disabled the translation for win32 systems:

function! s:translate(id) dict abort
  let cache = lh#dict#let(s:k_cached_translations, self._env.LANG .'.'. self._env.TEXTDOMAIN, {})
  if !has_key(cache, a:id)
    if executable('bash') && !has('win32')
      let cache[a:id] = lh#os#system('bash -c '.shellescape('echo $"'.a:id.'"'), self._env)
    else
      " TODO: support windows
      let cache[a:id] = a:id
    endif
  endif
  return cache[a:id]
endfunction

Let me know if you would be interested by a Pull Request.

[LucHermitte]

Actually, it's a bit more subtle as this works well with gvim-win32 + cygwin-bash + &shell==bash.

What I see is that it doesn't work at all when I revert my 'shell' option to cmd -- as I then execute a command file (with &shell) that sets environment variables with the syntax used with Unix shells instead of the Windows CMD syntax.

A quick and dirty patch would be to test &shell instead of has('win*'), but instead I'll try to fix this improper code behind lh#os#new_runner_script() if possible.

BTW, if you change your &shell to bash (it'll require to change a few other settings along the way -- see for instance my :GoBash command from system-tools), can you correctly execute things like : echo lh#po#context().translate('[Quickfix List]') ?

[LucHermitte]

Let me know if the fix does the job for you.

[vds2212]

If I trigger that command I get a cmd window with the following content hanging:

C:\Windows\system32\cmd.exe /c (^(C:\Windows\system32\cmd.exe C:\Users\vds\AppData\Local\Temp\VSK604D.tmp^) ^>C:\Users\vds\AppData\Local\Temp\VZ7604E.tmp 2^>^&1)

I'll try the same with :set shell=bash and let you know.

[vds2212]

With :set shell=bash I got the following error message:

Error detected while processing function <SNR>165_translate[4]..lh#os#system[14]..function <SNR>165_translate[4]..lh#os#system[11]..<SNR>167_run_script:
line    2:
E282: Cannot read from "C:\Users\vds\AppData\Local\Temp\VZ7616D.tmp"

[LucHermitte]

Hum. I see we don't use the same settings for the shell options -- I've forfeited on the official standard ones 2 decades ago.

Have you updated lh-vim-lib to use the latest version? I've tried to address a few issues. Even with shell==cmd, hopefully it should works (as long as it was already possible to run system("somebatchscript.cmd") with your configuration.

[vds2212]

Here is the error message I got when I run: echo lh#po#context().translate('[Quickfix List]')

/bin/bash: line 1: Quickfix List: syntax error in expression (error token is "List")

Here is the error message I got with the new version when I try to decode a trace (WTF)

Error detected while processing function lh#exception#say_what:
line    3:
E767: Too many arguments for printf()

[vds2212]

I have uninstalled WSL2 that was providing bash and without WSL2 it now works. But I suppose if I install it back I'll have the problem.
But this version is already more robust :-)

Thanks for the support.
Vivian.

[LucHermitte]

Unfortunately I don't have WSL/2 installed on my machines. I'll try to come up with a command file to execute from DOS console in a first time to see whether the problem comes from executing WSL2's bash from the command file or executing the command file with standard Vim settings for the &shell options.

Also, your Vim seems to be in English, I should be able to detect that and completely shortcut the localization of the messages.

I'll see what I can do.

[vds2212]

Thanks for the update. If you have something I'll be glad to test. Do I understand correctly that Vim use the shell to translate its messages? Why do we need to run a shell command to get the translation?

[LucHermitte]

That's the only way I've found to translate messages from the .po files -- without requiring to install 3rd party executables.

I guess I could have used python... hum...

[LucHermitte]

Would you mind trying to execute the following .cmd file from a DOS prompt?

@echo off
set LANG=fr_FR.UTF-8
set TEXTDOMAINDIR=C:\Progra~1\dev\Vim\vim81/lang
set TEXTDOMAIN=vim
bash -c 'echo $"[Quickfix List]"'

You'll have to replace the TEXTDOMAINDIR with the lang/ directory under $VIMRUNTIME

If this works, the next step would be to execute it from Vim with system() command:

" Expecting &shell to be cmd
:echo system('test-po-translate.cmd')