[PATCH] Add %includeifexists, a new type of hgrc include
Matt Mackall
mpm at selenic.com
Tue May 31 16:01:08 CDT 2011
On Tue, 2011-05-31 at 17:57 +0200, Martin Geisler wrote:
> Matt Mackall <mpm at selenic.com> writes:
>
> > On Tue, 2011-05-31 at 17:30 +0200, Martin Geisler wrote:
> >> Matt Mackall <mpm at selenic.com> writes:
> >>
> >> > On Tue, 2011-05-31 at 01:19 +0100, Tom Fitzhenry wrote:
> >> >> # HG changeset patch
> >> >> # User Tom Fitzhenry <tom at tom-fitzhenry.me.uk>
> >> >> # Date 1306801124 -3600
> >> >> # Node ID 029faced02ef51b7de5c6603f81580267d669c1e
> >> >> # Parent f6a433671c06ea7986e0cf2c54686d1af7f22ba1
> >> >> Add %includeifexists, a new type of hgrc include
> >> >>
> >> >> Use case: a user has a hgrc which is shared across multiple machines,
> >> >> /some/ of which have configuration specific to that machine. %include
> >> >> allows the user to include another configuration file, but causes a
> >> >> parse error if the included file does not exist.
> >> >
> >> > Perhaps we should just drop the parse error. This might make it harder
> >> > to debug problems with specifying the filename, but would otherwise be
> >> > harmless. Thoughts?
> >>
> >> I would like that, along with a ui.debug message. Maybe we could then
> >> let showconfig output a warning so that it's very clear what's going on.
> >
> > Unfortunately, there are no ui objects at this layer.
>
> Then I prefer just ignoring the exception -- I prefer that over
> introducing a new %tryinclude directive.
Ok, I've queued a fix that ignores include errors for nonexistent files.
--
Mathematics is the supreme nostalgia of our time.
More information about the Mercurial-devel
mailing list