-
-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
Build failure: luaPackages.lua-resty-openidc #291995
Comments
There doesn't seem to be much interest in this package as I've seen it broken for a long time, dont hesitate to mark yourself as maintainer.
It would be the first in the lua package set (haskell does it a lot) but yes it's acceptable. Alternatively you can use your own .csv file with the nixpkgs script to maintain an out of tree lua overlay if that's more convenient for you https://nixos.org/manual/nixpkgs/stable/#developing-with-lua |
@Qyriad with ZHF coming up, I wonder what to do. Which way did you go ? Should I mark the package as broken or were you able to find a solution ? |
@teto thank you for reminding me about this. I had put an inline package definition for lua-resty-session 3.10 in my config, but it seems that in the meantime |
It seems that Edit: apparently there's already a PR #314863 |
I think this can be closed now since #294057 has been merged. |
Steps To Reproduce
pkgs.lua.pkgs.lua-resty-openidc
, alternatively with any of the other Lua package sets likeluajit.pkgs
oropenresty_luajit.pkgs
nix build 'github:NixOS/nixpkgs/master#pkgs.luajit.pkgs.lua-resty-openidc'
, and also with currentnixpkgs-unstable
Build log
Additional context
lua-resty-session's README mentions
lua-resty-openidc requires <= 3.10, and given that v4 was a major breaking change, would it make sense to maintain a
lua.pkgs.lua-resty-session_3
or similar? I can PR this if that's acceptable.Notify maintainers
@teto
Metadata
Add a 👍 reaction to issues you find important.
The text was updated successfully, but these errors were encountered: