-
Notifications
You must be signed in to change notification settings - Fork 25
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
fix build on x86_64-darwin #80
base: master
Are you sure you want to change the base?
Conversation
46bf60e
to
bee1b0f
Compare
Should we add a Darwin build to CI? Does the free GHA plan allow this? |
@fricklerhandwerk does |
@phanirithvij indeed, the network mocks seem to be failing, but at first glance that looks more like a race condition. Specifically, there's
... and then the git fetcher seems to get 404 on some items. |
I did some basic changes and here's the diff master...phanirithvij:npins:drawin-ci-temp and the test is hanging https://github.com/phanirithvij/npins/actions/runs/9439870618/job/25998498494, not sure if it helps. not helpfulI have never owned/used a mac or plan to, I was just curious if I could get it working.coreutils dependency is for the timeout command. |
@phanirithvij thanks for taking a stab. The change doesn't seem to make a difference for me though. |
Perhaps worth considering a random port for these kinds of tests. That might already fix it? not sure what might be running on random macos box without a sandbox. |
I changed the all the mkGitTest invocations to use a different port 9000 and the githubtest ones still use 8000. This change let the tests run but they didn't exit properly and I had to stop the workflow run after 30+ mins. (Not getting 404 anymore) #80 (comment) |
also "works on my machine" now