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

Update golang.org/x/sys. #655

Merged
merged 1 commit into from Dec 13, 2022
Merged

Update golang.org/x/sys. #655

merged 1 commit into from Dec 13, 2022

Conversation

tjni
Copy link
Contributor

@tjni tjni commented Nov 14, 2022

This picks up a new 2022 version of golang.org/x/sys which is caused by golang/go#49219 and is needed to fix building using Go 1.18 on aarch64-darwin. Fixes #654.

As a side note: We cannot upgrade to the latest version of golang.org/x/sys because, at least for now, it requires a minimum Go version of 1.17.

This picks up a new 2022 version of golang.org/x/sys which is caused by
golang/go#49219 and is needed to fix building
using Go 1.18 on aarch64-darwin.
@tjni tjni mentioned this pull request Dec 3, 2022
13 tasks
@tjni
Copy link
Contributor Author

tjni commented Dec 12, 2022

Would it be possible to review and merge if this looks acceptable? Because the dependencies here are vendored, leading to many changes, they are not easy to patch as a package manager. In my case, I'd like to unbreak this package on Darwin for nixpkgs users.

/cc @astromechza @harryjsmith @metamemelord @JadeRedworth as the 4 most recent folks who committed to this repository. I envision many of you are quite busy, so I'll only ping occasionally.

Copy link
Contributor

@astromechza astromechza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, builds on my x86 darwin machine too

@astromechza astromechza merged commit 740a8f3 into fnproject:master Dec 13, 2022
@tjni tjni deleted the update-x-sys branch December 13, 2022 09:28
@tjni tjni mentioned this pull request Dec 13, 2022
13 tasks
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.

Go package x/sys needs an update for darwin
2 participants