Only build ports/servo by default (except Android), add `./mach build --libsimpleservo` #21809

SimonSapin · 2018-09-25T13:01:51Z

This change is

highfive · 2018-09-25T13:01:56Z

Heads up! This PR modifies the following files:

@wafflespeanut: python/servo/command_base.py, python/servo/devenv_commands.py, python/servo/build_commands.py

SimonSapin · 2018-09-25T13:01:58Z


        Auto merge of #21809 - servo:features, r=<try>

Only build ports/servo by default (except Android), add `./mach build --libsimpleservo` Fixes #21314

bors-servo · 2018-09-25T13:02:05Z

⌛ Trying commit 8da8250 with merge d4ed7cf...

bors-servo · 2018-09-25T13:17:25Z

💔 Test failed - linux-dev

paulrouget · 2018-09-25T13:18:01Z

Can we make sure our CI keeps building libsimpleservo on at least one non-android platform?


        Fix default-except-unstable build


        Only build ports/servo by default (except Android), add `./mach build…

… --libsimpleservo` Fixes #21314


        Build libsimpleservo on CI

SimonSapin · 2018-09-25T17:22:17Z

Good point, done.

@bors-servo try

bors-servo · 2018-09-25T17:22:24Z

⌛ Trying commit 4e85f9c with merge 12609d7...


        Auto merge of #21809 - servo:features, r=<try>

Only build ports/servo by default (except Android), add `./mach build --libsimpleservo` Fixes #21314  --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/21809)

bors-servo · 2018-09-25T18:45:04Z

☀️ Test successful - android, android-x86, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, status-taskcluster, windows-msvc-dev
State: approved= try=True

jdm · 2018-09-25T19:00:59Z

@bors-servo r+

bors-servo · 2018-09-25T19:00:59Z

📌 Commit 8aed0a7 has been approved by jdm

bors-servo · 2018-09-25T19:19:50Z

⌛ Testing commit 8aed0a7 with merge 52667cc...


        Auto merge of #21809 - servo:features, r=jdm

Only build ports/servo by default (except Android), add `./mach build --libsimpleservo` Fixes #21314  --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/21809)

bors-servo · 2018-09-25T19:20:07Z

💔 Test failed - linux-rel-css

SimonSapin · 2018-09-26T12:19:04Z

11 timeouts out of 62 tests is rather suspicious :/ (This is the subset of the test suite run with --multiprocess.) @jdm, what do you think?

SimonSapin · 2018-09-27T16:46:22Z

Let’s see if it happens again.

@bors-servo r- try retry

bors-servo · 2018-09-27T16:46:30Z

⌛ Trying commit deeb796 with merge 2075afb...


        Auto merge of #21809 - servo:features, r=<try>

Only build ports/servo by default (except Android), add `./mach build --libsimpleservo` Fixes #21314  --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/21809)

SimonSapin · 2018-09-27T17:30:55Z

jdm pointed out on IRC that the error messages included Failed to start unsandboxed child process!: Os { code: 12, kind: Other, message: "Cannot allocate memory" }, so this was likely another occurrence of remaining processes on the builder taking up RAM.

@bors-servo try- r+

(Does this command work?)

bors-servo · 2018-09-27T17:30:55Z

📌 Commit deeb796 has been approved by SimonSapin

bors-servo · 2018-09-27T17:35:11Z

⌛ Testing commit deeb796 with merge 0964d05...


        Auto merge of #21809 - servo:features, r=SimonSapin

Only build ports/servo by default (except Android), add `./mach build --libsimpleservo` Fixes #21314  --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/21809)

bors-servo · 2018-09-27T18:35:19Z

💔 Test failed - mac-rel-css2

jdm · 2018-09-27T18:50:49Z

@bors-servo retry

#21795

bors-servo · 2018-09-27T18:50:52Z

⚡ Previous build results for android, android-x86, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, status-taskcluster, windows-msvc-dev are reusable. Rebuilding only mac-rel-css2...

bors-servo · 2018-09-27T19:25:08Z

💔 Test failed - mac-rel-css2

jdm · 2018-09-27T19:33:11Z

@bors-servo retry

#21795

bors-servo · 2018-09-27T19:33:14Z

⚡ Previous build results for android, android-x86, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, status-taskcluster, windows-msvc-dev are reusable. Rebuilding only mac-rel-css2...

bors-servo · 2018-09-27T20:13:53Z

💔 Test failed - mac-rel-css2

jdm · 2018-09-27T20:23:54Z

@bors-servo retry

#21795

bors-servo · 2018-09-27T20:23:57Z

⚡ Previous build results for android, android-x86, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, status-taskcluster, windows-msvc-dev are reusable. Rebuilding only mac-rel-css2...

bors-servo · 2018-09-27T21:00:19Z

☀️ Test successful - android, android-x86, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, status-taskcluster, windows-msvc-dev
Approved by: SimonSapin
Pushing 0964d05 to master...

highfive assigned avadacatavra Sep 25, 2018

highfive added the S-awaiting-review label Sep 25, 2018

highfive added the S-tests-failed label Sep 25, 2018

SimonSapin added 3 commits Sep 25, 2018

Fix default-except-unstable build

6842553

Only build ports/servo by default (except Android), add `./mach build…

39079cf

… --libsimpleservo` Fixes #21314

Build libsimpleservo on CI

Loading status checks…

4e85f9c

SimonSapin force-pushed the features branch from 8da8250 to 4e85f9c Sep 25, 2018

highfive removed the S-tests-failed label Sep 25, 2018

highfive assigned jdm and unassigned avadacatavra Sep 25, 2018

highfive added S-awaiting-merge and removed S-awaiting-review labels Sep 25, 2018

highfive added S-tests-failed and removed S-awaiting-merge labels Sep 25, 2018

SimonSapin force-pushed the features branch from 8aed0a7 to 62c593d Sep 26, 2018

highfive added the S-tests-failed label Sep 26, 2018

highfive added S-awaiting-merge and removed S-tests-failed labels Sep 27, 2018

highfive assigned SimonSapin and unassigned jdm Sep 27, 2018

highfive added S-tests-failed and removed S-awaiting-merge labels Sep 27, 2018

highfive added the S-tests-failed label Sep 27, 2018

bors-servo mentioned this pull request Sep 27, 2018

[WIP] Optional compositor #21808

Open

SimonSapin deleted the features branch Sep 28, 2018

jdm mentioned this pull request Sep 28, 2018

Appveyor is broken #21832

Closed

servo / servo

Only build ports/servo by default (except Android), add `./mach build --libsimpleservo` #21809

Only build ports/servo by default (except Android), add `./mach build --libsimpleservo` #21809

SimonSapin commented Sep 25, 2018 •

edited

highfive commented Sep 25, 2018

SimonSapin commented Sep 25, 2018

bors-servo commented Sep 25, 2018

bors-servo commented Sep 25, 2018

paulrouget commented Sep 25, 2018

SimonSapin commented Sep 25, 2018

bors-servo commented Sep 25, 2018

bors-servo commented Sep 25, 2018

jdm commented Sep 25, 2018

bors-servo commented Sep 25, 2018

bors-servo commented Sep 25, 2018

bors-servo commented Sep 25, 2018

SimonSapin commented Sep 26, 2018

SimonSapin commented Sep 27, 2018

bors-servo commented Sep 27, 2018

SimonSapin commented Sep 27, 2018

bors-servo commented Sep 27, 2018

bors-servo commented Sep 27, 2018

bors-servo commented Sep 27, 2018

jdm commented Sep 27, 2018

bors-servo commented Sep 27, 2018

bors-servo commented Sep 27, 2018

jdm commented Sep 27, 2018

bors-servo commented Sep 27, 2018

bors-servo commented Sep 27, 2018

jdm commented Sep 27, 2018

bors-servo commented Sep 27, 2018

bors-servo commented Sep 27, 2018

servo / servo

Join GitHub today

Only build ports/servo by default (except Android), add `./mach build --libsimpleservo` #21809

Only build ports/servo by default (except Android), add `./mach build --libsimpleservo` #21809

Conversation

SimonSapin commented Sep 25, 2018 • edited

highfive commented Sep 25, 2018

SimonSapin commented Sep 25, 2018

bors-servo commented Sep 25, 2018

bors-servo commented Sep 25, 2018

paulrouget commented Sep 25, 2018

SimonSapin commented Sep 25, 2018

bors-servo commented Sep 25, 2018

bors-servo commented Sep 25, 2018

jdm commented Sep 25, 2018

bors-servo commented Sep 25, 2018

bors-servo commented Sep 25, 2018

bors-servo commented Sep 25, 2018

SimonSapin commented Sep 26, 2018

SimonSapin commented Sep 27, 2018

bors-servo commented Sep 27, 2018

SimonSapin commented Sep 27, 2018

bors-servo commented Sep 27, 2018

bors-servo commented Sep 27, 2018

bors-servo commented Sep 27, 2018

jdm commented Sep 27, 2018

bors-servo commented Sep 27, 2018

bors-servo commented Sep 27, 2018

jdm commented Sep 27, 2018

bors-servo commented Sep 27, 2018

bors-servo commented Sep 27, 2018

jdm commented Sep 27, 2018

bors-servo commented Sep 27, 2018

bors-servo commented Sep 27, 2018

SimonSapin commented Sep 25, 2018 •

edited