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

Javadoc may not logging all the errors #415

Closed
raboof opened this Issue Sep 26, 2017 · 4 comments

Comments

Projects
None yet
3 participants
@raboof
Contributor

raboof commented Sep 26, 2017

When debugging an instance where javadoc produced the error "javadoc: error - invalid flag: -target", this error was not printed on javacLogger.flush(exitCode).

It was correctly added to JavacLogger.msgs with level error, so I'd expect parseAndDumpSemanticErrors to fail to parse it and log it instead, not sure why that doesn't appear to be happening.

@jvican jvican changed the title from LocalJavadoc may not log all errors to Javadoc may not logging all the errors Sep 28, 2017

@jvican

This comment has been minimized.

Show comment
Hide comment
@jvican

jvican Sep 28, 2017

Member

@raboof Any ideas how we can reproduce/test this?

Member

jvican commented Sep 28, 2017

@raboof Any ideas how we can reproduce/test this?

@raboof

This comment has been minimized.

Show comment
Hide comment
@raboof

raboof Oct 11, 2017

Contributor
Contributor

raboof commented Oct 11, 2017

@jvican

This comment has been minimized.

Show comment
Hide comment
@jvican

jvican Oct 11, 2017

Member

That looks good. Hopefully we fix this soon. Thanks @raboof! 😄

Member

jvican commented Oct 11, 2017

That looks good. Hopefully we fix this soon. Thanks @raboof! 😄

@jvican

This comment has been minimized.

Show comment
Hide comment
@jvican

jvican Sep 15, 2018

Member

Closing in favor of #520 which is richer in details. We'll add the provided reproduced case in this ticket to our test suite, thank you @raboof

Member

jvican commented Sep 15, 2018

Closing in favor of #520 which is richer in details. We'll add the provided reproduced case in this ticket to our test suite, thank you @raboof

@jvican jvican closed this Sep 15, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment