ekho / capistrano-git-submodule-strategy

Git submodule support for Capistrano 3
MIT License
47 stars 20 forks source link

fix for double-check git release logic #4

Closed brushbox closed 9 years ago

brushbox commented 10 years ago

Here's a PR with my suggested change.

As for testing, all I can say is "it works in my environment".

brushbox commented 10 years ago

I was just thinking about how my solution works (googled for it, so I pretty much copied it from somewhere else)...and I realised that, apart from the slightly different logic in the conditional, the second part of the expression could possibly use ls -A | read instead of the find ... | read.

ekho commented 10 years ago

In my environment release_path does not exist prior to the operation.

I think condition should check what directory does not exist OR it's empty