Fix (#10440) by making all commands optional

[webfactory]

Make all commands optional. This will work around the symptoms described in http://projects.puppetlabs.com/issues/6907 (for the vcsrepo type), as the agent run will no longer fail if one of the commands is not installed.

This gives you enough time to install the necessary packages through Puppet (make sure you get the dependencies right), as the agent won't fail unless you really start accessing the repository.

[kbarber]

Hey thanks for this ... sorry for the delay in response.

Just for housekeeping and triage purposes - can you raise a ticket on this in redmine?

http://projects.puppetlabs.com/projects/modules/issues/new

Sorry about the hassle but we prefer to have the commit message align with a ticket. Also - you need to read and sign the CLA, but from memory you've submitted other things before so this may have already been done :-).

[webfactory]

http://projects.puppetlabs.com/issues/10440

How can I change the commit message accordingly, or shall I just change the title of this pull req?

[kbarber]

If you do:

git commit --amend

In your branch it will let you change the title of a commit message. Then just push it back into your branch. You will probably need to force it however since you are overwriting history ... something like:

git push origin +topic_optional_commands:topic_optional_commands

Will do it. You won't need to open a new pull request, this pull request will just get updated.

[kbarber]

Oh yeah - and update the ticket or this pull request when done ... as I don't get notified when you rebase on an open pull request. Thanks mate!

[webfactory]

Done, thanks for the help!