[pbs-devel] [PATCH proxmox-backup] pxar-bin: remove `log` dependency, use `tracing` directly
    Christian Ebner 
    c.ebner at proxmox.com
       
    Wed Sep  4 14:31:06 CEST 2024
    
    
  
On 9/4/24 10:06, Christian Ebner wrote:
> small nits inline
> 
> As you already discussed with Wolfgang in the other thread, any user 
> facing output not intended for debugging, e.g. the output of the `pxar 
> list` might be better displayed by writing to stdout using a `println` 
> instead.
> 
> Tested by listing and extracting a pxar archive, with and without the 
> `PXAR_LOG=debug` environment variable set.
> 
> Other than that, please consider this and the patch to `proxmox-log`:
> 
> Tested-by: Christian Ebner <c.ebner at proxmox.com>
> Reviewed-by: Christian Ebner <c.ebner at proxmox.com>
> 
Also, this needs a `cargo fmt` to correctly sort the entities in the 
proxmox_log use statement.
    
    
More information about the pbs-devel
mailing list