[PATCH 1 of 1] Support for SMTP TLS and login authentication
mpm at selenic.com
Fri Sep 9 13:14:43 CDT 2005
On Fri, Sep 09, 2005 at 11:02:19AM -0700, Lee Cantey wrote:
> On 9/9/05, Matt Mackall <mpm at selenic.com> wrote:
> > Tabs are lurking in the above. Please add some sort of documenting
> > comment at the top for this stuff.
> I'm fixing it up now. We use tabs at work (against my will) so they
> accidentally snuck in.
> Do you have a preference for the new variable names in the smtp section? I
> thought username and password were pretty obvious but debated among: tls,
> enabletls, usetls, and encrypted (my favorite this morning).
Don't particularly care about the precise names as it lives in
But it does raise an interesting point, which is that the hgrc
namespace needs to be controlled - we need some scheme to keep
extensions and add-ons from polluting the namespace for future core
It would seem the sensible thing is for variables added by extensions
or other add-ons to go in a section with the same name as the add-on.
smtp = foo
usetls = true
Or if something needs multiple sections, it can use a prefix:
smtp = foo
diffstat = true
Extensions are of course free to use existing variables where
Then when we finally add a mail reader to mercurial, we won't have
a conflict over this smtp section.
> PS. Should I resubmit the whole patch? I'd think so, but I don't know the
> cleanest way to remove/revert my current one from my repository.
Looks like Bryan beat you to it.
Mathematics is the supreme nostalgia of our time.
More information about the Mercurial