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

Build on ppc64(le), s390x #66

Merged
merged 1 commit into from
May 2, 2017

Conversation

sul3n3t
Copy link
Contributor

@sul3n3t sul3n3t commented Apr 30, 2017

These stubs will allow compilation on ppc64, ppc64le, and s390x architectures.

@sul3n3t
Copy link
Contributor Author

sul3n3t commented Apr 30, 2017

@stevvooe PTAL

@stevvooe
Copy link
Member

stevvooe commented May 1, 2017

LGTM

These probably need support. cc @estesp

@sul3n3t
Copy link
Contributor Author

sul3n3t commented May 1, 2017

Thanks @stevvooe! Do you think this is mergeable or would you like to wait on @estesp feedback?

@stevvooe
Copy link
Member

stevvooe commented May 1, 2017

@sul3n3t I'd like to get a little feedback to know if we need particular "road map" issues.

@estesp
Copy link
Member

estesp commented May 2, 2017

A quick spot check seems to show these syscalls (the xattr family of calls) are available in current Golang implementation for ppc64le and s390x (for example: https://github.com/golang/go/blob/master/src/syscall/zsyscall_linux_ppc64le.go)

So; obviously this PR is one way to get a building project for those architectures, but seems the long term solution is an update to the generate scripts to add the pairs for GOARCH/GOOS to get actual proper generation for these syscalls.

This was referenced May 2, 2017
@stevvooe stevvooe merged commit 67a877d into containerd:master May 2, 2017
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