Skip to content
This repository was archived by the owner on Feb 14, 2018. It is now read-only.

Luarocks Support #62

Open
mrrobby opened this issue Oct 11, 2016 · 2 comments
Open

Luarocks Support #62

mrrobby opened this issue Oct 11, 2016 · 2 comments

Comments

@mrrobby
Copy link

mrrobby commented Oct 11, 2016

Is there any chance you'll be adding support for a luarocks install (i.e. rockspec)?

@twistedstream
Copy link
Contributor

Hi Rob, if I recall one reason why we didn't attempt to package this library up as a rockspec was that the handful of libraries it depends on were also not registered in LuaRocks. So we would have had to either bundle them directly into one package or ideally get those other libraries into LuaRocks first and reference them as dependencies. The short answer is that it wasn't exactly on the roadmap but I'm sure we'd consider a PR that makes it work.

@mrrobby
Copy link
Author

mrrobby commented Oct 13, 2016

@twistedstream I noticed today that OpenResty has deprecated its usage of LuaRocks in favor of their new package manager OPM. Having said that, I was able to track down the packages used which are published on LuaRocks, that being basexx and lua-resty-jwt. The latter uses it's own fork of an hmac library, which gets packaged if you install using LuaRocks. The problem is the file here nginx-jwt.lua depends on a version of lua-resty-jwt which is not published. There have been code breaking updates since then, so perhaps that would be a first step, to update nginx-jwt.lua to work with lua-resty-jwt v1.0.7? It might jumpstart the desire get the other two libraries on OPM too.

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