Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error detected while processing function <SNR>28_InstallCLI[7]..<SNR>28_GetPythonBinary #115

Closed
GopherJ opened this issue Dec 13, 2021 · 5 comments

Comments

@GopherJ
Copy link

GopherJ commented Dec 13, 2021

latest vim-wakatime reported this error in macbook m1

@alanhamlett
Copy link
Member

That doesn't look like the full error message... usually there's some more?

@nganhkhoa
Copy link

This I believe is the full log. Artix linux, NVIM v0.6.0-dev+284-gc2a65921d

$ nvim -V9log.txt

sourcing "$HOME/.local/share/nvim/site/pack/packer/start/vim-wakatime/plugin/wakatime.vim"
Executing command: "set nonomatch; vimglob() { while [ $# -ge 1 ]; do echo "$1"; shift; done }; vimglob >/tmp/nvimHqzmqB/1 $WAKATIME_HOME"

Executing command: "'/bin/bash' '-c' 'uname -s'"


Executing command: "'/bin/bash' '-c' 'uname -m'"


Error detected while processing function <SNR>75_InstallCLI[7]..<SNR>75_GetPythonBinary:
line    5:
E928: String required
finished sourcing $HOME/.local/share/nvim/site/pack/packer/start/vim-wakatime/plugin/wakatime.vim

Just updated today.

@nganhkhoa
Copy link

This could be related: hsanson/ale@53ff5f2

To which I fixed the bug with:

@@ -371,7 +371,7 @@ EOF
         if has('g:wakatime_PythonBinary')
             let python_bin = g:wakatime_PythonBinary
         endif
-        if !filereadable(python_bin) && !executable(python_bin)
+        if empty(python_bin)
             if executable('python3')
                 let python_bin = 'python3'
             elseif executable('python')

@nganhkhoa
Copy link

But this is a trivial case because I don't set g:wakatime_PythonBinary ever. I'm not possitive that the fix is correct (to wakatime-vim).

Also, a quick look through the repo suggests that wakatime-vim will be using go binary in future release? That would mean removing all related python code.

@alanhamlett
Copy link
Member

Thanks! Just pushed up a fix.

alanhamlett added a commit that referenced this issue Dec 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants