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

Install script: wrap vim.system calls to output nicer error messages if binary can't be found #359

Closed
mrcjkb opened this issue May 28, 2024 · 3 comments

Comments

@mrcjkb
Copy link
Member

mrcjkb commented May 28, 2024

Hey 👋

The error is thrown here, because you don't have sh or git bash installed:

rocks.nvim/installer.lua

Lines 192 to 194 in ccdaaa1

sc = vim.system({
"sh",
"configure",

We definitely need to wrap those calls to output nicer error messages...

Originally posted by @mrcjkb in #358 (comment)

ENOENT: No such file or directory is very confusing...

@mrcjkb
Copy link
Member Author

mrcjkb commented May 28, 2024

Another idea:

If any installer dependencies are missing, show a warning in the intaller UI.

@softmoth
Copy link

softmoth commented Oct 4, 2024

Also this configure call in bootstrap.lua can fail if the luajit-devel package is missing. There's no message about why, it just says line it failed at the line that sources bootstrap.lua.

This exact issue may no longer be a problem once no longer depending on luarocks, but I guess the need for better error reporting will still be there.

@mrcjkb
Copy link
Member Author

mrcjkb commented Oct 4, 2024

Also this configure call in bootstrap.lua can fail if the luajit-devel package is missing.

Unfortunately, that's an issue with the luarocks build scripts. Not much we can do in our install/bootstrap scripts.

Closing this issue, because it's been implemented in #549.
Also, we are focusing almost all of our resources on rocks development now. I don't think it'll be too long before we can replace luarocks 😄

@mrcjkb mrcjkb closed this as completed Oct 4, 2024
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

2 participants