[PATCH 7 of 7] configitems: adds a developer warning when accessing undeclared configuration

2017-10-16 Thread Boris Feld
# HG changeset patch # User Boris Feld # Date 1508168487 -7200 # Mon Oct 16 17:41:27 2017 +0200 # Node ID 7a2c3832349499f8b00b9db64e6b87ff644faa9d # Parent d64632aed1d71fd2750aca29fe09d8a2e86921cd # EXP-Topic config.register.ready # Available At https://bitbucket.org/octobus/mercurial-devel/

Re: [PATCH 7 of 7] configitems: adds a developer warning when accessing undeclared configuration

2017-10-17 Thread Augie Fackler
On Mon, Oct 16, 2017 at 06:53:32PM +0200, Boris Feld wrote: > # HG changeset patch > # User Boris Feld > # Date 1508168487 -7200 > # Mon Oct 16 17:41:27 2017 +0200 > # Node ID 7a2c3832349499f8b00b9db64e6b87ff644faa9d > # Parent d64632aed1d71fd2750aca29fe09d8a2e86921cd > # EXP-Topic config.re

Re: [PATCH 7 of 7] configitems: adds a developer warning when accessing undeclared configuration

2017-10-17 Thread Ryan McElroy
On 10/17/17 4:27 PM, Augie Fackler wrote: On Mon, Oct 16, 2017 at 06:53:32PM +0200, Boris Feld wrote: # HG changeset patch # User Boris Feld # Date 1508168487 -7200 # Mon Oct 16 17:41:27 2017 +0200 # Node ID 7a2c3832349499f8b00b9db64e6b87ff644faa9d # Parent d64632aed1d71fd2750aca29fe09d

Re: [PATCH 7 of 7] configitems: adds a developer warning when accessing undeclared configuration

2017-10-18 Thread Boris Feld
On Tue, 2017-10-17 at 11:27 -0400, Augie Fackler wrote: > On Mon, Oct 16, 2017 at 06:53:32PM +0200, Boris Feld wrote: > > # HG changeset patch > > # User Boris Feld > > # Date 1508168487 -7200 > > #  Mon Oct 16 17:41:27 2017 +0200 > > # Node ID 7a2c3832349499f8b00b9db64e6b87ff644faa9d > > # Pa

Re: [PATCH 7 of 7] configitems: adds a developer warning when accessing undeclared configuration

2017-10-18 Thread Augie Fackler
> On Oct 18, 2017, at 09:18, Boris Feld wrote: > > On Tue, 2017-10-17 at 11:27 -0400, Augie Fackler wrote: >> On Mon, Oct 16, 2017 at 06:53:32PM +0200, Boris Feld wrote: >>> # HG changeset patch >>> # User Boris Feld >>> # Date 1508168487 -7200 >>> # Mon Oct 16 17:41:27 2017 +0200 >>> # No

Re: [PATCH 7 of 7] configitems: adds a developer warning when accessing undeclared configuration

2017-10-19 Thread Ryan McElroy
On 10/18/17 11:40 AM, Augie Fackler wrote: We would prefer to have it in this cycle. We want all extensions to have their configuration registered to start beginning working on the cool stuff. Closing the loop from an irc conversation: my concern was that there be a transitional release that ha

Re: [PATCH 7 of 7] configitems: adds a developer warning when accessing undeclared configuration

2017-10-19 Thread Martin von Zweigbergk via Mercurial-devel
On Thu, Oct 19, 2017 at 1:49 PM, Ryan McElroy wrote: > On 10/18/17 11:40 AM, Augie Fackler wrote: > > We would prefer to have it in this cycle. We want all extensions to > have their configuration registered to start beginning working on the > cool stuff. > > Closing the loop from an irc conversa

Re: [PATCH 7 of 7] configitems: adds a developer warning when accessing undeclared configuration

2017-10-19 Thread Gregory Szorc
On Wed, Oct 18, 2017 at 5:40 PM, Augie Fackler wrote: > > > On Oct 18, 2017, at 09:18, Boris Feld wrote: > > > > On Tue, 2017-10-17 at 11:27 -0400, Augie Fackler wrote: > >> On Mon, Oct 16, 2017 at 06:53:32PM +0200, Boris Feld wrote: > >>> # HG changeset patch > >>> # User Boris Feld > >>> # Da

Re: [PATCH 7 of 7] configitems: adds a developer warning when accessing undeclared configuration

2017-10-19 Thread Augie Fackler
> On Oct 19, 2017, at 17:25, Gregory Szorc wrote: > > On Wed, Oct 18, 2017 at 5:40 PM, Augie Fackler wrote: > > > On Oct 18, 2017, at 09:18, Boris Feld wrote: > > > > On Tue, 2017-10-17 at 11:27 -0400, Augie Fackler wrote: > >> On Mon, Oct 16, 2017 at 06:53:32PM +0200, Boris Feld wrote: > >>>

Re: [PATCH 7 of 7] configitems: adds a developer warning when accessing undeclared configuration

2017-10-19 Thread Gregory Szorc
On Thu, Oct 19, 2017 at 11:26 PM, Augie Fackler wrote: > > > On Oct 19, 2017, at 17:25, Gregory Szorc > wrote: > > > > On Wed, Oct 18, 2017 at 5:40 PM, Augie Fackler wrote: > > > > > On Oct 18, 2017, at 09:18, Boris Feld wrote: > > > > > > On Tue, 2017-10-17 at 11:27 -0400, Augie Fackler wrote

Re: [PATCH 7 of 7] configitems: adds a developer warning when accessing undeclared configuration

2017-10-20 Thread Boris Feld
On Fri, 2017-10-20 at 00:45 +0200, Gregory Szorc wrote: > On Thu, Oct 19, 2017 at 11:26 PM, Augie Fackler > wrote: > > > On Oct 19, 2017, at 17:25, Gregory Szorc > > wrote: > > > > > > > > > > On Wed, Oct 18, 2017 at 5:40 PM, Augie Fackler > > wrote: > > > > > > > > > > > On Oct 18, 2017, at