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

[Env] Print shebang when it doesnt match Gem.ruby #5703

Merged
merged 3 commits into from
Jun 13, 2017

Conversation

segiddins
Copy link
Member

Closes #5616.

Prints out Gem.ruby and the bundle binstub shebang when they don't match, which should help us pinpoint when ruby installation shenanigans are the root cause of an issue

specs = Bundler.rubygems.find_name(name)
out << "#{name} (#{specs.map(&:version).join(",")})\n" unless specs.empty?
env = environment
environment_ljust = env.map {|(k, _v)| k.to_s.length }.max
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

_v is giving an unused variable warning on < 2

@segiddins segiddins force-pushed the seg-environment-shebang-mismatch branch from b897fec to b272df0 Compare June 12, 2017 22:19
@indirect
Copy link
Member

oh my god what a good idea yes please @bundlerbot r+

@bundlerbot
Copy link
Collaborator

📌 Commit b272df0 has been approved by indirect

@bundlerbot
Copy link
Collaborator

⌛ Testing commit b272df0 with merge 85cd5d8...

bundlerbot added a commit that referenced this pull request Jun 12, 2017
…irect

[Env] Print shebang when it doesnt match Gem.ruby

Closes #5616.

Prints out `Gem.ruby` and the `bundle` binstub shebang when they don't match, which should help us pinpoint when ruby installation shenanigans are the root cause of an issue
@bundlerbot
Copy link
Collaborator

☀️ Test successful - status-travis
Approved by: indirect
Pushing 85cd5d8 to master...

@bundlerbot bundlerbot merged commit b272df0 into master Jun 13, 2017
@segiddins segiddins deleted the seg-environment-shebang-mismatch branch June 13, 2017 03:42
bundlerbot added a commit that referenced this pull request Jun 16, 2017
Print build metadata when running `bundle version`

Closes #5049.

Will get all the build metadata into `bundle env` once #5703 lands, since I don't want conflicts and want to use that code for generating the "tables"
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.

3 participants