Ken wrote:

> I ... do not think that is safe to put in?  It seems to me like that
> would fail if your command line looks like:
>
> rcvdist -server foo.bar -tls us...@example.com us...@example.com

I'm not sure what you mean.  How is this not safe?

    $ rcvdist -server foo.bar -tls us...@example.com us...@example.com 
</dev/null
    post: only one message at a time!
    /home/levine/lib/nmh/libexec/nmh/post: exited 1

> us...@example.com would be swallowed and passed to post.  If you just
> had one address rcvdist would fail.

Right:

    $ rcvdist -server foo.bar -tls us...@example.com </dev/null
    rcvdist: usage: rcvdist [switches] [switches for postproc] address ...

> You could maybe make sure that you
> always put a switch that takes an argument last, but that just seems
> lousy as well.

I don't see a need to do that.

It's up to the user to write an rcvdist command that does what
they want.  The same is true for other nmh commands that forward
arguments and switches to other programs, e.g., show -form 1 2 3.
(In that case, show(1) passes "-form 1" to its showproc.)

The way things are now (master and 1.7 branch), users can no
longer write an rcvdist command that does what they want.  One
fix, to pass along switch arguments to post as documented, is
straightforward.  And results in switch handling that is no
worse than for other nmh commands.

I don't think that we should attempt more invasive changes to rcvdist for
1.7.1.

David

-- 
Nmh-workers
https://lists.nongnu.org/mailman/listinfo/nmh-workers

Reply via email to