[Swan-dev] oddly redundant flags Pluto accepts

[Paul Wouters]

On Wed, 19 Feb 2014, D. Hugh Redelmeier wrote:

> Pluto still takes --foodgroupsdir.  Why?
>
> It is a synonym for --ipsecdir and --ipsec_dir.

Because it is a different concept, and might not be the same dir in the
future? The old OE code was left in until the new OE code is ready, so
this will most likely get cleaned up in the next few months.

> It also looks as if these are all synonyms:

I'm not sure of the entire history for these, but in general we only
document the one you should use, and have some undocumented ones for
backwards compatibility. Likely, these got renamed in the past. So yes,
at some point they can go. Probably the fork would have been a good
time, so I think we can remove these redundant ones.

> 	{ "coredir", required_argument, NULL, 'C' },
> 	{ "dumpdir", required_argument, NULL, 'C' }, /* alias for coredir */
> Only coredir is documented.
> But it is called dumpdir in the configfile, I think.
> It would make sense to pick one and make both use the same one.
> Since almost nobody uses the pluto flags (as far as I know), we should
> use "dumpdir".  In fact, Pluto's diagnostic concerning this directory
> calls it dumpdir.

But coredir is more obvious. People on embedded systems often start
pluto outside the scripts, using pluto options.

Paul