search

voxpupuli / puppet-staging

⛔️ Deprecated in favor of puppet-archive
https://forge.puppet.com/puppet/archive
Apache License 2.0
51 stars 109 forks source link

(fix) Add target -d to unzip method #85

Closed cyberious closed 9 years ago

cyberious commented 9 years ago
cyberious commented 9 years ago

@daenney @nanliu thoughts?

daenney commented 9 years ago

So now you can't specify not a target?

cyberious commented 9 years ago

The way I read it previously is that this is actually a bug and would be the target it should be expanding to. But that is the way I read it.

mkrakowitzer commented 9 years ago

How do you reproduce this issue? I have never experienced it myself. Is it specific to a distro or an os?

nibalizer commented 9 years ago

Target is a required parameter so requiring it isn't an issue. However with these execs it looks like they ensure location by setting cwd on the exec. Why doesn't that work @cyberious I think @mkrakowitzer is right that we need a better bug report :)

nanliu commented 9 years ago

Agreed, cwd should take care of the directory, so need to understand the issue and how to reproduce.