Re: [PATCH 4/4] send-email: add support for mutt aliases files
On Sat, Mar 25, 2006 at 12:31:18PM -0800, Junio C Hamano wrote:
Show 17 quoted lines
> Eric Wong <normalperson@yhbt.net> writes:
>
> > More email clients/address book formats can easily be supported
> > in the future.
>
> > +if (my $mutt_aliases = `git-repo-config sendemail.muttaliases`) {
> > + chomp $mutt_aliases;
> > + open my $ma, '<', $mutt_aliases or die "opening $mutt_aliases: $!\n";
> > + while (<$ma>) { if (/^alias\s+(\S+)\s+(.*)/) { $aliases{$1} = $2 } }
> > + close $ma;
> > +}
> > +# aliases for more mail clients can be supported here:
> > +
>
> I'd rather avoid proliferation of sendemail.{foo,bar,mutt,pine,...}aliases
> variables. Can we autodetect the alias file format and parse
> the given file accordingly?Don't bother - instead of lots of variables, just have two:
sendemail.aliasesfile
sendemail.aliasfiletype
--
Ryan Anderson
sometimes Pug Majere