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
python3Packages.simpleldap: remove #43782
Conversation
|
Judging from https://github.com/gdub/python-simpleldap/blob/0.8/setup.py#L12-L14, we should set |
pname = "simpleldap"; | ||
version = "0.8"; | ||
|
||
src = fetchFromGitHub { |
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.
What's the reason you switched to fetchFromGitHub
? Please add an inline comment.
I think we should just remove this package: It hasn't been updated since 2014 and its only dependent, inginious, is marked as broken since 6111168. Inginious should either be updated to the most recent version, which replaces |
f43cf61
to
1a8bbb4
Compare
@dotlambda thanks for elaborating, I dropped both of these packages, a broken one is obviiously not in use and |
Please adjust the commit message of the first commit accordingly. |
Fairly outdated and incompatible with Python 3, see the following links for further reference: * https://github.com/gdub/python-simpleldap/blob/0.8/setup.py#L12-L14 * NixOS#43782 (comment)
Fallout of 9db7f15ea3ce57eee25458daf87cce7a04ea98b2 which removed simpleldap. @layus suggests to remove: * NixOS#43782 (comment) * NixOS#31975 (comment)
1a8bbb4
to
0adca53
Compare
argh, good catch, sorry :/ |
Motivation for this change
Fairly outdated and incompatible with Python 3, see the following links
for further reference:
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)