search

websterclay / knife-github-cookbooks

A knife plugin facilitating installing, updating, and tracking Chef cookbooks found on Github.
Apache License 2.0
220 stars 35 forks source link

Error when installing new repo #8

Open patcon opened 13 years ago

patcon commented 13 years ago

Hey Jesse,

Any idea what might be causing this? I'm trying to install a new repo, and it seems like it's expecting the dir to be there already... I've tried both fresh (without a chef-vendor-drush branch being present, and with it).

Thoughts?

{code} $ knife cookbook github install msonnabaum/chef-drush zsh: correct 'cookbook' to 'cookbooks' [nyae]? n Installing drush from git://github.com/msonnabaum/chef-drush.git to site-cookbooks Checking out the master branch. Pristine copy branch (chef-vendor-drush) exists, switching to it. Removing pre-existing version. ERROR: Chef::Exceptions::ShellCommandFailed: Expected process to exit 0, but it exited with 1 ---- Begin output of mv _tmp_chef_drush site-cookbooks/drush ---- STDOUT: STDERR: mv: rename _tmp_chef_drush to site-cookbooks/drush: No such file or directory ---- End output of mv _tmp_chef_drush site-cookbooks/drush ---- Ran mv _tmp_chef_drush site-cookbooks/drush returned 1 {code}

patcon commented 13 years ago

Ahh... figured it out. knife.rb allows for you to just put in cookbooks like "site-cookbooks", and it takes this as root of the chef-repo, but it seems knife-github-cookbooks is expecting an absolute path

patcon commented 13 years ago

Goes without saying, but for those who commit their .chef folder as well, it makes it tough. I'll see if I can dig in later and figure it out :)

sumbach commented 13 years ago

Submitted a pull request with a proposed fix. See #14 for details.