On Fri, Jun 22 2018, Vadim Zhukov <persg...@gmail.com> wrote: > So after a few discussions I propose to add --dry-run as synonim for -C > in our patch(1). Quick summary: > > * GNU got --dry-run earlier than us; > * --dry-run way more popular name than --check for such functionality; > * FreeBSD and NetBSD has the same for a long time already; > * We don't care about long option names generally, anyway. > > On the second option, --binary, since we have no functionality and > I see no demand for it, lets just keep things as is. Failing with > "unkonwn option" message is clear indicator that requested functionality > isn't supported. > > Builds and runs on (almost) -CURRENT. Okay or not?
ok jca@ but please update the manpage, something like that I guess... Index: patch.1 =================================================================== RCS file: /d/cvs/src/usr.bin/patch/patch.1,v retrieving revision 1.31 diff -u -p -p -u -r1.31 patch.1 --- patch.1 11 Apr 2018 10:06:50 -0000 1.31 +++ patch.1 22 Jun 2018 09:46:27 -0000 @@ -99,7 +99,7 @@ This is equivalent to specifying This option is currently the default, unless .Fl -posix is specified. -.It Fl C , Fl Fl check +.It Fl C , Fl Fl check , Fl Fl dry-run Checks that the patch would apply cleanly, but does not modify anything. .It Fl c , Fl Fl context Forces -- jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE
