Make checking for existing executable respect format_executable 

With the current code, it checks against the unformatted name instead of the formatted name, which is definitely a bug.

Sorry for the lack of tests on this. I couldn't figure out a simple way to add tests for this behavior that didn't involve a copy/pasting a large chunk of code or refactoring the test suite.