nvim-neorocks / rocks.nvim

🌒 Neovim plugin management inspired by Cargo, powered by luarocks
GNU General Public License v3.0
655 stars 11 forks source link

[QOL] Installer/Bootstrap scripts: Better error messages #368

Open mrcjkb opened 3 months ago

mrcjkb commented 3 months ago

If a command fails, the installer and bootstrap scripts should output the command that failed.

zhenhua-zhang commented 1 month ago

Attached to this issue. Here is an example.

No liblua5.1-0-dev was available in my OS. While the bootstrap.lua is trying to install luarocks via the configure script, the error message is Vim:Configuring luarocks failed., which is confusing. However, it was just due to a library missing in the system. Thus, better error messages definitely help to make the installation easier and smoother.

mrcjkb commented 1 month ago

While the bootstrap.lua is trying to install luarocks via the configure script, the error message is Vim:Configuring luarocks failed., which is confusing.

🤔 It should also output stderr and stdout of the configure command.