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? -- WBR, Vadim Zhukov Index: patch.c =================================================================== RCS file: /cvs/src/usr.bin/patch/patch.c,v retrieving revision 1.65 diff -u -p -r1.65 patch.c --- patch.c 7 Apr 2018 14:55:13 -0000 1.65 +++ patch.c 22 Jun 2018 09:12:53 -0000 @@ -454,6 +454,7 @@ get_some_switches(void) {"context", no_argument, 0, 'c'}, {"debug", required_argument, 0, 'x'}, {"directory", required_argument, 0, 'd'}, + {"dry-run", no_argument, 0, 'C'}, {"ed", no_argument, 0, 'e'}, {"force", no_argument, 0, 'f'}, {"forward", no_argument, 0, 'N'},