Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Ensure gempath.sh and livery command are executable #63

Merged
merged 1 commit into from Dec 12, 2012

Conversation

Projects
None yet
3 participants
Contributor

jblaine commented Dec 12, 2012

No description provided.

R. Tyler Croy » blimpy #51 SUCCESS
This pull request looks good
(what's this?)

@rtyler rtyler added a commit that referenced this pull request Dec 12, 2012

@rtyler rtyler Merge pull request #63 from jblaine/patch-1
Ensure gempath.sh and livery command are executable
8ee0c96

@rtyler rtyler merged commit 8ee0c96 into rtyler:master Dec 12, 2012

I just noticed that this won't work for the Puppet livery properly. Oops.

Contributor

jblaine commented on 6853b1b Dec 19, 2012

Oh crap. I'm looking into this.

Contributor

jblaine replied Dec 19, 2012

Wow, what a shitty hasty PR. My apologies.

The following works for me if you care to try it out or bother with it:

box.ssh_into("cd #{dir_name} && chmod 755 ./gempath ../#{manifest_path} && #{run_sudo} ./gempath.sh #{command}")
Owner

rtyler replied Dec 19, 2012

I've fixed this in the most recent HEAD of master, I'm curious why chmod'ing the manifest file is necessary as well?

Contributor

jblaine replied Dec 19, 2012

It's not.

sighs at self

I was criss-crossing CWD with Puppet liveries in my head. Take-away: livery/cwd.rb should have a chmod fix as well before it tries to run its script.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment