Re: [PATCH 3 of 3 V2] manifest: change manifestctx to not inherit from manifestdict

2016-08-31 Thread Martin von Zweigbergk via Mercurial-devel
Queued, thanks! I've said it before, but I really like the direction this is going. I'm a little embarrassed I didn't think of creating the context objects myself. It seems so obvious now :-) Now I just need to figure out how to make narrowhg deal with these changes. Shouldn't be too hard, but

[PATCH 2 of 3] evolve: removed redundant try/finally blocks

2016-08-31 Thread Pulkit Goyal
# HG changeset patch # User Pulkit Goyal <7895pul...@gmail.com> # Date 1472684936 -19800 # Thu Sep 01 04:38:56 2016 +0530 # Branch stable # Node ID 2ab652cca0c72df663446cc5e79bef2afa0e9854 # Parent 20689e0835c621df4cd19efe7bd06127ab1e0bf8 evolve: removed redundant try/finally blocks There

[PATCH 3 of 3] evolve: drop point release of versions

2016-08-31 Thread Pulkit Goyal
# HG changeset patch # User Pulkit Goyal <7895pul...@gmail.com> # Date 1472695827 -19800 # Thu Sep 01 07:40:27 2016 +0530 # Branch stable # Node ID 3f1d6170cb2a3a967d0eaa08c8d3a80037948bf0 # Parent 2ab652cca0c72df663446cc5e79bef2afa0e9854 evolve: drop point release of versions Modern

Re: [PATCH 1 of 2] evolve: update bugzilla link

2016-08-31 Thread Pulkit Goyal
> While you're here, the point release of the version can be dropped, as > modern Mercurial only looks at the X.Y versions. Okay I am sending a V2 with this patch queued. ___ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org

Re: [PATCH 1 of 2] evolve: update bugzilla link

2016-08-31 Thread Gregory Szorc
On Wed, Aug 31, 2016 at 4:14 PM, Pulkit Goyal <7895pul...@gmail.com> wrote: > # HG changeset patch > # User Pulkit Goyal <7895pul...@gmail.com> > # Date 1472680411 -19800 > # Thu Sep 01 03:23:31 2016 +0530 > # Branch stable > # Node ID 20689e0835c621df4cd19efe7bd06127ab1e0bf8 > # Parent

[PATCH] py3: remove use of *L syntax

2016-08-31 Thread Pulkit Goyal
# HG changeset patch # User Pulkit Goyal <7895pul...@gmail.com> # Date 1472677186 -19800 # Thu Sep 01 02:29:46 2016 +0530 # Node ID 5ffa4cfc09d7c563b09bc3d4fbd50bccff16aa6a # Parent 8a84347b9907ada91f9f3a21aca1fb62cac0fed5 py3: remove use of *L syntax The int in Python 3 behaves as long so

[PATCH 2 of 2] evolve: removed redundant try/finally blocks

2016-08-31 Thread Pulkit Goyal
# HG changeset patch # User Pulkit Goyal <7895pul...@gmail.com> # Date 1472684936 -19800 # Thu Sep 01 04:38:56 2016 +0530 # Branch stable # Node ID 2ab652cca0c72df663446cc5e79bef2afa0e9854 # Parent 20689e0835c621df4cd19efe7bd06127ab1e0bf8 evolve: removed redundant try/finally blocks There

Re: [PATCH 4 of 8 v2] commands: stop abusing int-ness of Python booleans

2016-08-31 Thread Augie Fackler
> On Aug 31, 2016, at 17:19, Kevin Bullock > wrote: > >> On Aug 30, 2016, at 15:16, Augie Fackler wrote: >> >> # HG changeset patch >> # User Augie Fackler >> # Date 1472586452 14400 >> # Tue Aug 30 15:47:32 2016

RFC: bitmap storage for hidden

2016-08-31 Thread Durham Goode
One of the performance costs that affects every command is the computehidden function (which computes what commits are hidden based on a combination of obsmarkers, bookmarks, workingcopy, phases, and tags). At Facebook this function alone can add 300ms to every command a user runs (depending

[PATCH 2 of 3 V2] manifest: make uses of _mancache aware of contexts

2016-08-31 Thread Durham Goode
# HG changeset patch # User Durham Goode # Date 1472518929 25200 # Mon Aug 29 18:02:09 2016 -0700 # Node ID 59bc68e1d78538bb83f60c0d4f9342ec0a8893bf # Parent abce9af35512d8589683d94f34f6d8aa21163568 manifest: make uses of _mancache aware of contexts In a future patch we will

[PATCH 1 of 3 V2] manifest: add treemanifestctx class

2016-08-31 Thread Durham Goode
# HG changeset patch # User Durham Goode # Date 1472675389 25200 # Wed Aug 31 13:29:49 2016 -0700 # Node ID abce9af35512d8589683d94f34f6d8aa21163568 # Parent 318e2b600b80e4ed3c6f37df46ec7544f60d4c0b manifest: add treemanifestctx class Before we start using repo.manifestlog

[PATCH 3 of 3 V2] manifest: change manifestctx to not inherit from manifestdict

2016-08-31 Thread Durham Goode
# HG changeset patch # User Durham Goode # Date 1472672813 25200 # Wed Aug 31 12:46:53 2016 -0700 # Node ID e65cbaceb14f4b522ab59cdccf3f4b2761b2e3bd # Parent 59bc68e1d78538bb83f60c0d4f9342ec0a8893bf manifest: change manifestctx to not inherit from manifestdict If manifestctx

Re: [PATCH 8 of 8 v2] help: mark boolean flags with a ^ and explain that they can be negated (RFC)

2016-08-31 Thread timeless
> options ([+] can be repeated, [?] are flags): This is closer to how I was envisioning it. > Having --no-noninteractive seems really sad. We could make --noninteractive > hidden and then advertise --no-interactive as the long version. Agreed > Also, I don't think --no-no-backup is a good

Re: [PATCH] bundle2: handleoutput i18n

2016-08-31 Thread Akihiko Odaki
On 2016-08-30 13:19, Matt Mackall wrote: On Tue, 2016-08-30 at 11:10 +0900, Akihiko Odaki wrote: Hi, Thanl you for reviewing my patch. On 2016-08-30 09:38, Matt Mackall wrote: > On Sun, 2016-08-28 at 10:57 +0900, Akihiko Odaki wrote: >> # HG changeset patch >> # User Akihiko Odaki

Re: [PATCH] histedit: correct output of error when 'base' is from the edit list

2016-08-31 Thread Yuya Nishihara
On Tue, 30 Aug 2016 13:18:54 -0400, Augie Fackler wrote: > # HG changeset patch > # User Augie Fackler > # Date 1472577230 14400 > # Tue Aug 30 13:13:50 2016 -0400 > # Node ID 4e6fc4921eca722aa5c7967408d93ca3c59b1593 > # Parent 12f8bef59bfa2739d0c5d8425ab494fd2fe38a81 >

Re: [PATCH 03 of 12 topics] flake8: fix E127 style

2016-08-31 Thread Augie Fackler
On Tue, Aug 30, 2016 at 08:58:09PM -0700, Sean Farley wrote: > # HG changeset patch > # User Sean Farley > # Date 1472591416 25200 > # Tue Aug 30 14:10:16 2016 -0700 > # Node ID a6f4b32f655ba895875640534b051c5ebe632695 > # Parent 78dd9f1db1b1cf5a869e1240bb4ed523e930c907 > #

Re: [PATCH 05 of 12 topics] flake8: fix W503 style

2016-08-31 Thread Augie Fackler
On Tue, Aug 30, 2016 at 08:58:11PM -0700, Sean Farley wrote: > # HG changeset patch > # User Sean Farley > # Date 1472595240 25200 > # Tue Aug 30 15:14:00 2016 -0700 > # Node ID 752a62b50ba31b768c52c6b423cd1a36d0c6eae2 > # Parent 8f1e18702ba107ffcb35be04dc8a1e650e5c1e7f > #

Re: [PATCH STABLE] bundle2: localize "remote: "

2016-08-31 Thread Matt Mackall
On Tue, 2016-08-30 at 17:49 -0700, Gregory Szorc wrote: > However, I'm going to posit that _() not being used was accidental. We > translate "remote: " everywhere else. This code is running locally (not on > the server). So I don't see why we wouldn't tranhslate. I agree it's almost certainly