akiyosi / goneovim

A GUI frontend for neovim.
MIT License
2.37k stars 62 forks source link

:h goneovim not working #415

Open Karthik-Dulam opened 1 year ago

Karthik-Dulam commented 1 year ago

No help found for goneovim.

gmankab commented 1 year ago

same error

akiyosi commented 1 year ago

@Karthik-Dulam @gmankab Hi, Sorry I haven't responded.

By default, goneovim references ./runtime/doc/goneovim.txt directory that exists within the directory where its own executable binaries reside. The cause of the problem could be that this reference is not working properly: how do you manage Goneovim's runtime binaries?

Also, since the current implementation is not flexible, it might be good to be able to change the path to Goneovim's Help, for example, via an environment variable. If you have any other good ideas in this regard, please let me know.

gmankab commented 1 year ago

i installed it from aur

akiyosi commented 1 year ago

What is the directory structure where the goneovim binaries are located?

gmankab commented 1 year ago

if you installing goneovim from aur, then it will be installed in /bin, and it doesn't creating runtime directory and it's files

it would be better for the installer to put docs in /usr/share/nvim/runtime/doc

since goneovim in aur is being maintained by @alerque, we should report it to him

i created an issue in alerque/aur repo - https://github.com/alerque/aur/issues/53

alerque commented 1 year ago

The goneovim-bin package from the AUR already puts the docs in /usr/share/nvim/runtime/doc/goneovim.txt. I see that the online help isn't working, but the file is already where it should be.

$ pacman -Ql goneovim-bin
goneovim-bin /usr/
goneovim-bin /usr/bin/
goneovim-bin /usr/bin/goneovim
goneovim-bin /usr/share/
goneovim-bin /usr/share/applications/
goneovim-bin /usr/share/applications/goneovim.desktop
goneovim-bin /usr/share/nvim/
goneovim-bin /usr/share/nvim/runtime/
goneovim-bin /usr/share/nvim/runtime/doc/
goneovim-bin /usr/share/nvim/runtime/doc/goneovim.txt
goneovim-bin /usr/share/pixmaps/
goneovim-bin /usr/share/pixmaps/goneovim.ico
gmankab commented 1 year ago

i just checked it

i was wrong, these files do exist

it turns out that @alerque pointed out the real problem here - https://github.com/alerque/aur/issues/53#issuecomment-1545294870

akiyosi commented 1 year ago

It may not be a direct solution, but I am thinking of allowing the environment variable $GONEOVIM_RUNTIME to control the runtime path where the help files reside.

alerque commented 1 year ago

I'm not convinced the Arch specific bug I linked in the AUR issue is actually the root cause here. There are RTP issues in the vim plugin ecosystem, but that bug isn't about the nvim RTP itself. It seems goneovim is not using anything at all (at least not anything new) placed in the standard NeoVIM docs location. Why would that be?

akiyosi commented 1 year ago

@gmankab I have improved the doc detection process for goneovim. I would appreciate your feedback.

https://github.com/akiyosi/goneovim/actions/runs/5033064055

gmankab commented 1 year ago

thank you

since this problem occurs when installing from aur, I have to wait for @alerque to update the aur package, and after that I can check if the problem is fixed or not

akiyosi commented 1 year ago

I thought we could check by replacing only the goneovim binary, keeping the structure of the goneovim installed in the AUR. Nevertheless, I understand. I will merge the changes then and release 0.6.5 in the near future.

alerque commented 1 year ago

@gmankab I haven't reviewed the PKGBUILD to make sure it installs stuff in the same places, but you could check out the goneovim-git package to get VCS HEAD builds to test fixes before releases. If it doesn't handle the help files the same way the stable one does you can clone it and tweak it yourself to be be more similar.

gmankab commented 1 year ago

i am getting error when building goneovim-git:

error ```text ERRO[0000] failed to run command _func=RunCmd cmd="go list -mod=vendor -find -e -f {{.Dir}} github.com/therecipe/env_linux_amd64_513" dir=/home/gmanka/.cache/yay/goneovim-git/src/goneovim env="SHELL=/bin/zsh LSCOLORS=Gxfxcxdxbxegedabagacad SESSION_MANAGER=local/arch:@/tmp/.ICE-unix/907,unix/arch:/tmp/.ICE-unix/907 COLORTERM=truecolor LESS=-R XDG_MENU_PREFIX=gnome- PYTHONHASHSEED=0 TEXTDOMAINDIR=/usr/share/locale SSH_AUTH_SOCK=/run/user/1000/keyring/ssh DESKTOP_SESSION=gnome PWD=/home/gmanka/.cache/yay/goneovim-git/src/goneovim/cmd/goneovim SOURCE_DATE_EPOCH=1684636489 LOGNAME=gmanka XDG_SESSION_DESKTOP=gnome XDG_SESSION_TYPE=wayland SYSTEMD_EXEC_PID=922 CXXFLAGS=-march=x86-64 -mtune=generic -O2 -pipe -fno-plt -fexceptions -Wp,-D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security -fstack-clash-protection -fcf-protection -Wp,-D_GLIBCXX_ASSERTIONS XAUTHORITY=/run/user/1000/.mutter-Xwaylandauth.SI8D51 QT_STYLE_OVERRIDE=kvantum MOTD_SHOWN=pam GDM_LANG=ru_RU.UTF-8 COMMAND_MODE=legacy LDFLAGS=-Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now HOME=/home/gmanka USERNAME=gmanka LANG=ru_RU.UTF-8 LS_COLORS=rs=0:di=01;34:ln=01;36:mh=00:pi=40;33:so=01;35:do=01;35:bd=40;33;01:cd=40;33;01:or=40;31;01:mi=00:su=37;41:sg=30;43:ca=00:tw=30;42:ow=34;42:st=37;44:ex=01;32:*.tar=01;31:*.tgz=01;31:*.arc=01;31:*.arj=01;31:*.taz=01;31:*.lha=01;31:*.lz4=01;31:*.lzh=01;31:*.lzma=01;31:*.tlz=01;31:*.txz=01;31:*.tzo=01;31:*.t7z=01;31:*.zip=01;31:*.z=01;31:*.dz=01;31:*.gz=01;31:*.lrz=01;31:*.lz=01;31:*.lzo=01;31:*.xz=01;31:*.zst=01;31:*.tzst=01;31:*.bz2=01;31:*.bz=01;31:*.tbz=01;31:*.tbz2=01;31:*.tz=01;31:*.deb=01;31:*.rpm=01;31:*.jar=01;31:*.war=01;31:*.ear=01;31:*.sar=01;31:*.rar=01;31:*.alz=01;31:*.ace=01;31:*.zoo=01;31:*.cpio=01;31:*.7z=01;31:*.rz=01;31:*.cab=01;31:*.wim=01;31:*.swm=01;31:*.dwm=01;31:*.esd=01;31:*.avif=01;35:*.jpg=01;35:*.jpeg=01;35:*.mjpg=01;35:*.mjpeg=01;35:*.gif=01;35:*.bmp=01;35:*.pbm=01;35:*.pgm=01;35:*.ppm=01;35:*.tga=01;35:*.xbm=01;35:*.xpm=01;35:*.tif=01;35:*.tiff=01;35:*.png=01;35:*.svg=01;35:*.svgz=01;35:*.mng=01;35:*.pcx=01;35:*.mov=01;35:*.mpg=01;35:*.mpeg=01;35:*.m2v=01;35:*.mkv=01;35:*.webm=01;35:*.webp=01;35:*.ogm=01;35:*.mp4=01;35:*.m4v=01;35:*.mp4v=01;35:*.vob=01;35:*.qt=01;35:*.nuv=01;35:*.wmv=01;35:*.asf=01;35:*.rm=01;35:*.rmvb=01;35:*.flc=01;35:*.avi=01;35:*.fli=01;35:*.flv=01;35:*.gl=01;35:*.dl=01;35:*.xcf=01;35:*.xwd=01;35:*.yuv=01;35:*.cgm=01;35:*.emf=01;35:*.ogv=01;35:*.ogx=01;35:*.aac=00;36:*.au=00;36:*.flac=00;36:*.m4a=00;36:*.mid=00;36:*.midi=00;36:*.mka=00;36:*.mp3=00;36:*.mpc=00;36:*.ogg=00;36:*.ra=00;36:*.wav=00;36:*.oga=00;36:*.opus=00;36:*.spx=00;36:*.xspf=00;36:*~=00;90:*#=00;90:*.bak=00;90:*.old=00;90:*.orig=00;90:*.part=00;90:*.rej=00;90:*.swp=00;90:*.tmp=00;90:*.dpkg-dist=00;90:*.dpkg-old=00;90:*.ucf-dist=00;90:*.ucf-new=00;90:*.ucf-old=00;90:*.rpmnew=00;90:*.rpmorig=00;90:*.rpmsave=00;90: XDG_CURRENT_DESKTOP=GNOME VTE_VERSION=7201 WAYLAND_DISPLAY=wayland-0 GNOME_TERMINAL_SCREEN=/org/gnome/Terminal/screen/109c5c9b_545e_444d_a4c7_58c8a614aa3b GNOME_SETUP_DISPLAY=:1 XDG_SESSION_CLASS=user TERM=xterm-256color ZSH=/home/gmanka/.oh-my-zsh USER=gmanka GNOME_TERMINAL_SERVICE=:1.83 DISPLAY=:0 SHLVL=2 PAGER=less XDG_RUNTIME_DIR=/run/user/1000 DEBUGINFOD_URLS=https://debuginfod.archlinux.org XDG_DATA_DIRS=/home/gmanka/.local/share/flatpak/exports/share:/var/lib/flatpak/exports/share:/usr/local/share/:/usr/share/ CHOST=x86_64-pc-linux-gnu PATH=/usr/local/bin:/usr/bin:/usr/local/sbin:/var/lib/flatpak/exports/bin:/usr/lib/jvm/default/bin:/usr/bin/site_perl:/usr/bin/vendor_perl:/usr/bin/core_perl GDMSESSION=gnome CFLAGS=-march=x86-64 -mtune=generic -O2 -pipe -fno-plt -fexceptions -Wp,-D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security -fstack-clash-protection -fcf-protection DBUS_SESSION_BUS_ADDRESS=unix:path=/run/user/1000/bus MAIL=/var/spool/mail/gmanka OLDPWD=/home/gmanka/.cache/yay/goneovim-git/src GOPATH=/home/gmanka/.cache/yay/goneovim-git/src/go TEXTDOMAIN=pacman-scripts _=/home/gmanka/.cache/yay/goneovim-git/src/go/bin/qtdeploy CGO_ENABLED=0" error="exit status 1" name="get env dir" go: inconsistent vendoring in /home/gmanka/.cache/yay/goneovim-git/src/goneovim: github.com/BurntSushi/toml@v1.2.1: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/akiyosi/goqtframelesswindow@v0.0.0-20230504150915-a0fc55d27d45: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/akiyosi/short_path@v0.2.1: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/atotto/clipboard@v0.1.4: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/bluele/gcache@v0.0.2: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/jessevdk/go-flags@v1.5.0: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/mattn/go-isatty@v0.0.18: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/mitchellh/go-homedir@v1.1.0: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/neovim/go-client@v1.2.2-0.20230301235731-1762f7f0fa25: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/therecipe/qt@v0.0.0-20200904063919-c0c124a5770d: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/akiyosi/w32@v0.0.0-20221111133244-de73be0c7b36: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/gopherjs/gopherjs@v0.0.0-20190411002643-bd77b112433e: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/konsorten/go-windows-terminal-sequences@v1.0.2: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/sirupsen/logrus@v1.4.1: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/therecipe/qt/internal/binding/files/docs/5.12.0@v0.0.0-20200904063919-c0c124a5770d: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/therecipe/qt/internal/binding/files/docs/5.13.0@v0.0.0-20200904063919-c0c124a5770d: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt golang.org/x/sys@v0.6.0: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt golang.org/x/tools@v0.0.0-20190420181800-aa740d480789: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt To ignore the vendor directory, use -mod=readonly or -mod=mod. To sync the vendor directory, run: go mod vendor ```

so i installed goneovim-bin, and then replaced binary manually

unfortunately, problem not fixed :(

akiyosi commented 1 year ago

@gmankab Thanks!

Sorry, there was a mistake in the fix code. I have fixed it and would appreciate it if you could check it again.

https://github.com/akiyosi/goneovim/actions/runs/5035731712

I would also appreciate it if you could provide information on the results of executing the following commands in your environment.

pacman -Ql goneovim-bin
gmankab commented 1 year ago

still no help found

➜  ~ goneovim --version
goneovim: /usr/lib/libtiff.so.5: no version information available (required by goneovim)
v0.6.4-78-gbe27753
➜  ~ pacman -Ql goneovim-bin 
goneovim-bin /usr/
goneovim-bin /usr/bin/
goneovim-bin /usr/bin/goneovim
goneovim-bin /usr/share/
goneovim-bin /usr/share/applications/
goneovim-bin /usr/share/applications/goneovim.desktop
goneovim-bin /usr/share/nvim/
goneovim-bin /usr/share/nvim/runtime/
goneovim-bin /usr/share/nvim/runtime/doc/
goneovim-bin /usr/share/nvim/runtime/doc/goneovim.txt
goneovim-bin /usr/share/pixmaps/
goneovim-bin /usr/share/pixmaps/goneovim.ico
akiyosi commented 1 year ago

@gmankab Oops, sorry, the root was missing from the path. Fixed again.

https://github.com/akiyosi/goneovim/actions/runs/5037936317

akiyosi commented 1 year ago

Ummm, In parallel I had also created an environment to check the operation, but if /usr/share/nvim/runtimeis not included in runtimepath, it is not possible to see the help. This is the installation configuration of the AUR package, but I feel the following is correct.

goneovim-bin /usr/
goneovim-bin /usr/bin/
goneovim-bin /usr/bin/goneovim
goneovim-bin /usr/share/
goneovim-bin /usr/share/applications/
goneovim-bin /usr/share/applications/goneovim.desktop
goneovim-bin $XDG_DATA_HOME/nvim/
goneovim-bin $XDG_DATA_HOME/nvim/site/
goneovim-bin $XDG_DATA_HOME/nvim/site/doc/
goneovim-bin $XDG_DATA_HOME/nvim/site/doc/goneovim.txt
goneovim-bin /usr/share/pixmaps/
goneovim-bin /usr/share/pixmaps/goneovim.ico

The default $XDG_DATA_HOME is ~/.local/share.

gmankab commented 1 year ago

still no help found :)

akiyosi commented 1 year ago

@gmankab Thanks for confirming. Yes, it is because the path (/usr/share/nvim/runtime) is not include in the runtimepath of nvim

Karthik-Dulam commented 10 months ago

:h goneovim.txt works for me now. Version v0.6.7 on Archlinux.