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

Improve ARM cross compilation #48

Merged
merged 3 commits into from Jul 28, 2015
Merged

Improve ARM cross compilation #48

merged 3 commits into from Jul 28, 2015

Conversation

@mmatyas
Copy link
Contributor

mmatyas commented Jul 2, 2015

The first patch fixes some configuration flags for ARM, which also means jemalloc is now disabled for all targets.
The second patch fixes zlib flags, which caused moz.build errors during cross compilation. I'm not sure if this is the correct way to solve this of if this works for all platforms, could you check it?

Requires servo/servo#6531
Part of servo/servo#6327

Review on Reviewable

@mmatyas mmatyas force-pushed the mmatyas:arm_build2 branch from 76c7d49 to 2c1c609 Jul 28, 2015
@mmatyas
Copy link
Contributor Author

mmatyas commented Jul 28, 2015

Updated as described in servo/servo#6537

@jdm
Copy link
Member

jdm commented Jul 28, 2015

@bors-servo
Copy link
Contributor

bors-servo commented Jul 28, 2015

📌 Commit 2c1c609 has been approved by jdm

@bors-servo
Copy link
Contributor

bors-servo commented Jul 28, 2015

Testing commit 2c1c609 with merge 496f9e2...

bors-servo pushed a commit that referenced this pull request Jul 28, 2015
bors-servo
Improve ARM cross compilation

The first patch fixes some configuration flags for ARM, which also means `jemalloc` is now disabled for all targets.
The second patch fixes zlib flags, which caused moz.build errors during cross compilation. I'm not sure if this is the correct way to solve this of if this works for all platforms, could you check it?

Requires servo/servo#6531
Part of servo/servo#6327

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/mozjs/48)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jul 28, 2015

☀️ Test successful - cargo-linux, cargo-mac

@bors-servo bors-servo merged commit 2c1c609 into servo:master Jul 28, 2015
1 of 2 checks passed
1 of 2 checks passed
continuous-integration/appveyor AppVeyor build failed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.