Re: [PATCH v5 proxmox-backup 10/16] client: treat minus sign as stdin

[Christian Ebner <c.ebner@proxmox.com>]

On 1/30/26 5:45 PM, Robert Obkircher wrote:
> Treat "-" as an alias for "/dev/stdin". If there is an actual file
> with that name it can still be read via "./-".
> 
> Signed-off-by: Robert Obkircher <r.obkircher@proxmox.com>
> ---
>   proxmox-backup-client/src/main.rs | 7 +++++++
>   1 file changed, 7 insertions(+)
> 
> diff --git a/proxmox-backup-client/src/main.rs b/proxmox-backup-client/src/main.rs
> index 7fc711fd..37878b01 100644
> --- a/proxmox-backup-client/src/main.rs
> +++ b/proxmox-backup-client/src/main.rs
> @@ -845,6 +845,13 @@ async fn create_backup(
>           }
>           target_set.insert(target.clone());
>   
> +        // one can still use ./- to refer to an actual file with that name
> +        let filename = if filename == "-" {
> +            String::from("/dev/stdin")

should we add a warning here?

This is actually a breaking change, but guarding it behind a flag is 
maybe a bit overkill. We could check if the file exists before 
continuing and if so fail with an appropriate warning/error?

> +        } else {
> +            filename
> +        };
> +
>           use std::os::unix::fs::FileTypeExt;
>   
>           let metadata = std::fs::metadata(&filename)