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

Alternate .busted for nlua #19

Open
Julian opened this issue Sep 5, 2024 · 2 comments
Open

Alternate .busted for nlua #19

Julian opened this issue Sep 5, 2024 · 2 comments

Comments

@Julian
Copy link

Julian commented Sep 5, 2024

I just started a new plugin and decided to give busted another shot after previously having found it not usable for plugin development (a separate story -- let's see if this time goes any better...)

I see the .busted file here contains a hardcoded path to nlua, which struck me as quite strange, as that path is not guaranteed to be where luarocks is installing things to, given it's user-configurable (and indeed in my own case, I do not have luarocks installing things to ~/.luarocks). I see simply putting "nlua" there and letting the PATH be searched as "usual" though works. That seems to me strictly better.

Secondly, despite the README saying that eval $(luarocks path --no-bin) is the fix for the busted.runner error message, that wasn't enough in my case, and all it did was give me a different error message, one saying that pl.path requires LuaFileSystem -- neither of these errors seem to have many search results for them.

But what did then work for me was changing to luarocks path --no-bin --lua-version 5.1 in both the shell sourcing as well as the luarocks test invocation. I assume whether this works as documented or doesn't for someone probably depends on what lua version their luarocks is installed to?

The final product that seems to work here (so far) is this commit

So TL;DR, I propose tweaking the .busted to match what I ended up with. Is there any drawback I'm missing for that?

(I'm happy to send a PR for either or both of the above if there's agreement.)

@Julian
Copy link
Author

Julian commented Sep 5, 2024

(The saga continues a bit, as I see in CI using the 2 Lua GitHub actions, that even what I had there is not enough https://github.com/Julian/runt.nvim/actions/runs/10726393103/job/29746328008 and that presumably I need a bit more to tell luarocks in GH Actions where Lua lives. So that'll be what I try to figure out later, as I've probably fought enough for the morning :) -- maybe I want to follow the section of the nlua readme saying how to configure luarocks to install directly to nlua...)

@Julian
Copy link
Author

Julian commented Sep 5, 2024

OK, telling the lua github action to install luajit seems to have gotten me over the line for now...

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

1 participant