genome/genome — pull request #44 by mark-burnett (2014-09-19 20:41:26+00:00)
Title: Fix G:D:A _cleanup_archive_directory

@nnutter and I discovered some questionable behavior while looking into a question he raised in #39, resulting in these changes.

It appears that this code would never actually execute `rm -rf $directory`, since the lack of a space between `$directory` and `]` resulted in a syntax error that caused the if statement to not execute the true code path. These changes make it possible for the test to succeed, and also notify us in the case of failure.

It might be the case that this function was intended to silently fail, though I'm not sure exactly why that would be needed right now.