Skip to content

Update dwrote-rs to pick up non-en-us language fix#14177

Merged
bors-servo merged 1 commit intoservo:masterfrom
vvuk:dwfix
Nov 12, 2016
Merged

Update dwrote-rs to pick up non-en-us language fix#14177
bors-servo merged 1 commit intoservo:masterfrom
vvuk:dwfix

Conversation

@vvuk
Copy link
Copy Markdown
Contributor

@vvuk vvuk commented Nov 11, 2016

Update dwrote-rs to not panic on non-en-US locales


This change is Reviewable

@highfive highfive added the S-awaiting-review There is new code that needs to be reviewed. label Nov 11, 2016
@emilio
Copy link
Copy Markdown
Member

emilio commented Nov 11, 2016

@bors-servo r+

  • Reviewing that fast because I saw the patch upstream before :)

Thanks for the fix vlad!

@bors-servo
Copy link
Copy Markdown
Contributor

📌 Commit 3cbf7aa has been approved by emilio

@highfive highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-awaiting-review There is new code that needs to be reviewed. labels Nov 11, 2016
@bors-servo
Copy link
Copy Markdown
Contributor

⌛ Testing commit 3cbf7aa with merge 75fa3df...

bors-servo pushed a commit that referenced this pull request Nov 12, 2016
Update dwrote-rs to pick up non-en-us language fix

Update dwrote-rs to not panic on non-en-US locales

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

💔 Test failed - mac-rel-wpt1

@highfive highfive added S-tests-failed The changes caused existing tests to fail. and removed S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. labels Nov 12, 2016
@bors-servo
Copy link
Copy Markdown
Contributor

⌛ Testing commit 3cbf7aa with merge 791cb27...

bors-servo pushed a commit that referenced this pull request Nov 12, 2016
Update dwrote-rs to pick up non-en-us language fix

Update dwrote-rs to not panic on non-en-US locales

<!-- 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/14177)
<!-- Reviewable:end -->
@highfive highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-tests-failed The changes caused existing tests to fail. labels Nov 12, 2016
@bors-servo
Copy link
Copy Markdown
Contributor

💔 Test failed - linux-rel-wpt

@highfive highfive added S-tests-failed The changes caused existing tests to fail. and removed S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. labels Nov 12, 2016
@nox
Copy link
Copy Markdown
Contributor

nox commented Nov 12, 2016

@bors-servo retry #13479

@bors-servo
Copy link
Copy Markdown
Contributor

⌛ Testing commit 3cbf7aa with merge 04ddac6...

bors-servo pushed a commit that referenced this pull request Nov 12, 2016
Update dwrote-rs to pick up non-en-us language fix

Update dwrote-rs to not panic on non-en-US locales

<!-- 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/14177)
<!-- Reviewable:end -->
@highfive highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-tests-failed The changes caused existing tests to fail. labels Nov 12, 2016
@bors-servo
Copy link
Copy Markdown
Contributor

💔 Test failed - mac-rel-css

@highfive highfive added S-tests-failed The changes caused existing tests to fail. and removed S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. labels Nov 12, 2016
@highfive
Copy link
Copy Markdown

  ▶ TIMEOUT [expected FAIL] /css-flexbox-1_dev/html/flexbox-basic-iframe-horiz-001.htm
  │ 
  └ Shutting down the Constellation after generating an output file or exit flag specified

@jdm
Copy link
Copy Markdown
Member

jdm commented Nov 12, 2016

@bors-servo
Copy link
Copy Markdown
Contributor

⌛ Testing commit 3cbf7aa with merge 3299137...

bors-servo pushed a commit that referenced this pull request Nov 12, 2016
Update dwrote-rs to pick up non-en-us language fix

Update dwrote-rs to not panic on non-en-US locales

<!-- 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/14177)
<!-- Reviewable:end -->
@highfive highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-tests-failed The changes caused existing tests to fail. labels Nov 12, 2016
@bors-servo
Copy link
Copy Markdown
Contributor

@bors-servo bors-servo merged commit 3cbf7aa into servo:master Nov 12, 2016
@highfive highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Nov 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants