Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Print error messages to stderr(System.err) from CLI #935

Merged
merged 1 commit into from Aug 27, 2015

Conversation

@karl82
Copy link
Contributor

@karl82 karl82 commented Jan 18, 2015

I need to redirect error messages from command line into separate file. It's possible to use |grep 'ERROR:' but it will not catch exception stacktraces, because they are not prefixed with ERROR:

I think this is a good behavior for command line tool

@karl82 karl82 force-pushed the karl82:error_loging_to_System.err branch from d094b01 to 04386a8 Jan 19, 2015
@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Feb 25, 2015

Thanks Karl! This makes a lot of sense. I'll merge it for 4.0 (due in April) as it isn't backwards compatible.

Cheers
Axel

@karl82
Copy link
Contributor Author

@karl82 karl82 commented Feb 25, 2015

great! thx

axelfontaine added a commit that referenced this pull request Aug 27, 2015
Print error messages to stderr(System.err) from CLI
@axelfontaine axelfontaine merged commit cc39cb7 into flyway:master Aug 27, 2015
1 check passed
1 check passed
continuous-integration/travis-ci The Travis CI build passed
Details
axelfontaine added a commit to flyway/flywaydb.org that referenced this pull request Aug 27, 2015
@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Aug 27, 2015

Merged. Thanks once again!

@karl82
Copy link
Contributor Author

@karl82 karl82 commented Aug 27, 2015

awesome!

On Thu, Aug 27, 2015 at 4:16 PM Axel Fontaine notifications@github.com
wrote:

Merged. Thanks once again!


Reply to this email directly or view it on GitHub
#935 (comment).

axelfontaine added a commit to flyway/flywaydb.org that referenced this pull request Dec 23, 2015
@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Aug 17, 2017

@karl82 Could you please get in touch with me via email? (axel at boxfuse.com) Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants