Skip to content
This repository
Browse code

Ensure gempath.sh and livery command are executable

  • Loading branch information...
commit 6853b1ba5a4f287d7d4b815d95cc5a66a2f46f02 1 parent 7c0745e
Jeff Blaine authored December 12, 2012

Showing 1 changed file with 1 addition and 1 deletion. Show diff stats Hide diff stats

  1. 2  lib/blimpy/livery/puppet.rb
2  lib/blimpy/livery/puppet.rb
@@ -44,7 +44,7 @@ def flight(box)
44 44
         run_sudo = ''
45 45
         run_sudo = 'sudo' if use_sudo?(box)
46 46
 
47  
-        box.ssh_into("cd #{dir_name} && #{run_sudo} ./gempath.sh #{command}")
  47
+        box.ssh_into("cd #{dir_name} && chmod 755 ./gempath.sh #{command} && #{run_sudo} ./gempath.sh #{command}")
48 48
       end
49 49
 
50 50
       def postflight(box)

4 notes on commit 6853b1b

R. Tyler Croy

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

Jeff Blaine

Oh crap. I'm looking into this.

Jeff Blaine

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}")
R. Tyler Croy
Owner

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

Jeff Blaine

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.

Please sign in to comment.
Something went wrong with that request. Please try again.