Re: [PATCH] rm(1): add -v option for verbosity

2017-06-25 Thread patrick keshishian
On Sun, Jun 25, 2017 at 11:16:00PM +0200, Ingo Schwarze wrote: > Hi, > > Paul de Weerd wrote on Sun, Jun 25, 2017 at 10:46:15PM +0200: > > On Sun, Jun 25, 2017 at 03:12:26PM +0200, Ingo Schwarze wrote: > > > | If you are really unsure, study the output of > > | $ find * > > | first, before

Re: [PATCH] rm(1): add -v option for verbosity

2017-06-25 Thread Ingo Schwarze
Hi, Paul de Weerd wrote on Sun, Jun 25, 2017 at 10:46:15PM +0200: > On Sun, Jun 25, 2017 at 03:12:26PM +0200, Ingo Schwarze wrote: > | If you are really unsure, study the output of > | $ find * > | first, before typing > | $ rm -rf * > | No non-standard option is needed at all for this. >

Re: [PATCH] rm(1): add -v option for verbosity

2017-06-25 Thread Job Snijders
On Sun, Jun 25, 2017 at 04:09:11PM +0200, Job Snijders wrote: > --- bin/rm/rm.1 > +++ bin/rm/rm.1 > @@ -95,6 +95,8 @@ that directory is skipped. > .It Fl r > Equivalent to > .Fl R . > +.It Fl v > +Explain what is being done. On second thought, "Display what files were removed." would

Re: [PATCH] rm(1): add -v option for verbosity

2017-06-25 Thread Paul de Weerd
Hi Ingo, On Sun, Jun 25, 2017 at 03:12:26PM +0200, Ingo Schwarze wrote: | Hi, | | Job Snijders wrote on Sun, Jun 25, 2017 at 02:06:16PM +0200: | | > This patch adds a '-v' option to rm(1) for more verbose output. | | Do not add new options to standard utilities, unless you can show | that they

Re: [PATCH] rm(1): add -v option for verbosity

2017-06-25 Thread Job Snijders
Hi Ingo, Thanks for taking the time to review this. On Sun, Jun 25, 2017 at 03:12:26PM +0200, Ingo Schwarze wrote: > Job Snijders wrote on Sun, Jun 25, 2017 at 02:06:16PM +0200: > > > This patch adds a '-v' option to rm(1) for more verbose output. > > Do not add new options to standard

Re: [PATCH] rm(1): add -v option for verbosity

2017-06-25 Thread Ingo Schwarze
Hi, Job Snijders wrote on Sun, Jun 25, 2017 at 02:06:16PM +0200: > This patch adds a '-v' option to rm(1) for more verbose output. Do not add new options to standard utilities, unless you can show that they are unusually useful in practice *and* practically every other system out there has