Re: [PATCH 1/9] send-email: further document missing sendmail aliases functionality

2015-06-01 Thread Allen Hubbe
This looks good. On Mon, Jun 1, 2015 at 2:22 PM, Eric Sunshine wrote: > On Mon, Jun 01, 2015 at 07:43:08AM -0400, Allen Hubbe wrote: >> On May 31, 2015 at 6:29 PM, Eric Sunshine wrote: >> > Sendmail aliases[1] supports expansion to a file ("/path/name") or >> > pipe ("|command"), as well as file

Re: [PATCH 1/9] send-email: further document missing sendmail aliases functionality

2015-06-01 Thread Eric Sunshine
On Mon, Jun 01, 2015 at 07:43:08AM -0400, Allen Hubbe wrote: > On May 31, 2015 at 6:29 PM, Eric Sunshine wrote: > > Sendmail aliases[1] supports expansion to a file ("/path/name") or > > pipe ("|command"), as well as file inclusion (":include: /path/name"), > > however, our implementation does not

Re: [PATCH 1/9] send-email: further document missing sendmail aliases functionality

2015-06-01 Thread Allen Hubbe
According to the documentation, the parser should print a warning for any explicitly unsupported constructs. These are now explicitly unsupported, so the parser should warn on |, /, and :include: . Perhaps the lines that match should be ignored like the others, too. On Sun, May 31, 2015 at 6:29 P

[PATCH 1/9] send-email: further document missing sendmail aliases functionality

2015-05-31 Thread Eric Sunshine
Sendmail aliases[1] supports expansion to a file ("/path/name") or pipe ("|command"), as well as file inclusion (":include: /path/name"), however, our implementation does not support such functionality. [1]: https://www.freebsd.org/cgi/man.cgi?query=aliases&sektion=5 Signed-off-by: Eric Sunshine