Skip to content
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

Fix gem resource on Windows #1650

Merged
merged 1 commit into from
Apr 12, 2017
Merged

Fix gem resource on Windows #1650

merged 1 commit into from
Apr 12, 2017

Conversation

adamleff
Copy link
Contributor

RubyGems on windows comes with a batch file that wraps the gem command
so it executes correctly. This change uses that batch file for windows
for our gem resource, and also properly handles when we receive no output
from the command.

Fixes #1645

Copy link
Contributor

@chris-rock chris-rock left a comment

Choose a reason for hiding this comment

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

Thank you @adamleff

RubyGems on windows comes with a batch file that wraps the `gem` command
so it executes correctly. This change uses that batch file for windows
for our `gem` resource, and also properly handles when we receive no output
from the command.

Signed-off-by: Adam Leff <adam@leff.co>
@adamleff adamleff merged commit 510bc17 into master Apr 12, 2017
@adamleff adamleff deleted the adamleff/fix-gem-on-windows branch April 12, 2017 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants