Skip to content
This repository has been archived by the owner on Oct 2, 2018. It is now read-only.

COOK-4183 Set $HOME to new owner as in the pip provider when shelling ou... #72

Merged
merged 4 commits into from
Jan 21, 2014

Conversation

comandrei
Copy link
Contributor

...t

@tinnet
Copy link

tinnet commented Jan 6, 2014

fixed the issue for us 👍

@andrewgross
Copy link

+1 for getting this fixed, running in to the exact same issue and the patch fixed it.

@guilhem
Copy link
Contributor

guilhem commented Jan 8, 2014

hi @comandrei
I do a PR comandrei#1 in your repository to improve your patch.
Feel free to comment / merge :)

@rgbkrk
Copy link
Contributor

rgbkrk commented Jan 9, 2014

👍

This might be happening as a result of pypa/virtualenv#532

@brint
Copy link

brint commented Jan 10, 2014

👍

@wolfg1969
Copy link

Dir.home will cause 'NoMethodError: undefined method `home' for Dir:Class' on Amazon Opsworks (Chef version 11.4.4)

@coderanger
Copy link
Member

@wolfg1969 Dir.home was added in Ruby 1.9, is Amazon using Ruby 1.8?

@wolfg1969
Copy link

@coderanger yes, I think so. Using File.expand_path should be ok.

@coderanger
Copy link
Member

@wolfg1969 That is unfortunate, Chef ships Ruby 1.9.3 so anything earlier than that is formally unsupported. As I rewrite this cookbook a bit it will definitely become 1.9+-only, so I would recommend avoiding Opsworks if they are indeed using an incorrect version of Ruby.

@wolfg1969
Copy link

@coderanger I understand. I'll do the trick in my cookbook.

@rgbkrk
Copy link
Contributor

rgbkrk commented Jan 11, 2014

Thank you @coderanger for upgrading this cookbook to Ruby 1.9+.

@seperman
Copy link

This patch fixes virtual env creation but the pip provider keeps raising the same OSError: [Errno 13] Permission denied: '/root/.distlib' for me.
I manually uninstalled virtualenv 1.11, removed the env and then installed 1.10.1 and that solved the problem.

@leogamas
Copy link

Fixed the issue for me 👍

@andrewgross
Copy link

Just curious if we are going to merge this or wait for a pip update instead?

coderanger added a commit that referenced this pull request Jan 21, 2014
COOK-4183 Set $HOME to new owner as in the pip provider when shelling ou...
@coderanger coderanger merged commit 4920ddd into poise:master Jan 21, 2014
@cpdean
Copy link

cpdean commented Feb 8, 2014

I'm getting the "NoMethodError: undefined method `home' for Dir:Class" error using Vagrant. When did chef move on to ruby 1.9? does this backwards-compatability snag mean Vagrant should update their dependencies?

@coderanger
Copy link
Member

@cpdean Chef omnibus has always used Ruby 1.9. The provisioner-included baseboxes have been deprecated by both Vagrant and Chef for a long time, use the vagrant-omnibus plugin.

@cpdean
Copy link

cpdean commented Mar 11, 2014

ah, thanks!
On Feb 8, 2014 12:04 PM, "Noah Kantrowitz" notifications@github.com wrote:

@cpdean https://github.com/cpdean Chef omnibus has always used Ruby
1.9. The provisioner-included baseboxes have been deprecated by both
Vagrant and Chef for a long time, use the vagrant-omnibus plugin.


Reply to this email directly or view it on GitHubhttps://github.com//pull/72#issuecomment-34550893
.

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.