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

Add powerpc builds #534

Merged
merged 1 commit into from Jul 2, 2016

Conversation

Projects
None yet
3 participants
@brson
Copy link
Contributor

brson commented Jun 15, 2016

Completely untested. How does this look @alexcrichton?

cc #531 @lu-zero

@brson brson referenced this pull request Jun 15, 2016

Closed

ppc64le not supported #531

@alexcrichton

This comment has been minimized.

Copy link
Member

alexcrichton commented Jun 15, 2016

Ah looks like the build for OpenSSL needs to be updated to build powerpc as well, but otherwise I think this should be all that's needed.

@brson brson force-pushed the brson:powerpc branch from 726d779 to d70a332 Jun 21, 2016

@brson

This comment has been minimized.

Copy link
Contributor Author

brson commented Jun 21, 2016

Updated the openssl builds.

@alexcrichton

This comment has been minimized.

Copy link
Member

alexcrichton commented Jun 21, 2016

Looks like the OS may be linux-ppc instead of linux-powerpc (and similar for others)

@brson brson force-pushed the brson:powerpc branch from d70a332 to b72d940 Jul 1, 2016

@brson

This comment has been minimized.

Copy link
Contributor Author

brson commented Jul 1, 2016

Updated again.

;;
powerpc64-*-linux-*)
OPENSSL_OS=linux-ppc64
OPENSSL_CC=powerpc64-linux-gnu-gcc

This comment has been minimized.

@alexcrichton

alexcrichton Jul 1, 2016

Member

Oh the failure now is:

powerpc64-linux-gnu-gcc: Command not found

I think that powerpc64 works by using the powerpc-linux-gnu compiler and then passing -m64 as a flag

@brson brson force-pushed the brson:powerpc branch from b72d940 to 435eba8 Jul 2, 2016

@brson

This comment has been minimized.

Copy link
Contributor Author

brson commented Jul 2, 2016

Updated again.

@alexcrichton

This comment has been minimized.

Copy link
Member

alexcrichton commented Jul 2, 2016

Hey look it works!

@alexcrichton alexcrichton merged commit 113965b into rust-lang:master Jul 2, 2016

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@lu-zero

This comment has been minimized.

Copy link

lu-zero commented Jul 2, 2016

Great !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.