-
Notifications
You must be signed in to change notification settings - Fork 841
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
Failure in upload integration tests on Mac #4191
Comments
Reproduced on master, with essentially the same log AFAICS.
|
Fixing a likely typo makes the test progress more, not sure if this is success or a failure, and gotta look into my diff --git a/test/integration/tests/upload/files/FakeHackage.hs b/test/integration/tests/upload/files/FakeHackage.hs
index 825e8ad7..eef4b934 100644
--- a/test/integration/tests/upload/files/FakeHackage.hs
+++ b/test/integration/tests/upload/files/FakeHackage.hs
@@ -18,7 +18,7 @@ main =
serve :: IO ()
serve = do
let hints = defaultHints {addrFlags = [AI_PASSIVE], addrSocketType = Stream}
- (addr:_) <- getAddrInfo Nothing Nothing (Just "12415")
+ (addr:_) <- getAddrInfo (Just hints) Nothing (Just "12415")
sock <- socket (addrFamily addr) (addrSocketType addr) (addrProtocol addr)
setSocketOption sock ReuseAddr 1
bind sock (addrAddress addr) Output:
|
The gpg output is expected - the test is substituted an always failing script named |
Awesome, then we'll close this once we have reports of the test passing on all 3 major systems. Thanks |
Reported passes on all three OSes. |
The upload integration test runs into network problems on my Mac, both via
stack build --flag stack:integration-tests --test stack:stack-integration-test
and by running the test directly.I've seen the error on top of #4190 and I'm retesting on master (78762f9), but that PR shouldn't affect this test (testing).
The key error is
From the bigger test suite:
The text was updated successfully, but these errors were encountered: