Skip to content
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

Fix fs tests on Windows systems with non-english locales. #38536

Merged
merged 1 commit into from
Dec 26, 2016

Conversation

retep998
Copy link
Member

@retep998 retep998 commented Dec 22, 2016

@rust-highfive
Copy link
Collaborator

r? @brson

(rust_highfive has picked a reviewer for you, use r? to override)

@petrochenkov
Copy link
Contributor

Does this fix #34628?
(That issue becomes more important because rustbuild doesn't allow manually skipping tests currently.)

@retep998
Copy link
Member Author

@petrochenkov It does now.

@petrochenkov
Copy link
Contributor

@bors r+

@bors
Copy link
Contributor

bors commented Dec 23, 2016

📌 Commit 23cfcdd has been approved by petrochenkov

@bors
Copy link
Contributor

bors commented Dec 26, 2016

⌛ Testing commit 23cfcdd with merge 8493dbe...

bors added a commit that referenced this pull request Dec 26, 2016
Fix fs tests on Windows systems with non-english locales.

Fixes #34628

r? @alexcrichton
@bors
Copy link
Contributor

bors commented Dec 26, 2016

☀️ Test successful - status-appveyor, status-travis
Approved by: petrochenkov
Pushing 8493dbe to master...

@bors bors merged commit 23cfcdd into rust-lang:master Dec 26, 2016
@daschl
Copy link

daschl commented Dec 26, 2016

@retep998 how did that branch name came to be? 😂

@est31
Copy link
Member

est31 commented Dec 26, 2016

@daschl I've stumbled across this error on a german locale, and the 🐇 has fixed it :)

@petrochenkov
Copy link
Contributor

Can confirm, with chcp 65001 all make check tests now pass on a totally non-English (Windows itself, locale, username, env vars) system.

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.

7 participants