[Swan-dev] oddly redundant flags Pluto accepts

Tuomo Soini tis at foobar.fi
Wed Feb 19 22:08:43 EET 2014


On Wed, 19 Feb 2014 10:34:18 -0500 (EST)
Paul Wouters <paul at nohats.ca> wrote:

> 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.

I'd still drop option coredir when we have dumpdir which is only real
one being used.

-- 
Tuomo Soini <tis at foobar.fi>
Foobar Linux services
+358 40 5240030
Foobar Oy <http://foobar.fi/>


More information about the Swan-dev mailing list