-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Shut down the media stack when shutting down the constellation #19225
Conversation
Heads up! This PR modifies the following files:
|
@bors-servo try This looks like it does nothing, but thanks to servo/gecko-media#69 it actually does. |
Shut down the media stack when shutting down the constellation <!-- 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/19225) <!-- Reviewable:end -->
💔 Test failed - linux-dev |
Linking fails with:
I have no idea why. |
At least, it's consistent and failed on all platforms. Could it be a Servo-specific argument passed to cargo, rustc or the linker that makes the operation fail? |
I misread, it only fails on Linux. |
While it's broken on linux, this also introduced a new source of intermittent failure in tests on macOS:
|
@bors-servo try |
Shut down the media stack when shutting down the constellation <!-- 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/19225) <!-- Reviewable:end -->
💔 Test failed - arm64 |
@bors-servo try |
Shut down the media stack when shutting down the constellation <!-- 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/19225) <!-- Reviewable:end -->
💔 Test failed - arm64 |
Cfg-gating too hard for me. @bors-servo try |
Shut down the media stack when shutting down the constellation <!-- 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/19225) <!-- Reviewable:end -->
💔 Test failed - linux-rel-wpt |
Not a very useful message, but at least there are no linking issues anymore. I'll try to catch panics in gecko-media and see what happens. |
☔ The latest upstream changes (presumably #19301) made this pull request unmergeable. Please resolve the merge conflicts. |
@bors-servo try |
Shut down the media stack when shutting down the constellation <!-- 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/19225) <!-- Reviewable:end -->
☀️ Test successful - android, 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, windows-msvc-dev |
@jdm r? |
@bors-servo: r+ |
📌 Commit 38fd964 has been approved by |
Shut down the media stack when shutting down the constellation <!-- 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/19225) <!-- Reviewable:end -->
☀️ Test successful - android, 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, windows-msvc-dev |
This change is