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

BUG: Out-of-date syntax in subprocess.lua #41

Open
mantacid opened this issue Oct 27, 2024 · 3 comments
Open

BUG: Out-of-date syntax in subprocess.lua #41

mantacid opened this issue Oct 27, 2024 · 3 comments

Comments

@mantacid
Copy link

The subprocess library uses the wrong syntax for require, current versions of lpty use the syntax lpty = require "lpty", not require("lpty").

@mantacid
Copy link
Author

This is probably not the only issue with the subprocess library; I'm still getting errors when trying to read from a process for example. But this is the main one that is stopping the library from working.

@presidentbeef
Copy link
Owner

But all the libraries are included in the Brat repo. Are you trying to use a newer version? Just tested and seems to be working okay.

@mantacid
Copy link
Author

mantacid commented Nov 9, 2024

I had installed the latest version of both brat and its dependencies to run this test.

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