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

Update dependencies #22221

Merged
merged 6 commits into from Nov 27, 2018
Merged

Update dependencies #22221

merged 6 commits into from Nov 27, 2018
+79 −64

Conversation

@Eijebong
Copy link
Member

Eijebong commented Nov 18, 2018 •
edited by SimonSapin

Those are all easy and unrelated dependency updates, hence the single PR

This change is Reviewable
@highfive
Copy link

highfive commented Nov 18, 2018

Heads up! This PR modifies the following files:

  • @asajeffrey: components/script/Cargo.toml, components/webdriver_server/Cargo.toml
  • @jgraham: components/webdriver_server/Cargo.toml
  • @KiChjang: components/net/Cargo.toml, components/script/Cargo.toml
  • @emilio: components/style/Cargo.toml
@highfive
Copy link

highfive commented Nov 18, 2018

warning Warning warning

  • These commits modify net, style, gfx, and script code, but no tests are modified. Please consider adding a test!
@Eijebong
Copy link
Member Author

Eijebong commented Nov 18, 2018

@bors-servo try
@bors-servo
Copy link
Contributor

bors-servo commented Nov 18, 2018

⌛ Trying commit 2a8af39 with merge d82077a...
bors-servo added a commit that referenced this pull request Nov 18, 2018
@bors-servo
Auto merge of #22221 - Eijebong:random-depup, r=<try>
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
d82077a 
Update dependencies

Those are all easy and unrelated dependency updates, hence the single PR
@bors-servo
Copy link
Contributor

bors-servo commented Nov 18, 2018

💔 Test failed - mac-rel-wpt3
@Eijebong
Copy link
Member Author

Eijebong commented Nov 18, 2018

Mhhh I guess the openssl changes are to blame here
@Eijebong
Copy link
Member Author

Eijebong commented Nov 18, 2018

Actually... It passed on linux-wpt.

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

bors-servo commented Nov 18, 2018

⌛ Trying commit 2a8af39 with merge 22f9245...
bors-servo added a commit that referenced this pull request Nov 18, 2018
@bors-servo
Auto merge of #22221 - Eijebong:random-depup, r=<try>
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
22f9245 
Update dependencies

Those are all easy and unrelated dependency updates, hence the single PR

<!-- 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/22221)
<!-- Reviewable:end -->
@emilio
emilio approved these changes Nov 18, 2018
View changes
Copy link
Member

emilio left a comment

r=me, assuming try is happy
@emilio
Copy link
Member

emilio commented Nov 18, 2018

@bors-servo r+
@bors-servo
Copy link
Contributor

bors-servo commented Nov 18, 2018

📌 Commit 2a8af39 has been approved by emilio
@highfive highfive assigned emilio and unassigned asajeffrey Nov 18, 2018
@bors-servo
Copy link
Contributor

bors-servo commented Nov 18, 2018

⌛ Testing commit 2a8af39 with merge 657248b...
bors-servo added a commit that referenced this pull request Nov 18, 2018
@bors-servo
Auto merge of #22221 - Eijebong:random-depup, r=emilio
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
657248b 
Update dependencies

Those are all easy and unrelated dependency updates, hence the single PR

<!-- 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/22221)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 18, 2018

💔 Test failed - mac-rel-wpt3
@Eijebong
Copy link
Member Author

Eijebong commented Nov 18, 2018

Ok, this isn't intermittent but only happens on mac. I'll have a look tomorrow
@bors-servo
Copy link
Contributor

bors-servo commented Nov 19, 2018

☔ The latest upstream changes (presumably #22200) made this pull request unmergeable. Please resolve the merge conflicts.
@Eijebong
Update bindgen
5fdcd8a
@atouchet
Copy link
Contributor

atouchet commented Nov 22, 2018

hyper 0.12.16 is now out.
@atouchet atouchet mentioned this pull request Nov 22, 2018
Open
@nox
Copy link
Member

nox commented Nov 27, 2018

@bors-servo r+
@bors-servo
Copy link
Contributor

bors-servo commented Nov 27, 2018

📌 Commit dafa2cf has been approved by nox
@highfive highfive assigned nox and unassigned emilio Nov 27, 2018
@bors-servo
Copy link
Contributor

bors-servo commented Nov 27, 2018

⌛ Testing commit dafa2cf with merge ef1b6ee...
bors-servo added a commit that referenced this pull request Nov 27, 2018
@bors-servo
Auto merge of #22221 - Eijebong:random-depup, r=nox
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
ef1b6ee 
Update dependencies

Those are all easy and unrelated dependency updates, hence the single PR

<!-- 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/22221)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 27, 2018

💔 Test failed - linux-rel-wpt
@CYBAI
Copy link
Collaborator

CYBAI commented Nov 27, 2018

@bors-servo retry

{"status": "FAIL", "group": "default", "message": "/_mozilla/css/inline_block_parent_padding_a.html 495641fc705b6af91aa9ff0d3235cd2c5342179c\n/_mozilla/css/inline_block_parent_padding_ref.html 4e6ec2a4e7504e5c4bb739eb4fad793251d3180f\nTesting 495641fc705b6af91aa9ff0d3235cd2c5342179c == 4e6ec2a4e7504e5c4bb739eb4fad793251d3180f", "stack": null, "subtest": null, "test": "/_mozilla/css/inline_block_parent_padding_a.html", "line": 31791, "action": "test_result", "expected": "PASS"}
@jdm
Copy link
Member

jdm commented Nov 27, 2018

That failure was #22062 for the record.
bors-servo added a commit that referenced this pull request Nov 27, 2018
@bors-servo
Auto merge of #22221 - Eijebong:random-depup, r=nox
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
482923c 
Update dependencies

Those are all easy and unrelated dependency updates, hence the single PR

<!-- 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/22221)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 27, 2018

⌛ Testing commit dafa2cf with merge 482923c...
@bors-servo
Copy link
Contributor

bors-servo commented Nov 27, 2018

💔 Test failed - mac-rel-css1
@jdm
Copy link
Member

jdm commented Nov 27, 2018

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

bors-servo commented Nov 27, 2018

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

bors-servo commented Nov 27, 2018

☀️ Test successful - android, android-mac, 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, magicleap, status-taskcluster, windows-msvc-dev
Approved by: nox
Pushing 482923c to master...
@bors-servo bors-servo merged commit dafa2cf into servo:master Nov 27, 2018
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
@bors-servo bors-servo mentioned this pull request Nov 27, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@emilio emilio

Assignees

@nox nox

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

Successfully merging this pull request may close these issues.

None yet

9 participants
@Eijebong @highfive @bors-servo @emilio @atouchet @nox @CYBAI @jdm @asajeffrey
You can’t perform that action at this time.