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

first stab at restoring the ~/.chefdk dir for installing user gems, but ... #71

Closed
wants to merge 2 commits into from

Conversation

tknerr
Copy link
Owner

@tknerr tknerr commented Jan 18, 2015

...it still fails the acceptance tests

Note: Keeping this PR open until the underlying issues are fixed.

Essentially I want to restore the default behaviour of ChefDK where gems are installed into ~/.chefdk rather than the chefdk embedded dir. This would be a clean separation between the chefdk shipped gems and the user / bundle installed gems again. It works on my ubuntu dev-box without problems that way, but in bills kitchen it still fails the acceptance tests:

"--> Running command: 'cd C:/Repos/_github/bills-kitchen/target/build/repo/vagrant-workflow-tests/playground/sample-toplevel-cookbook && rake test':"
knife cookbook test sample-toplevel-cookbook -o ..
'"C:\Repos\_github\bills-kitchen\target\build\home\.chefdk\gem\ruby\2.0.0\bin\ruby.exe"' is not recognized as an internal or external command,
operable program or batch file.
rake aborted!

Might have to do or is related to the chefdk mangling we do here:
https://github.com/tknerr/bills-kitchen/blob/master/Rakefile#L117-132

Related ChefDK issues:

@tknerr
Copy link
Owner Author

tknerr commented Jan 18, 2015

As mentioned here, the upstream fix in RubyGems 2.4.4 should resolve the ruby path issues.

So it will likely be fixed with the next ChefDK and we could probably get rid of lots of the cruft in here:
https://github.com/tknerr/bills-kitchen/blob/master/Rakefile#L117-132

@tknerr tknerr mentioned this pull request Jan 22, 2015
@tknerr
Copy link
Owner Author

tknerr commented Jan 22, 2015

fixed via #72 (original commits here cherry-picked)

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.

1 participant