cprice-puppet/puppet-acceptance — comment on commit fcc8b4feef4dc2e73778fb7230e49c4a8e9665b3
File:  line 0
By djm68:

Chris,

I have review this pull request and I am a little confused. Many of the changes/additions you made move "bashism" in the harness. In our recent meeting regarding the harness direction I believe we all agreed that moving bash commands into the harness was a bad idea. Invoking ruby on a target system to perform a simple "test -f" hardly seems efficient. Additionally, your proposed changes duplicates existing harness functions -- such as creating a remote file.