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

Clean RUBYLIB as well in with_clean_env #4002

Merged
merged 2 commits into from
Sep 20, 2015
Merged

Clean RUBYLIB as well in with_clean_env #4002

merged 2 commits into from
Sep 20, 2015

Conversation

agis
Copy link
Contributor

@agis agis commented Sep 19, 2015

Fixes #3982.

@agis agis mentioned this pull request Sep 19, 2015
@agis
Copy link
Contributor Author

agis commented Sep 19, 2015

@indirect The original branch (3982-clean-rubylib) was based on 1-10-stable. However, since the issue is also present on master also, this PR targets master (this additionally required fixing some rubocop offenses).

I didn't yet squash for clarity (I'll squash after the review).

@segiddins
Copy link
Member

Looks good! 👍🏻

@agis
Copy link
Contributor Author

agis commented Sep 19, 2015

Squashed.

@indirect
Copy link
Member

@homu r+

@homu
Copy link
Contributor

homu commented Sep 20, 2015

📌 Commit 42b5072 has been approved by indirect

@homu
Copy link
Contributor

homu commented Sep 20, 2015

⌛ Testing commit 42b5072 with merge a136afe...

homu added a commit that referenced this pull request Sep 20, 2015
Clean RUBYLIB as well in `with_clean_env`

Fixes #3982.
@homu
Copy link
Contributor

homu commented Sep 20, 2015

☀️ Test successful - status

@homu homu merged commit 42b5072 into rubygems:master Sep 20, 2015
@agis agis deleted the 3982-clean-rubylib branch September 20, 2015 11:16
@ojab
Copy link
Contributor

ojab commented Dec 16, 2015

Please take a look @ bundler/issues#4165, looks like this commit has broken rails's spring.

sonalkr132 added a commit to sonalkr132/spring that referenced this pull request Jan 23, 2016
ENV["GEM_PATH"] was empty string, so GEM_PATH and GEM_HOME passed to
server process were same path.
It used to work before bundler 1.11.0 release cause bundler didn't clean
ENV["RUBYLIB"] and `require bundler/setup` was using that instead.
Change in bundler: rubygems/bundler#4002
sonalkr132 added a commit to sonalkr132/spring that referenced this pull request Jan 23, 2016
ENV["GEM_PATH"] was empty string, so GEM_PATH and GEM_HOME passed to
server process were same path.
It used to work before bundler 1.11.0 release cause bundler didn't clean
ENV["RUBYLIB"] and `require bundler/setup` was using that instead.
Change in bundler: rubygems/bundler#4002
@coilysiren coilysiren modified the milestone: Release Archive Oct 9, 2016
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.

6 participants