Skip to content

[Frontend] Don't ask to report bugs for a script failure #39619

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

Merged
merged 1 commit into from
Oct 8, 2021

Conversation

xymus
Copy link
Contributor

@xymus xymus commented Oct 6, 2021

A script crash.swift failing on an assert such as:

assert(false)

Leads to a full stacktrace starting with a request to file a bug to swift.org:

% swift crash.swift
crash/crash.swift:1: Assertion failed
Please submit a bug report (https://swift.org/contributing/#reporting-bugs) and include the project and the crash backtrace.
Stack dump:
...

This message is confusing as the issue here is an expected user-side script failure. Let's remove the "Please submit a bug report..." line in immediate mode and keep the stacktrace as it has useful information, even if likely too much for the need of the script developer. Note that this will apply to failures in both client code and in the compiler.

rdar://81233463

@xymus xymus requested review from eeckstein and lhames October 6, 2021 21:26
@xymus
Copy link
Contributor Author

xymus commented Oct 6, 2021

cc @lhames This is casting a wide net, differentiating compiler and script-side failures could be an interesting improvement in future JIT work.

@xymus xymus changed the title [Frontend] Don't ask to report bugs for scripts failures [Frontend] Don't ask to report bugs for a script failure Oct 6, 2021
@xymus
Copy link
Contributor Author

xymus commented Oct 7, 2021

@swift-ci Please smoke test

@xymus xymus merged commit baf032b into swiftlang:main Oct 8, 2021
@xymus xymus deleted the pls-no-script-bug-reports branch October 8, 2021 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant