-
Notifications
You must be signed in to change notification settings - Fork 532
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
Question about LUA_PATH #203
Comments
One thing I just noticed in looking at this... the "build-from-source" flavors include that directive, but the "upstream" flavors do not. I did some archaeology to see why this is... and it's from #53 where I added it for LuaRocks. So only images that use LuaRocks has that built-in. You can try
Other than that, I think it should work fine if you use an image with that... here's a test with
Related also is #186 |
Thanks for the response. The question was whether you had any intention for that directory to hold anything system specific now or in future or was it included for developer usage so from your answer I assume the latter. I've tested it myself and worked like a charm 👍 saved me one line of updating |
Hey! Not strictly an issue but I've been looking at the LUA_PATH in docker images and it's set like this.
I noticed that
/usr/local/openresty/site/lualib
is empty - would it be a good practice if I mounted my libs directory there?Currently I'm adding a custom path in nginx.conf
lua_package_path "/etc/nginx/libs/?.lua;;";
and mount my libs to/etc/nginx/libs
and it works too but I'd much rather like to drop the custom path and use built in if that's what it was created for.Best, Karol
The text was updated successfully, but these errors were encountered: