undefined | Better HN

story

0 pointsmajewsky7y ago0 comments

This does not work. The arguments to `del` get appended at the end, after `find`. You want this:

  del() { mv "$@" ~/.Trash/ && find ~/.Trash -mtime +30 -delete }

I also replaced the unnecessary exec by find's builtin -delete

0 comments

Thank you, it's painful to see people providing replacements for critical commands like `rm` without understanding basic shell features like aliases.

And just a reminder everybody please don't run random scripts you find on the internet.

P.S. you need a semicolon before the close brace :)

j / k navigate · click thread line to collapse

Thank you, it's painful to see people providing replacements for critical commands like `rm` without understanding basic shell features like aliases.

And just a reminder everybody please don't run random scripts you find on the internet.

P.S. you need a semicolon before the close brace :)

j / k navigate · click thread line to collapse