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
luaPackages.luadbi: 0.7.1 -> 0.7.2 #62964
Conversation
there are already 2 open PRs for this https://github.com/NixOS/nixpkgs/pulls?utf8=%E2%9C%93&q=is%3Apr+is%3Aopen+luadbi . I would love to get some guidance on how to properly test that prosody works as I doubt my generated plugin works #62663 but yet prosody doesn't complain on start. |
@ teto me used this test:
|
would be super cool if you could make #62663 work, using my PR and adding your changes to https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/lua- modules/overrides.nix . If you can't I will but you might be faster. Btw there seems to be a prosody test in nixpkgs already. Does yours differ ? |
@teto with buildLuarocksPackage plugin luadbi error worked- not loaded sql dirivers:
Yes, modified test script. Writed to find error work prosody. I am using prosody with mysql. |
As there is a demand for a fix, I merged this and will try to move to generated later. |
@teto need create test prosody with mysql? Replace test sql to mysql or add test prosody with sql + mysql? |
Added test check work prosody with MySQL database - #63150 |
Motivation for this change
Update lua package luadbi to v0.7.2 and fix worked prosody with mysql database.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)