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

Cgo without cgo #1322

Merged
merged 2 commits into from
Oct 21, 2020
Merged

Conversation

sagikazarmark
Copy link
Contributor

CGO is often not required for all platforms. This PR improves cross-compilation support by allowing to build regular go libraries when no cgo sources are provided (for example by using select to return an empty list on platforms not requiring cgo)

Signed-off-by: Mark Sagi-Kazar <mark.sagikazar@gmail.com>
Signed-off-by: Mark Sagi-Kazar <mark.sagikazar@gmail.com>
@sagikazarmark
Copy link
Contributor Author

Breaking test seems unrelated

Copy link
Member

@Tatskaari Tatskaari left a comment

Choose a reason for hiding this comment

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

I guess creating a cgo library without any c sources is a bit weird but not a problem. Thanks for that!

@Tatskaari
Copy link
Member

Breaking test seems unrelated

Yeah I think cirrus just hates us...

@Tatskaari Tatskaari merged commit 006bfa5 into thought-machine:master Oct 21, 2020
@sagikazarmark sagikazarmark deleted the cgo-without-cgo branch October 21, 2020 17:03
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.

2 participants