On Mon, May 14, 2018 at 10:14 PM Matt Harbison <mharbiso...@gmail.com> wrote:
> > On May 15, 2018, at 1:05 AM, Martin von Zweigbergk <martinv...@google.com> > wrote: > > > > On Mon, May 14, 2018 at 8:43 PM Matt Harbison <mharbiso...@gmail.com> > wrote: > >> # HG changeset patch >> # User Matt Harbison <matt_harbi...@yahoo.com> >> # Date 1526353230 14400 >> # Mon May 14 23:00:30 2018 -0400 >> # Node ID b7849cc51d8d8040f233328ee46a6466dff207cd >> # Parent b5d62ec33438171e00030ec1442f91fbf496adf6 >> phabricator: split auth.url into the standard auth.schemes and auth.prefix >> > > Kyle (on CC) noticed earlier today that the fact that the > "phabricator.auth" section has a period in its name means it can't be > specified on the command line using --config (as far as he could tell, > anyway). The timing of this patch made me assume it was in response to that > finding (maybe Kyle had reached out to you, I thought), but it's not about > fixing that issue, as far as I can tell, which confused me quite a bit for > a while :) Anyway, I thought I might at least use this as an opportunity to > make you aware of that issue. > > > Yuya mentioned that over the weekend, so I moved it to [auth]. When he > queued that this morning, he mentioned that [auth] has these fields instead > of a URL. So, this is the follow up to the fix you mentioned. > Aha, that's why the example in your patch said just "[auth]". Great :) Thanks for fixing!
_______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
