Re: [PATCH] Added get sendmail from .mailrc

From: Brilliantov Kirill Vladimirovich <brilliantov@inbox.ru>
Date: Jan 26, 2014, 07:34 UTC
Message-ID: <20140126073438.GA3858@kirill.netbynet.ru>
In-Reply-To: <20140125223721.GA2684@dcvr.yhbt.net>

On 2014-01-25 22:37:21, Eric Wong wrote:

Show 8 quoted lines

> Brilliantov Kirill Vladimirovich <brilliantov@inbox.ru> wrote:
> > Signed-off-by: Brilliantov Kirill Vladimirovich <brilliantov@inbox.ru>
> > ---
> >  git-send-email.perl | 18 ++++++++++++++++++
> >  1 file changed, 18 insertions(+)
> 
> Some documentation references to .mailrc and its format would be nice.
>

Unfortunally I can't found official documentation on this option: http://linux.die.net/man/1/mailx http://publib.boulder.ibm.com/infocenter/aix/v6r1/topic/com.ibm.aix.files/doc/aixfiles/mailrc.htm

On my system (Debian GNU/Linux 7.3) documentation on mailx not conteins description senmail options.

Show 14 quoted lines

> > --- a/git-send-email.perl
> > +++ b/git-send-email.perl
> > @@ -28,6 +28,7 @@ use File::Temp qw/ tempdir tempfile /;
> >  use File::Spec::Functions qw(catfile);
> >  use Error qw(:try);
> >  use Git;
> > +use File::HomeDir;
> 
> We should probably avoid a new dependency and also remain consistent
> with the rest of git handles home directories.
> 
> Unfortunately, expand_user_path()/git_config_pathname() isn't currently
> exposed to scripters right now...
>

Ok, if new dependency is not allowed I see next ways:
- add new argument
- add new configuration parameters