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

Deploy an OPM package? #45

Open
brendon opened this issue Feb 10, 2017 · 9 comments
Open

Deploy an OPM package? #45

brendon opened this issue Feb 10, 2017 · 9 comments

Comments

@brendon
Copy link

brendon commented Feb 10, 2017

According to the LuaRocks page, its use is now discouraged:

http://openresty.org/en/using-luarocks.html

Would you be interested in releasing an OPM package also? :)

@GUI
Copy link
Collaborator

GUI commented Feb 25, 2017

I'm interested in publishing to OPM (I've published my other lua-resty module to OPM), but lua-resty-auto-ssl's installation process currently builds the C library sockproc, and OPM doesn't yet support compiling C components (although it's on their TODO list): openresty/opm#30

So I had been waiting on support for building C things to land in OPM before publishing lua-resty-auto-ssl to OPM.

That being said, we could technically publish lua-resty-auto-ssl to OPM, but without sockproc. You'd then have to manually build and install sockproc yourself, so I'm not sure it would be more convenient. But let me know if that seems preferable.

@brendon
Copy link
Author

brendon commented Feb 27, 2017

I don't really have an opinion either way. I found the luarocks method easy enough :)

It was just the warning message on the openresty website that prompted me to ask :) This can probably serve as an answer to others in the meantime until OPM supports everything you need.

@luto
Copy link
Collaborator

luto commented Jun 5, 2017

Since OPM seems to be the direction openresty itself is moving, we should keep this open as a tracking bug. But can't actually implement anything until the C support in OPM is done.

@luto luto reopened this Jun 5, 2017
This was referenced Jul 18, 2017
@byrnedo
Copy link

byrnedo commented Sep 29, 2017

Any idea when this might be done? Is the best way in the meantime to install manually?

@GUI
Copy link
Collaborator

GUI commented Oct 1, 2017

@byrnedo: Using LuaRocks is still the preferred installation method for lua-resty-auto-ssl. Until OPM has C support, there's still various things, like lua-resty-auto-ssl, that can't be installed via OPM. So while OpenResty prefers OPM these days, LuaRocks still works great and is still the de facto package manager actively supported by the broader Lua community.

But since LuaRocks doesn't come bundled with OpenResty, you will need to install it first: http://openresty.org/en/using-luarocks.html#install-luarocks

@byrnedo
Copy link

byrnedo commented Oct 1, 2017 via email

@aftabnaveed
Copy link

But since LuaRocks doesn't come bundled with OpenResty, you will need to install it first: http://openresty.org/en/using-luarocks.html#install-luarocks

This is discouraged by openresty and I am just wondering if still using the luarock would have any implications?

@brendon
Copy link
Author

brendon commented Aug 1, 2018

It shouldn't. It works fine for me in my production environment. I think it's discouraged because they want to shift people to OPM packages.

@kajmagnus
Copy link

kajmagnus commented Jan 10, 2019

Maybe mention all this briefly in the readme? I'm thinking everyone who takes a closer look at lua-resty-auto-ssl will get slightly confused about using LuaRocks, although deprecated — slightly gave me the incorrect impression that lua-resty-auto-ssl wasn't being kept up-to-date. And not everyone will find this discussion and explanation.

Example:

Requirements:
...
LuaRocks (LuaRocks is deprecated, but OpenResty's new package manager, OPM, cannot currently compile C components)

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

No branches or pull requests

6 participants