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

Apply patch from rubygems/rubygems#1685 #4198

Merged
merged 1 commit into from Oct 5, 2016

Conversation

Projects
None yet
2 participants
@etehtsea
Contributor

etehtsea commented Oct 1, 2016

Fixes #4019

rubygems/rubygems#1685 didn't get into release so I added it manually like in bdb4a36

@etehtsea

This comment has been minimized.

Show comment
Hide comment
@etehtsea

etehtsea Oct 1, 2016

Contributor

It looks like smth went wrong. I'll dig into it.

Contributor

etehtsea commented Oct 1, 2016

It looks like smth went wrong. I'll dig into it.

@etehtsea

This comment has been minimized.

Show comment
Hide comment
@etehtsea

etehtsea Oct 1, 2016

Contributor

Ok, I reverted update to 2.6.7 because it breaks default behavior.

When I ran:

 bin/jruby -S gem list --no-version |xargs bin/jruby -S gem uninstall -aI

it removed all default gems:

bin/jruby -S gem list

*** LOCAL GEMS ***
Contributor

etehtsea commented Oct 1, 2016

Ok, I reverted update to 2.6.7 because it breaks default behavior.

When I ran:

 bin/jruby -S gem list --no-version |xargs bin/jruby -S gem uninstall -aI

it removed all default gems:

bin/jruby -S gem list

*** LOCAL GEMS ***

@etehtsea etehtsea changed the title from RubyGems 2.6.7 + patch in rubygems/rubygems#1685 to Apply patch in rubygems/rubygems#1685 Oct 1, 2016

@etehtsea etehtsea changed the title from Apply patch in rubygems/rubygems#1685 to Apply patch from rubygems/rubygems#1685 Oct 1, 2016

@headius

This comment has been minimized.

Show comment
Hide comment
@headius

headius Oct 5, 2016

Member

Seems fine to let this bake on master. I'll file an issue for updating to a stock release before 9.1.6.0.

Member

headius commented Oct 5, 2016

Seems fine to let this bake on master. I'll file an issue for updating to a stock release before 9.1.6.0.

@headius

headius approved these changes Oct 5, 2016

@headius headius added this to the JRuby 9.1.6.0 milestone Oct 5, 2016

@headius headius merged commit 7c4d118 into jruby:master Oct 5, 2016

0 of 2 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details

@headius headius referenced this pull request Oct 5, 2016

Closed

Update RubyGems #4205

@etehtsea etehtsea deleted the etehtsea:gh-4019-rubygems-update branch Oct 5, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment