Vague and unhelpful error message if log cannot be created about airbrussh HOT 4 OPEN

The problem is that Airbrussh is trying to create its default log file at log/capistrano.log, and it cannot because log exists and is not a directory.

You can solve this by changing the log file location to something else, or disable it entirely, using this option:

set :format_options, log_file: "a/different/path.log"
# or disable it
set :format_options, log_file: nil

That aside, I agree that the error behavior in this scenario could be improved. Should Airbrussh print an error that it cannot create the log file and then exit? Or print a warning and continue without writing a log? What is your expectation?

from airbrussh.

The problem is that Airbrussh is trying to create its default log file at log/capistrano.log, and it cannot because log exists and is not a directory.

indeed, I figured that much out, but the error message wasn't very clear.

Should Airbrussh print an error that it cannot create the log file and then exit? Or print a warning and continue without writing a log? What is your expectation?

Me personally, I'd expect it to "print an error that it cannot create the log file and then exit", maybe telling the user how to fix it (something like "you can change the log file via the :format_options option, check the help to know how")

from airbrussh.

Good suggestion. Thanks for the bug report! 🙇

from airbrussh.

I will accept a PR to improve the error message if anyone is interested! 😃

from airbrussh.