deemix: init at 2.0.16 #108942
deemix: init at 2.0.16 #108942
Conversation
|
Result of 6 packages built:
|
| eventlet | ||
| ]; | ||
|
|
||
| doCheck = false; # tcp: protocol not found |
There was a problem hiding this comment.
| doCheck = false; # tcp: protocol not found | |
| doCheck = false; # tcp: protocol not found | |
| pythonImportsCheck = [ "deezer" ]; |
There was a problem hiding this comment.
It fails the same way tests do:
File "/nix/store/rdgah0m8hww9pidmjbay2hjf81hnyzmj-python3.8-dnspython-1.16.0/lib/python3.8/site-packages/dns/rdtypes/IN/WKS.py", line 25, in <module>
_proto_tcp = socket.getprotobyname('tcp')
OSError: protocol not found
There was a problem hiding this comment.
That shouldn't happen. If this is not working the package wont work.
There was a problem hiding this comment.
But it happens, and the package works.
Why shouldn't it be happening? The build is sandboxed, so it is not surprising it cannot have access to tcp connections.
There was a problem hiding this comment.
Importing a package should not require a network connection.
There was a problem hiding this comment.
I agree, it should not, but this is Python, and I guess imports can have side effects…
There was a problem hiding this comment.
You could try having /etc/protocols in the sandbox and see what happens.
There was a problem hiding this comment.
Even with this it fails with the same error when importing eventlet.
|
This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 6 packages built:
|
|
Superseded by #132580 |
Motivation for this change
I will package deemix-pyweb
Things done
sandboxinnix.confon non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"./result/bin/)nix path-info -Sbefore and after)