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
python.pkgs.slackclient: 1.2.1 -> 2.5.0 #80755
Conversation
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.
It builds with Python 3.7 on NixOS. Python 2.7 fails
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.
It doesn't build in python 2.7, since it
@JeffLabonte Thanks for the review! I adressed the comments, PTAL. |
I am wondering if the python in python-slackclient is necessary. I mean you already have python3.7 or python3.8 to add already. Also it is already in python-module, so I believe that it is one too many repetition for me. What do you think @flokli? |
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.
regression in errbot
[2 built (1 failed), 17 copied (7.6 MiB), 2.1 MiB DL]
error: build of '/nix/store/0446y6a7q6x3g85la0anjjj54arv14jz-env.drv' failed
https://github.com/NixOS/nixpkgs/pull/80755
1 package failed to build:
errbot
1 package built:
python37Packages.slackclient
|
since errbot is an application, do you mind pinning the version similar to |
@jonringer I pinned |
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.
otherwise LGTM
43deafe
to
01f9faf
Compare
I don't really get why eval fails here, seems to be unrelated. |
seeing as the latest 15 or so PRs are affected by this, I don't think this was your doing |
I formalized a ticket #81325 |
@GrahamcOfBorg eval |
@GrahamcOfBorg build python3.pkgs.slackclient errbot |
errbot requires slackclient 1.x, see errbotio/errbot#1367 latest 1.x release would be 1.3.2, but it requires an older websocket_client than the one in nixpkgs so let's just vendor the known-working version until they've migrated to 2.x.
@GrahamcOfBorg build python3.pkgs.slackclient errbot |
looks like this is a regression, errbot.darwin builds on hydra https://hydra.nixos.org/build/113402669 |
@GrahamcOfBorg build errbot |
1 similar comment
@GrahamcOfBorg build errbot |
@jonringer I couldn't see how errbot could have regressed - we still ship the same dependencies afterwards. Digging a bit more trough hydra, this seems to be flaky on darwin already: https://hydra.nixos.org/build/113098164/nixlog/1 I could mark it as broken for darwin, and add a comment explaining the kind of flakyness. |
At least tests seem to be flaky there.
Pushed a commit doing that. Feel free to merge :-) |
nah I'm fine with that. It's unfortunate that flaky tests exist, and on hardware most people don't have. |
that being said, I have no motivation to ever get another mac |
This bumps
slackclient
to the latest PyPi release, and usespytestCheckHook to run pytest tests.
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)