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
python3Packages.fastapi: fix build #122457
Conversation
Result of 5 packages built successfully:
1 suggestion:
Result of 5 packages built successfully:
1 suggestion:
|
Let's give the author a few hours here. Also there is https://github.com/tiangolo/fastapi/releases/tag/0.64.0 (3 days ago), so even if we would go with the patch an update would be appropriate. |
https://github.com/tiangolo/fastapi/releases/tag/0.65.0 And here we are. |
disabledTests = [ | ||
"quart" # Quart is not packaged for Nixpkgs | ||
] ++ lib.optionals (pythonAtLeast "3.9") [ | ||
"test_api_example" # network access |
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.
This is really strange.
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.
Did urllib3 somehow change in Python 3.9 in a way such that local networking doesn't work in the sandbox anymore?
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPConnection object at 0x7ffff50a5f70>: Failed to establish a new connection: [Errno 111] Connection refused
cc @FRidh @jonringer
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.
Never mind, it works now. Though the test looks a little flaky:
https://github.com/bobbui/json-logging-python/blob/1e887ca1dc949893e1757d29dc8a1d75fccc8dd1/tests/smoketests/test_run_smoketest.py#L42-L55
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.
LGTM, save for the leftover pythonAtLeast
.
Motivation for this change
tiangolo/fastapi#2335 was merged.
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)