-
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
Stack integration test suite failing for master branch #4142
Comments
Fixed the second test failure in #4149 and I guess the last one is just a matter of changing the resolver at https://github.com/commercialhaskell/stack/pull/4114/files#diff-d4f605d4bd11f79da81131332c55f32aR1 Will do that in around 12-13 hours from now. |
Solve one of the failing integration tests from #4142
I've gone ahead and merged @mihaimaruseac's fixes in #4146, #4149, and #4152 (thanks Mihai). I'll confirm that all the tests pass once the CI builds have completed then close this out. |
Thanks @mattaudesse. Did you get a chance to check if CI builds and passes all tests? |
Hey @mihaimaruseac - I think we're all set but I ran out of time last night trying to verify the integration tests succeed locally. Using FreeBSD might be adding some additional variables to the mix in my case so I'm going to re-run in a Debian VM and see how it goes. I have to leave my computer for a few hours but hopefully I'll return to a 100% successful test run. Or you guys can just close this out if you know it's working - sorry for the delay! |
Cool, I'm seeing fully successful integration test runs now - closing. |
General summary/comments
Observed on Linux and macOS (not tested on any other platforms). This blocks the release of stack-1.8.x.
Steps to reproduce
Run:
Expected
All integration tests pass.
Actual
Some of the integration tests fail:
Stack version
master
branch, as of commit bcbfb1e.Method of installation
Build from source.
The text was updated successfully, but these errors were encountered: