-
Notifications
You must be signed in to change notification settings - Fork 138
error-chain deprecation #181
Comments
Hi. Thanks for your update.
I think the biggest problem I see with this, is that the paradigms of both crates are a bit different, and in general going from error-chain to failure has involved rethinking my error handling. What's usually happened when using error-chain for me is that I end up creating My personal experience is that failure leads me to writing cleaner error handling, however I have yet to try how well it proves for debugging. Because on of the biggest selling points of error-chain for me has always been, use it everywhere (which is really easy) and you will get backtraces everywhere when you need them. |
Yeah, I didn't mean a 1-1 feature mapping. I meant that the use-cases are covered. |
Should this discussion be revisited now that the path forward for |
I feel like the issue here is now more complex as error is now being fixed in the stdlib so it's not even clear what the failure story is going to be going forward. |
error-chain seems to be deprecated (rust-lang-deprecated/failure#181), but its "successor", failure, is not in good shape either. Following guidance from https://blog.burntsushi.net/rust-error-handling/, use Rust's standard library error managment system.
Hey,
error-chain
's maintainer here.I have not been as active on error-chain maintenance as I should have (cf all the issues and PR). I'm also working less with Rust these times (I keep an eye on it though). Since it's not perfect anyway (rust-lang-deprecated/error-chain#190) and
failure
is an improvement onerror-chain
, I guess the best way to go would be to deprecate it and go fullfailure
. It could be done with the 1.0failure
release.I see a few points that should be solved before the full deprecation:
error-chain
. I'm thinking something like "if you do this inerror-chain
, replace it by that infailure
.error-chain
is supported byfailure
.Do you think of it?
Not sure who should be included in this discussion.
The text was updated successfully, but these errors were encountered: