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

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

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: >

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

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

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

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

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

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

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

[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