Merge commit was d496655b

Adding a $throw_drush_error parameter to provision_process(), to allow implementors to avoid crashing drush if a process fails. Also adding doc block.

Default the "error message" to NULL. Once the logs are rendered nicely, we don't need an extra "Process failed" message.

This reverts commit 454502f0.

that way we do not need to check if the path is in the right directory
at all, we just remove the link, who cares if it points to a place
outside the site dir if we do not dereference it?

improvements;

 * we close directories as we go
 * coding style