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
Marcus7070/spyder #80084
Marcus7070/spyder #80084
Conversation
So Spyder wants a later version of python-language-server (PLS). PLS now wants numpy, pandas, matplotlib, and pyqt5 for testing. Is there any concern with adding huge attributes to checkInputs, like there would be for closure size if they were added to buildInputs? |
Builds but needs more checking. Still to do (notes to myself):
|
I've tried enabling the tests on Jedi, 786 failed, 1904 passed. I'm not sure what's wrong with it, I've tried adding in On the plus side, master's version of jedi produces similar numbers of fails and passes, so I guess I haven't made anything worse? |
Todo:
Almost there, but I'm getting test failures with pyls version 29.1 was the previous nixpkgs version and it still builds successfully. In 29.2 pyls started using subprocess. palantir/python-language-server@5850ac0#diff-12a2d0e8313818662c49cc658cd8e8ae I don't think I know enough to fix this one. @Mic92 (the maintainer), do you have any suggestions? |
Was flake8 in the $PATH during check phase? |
Thanks @Mic92, I had previously put flake8 into checkInputs and then accidentally reverted it. Fixed now, and pyls builds for 2.7, 3.6, 3.7 and 3.8. But Spyder in nixpkgs has been reverted to 3.3, so I no longer need to bump pyls' version. Since I've got it done, should I submit a new PR for python-language-server: 29.1 -> 31.8? Or should we leave nixpkgs at 29.1 until someone actually needs a later version? |
Since you put some effort in finding out how to upgrade pyls it might be beneficial if that information does not get lost. |
Motivation for this change
#80083
Summary: Spyder version was bumped in a batch process, but dependencies were not updated.
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)I'm working through the giant web of associated packages that need version bumps and changes to satisfy spyder's setup.py.
Commits will be tidied up before finalising this PR.