-
-
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
Remove the msg dependency from style #9261
Conversation
@bors-servo try |
Remove the msg dependency from style This is part of slimming down the dependencies of geckolib. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9261) <!-- Reviewable:end -->
r? @SimonSapin |
@@ -39,6 +39,7 @@ | |||
os.path.join(".", "target", "*"), | |||
os.path.join(".", "ports", "gonk", "src", "native_window_glue.cpp"), | |||
os.path.join(".", "ports", "cef", "*"), | |||
os.path.join(".", "ports", "geckolib", "*"), |
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.
I don't think we need to ignore this, do we?
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.
Good catch - that was an accidental leftover from rebasing this over what actually landed in #9209.
☔ The latest upstream changes (presumably #9249) made this pull request unmergeable. Please resolve the merge conflicts. |
💔 Test failed - gonk |
@bors-servo try |
f99078f
to
e93effe
Compare
@bors-servo try |
🔒 Merge conflict |
e93effe
to
390d47e
Compare
@bors-servo try |
Remove the msg dependency from style This is part of slimming down the dependencies of geckolib. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9261) <!-- Reviewable:end -->
💔 Test failed - mac-dev-ref-unit |
This is puzzling... #9254 didn't even land yet! |
The pipeline id stuff is currently unused. If someone needs it, they can add an additional trait bound on their css error reporter to get the pipeline id.
390d47e
to
384cdfc
Compare
@bors-servo try |
Remove the msg dependency from style This is part of slimming down the dependencies of geckolib. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9261) <!-- Reviewable:end -->
☀️ Test successful - android, gonk, linux-dev, linux-rel, mac-dev-ref-unit, mac-rel-css, mac-rel-wpt |
@bors-servo r+ |
📌 Commit 384cdfc has been approved by |
Remove the msg dependency from style This is part of slimming down the dependencies of geckolib. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9261) <!-- Reviewable:end -->
fixes #9246 |
☀️ Test successful - android, gonk, linux-dev, linux-rel, mac-dev-ref-unit, mac-rel-css, mac-rel-wpt |
This is part of slimming down the dependencies of geckolib.