Skip to content
This repository has been archived by the owner on Aug 2, 2023. It is now read-only.

Needs to check luac is 5.3 before running #47

Open
take-cheeze opened this issue Dec 5, 2018 · 1 comment
Open

Needs to check luac is 5.3 before running #47

take-cheeze opened this issue Dec 5, 2018 · 1 comment

Comments

@take-cheeze
Copy link

Sometimes luac isn't 5.3.

cmd := exec.Command("luac", "-o", tmp, "-")

@artob
Copy link

artob commented Jun 29, 2019

This function (State.load()) seems the weakest part of the current implementation in general. When attempting to use the library on Android (via GoMobile), the creation of a temporary directory/file fails due to permission denied, unsurprisingly. And, of course, there is no luac binary available even if the previous did succeed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants