Skip to content
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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fricklerhandwerk
Copy link
Contributor

also "works on my machine" now

@andir
Copy link
Owner

andir commented Jun 8, 2024

Should we add a Darwin build to CI? Does the free GHA plan allow this?

@phanirithvij phanirithvij mentioned this pull request Jun 9, 2024
@phanirithvij
Copy link
Contributor

phanirithvij commented Jun 9, 2024

@fricklerhandwerk does nix-build -A meta.tests work on your mac? It is failing in gha, I don't have a mac just used the gha to debug. (see actions in my fork)

@fricklerhandwerk
Copy link
Contributor Author

@phanirithvij indeed, the network mocks seem to be failing, but at first glance that looks more like a race condition. Specifically, there's

OSError: [Errno 48] Address already in use
Connection to 127.0.0.1 8000 port [tcp/irdmi] succeeded!

... and then the git fetcher seems to get 404 on some items.

@phanirithvij
Copy link
Contributor

phanirithvij commented Jun 10, 2024

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 helpful I 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.

@fricklerhandwerk
Copy link
Contributor Author

@phanirithvij thanks for taking a stab. The change doesn't seem to make a difference for me though.

@andir
Copy link
Owner

andir commented Jun 19, 2024

@phanirithvij indeed, the network mocks seem to be failing, but at first glance that looks more like a race condition. Specifically, there's

OSError: [Errno 48] Address already in use
Connection to 127.0.0.1 8000 port [tcp/irdmi] succeeded!

... and then the git fetcher seems to get 404 on some items.

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.

@phanirithvij
Copy link
Contributor

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)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants