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
perlPackages: init Catalyst::Authentication::Store::LDAP at 1.016 (and dependencies) #97673
Conversation
…d dependencies) This is part of and effort of adding LDAP support to hydra.
f4c93db
to
24aedc9
Compare
@GrahamcOfBorg build perlPackages.CatalystAuthenticationStoreLDAP perldevelPackages.CatalystAuthenticationStoreLDAP |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi there, just a few things:
The commit message should be consistent with CONTRIBUTING.md, like perlPackages.CatalystAuthenticationStoreLDAP: init at 1.016
sha256 = "0cm399vxqqf05cjgs1j5v3sk4qc6nmws5nfhf52qvpbwc4m82mq8"; | ||
}; | ||
propagatedBuildInputs = [ NetLDAP CatalystPluginAuthentication ClassAccessorFast ]; | ||
buildInputs = [ TestMore TestMockObject TestException NetLDAPServerTest ]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
buildInputs = [ TestMore TestMockObject TestException NetLDAPServerTest ]; | |
buildInputs = [ TestMockObject TestException NetLDAPServerTest ]; |
NetLDAPServerTest
includes TestMore
url = "mirror://cpan/authors/id/K/KA/KARMAN/Net-LDAP-Server-Test-0.22.tar.gz"; | ||
sha256 = "13idip7jky92v4adw60jn2gcc3zf339gsdqlnc9nnvqzbxxp285i"; | ||
}; | ||
propagatedBuildInputs = [ NetLDAP NetLDAPServer TestMore DataDump NetLDAPSID ]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
propagatedBuildInputs = [ NetLDAP NetLDAPServer TestMore DataDump NetLDAPSID ]; | |
propagatedBuildInputs = [ NetLDAPServer TestMore DataDump NetLDAPSID ]; |
NetLDAPServer
includes NetLDAP
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM apart from minor nitpics
Result of nixpkgs-review pr 97673
1
8 packages built:
- perl530Packages.CatalystAuthenticationStoreLDAP
- perl530Packages.NetLDAPSID
- perl530Packages.NetLDAPServer
- perl530Packages.NetLDAPServerTest
- perl532Packages.CatalystAuthenticationStoreLDAP
- perl532Packages.NetLDAPSID
- perl532Packages.NetLDAPServer
- perl532Packages.NetLDAPServerTest
Motivation for this change
This is part of and effort of adding LDAP support to hydra: NixOS/hydra#805
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)