martinvonz added a comment.
In D7627#112577 <https://phab.mercurial-scm.org/D7627#112577>, @mharbison72 wrote: > In D7627#112373 <https://phab.mercurial-scm.org/D7627#112373>, @martinvonz wrote: > >> In D7627#112223 <https://phab.mercurial-scm.org/D7627#112223>, @martinvonz wrote: >> >>> In D7627#112220 <https://phab.mercurial-scm.org/D7627#112220>, @mharbison72 wrote: >>> >>>> On Windows, you can set a registry key to one or more *.ini files or directories that get read in, so it seems slightly handy to know the order in which files are processed. I know I've used that mechanism on various systems to find directories in which to stash things too. >>>> That said, I don't think I feel strongly about keeping it if it makes the next patch messy. (I did a quick skim of it, but didn't see why that would be.) >>> >>> It means that every line in merge-tools.rc would get two lines in this output (one line saying "set value from ..." or something like that). >> >> I'm also happy to drop just the "set item" lines. Better? > > I think so, thanks. > One of the last hunks dropped here mentions "set by $EDITOR", but then never mentions EDITOR again. That seems confusing if you don't know what that setting applies to and the precedence rules, and not real helpful if you do. Okay, will upload an update that adds back the `read config from:` lines a few minutes. REPOSITORY rHG Mercurial CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D7627/new/ REVISION DETAIL https://phab.mercurial-scm.org/D7627 To: martinvonz, #hg-reviewers Cc: mharbison72, mercurial-devel _______________________________________________ Mercurial-devel mailing list [email protected] https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
