Skip to content

Conversation

@heath
Copy link
Contributor

@heath heath commented Nov 23, 2014

Only generic messages are printed to the screen currently.

@laanwj
Copy link
Member

laanwj commented Nov 24, 2014

I agree that it makes sense to show a more detailed error message, but letting exceptions hang out with raise is not an acceptable way to show errors to the user IMO. Also, now you're catching the exceptions just to raise them again 🐰

@kanzure
Copy link
Contributor

kanzure commented Nov 24, 2014

So it seems to me that you want a --verbose option for the script, and the default is to swallow detailed error messages or detailed log messages.

@heath
Copy link
Contributor Author

heath commented Nov 24, 2014

please close this ticket or merge #5360, sorry it wasn't an update to this PR
that new PR addresses your issue @laanwj

@sipa
Copy link
Member

sipa commented Nov 24, 2014

You can close this pull request yourself, no?

@heath
Copy link
Contributor Author

heath commented Nov 24, 2014

@sipa sure

@heath heath closed this Nov 24, 2014
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants