On Wed, Jan 15, 2020 at 07:41:46PM +0000, Stuart Henderson wrote:
> On 2020/01/14 21:48, Stuart Henderson wrote:
> > > while ((c = getopt(argc, argv, "6dD:nf:vSTt")) != -1) {
> > > switch (c) {
> > > case '6':
> > > - opts |= IKED_OPT_NOIPV6BLOCKING;
> > > + log_warnx("the -6 option is deprecated and will be "
> > > + "removed in the future.");
> >
> > "deprecated" implies that it still works but you shouldn't use it any more.
> >
> > Perhaps "ignored" or "no longer supported" instead?
> >
>
> Now that this is committed anyway - can I do this or something similar?
>
> deprecate -> "discouragement of use of some terminology, feature,
> design, or practice, typically because it has been superseded or is no
> longer considered efficient or safe, without completely removing it or
> prohibiting its use"
>
> Index: iked.c
> ===================================================================
> RCS file: /cvs/src/sbin/iked/iked.c,v
> retrieving revision 1.39
> diff -u -p -r1.39 iked.c
> --- iked.c 14 Jan 2020 22:28:29 -0000 1.39
> +++ iked.c 15 Jan 2020 19:39:37 -0000
> @@ -76,7 +76,7 @@ main(int argc, char *argv[])
> while ((c = getopt(argc, argv, "6dD:nf:vSTt")) != -1) {
> switch (c) {
> case '6':
> - log_warnx("the -6 option is deprecated and will be "
> + log_warnx("the -6 option is ignored and will be "
> "removed in the future.");
> break;
> case 'd':
>
I totally missed that part of your previous mail, sorry.
Reading the definition it seems you are right that "ignored" is the better
word here.
ok tobhe@
