cargo-sweep icon indicating copy to clipboard operation
cargo-sweep copied to clipboard

`-i` is broken on master

Open birkenfeld opened this issue 3 years ago • 1 comments

$ cargo run -- sweep -d -r -i      
   Compiling cargo-sweep v0.5.0 (/home/gbr/devel/ext/cargo-sweep)
    Finished dev [unoptimized + debuginfo] target(s) in 1.55s
     Running `target/debug/cargo-sweep sweep -d -r -i`
thread 'main' panicked at '--time argument missing', src/main.rs:219:18
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

birkenfeld avatar Oct 22 '20 07:10 birkenfeld

The panic was introduced in 4430478, which was written in response to e53601a. The combination of these seems to have effectively made --time required, despite it not being marked as required (ca. 037155c).

Personally, I'd expect cargo-sweep to simply not delete any files based on time if --time is not provided, but this might disrupt the workflow of people who have come to expect cargo-sweep to delete old files by default.

PatchMixolydic avatar Dec 22 '20 18:12 PatchMixolydic

This seems to be fixed on master.

jyn514 avatar Dec 01 '22 20:12 jyn514