search

z0rzi / copilot-help.nvim

MIT License
34 stars 1 forks source link

Allow bun executable to be set by user #1

Closed jonathanmorris180 closed 9 months ago

jonathanmorris180 commented 9 months ago

Hi @z0rzi, thanks for making this plugin! I tried it out today but unfortunately I wasn't able to make it work out of the box. After some debugging, I found that the executable that's included is for Linux. Running file ./lua/copilot/bun gave me: ./lua/copilot/bun: ELF 64-bit LSB pie executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, for GNU/Linux 3.2.0, BuildID[xxHash]=f0cd378df2f73266, stripped

I've added some code here that should solve the issue for any other users. It allows the bun executable path to be either set in the config or found on the $PATH, but Linux users can still use the shipped executable out of the box. I've tested it out and it seems to be working well.

I also updated the readme with this information and added a few missing keymaps.

z0rzi commented 9 months ago

Hey @jonathanmorris180, thanks a lot for the investigation work, and the changes! :)