Skip to content
This repository was 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
coderanger merged 4 commits into
poise:masterfrom
comandrei:COOK-4183
Jan 21, 2014
Merged

COOK-4183 Set $HOME to new owner as in the pip provider when shelling ou...#72
coderanger merged 4 commits into
poise:masterfrom
comandrei:COOK-4183

Conversation

@comandrei

Copy link
Copy Markdown
Contributor

...t

@tinnet

tinnet commented Jan 6, 2014

Copy link
Copy Markdown

fixed the issue for us 👍

@andrewgross

Copy link
Copy Markdown

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

@guilhem

guilhem commented Jan 8, 2014

Copy link
Copy Markdown
Contributor

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

@rgbkrk

rgbkrk commented Jan 9, 2014

Copy link
Copy Markdown
Contributor

👍

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

@brint

brint commented Jan 10, 2014

Copy link
Copy Markdown

👍

@wolfg1969

Copy link
Copy Markdown

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

@coderanger

Copy link
Copy Markdown
Member

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

@wolfg1969

Copy link
Copy Markdown

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

@coderanger

Copy link
Copy Markdown
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
Copy Markdown

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

@rgbkrk

rgbkrk commented Jan 11, 2014

Copy link
Copy Markdown
Contributor

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

@seperman

Copy link
Copy Markdown

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
Copy Markdown

Fixed the issue for me 👍

@andrewgross

Copy link
Copy Markdown

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

cpdean commented Feb 8, 2014

Copy link
Copy Markdown

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
Copy Markdown
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

cpdean commented Mar 11, 2014

Copy link
Copy Markdown

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.