Comment by mycall

Comment by mycall 5 hours ago

21 replies

I like the opposite too, -commit or -execute as it is assumed running it with defaults is immutable as the dry run, simplifying validation complexity and making the go live explicit.

Twirrim 5 hours ago

I've biased towards this heavily in the last 8 or so years now.

I've yet to have anyone mistakenly modify anything when they need to pass --commit, when I've repeatedly had people repeatedly accidentally modify stuff because they forgot --dry-run.

  • IgorPartola 4 hours ago

    I wouldn’t want most things to work this way:

        $ rm file.bin
        $ rm —-commit file.bin
        $ cat foo.txt > bar.txt
        $ cat foo.txt | tee —-write-for-real bar.txt
        $ cp balm.mp3 pow.mp3
        $ cp —-i-mean-it balm.mp3 pow.mp3
    
    There is a time and a place for it but it should not be the majority of use cases.
    • Darfk 4 hours ago

      Totally agree it shouldn't be for basic tools; but if I'm ever developing a script that performs any kind of logic before reaching out to a DB or vendor API and modifies 100k user records, creating a flag to just verify the sanity of the logic is a necessity.

      • Joker_vD 4 hours ago

            if [ -n "$DRY_RUN" ] ; then
                alias rm='echo rm'
                alias cp='echo cp'
            fi
        
        Of course, output redirects will still overwrite the files, since the shell does it and IIRC this behaviour can't be changed.
      • james_marks 4 hours ago

        Yep. First thing I do for this kind thing is make a preview=true flag so I don’t accidentally run destructive actions.

    • digiown 4 hours ago

      For most of these local data manipulation type of commands, I'd rather just have them behave dangerously, and rely on filesystems snapshots to rollback when needed. With modern filesystems like zfs or btrfs, you can take a full snapshot every minute and keep it for a while to negate the damage done by almost all of these scripts. They double as a backup solution too.

    • ronjakoi an hour ago

      I used to have alias rm='rm -i' for a few years to be careful, but I took it out once I realised that I had just begun adding -f all the time

    • hdjrudni 4 hours ago

      Even in those basic examples, it probably would be useful. `cp` to a blank file? No problem. `cp` over an existing file? Yeah, I want to be warned.

      `rm` a single file? Fine. `rm /`? Maybe block that one.

torstenvl 2 hours ago

I have a parallel directory deduper that uses hard links and adopted this pattern exactly.

By default it'll only tell you which files are identical between the two parallel directory structures.

If you want it to actually replace the files with hard links, you have to use the --execute flag.

spike021 3 hours ago

There was a tool I used some time ago that required typing in a word or phrase to acknowledge that you know it's doing the run for real.

Pros and cons to each but I did like that because it was much more difficult to fat finger or absentmindedly use the wrong parameter.

xyse53 5 hours ago

Yeah I'm more of a `--wet-run` `-w` fan myself. But it does depend on how serious/annoying the opposite is.

  • aqme28 5 hours ago

    I've done that, but I hate the term "wet run."

    I use "live run" now, which I think gets the point across without being sort of uncomfortable.

    • IgorPartola 4 hours ago

      --with-danger

      --make-it-so

      --do-the-thing

      --go-nuts

      --safety-off

      So many fun options.

      • Darfk 4 hours ago

        I'm a fan of --safety-off. It gives off a 'aim away from face' or 'mishandle me and I'll blow a chunk out of your DB' vibe.

      • torstenvl 2 hours ago

        It's in the UI not the command line, but I like Chromium's thisisunsafe

      • JsonCameron 4 hours ago

        I've done a few --execute --i-know-what-im-doing for some more dangerous scripts

        • altairprime 3 hours ago

          May I recommend --I-take-responsibility-for-the-outcome-of-proceeding and require a capital I?

lazide 3 hours ago

Just don’t randomly mix and match the approaches or you are in for a bad time.