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
pythonPackages.discordpy: Use websockets==3.4 #34353
Conversation
discord.py only works with websockets<4 See Rapptz/discord.py#973
@GrahamcOfBorg build python3Packages.discordpy |
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.
Success for system: x86_64-linux
Installing collected packages: discord.py
Successfully installed discord.py-0.16.12
/tmp/nix-build-python3.6-discord.py-0.16.12.drv-0/discord.py-0.16.12
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/0jlffjsyfnxq2vb3avgm3an4v7m4c1sy-python3.6-discord.py-0.16.12
strip is /nix/store/mdyy001q67hiks0g24ra53z7ckm4jfr4-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/0jlffjsyfnxq2vb3avgm3an4v7m4c1sy-python3.6-discord.py-0.16.12/lib
patching script interpreter paths in /nix/store/0jlffjsyfnxq2vb3avgm3an4v7m4c1sy-python3.6-discord.py-0.16.12
checking for references to /tmp/nix-build-python3.6-discord.py-0.16.12.drv-0 in /nix/store/0jlffjsyfnxq2vb3avgm3an4v7m4c1sy-python3.6-discord.py-0.16.12...
/nix/store/0jlffjsyfnxq2vb3avgm3an4v7m4c1sy-python3.6-discord.py-0.16.12
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.
Success for system: aarch64-linux
Installing collected packages: discord.py
Successfully installed discord.py-0.16.12
/build/discord.py-0.16.12
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/dr8bayf7qh48gv3aqyim6dv3hvz804n1-python3.6-discord.py-0.16.12
strip is /nix/store/jwz859pxqj7sl2dbwvmxkx68jp774izb-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/dr8bayf7qh48gv3aqyim6dv3hvz804n1-python3.6-discord.py-0.16.12/lib
patching script interpreter paths in /nix/store/dr8bayf7qh48gv3aqyim6dv3hvz804n1-python3.6-discord.py-0.16.12
checking for references to /build in /nix/store/dr8bayf7qh48gv3aqyim6dv3hvz804n1-python3.6-discord.py-0.16.12...
/nix/store/dr8bayf7qh48gv3aqyim6dv3hvz804n1-python3.6-discord.py-0.16.12
, asyncio | ||
, aiohttp | ||
, websockets | ||
, pynacl | ||
}: | ||
|
||
let |
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.
The purpose of python-packages.nix
is to provide a coherent set of packages that work together. We cannot have any overrides inside python-packages.nix
because that may cause collisions. In this case we likely have to mark the package as broken.
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.
I am doing the same in #34188 which probably means it can only be packaged as an application.
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.
Correct.
Motivation for this change
discord.py only works with websockets<4
See Rapptz/discord.py#973
I have not tested this, since I have no use for discord.py.
I only came across the problem using
nox-review
on #34352.Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)