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
Switch to upstream servo/osmesa-src. #3308
Conversation
@bors-servo r+ |
📌 Commit e0afb44 has been approved by |
r? @jrmuizel |
This has the same Permission denied problem
|
Could it be something like the +x file permission bit is somehow not set on that script? |
It'd be really weird that that only happens when building in parallel mode. Looks like OSX make doesn't really respect MAKEFLAGS, but I'll find a different solution. @bors-servo r- |
This is what mozjs does, and allows me to fix up servo/webrender#3308.
Allow specifying make via an environment variable. This is what mozjs does, and allows me to fix up servo/webrender#3308.
Let's see how that goes. |
Make the mac hack for servo#2638 specific for CI, via `CARGO_MAKEFLAGS`.
bc20b91
to
d477ecc
Compare
https://tools.taskcluster.net/groups/RlTzFrmjSfyDJykTgmkftw is green on OSX. |
@bors-servo r+ |
📌 Commit d477ecc has been approved by |
@@ -156,6 +156,9 @@ tasks: | |||
export RUSTFLAGS='--deny warnings' | |||
export PKG_CONFIG_PATH="/usr/local/opt/zlib/lib/pkgconfig:$PKG_CONFIG_PATH" | |||
export RUSTC_WRAPPER=sccache | |||
echo 'exec make -j1 "$@"' > $HOME/make # See #2638 | |||
chmod +x $HOME/make |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is terrible.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
True that.
Switch to upstream servo/osmesa-src. Make the mac hack for #2638 specific for CI, via `CARGO_MAKEFLAGS`. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/3308) <!-- Reviewable:end -->
☀️ Test successful - status-appveyor, status-taskcluster |
Make the mac hack for #2638 specific for CI, via
CARGO_MAKEFLAGS
.This change is