[pbs-devel] [PATCH proxmox{, -backup} v4 0/4] output full anyhow context in client
Gabriel Goller
g.goller at proxmox.com
Tue Jun 18 16:21:59 CEST 2024
Instead of using format_err! and format! to create error messages, we
want to move to anyhow::Context and add information to already
existing anyhow::Error's.
Before we start to gradually phase out the format! error calls, we
need to print the whole context + error, because the default
anyhow::Error Display implementation doesn't print the actual error.
This series starts with the proxmox-backup-client and includes a patch
from Fabian that introduces pretty-printing of errors (including
the context) and some other high-level stuff from me. The scope
is not to remove every format call and add a context everywhere,
but to enable this change in the future.
v4:
- inlined eprintln's arguments
- removed readme.rst (to be moved to the Developer Documentation)
v3:
- rebased
- applied Max's fixup
v2:
- moved explanation to readme.rst file
- remove unwrap, use take() in `if let`
- cargo fmt
proxmox:
Gabriel Goller (1):
CLI: print fatal errors including causes
proxmox-router/src/cli/command.rs | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
proxmox-backup:
Gabriel Goller (3):
pxar: remove ArchiveError
pxar: add UniqueContext helper
pxar: use anyhow::Error in PxarBackupStream
pbs-client/src/pxar/create.rs | 48 +++++++++++++---------------
pbs-client/src/pxar_backup_stream.rs | 18 +++++------
2 files changed, 32 insertions(+), 34 deletions(-)
Summary over all repositories:
3 files changed, 34 insertions(+), 36 deletions(-)
--
Generated by git-murpp 0.5.0
More information about the pbs-devel
mailing list