-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Backed out changeset dd834d38eaab for Windows debug build bustage #19143
Conversation
@bors-servo r+ force p=9001 treeclosed=9000 |
📌 Commit 38b9a54 has been approved by |
Thanks for the pull request, and welcome! The Servo team is excited to review your changes, and you should hear from @glennw (or someone else) soon. |
Heads up! This PR modifies the following files:
|
@bors-servo treeclosed- |
Backed out changeset dd834d38eaab for Windows debug build bustage Backed out changeset dd834d38eaab for Windows debug build bustage Backs out #19133 <!-- 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/19143) <!-- Reviewable:end -->
@indygreg It doesn't seem to me this backout is reasonable. Judging from the log, it seems the bustage happened on the original push was just some infra intermittent, and the bustage on your push after that one was some sccache issue. The original push also doesn't touch anything we build in Gecko at all... |
💔 Test failed - linux-rel-css |
@bors-servo retry |
⚡ Previous build results for android, arm32, arm64, linux-dev, linux-rel-wpt, mac-dev-unit, mac-rel-css2, mac-rel-wpt3, windows-msvc-dev are reusable. Rebuilding only linux-rel-css, mac-rel-css1, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt4... |
It looks like I made a mistake. Sorry for the mix-up. Please re-land at your earliest convenience, |
@bors-servo: retry |
⚡ Previous build results for android, arm32, arm64, linux-dev, linux-rel-wpt, mac-dev-unit, mac-rel-css2, mac-rel-wpt2, mac-rel-wpt3, windows-msvc-dev are reusable. Rebuilding only linux-rel-css, mac-rel-css1, mac-rel-wpt1, mac-rel-wpt4... |
💔 Test failed - linux-rel-css |
@bors-servo treeclosed+ |
@bors-servo r+ force p=9001 treeclosed=9000 |
💡 This pull request was already approved, no need to approve it again.
|
📌 Commit 38b9a54 has been approved by |
We should not land anything else before backout finishes... which is part of why a servo backout sucks. But making the two repos unsync only makes things suck more. |
@bors-servo r=moz-servo-sync force p=9001 treeclosed=9000 |
💡 This pull request was already approved, no need to approve it again. |
📌 Commit 38b9a54 has been approved by |
@bors-servo retry |
@bors-servo treeclosed- |
⚡ Previous build results for android, arm32, arm64, linux-dev, linux-rel-wpt, mac-dev-unit, mac-rel-css2, mac-rel-wpt2, mac-rel-wpt3, windows-msvc-dev are reusable. Rebuilding only linux-rel-css, mac-rel-css1, mac-rel-wpt1, mac-rel-wpt4... |
☀️ 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 |
Import sequence inner types if it appears in dictionary members Relands #19133, as it was backed out wrongly in #19143. <!-- 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/19148) <!-- Reviewable:end -->
Backed out changeset dd834d38eaab for Windows debug build bustage
Backs out #19133
This change is