I strongly agree that we should avoid use of the word 'deprecated'
towards the public.  People interpret what it means differently,
so try to be EXACT.

'deprecated' is our choice to make the change, but 'ignored' is
the result of that decision upon the people.

Stuart Henderson <s...@spacehopper.org> 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':
> 

Reply via email to