Skip to content
This repository was archived by the owner on Apr 14, 2021. It is now read-only.

Removed win32-open3 workaround.#6740

Merged
1 commit merged intomasterfrom
remove-old-open3-hack
Oct 16, 2018
Merged

Removed win32-open3 workaround.#6740
1 commit merged intomasterfrom
remove-old-open3-hack

Conversation

@hsbt
Copy link
Copy Markdown
Member

@hsbt hsbt commented Oct 16, 2018

What was the end-user problem that led to this PR?

win32-open3 gem was required Ruby 1.8 with Windows environment. Because RubyInstaller2 only provides the version of Ruby 2.1+.

What is your fix for the problem, implemented in this PR?

Removed needless workaround.

  win32-open3 gem was required Ruby 1.8 with Windows environment.
  Because RubyInstaller2 only provides the version of Ruby 2.1+.
@hsbt
Copy link
Copy Markdown
Member Author

hsbt commented Oct 16, 2018

See original commit. ruby/ruby@f35fb6d

@colby-swandale
Copy link
Copy Markdown
Member

Great commit! it's nice to be able to cleanup 1.8

@bundlerbot r+

ghost pushed a commit that referenced this pull request Oct 16, 2018
6740: Removed win32-open3 workaround. r=colby-swandale a=hsbt

### What was the end-user problem that led to this PR?

`win32-open3` gem was required Ruby 1.8 with Windows environment. Because RubyInstaller2 only provides the version of Ruby 2.1+.

### What is your fix for the problem, implemented in this PR?

Removed needless workaround.



Co-authored-by: SHIBATA Hiroshi <hsbt@ruby-lang.org>
@ghost
Copy link
Copy Markdown

ghost commented Oct 16, 2018

Build succeeded

@ghost ghost merged commit 0bc55ba into master Oct 16, 2018
@colby-swandale colby-swandale deleted the remove-old-open3-hack branch October 16, 2018 03:39
@hsbt hsbt added this to the 1.17.0 milestone Oct 17, 2018
@hsbt hsbt modified the milestones: 1.17.0, 2.0.0 Oct 25, 2018
colby-swandale pushed a commit that referenced this pull request Oct 29, 2018
6740: Removed win32-open3 workaround. r=colby-swandale a=hsbt

### What was the end-user problem that led to this PR?

`win32-open3` gem was required Ruby 1.8 with Windows environment. Because RubyInstaller2 only provides the version of Ruby 2.1+.

### What is your fix for the problem, implemented in this PR?

Removed needless workaround.



Co-authored-by: SHIBATA Hiroshi <hsbt@ruby-lang.org>
(cherry picked from commit 6063718)
colby-swandale pushed a commit that referenced this pull request Oct 29, 2018
6740: Removed win32-open3 workaround. r=colby-swandale a=hsbt

### What was the end-user problem that led to this PR?

`win32-open3` gem was required Ruby 1.8 with Windows environment. Because RubyInstaller2 only provides the version of Ruby 2.1+.

### What is your fix for the problem, implemented in this PR?

Removed needless workaround.



Co-authored-by: SHIBATA Hiroshi <hsbt@ruby-lang.org>
(cherry picked from commit 6063718)
hsbt added a commit that referenced this pull request Nov 10, 2018
6740: Removed win32-open3 workaround. r=colby-swandale a=hsbt

### What was the end-user problem that led to this PR?

`win32-open3` gem was required Ruby 1.8 with Windows environment. Because RubyInstaller2 only provides the version of Ruby 2.1+.

### What is your fix for the problem, implemented in this PR?

Removed needless workaround.



Co-authored-by: SHIBATA Hiroshi <hsbt@ruby-lang.org>
(cherry picked from commit 6063718)
hsbt added a commit that referenced this pull request Dec 11, 2018
6740: Removed win32-open3 workaround. r=colby-swandale a=hsbt

### What was the end-user problem that led to this PR?

`win32-open3` gem was required Ruby 1.8 with Windows environment. Because RubyInstaller2 only provides the version of Ruby 2.1+.

### What is your fix for the problem, implemented in this PR?

Removed needless workaround.



Co-authored-by: SHIBATA Hiroshi <hsbt@ruby-lang.org>
(cherry picked from commit 6063718)
(cherry picked from commit ca48374)
colby-swandale pushed a commit that referenced this pull request Dec 11, 2018
* 1-17-stable:
  Version 1.17.2 with changelog
  Merge #6798
  Merge #6818
  Merge #6786
  Merge #6743
  Merge #6742
  Merge #6740
  Merge #6834
  Merge pull request #6752 from bundler/indirect/backport-6737
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants