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

Warn if RUBYGEMS_GEMDEPS is set #3656

Closed
segiddins opened this issue May 17, 2015 · 6 comments
Closed

Warn if RUBYGEMS_GEMDEPS is set #3656

segiddins opened this issue May 17, 2015 · 6 comments

Comments

@segiddins
Copy link
Member

Should we do this so there's less confusion when rubygems and bundler clash?

@indirect
Copy link
Member

How do they clash? And yes, that seems likely :/

On Sat, May 16, 2015 at 7:38 PM, Samuel E. Giddins
notifications@github.com wrote:

Should we do this so there's less confusion when rubygems and bundler clash?

Reply to this email directly or view it on GitHub:
#3656

@segiddins
Copy link
Member Author

If rubygems in any resolves things differently than bundler, we might not be able to setup our environment correctly. Haven't we also had issues with things like rubygems-bundler in the past?

@indirect
Copy link
Member

I have no idea what happens if you set RUBYGEMS_GEMDEPS and also use Bundler. I guess we should check?

As for rubygems-bundler, that was broken because of trying to automatically bundle exec not because of RUBYGEMS_GEMDEPS. But yes, definitely issues.

@agis
Copy link
Contributor

agis commented Oct 22, 2015

Does a warning on every CLI command along the lines of

RUBYGEMS_GEMDEPS` is set and this may cause bundler to conflict with rubygems

sounds sufficient?

@segiddins
Copy link
Member Author

@agis- yeah

agis added a commit that referenced this issue Oct 22, 2015
agis added a commit that referenced this issue Oct 25, 2015
agis added a commit that referenced this issue Oct 25, 2015
homu added a commit that referenced this issue Oct 26, 2015
Warn if RUBYGEMS_GEMDEPS env. variable is set

Closes #3656.
@ioquatix
Copy link

Is this still relevant? It's nice that I don't have to use bundle exec everywhere. With bundler and rubygems merging, can we find some consensus on this?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants