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
[20.03]: python38Packages.uvloop: enable build #84224
[20.03]: python38Packages.uvloop: enable build #84224
Conversation
This doesn't appear to build yet, per |
Weird. I'll fix it.
…On Fri, Apr 3, 2020, 9:30 PM Benjamin Hipple ***@***.***> wrote:
This doesn't appear to build yet, per nixpkgs-review. Perhaps there's
something else that needs to be backported?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#84224 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACNZYI5AQVE6SUFEVIYD2JTRK2ETRANCNFSM4L4VNBCA>
.
|
I think the issue is #80595, and this can't find the Thoughts on if I should amend this PR (e.g. add |
oh yea, i didnt backport that because it could be a breaking change in a few cases. This only really comes into play if there's compiled code. Feel free to alter the commit so that it works on 20.03. No use in adding something that is unable to build. Thanks for doing this! |
also, backporting #80595 would cause a rebuild of everything that depends on python. |
all you need is:
assuming that $TMP_DIR is right, you may need to copy it to $out. |
88c02fc
to
2511827
Compare
Allow build pass by disabling test. Isolated issue to test_sockets.py::TestAIOSockets::test_sock_close_add_reader_race. This test is supposed to be skipped, but it isn't for some reason, so we disable it instead. See uvloop#284 (MagicStack/uvloop#284) for full details. Don't know why this test isn't properly skipped. (cherry picked from commit 364909d)
2511827
to
6118f04
Compare
That builds properly. |
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.
Result of nixpkgs-review pr 84224 1
Motivation for this change
Backport #84223 to 20.03 (see for details)
ZHF: #80379
Will enable once #84223 accepted to get most accurate commit.
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)