Skip to content

/ servo

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

Revert "Enable multiprocess and sandbox" #12867

Merged
merged 1 commit into from Aug 15, 2016
Merged

Revert "Enable multiprocess and sandbox" #12867

merged 1 commit into from Aug 15, 2016

Conversation

@paulrouget
Copy link
Contributor

paulrouget commented Aug 15, 2016 •
edited

Multiprocess and sandboxing still have some issues:

Let's disable these features for now and reenable them later: #12868.

This change is Reviewable
@paulrouget
Revert "Enable multiprocess and sandbox"
Loading status checks…
c40a658 
This reverts commit 663f853.
@highfive
Copy link

highfive commented Aug 15, 2016

Heads up! This PR modifies the following files:

  • @wafflespeanut: python/servo/package_commands.py, python/servo/post_build_commands.py
This was referenced Aug 15, 2016
Open
Open
@emilio
Copy link
Member

emilio commented Aug 15, 2016

Also #12840, fwiw.

@bors-servo: r+
@highfive highfive assigned emilio and unassigned cbrewster Aug 15, 2016
@bors-servo
Copy link
Contributor

bors-servo commented Aug 15, 2016

📌 Commit c40a658 has been approved by emilio
@bors-servo
Copy link
Contributor

bors-servo commented Aug 15, 2016

⌛ Testing commit c40a658 with merge a617ea3...
bors-servo added a commit that referenced this pull request Aug 15, 2016
@bors-servo
Auto merge of #12867 - paulrouget:noMS, r=emilio
Loading status checks…
a617ea3 
Revert "Enable multiprocess and sandbox"

Multiprocess and sandboxing still have some issues:
* #12791
* #12807

Let's disable these features for now and reenable them later: #12868.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/12867)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Aug 15, 2016

💔 Test failed - linux-rel
@KiChjang
Copy link
Member

KiChjang commented Aug 15, 2016

@bors-servo retry
@bors-servo
Copy link
Contributor

bors-servo commented Aug 15, 2016

⚡ Previous build results for arm32, arm64, linux-dev, mac-dev-unit, mac-rel-css, mac-rel-wpt, windows-dev are reusable. Rebuilding only linux-rel...
@bors-servo
Copy link
Contributor

bors-servo commented Aug 15, 2016

💔 Test failed - linux-rel
@wafflespeanut
Copy link
Member

wafflespeanut commented Aug 15, 2016

@bors-servo retry #12874
@bors-servo
Copy link
Contributor

bors-servo commented Aug 15, 2016

⚡ Previous build results for arm32, arm64, linux-dev, mac-dev-unit, mac-rel-css, mac-rel-wpt, windows-dev are reusable. Rebuilding only linux-rel...
@bors-servo
Copy link
Contributor

bors-servo commented Aug 15, 2016

☀️ Test successful - arm32, arm64, linux-dev, linux-rel, mac-dev-unit, mac-rel-css, mac-rel-wpt, windows-dev
@bors-servo bors-servo merged commit c40a658 into servo:master Aug 15, 2016
3 checks passed
3 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@emilio emilio

Labels
S-tests-failed
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

7 participants
@paulrouget @highfive @emilio @bors-servo @KiChjang @wafflespeanut @cbrewster
You can’t perform that action at this time.