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

[Rails4 regression] prevent thin and puma cause error in Non ASCII URL on Windows #19689

Closed
wants to merge 7 commits into from
Closed

[Rails4 regression] prevent thin and puma cause error in Non ASCII URL on Windows #19689

wants to merge 7 commits into from

Conversation

@rafaelfranca
Copy link
Member

Could you please squash your commits?

@marutosi
Copy link
Contributor Author

marutosi commented Apr 8, 2015

Which revisions?

@rafaelfranca
Copy link
Member

The gemfile changes can be grouped in one commit and the last two commits also.

@rafaelfranca
Copy link
Member

BTW, awesome work ❤️

@marutosi
Copy link
Contributor Author

marutosi commented Apr 9, 2015

Could you please squash your commits?

Done.

rafaelfranca pushed a commit that referenced this pull request Apr 9, 2015
… on Linux

Related: #19617, #19187, #19533, #19689, #19675.

This is POC (Proof Of Concept) which bundler does not remove mingw lines.
#19617 (comment)
rafaelfranca pushed a commit that referenced this pull request Apr 9, 2015
rafaelfranca added a commit that referenced this pull request Apr 9, 2015
[Rails4 regression] prevent thin and puma cause error in Non ASCII URL
on Windows
@rafaelfranca
Copy link
Member

Merged at bfe6507

rafaelfranca added a commit that referenced this pull request Apr 9, 2015
[Rails4 regression] prevent thin and puma cause error in Non ASCII URL
on Windows
@rafaelfranca
Copy link
Member

Backported as 4df216c

@rafaelfranca
Copy link
Member

@marutosi thank you so much for the fix

@marutosi marutosi deleted the master-issue-19187 branch June 28, 2015 15:56
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.

None yet

2 participants