[PATCH 2 of 6 ctx-minor-fixes] context: fix typo in workingcommitctx

# HG changeset patch # User Sean Farley # Date 1528775327 25200 # Mon Jun 11 20:48:47 2018 -0700 # Branch ctx-minor-fixes # Node ID c21a9a45b1a718d67d294fb822b58c7d6a2aa207 # Parent 03041a020368729345f9b72e6914122c5e88a302 context: fix typo in workingcommitctx This was probably a copy

[PATCH 6 of 6 ctx-minor-fixes] context: define contract for parents

# HG changeset patch # User Sean Farley # Date 1528497109 25200 # Fri Jun 08 15:31:49 2018 -0700 # Branch ctx-minor-fixes # Node ID 58496e395dd324297748551297d1d572ab762496 # Parent 56222df6e5c8738b9b04881477c166540c50b17a context: define contract for parents From now on, context._parents

[PATCH 4 of 6 ctx-minor-fixes] memctx: simplify _manifest with new revlog nodeids

# HG changeset patch # User Sean Farley # Date 1526998571 -7200 # Tue May 22 16:16:11 2018 +0200 # Branch ctx-minor-fixes # Node ID 0173c0dca9d389d55a3f0e8ff3bcf43dc7ef659d # Parent 2a524ac6b6e5da11f0f8ab9c9137d44c16f724b5 memctx: simplify _manifest with new revlog nodeids This was

[PATCH 3 of 6 ctx-minor-fixes] context: remove unused overlayctx

# HG changeset patch # User Sean Farley # Date 1526985338 -7200 # Tue May 22 12:35:38 2018 +0200 # Branch ctx-minor-fixes # Node ID 2a524ac6b6e5da11f0f8ab9c9137d44c16f724b5 # Parent c21a9a45b1a718d67d294fb822b58c7d6a2aa207 context: remove unused overlayctx It seems that this was maybe used

[PATCH 1 of 6 ctx-minor-fixes] filectx: fix return of renamed

# HG changeset patch # User Sean Farley # Date 1531872982 25200 # Tue Jul 17 17:16:22 2018 -0700 # Branch ctx-minor-fixes # Node ID 03041a020368729345f9b72e6914122c5e88a302 # Parent 3d22aef3ecd5fe02c9411503b04dfbdcb6240cc1 filectx: fix return of renamed How is this not blowing up

[PATCH 5 of 6 ctx-minor-fixes] context: use object-oriented programming

# HG changeset patch # User Sean Farley # Date 1528495331 25200 # Fri Jun 08 15:02:11 2018 -0700 # Branch ctx-minor-fixes # Node ID 56222df6e5c8738b9b04881477c166540c50b17a # Parent 0173c0dca9d389d55a3f0e8ff3bcf43dc7ef659d context: use object-oriented programming We have abstracted

D4663: filelog: stop proxying headrevs() (API)

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY The previous commit removed the last user of this method. It is redundant with heads() and adds little to no value other than convenience. Let's nuke it.

D4662: hgweb: use heads() instead of headrevs()

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY These appear to be the only callers of headrevs() on file storage objects. Let's port to heads() so we can remove headrevs(). REPOSITORY rHG Mercurial

D4656: revlog: move censor logic out of censor extension

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY The censor extension is doing very low-level things with revlogs. It is fundamentally impossible for this logic to remain in the censor extension while support

D4661: filelog: record what's using attributes

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY filelog and the file storage interface have some attributes that ideally shouldn't be there. This commit annotates some of those attributes with their users so

D4660: filelog: stop proxying datafile (API)

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY It appears the censor code was the last user of this proxy. With there being a dedicated censor API, we can drop the proxy. REPOSITORY rHG Mercurial REVISION

D4659: filelog: stop proxying _addrevision() (API)

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY There are no callers of this API in core. And I'm not sure why this proxy was added in the first place, as the commit that added it

D4658: filelog: stop proxying compress() (API)

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY The censoring code was previously relying on this. With a dedicated censoring API on the interface, no consumers are left and we can stop proxying this method.

D4657: filelog: stop proxying start(), end(), and length() (API)

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY These were needed by the censoring code, which formerly lived in the censor extension. Now that there is a censoring API on the file storage interface, nothing

Re: [PATCH 3 of 3] py3: allow run-tests.py to run on Windows

On Tue, 18 Sep 2018 09:40:58 -0400, Yuya Nishihara wrote: It looks like part of the problem with not finding these files is TESTDIR is not defined when running the *.t file. Inside Test.runTest(), 'TESTDIR' is not in env, osenvironb, or os.environ on py3 (but it is on py2). I'm assuming

Re: [PATCH] about: update URLs and prefer https://

Sean Farley writes: > Augie Fackler writes: > >> (+smf - this is for the website) >> >>> On Sep 6, 2018, at 22:07, Bradley M. Jones wrote: >>> >>> # HG changeset patch >>> # User Bradley Jones >>> # Date 1536285273 -43200 >>> # Fri Sep 07 13:54:33 2018 +1200 >>> # Node ID

mercurial@39686: 59 new changesets

59 new changesets in mercurial: https://www.mercurial-scm.org/repo/hg/rev/a5de21c9e370 changeset: 39628:a5de21c9e370 user:Gregory Szorc date:Tue Aug 21 15:33:11 2018 -0700 summary: httppeer: expose capabilities for each command

D4655: global: replace most uses of RevlogError with StorageError (API)

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY When catching errors in storage, we should be catching StorageError instead of RevlogError. When throwing errors related to storage, we shouldn't be using

D4654: error: introduce StorageError

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY Errors in revlogs are often represented by RevlogError. It's fine for revlogs to raise a revlog-specific exception. But in the context of multiple storage

D4652: revlog: drop some more error aliases (API)

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY These should be lightly used and I doubt that will be any strong objections to removing the aliases. Note that some uses of ProgrammingError in this file

D4653: revlog: drop LookupError alias (API)

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY This alias is especially bad because it shadows the built-in LookupError type. This has caused me confusion in the past when reading revlog code. Qualifying

D4651: revlog: drop RevlogError alias (API)

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY error.RevlogError was moved from revlog.py in https://phab.mercurial-scm.org/rHG08cabecfa8a89f14b17dd5b68ef297f9ab6f653a in 2009. revlog.RevlogError has

D4650: testing: add interface unit tests for file storage

indygreg created this revision. Herald added subscribers: mercurial-devel, mjpieters. Herald added a reviewer: hg-reviewers. REVISION SUMMARY Our strategy for supporting alternate storage backends is to define interfaces for everything then "code to the interface." We already have

D4483: wireprotov2: add phases to "changesetdata" command

indygreg added a comment. In https://phab.mercurial-scm.org/D4483#70789, @martinvonz wrote: > > It's worth noting that when phase data is being requested, the > > server will also emit changeset records for nodes in the bases > > specified by the "noderange" argument. This is to

D4483: wireprotov2: add phases to "changesetdata" command

martinvonz added a comment. > It's worth noting that when phase data is being requested, the > server will also emit changeset records for nodes in the bases > specified by the "noderange" argument. This is to ensure that > phase-only updates for nodes the client has are available to

D4648: localrepo: enable ellipsis flag on revlogs when repo is narrow

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY If the narrow requirement is present, revlogs created for that repository will have the ellipsis flag enabled. This is the same behavior that the narrow

D4649: narrow: remove narrowrevlog

indygreg created this revision. Herald added a reviewer: durin42. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY Core now automatically enables ellipsis support on revlogs when repositories have narrow enabled. So, we no longer need to

D4647: revlog: add opener option to enable ellipsis flag processor

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY The ellipsis flag processor can now be registered by specifying an opener option when constructing a revlog instance. This allows us to enable ellipsis flags

D4641: localrepo: pass root manifest into manifestlog.__init__

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY Today, localrepository has a method that can be overloaded which returns an instance of the root manifest storage object. When a manifestlog is created, it

D4646: revlog: store flag processors per revlog

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY Previously, revlog flag processing would consult a global dict when processing flags. This was simple. But it had the undesired side-effect that any extension

D4645: revlog: define ellipsis flag processors in core

indygreg created this revision. Herald added a reviewer: durin42. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY We will soon be teaching core to honor the ellipsis flag on revlogs. Moving the definition of the processor functions to core

D4643: filelog: custom filelog to be used with narrow repos

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY Narrow repos may have file revisions whose copy/rename metadata references files not in the store. This can pose problems when consumers attempt to access a

D4642: localrepo: iteratively derive local repository type

indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY This commit implements the dynamic local repository type derivation that was explained in the recent commit

D4644: narrow: remove custom filelog type

indygreg created this revision. Herald added a reviewer: durin42. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY This functionality is now handled by core as of the previous commit. I wanted this to be a standalone commit because the

mercurial@39627: 5 new changesets

5 new changesets in mercurial: https://www.mercurial-scm.org/repo/hg/rev/34ecc0a09c76 changeset: 39623:34ecc0a09c76 user:Yuya Nishihara date:Thu Jun 07 21:48:11 2018 +0900 summary: formatter: populate ctx from repo and node value

D4576: localrepo: extract resolving of opener options to standalone functions

indygreg added a comment. In https://phab.mercurial-scm.org/D4576#70668, @durin42 wrote: > I kind of hope that eventually you're going to break all the requirements-handling nonsense out into its own module somehow? I suspect I'm trending in the direction of creating a `storage`

D4576: localrepo: extract resolving of opener options to standalone functions

This revision was automatically updated to reflect the committed changes. Closed by commit rHGb10d145837bc: localrepo: extract resolving of opener options to standalone functions (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4575: localrepo: use boolean in opener options

This revision was automatically updated to reflect the committed changes. Closed by commit rHG6f26417b71bb: localrepo: use boolean in opener options (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D4575?vs=11013=11173

D4574: localrepo: move store() from store module

This revision was automatically updated to reflect the committed changes. Closed by commit rHGf44187605315: localrepo: move store() from store module (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D4574?vs=11012=11171

D4572: localrepo: document and test bug around opening shared repos

This revision was automatically updated to reflect the committed changes. Closed by commit rHG9de1a1c83cd7: localrepo: document and test bug around opening shared repos (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4573: localrepo: resolve store and cachevfs in makelocalrepository()

This revision was automatically updated to reflect the committed changes. Closed by commit rHG98ca9078807a: localrepo: resolve store and cachevfs in makelocalrepository() (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4571: localrepo: move requirements reasonability testing to own function

This revision was automatically updated to reflect the committed changes. Closed by commit rHGcb2dcfa5cade: localrepo: move requirements reasonability testing to own function (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4570: statichttprepo: use new functions for requirements validation

This revision was automatically updated to reflect the committed changes. Closed by commit rHG6192980553b4: statichttprepo: use new functions for requirements validation (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4569: localrepo: validate supported requirements in makelocalrepository()

This revision was automatically updated to reflect the committed changes. Closed by commit rHG5b8e9b2060ef: localrepo: validate supported requirements in makelocalrepository() (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4568: localrepo: read requirements file in makelocalrepository()

This revision was automatically updated to reflect the committed changes. Closed by commit rHG6a3162ed881d: localrepo: read requirements file in makelocalrepository() (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4567: localrepo: check for .hg/ directory in makelocalrepository()

This revision was automatically updated to reflect the committed changes. Closed by commit rHG2f067e365532: localrepo: check for .hg/ directory in makelocalrepository() (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4566: localrepo: load extensions in makelocalrepository()

This revision was automatically updated to reflect the committed changes. Closed by commit rHGe0c5017124b3: localrepo: load extensions in makelocalrepository() (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4565: localrepo: copy ui in makelocalrepository()

This revision was automatically updated to reflect the committed changes. Closed by commit rHGf19bba3f4d3f: localrepo: copy ui in makelocalrepository() (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4564: localrepo: move some vfs initialization out of __init__

This revision was automatically updated to reflect the committed changes. Closed by commit rHG2f9cdb5b3644: localrepo: move some vfs initialization out of __init__ (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4563: localrepo: create new function for instantiating a local repo object

This revision was automatically updated to reflect the committed changes. Closed by commit rHGbfeab472e3c0: localrepo: create new function for instantiating a local repo object (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4576: localrepo: extract resolving of opener options to standalone functions

durin42 added a comment. I kind of hope that eventually you're going to break all the requirements-handling nonsense out into its own module somehow? REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D4576 To: indygreg, #hg-reviewers Cc: durin42, mercurial-devel

D4615: wireprotov2: declare command arguments richly

indygreg added inline comments. INLINE COMMENTS > durin42 wrote in wireprotov2server.py:543-546 > Shouldn't there be a default XOR required == true? That is, could we get away > with `default=None` and then deriving requiredness from default-availability? We could do that, sure. Would it be OK

D4615: wireprotov2: declare command arguments richly

durin42 added inline comments. INLINE COMMENTS > wireprotov2server.py:543-546 > + ``default`` > + A callable returning the default value for this argument. > + > + ``required`` Shouldn't there be a default XOR required == true? That is, could we get away with

D4634: transaction: make entries a private attribute (API)

This revision was automatically updated to reflect the committed changes. Closed by commit rHG3d22aef3ecd5: transaction: make entries a private attribute (API) (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4632: transaction: make map a private attribute

This revision was automatically updated to reflect the committed changes. Closed by commit rHG1a68c9b1920d: transaction: make map a private attribute (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D4632?vs=11131=11159

D4633: transaction: make names a private attribute

This revision was automatically updated to reflect the committed changes. Closed by commit rHG4024c363cd33: transaction: make names a private attribute (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4631: transaction: make report a private attribute

This revision was automatically updated to reflect the committed changes. Closed by commit rHG337d6e0fd9c9: transaction: make report a private attribute (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4630: transaction: make opener a private attribute

This revision was automatically updated to reflect the committed changes. Closed by commit rHGb590f4763aba: transaction: make opener a private attribute (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4629: transaction: make after a private attribute

This revision was automatically updated to reflect the committed changes. Closed by commit rHG0d7b9db85675: transaction: make after a private attribute (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4628: transaction: make checkambigfiles a private attribute

This revision was automatically updated to reflect the committed changes. Closed by commit rHG264d56954dda: transaction: make checkambigfiles a private attribute (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4625: transaction: make file a private attribute

This revision was automatically updated to reflect the committed changes. Closed by commit rHGd27fde3e023e: transaction: make file a private attribute (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4627: transaction: make validator a private attribute

This revision was automatically updated to reflect the committed changes. Closed by commit rHGebbba3ba3f66: transaction: make validator a private attribute (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4626: transaction: make releasefn a private attribute

This revision was automatically updated to reflect the committed changes. Closed by commit rHG040007cd3d81: transaction: make releasefn a private attribute (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4624: transaction: make journal a private attribute

This revision was automatically updated to reflect the committed changes. Closed by commit rHG77c4e2ae9f07: transaction: make journal a private attribute (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4623: transaction: make undoname a private attribute

This revision was automatically updated to reflect the committed changes. Closed by commit rHGda9ce63bfa9b: transaction: make undoname a private attribute (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4622: transaction: make count and usages private attributes

This revision was automatically updated to reflect the committed changes. Closed by commit rHG3e8952c0cb45: transaction: make count and usages private attributes (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4640: narrow: add test showing that local-to-local narrow clones don't work

durin42 created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY It turns out they've never actually worked: prior to some recent refactoring they just unintentionally followed the full-clone path, which we unintentionally

D4638: changegroup: add functionality to skip adding changelog data to changegroup

pulkit added a comment. In https://phab.mercurial-scm.org/D4638#70613, @indygreg wrote: > I think the code as written makes sense: I'd rather have 1 loop over `deltas` than 2. > > Regarding the other potential improvement, the reality of changegroups is that both they and bundle2

D4639: narrow: don't send the changelog information when widening without ellipses

This revision was automatically updated to reflect the committed changes. Closed by commit rHGf1844a10ee19: narrow: dont send the changelog information when widening without ellipses (authored by pulkit, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4638: changegroup: add functionality to skip adding changelog data to changegroup

This revision was automatically updated to reflect the committed changes. Closed by commit rHGa1942015c10e: changegroup: add functionality to skip adding changelog data to changegroup (authored by pulkit, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4638: changegroup: add functionality to skip adding changelog data to changegroup

indygreg accepted this revision. indygreg added a comment. This revision is now accepted and ready to land. I think the code as written makes sense: I'd rather have 1 loop over `deltas` than 2. Regarding the other potential improvement, the reality of changegroups is that both they and

D4637: tests: add debug output in test-narrow-widen-no-ellipsis.t

This revision was automatically updated to reflect the committed changes. Closed by commit rHGdee887072f27: tests: add debug output in test-narrow-widen-no-ellipsis.t (authored by pulkit, committed by ). CHANGED PRIOR TO COMMIT https://phab.mercurial-scm.org/D4637?vs=11136=11144#toc

D4636: changegroup: improve the devel-warn to specify changelog was empty

This revision was automatically updated to reflect the committed changes. Closed by commit rHG9b07ee0a1054: changegroup: improve the devel-warn to specify changelog was empty (authored by pulkit, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4637: tests: add debug output in test-narrow-widen-no-ellipsis.t

indygreg added inline comments. INLINE COMMENTS > test-narrow-widen-no-ellipsis.t:93 >comparing with ssh://user@dummy/master > + running python "/place/home/pulkit/repo/hg-committed/tests/dummyssh" > 'user@dummy' 'hg -R master serve --stdio' > + sending hello command This needs some

D4612: zsh_completion: add -b/--branch and -B/--bookmark(s) flags properly

This revision was automatically updated to reflect the committed changes. Closed by commit rHG3f11cb1aeb90: zsh_completion: add -b/--branch and -B/--bookmark(s) flags properly (authored by av6, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4606: narrow: when writing treemanifests, skip inspecting directories outside narrow

This revision was automatically updated to reflect the committed changes. Closed by commit rHG24870f1be088: narrow: when writing treemanifests, skip inspecting directories outside narrow (authored by spectral, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4635: tests: fix a couple of drawdag.py references

This revision was automatically updated to reflect the committed changes. Closed by commit rHG0b7594ada0db: tests: fix a couple of drawdag.py references (authored by durin42, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4588: py3: fix kwargs handling in hgext/fastannotate.py

This revision was automatically updated to reflect the committed changes. Closed by commit rHGb77f2ea51da5: py3: fix kwargs handling in hgext/fastannotate.py (authored by pulkit, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4635: tests: fix a couple of drawdag.py references

yuja added a comment. No idea which is more correct, but drawdag.py is generally paired with `$TESTDIR`. Queued, thanks. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D4635 To: durin42, #hg-reviewers Cc: yuja, mercurial-devel

Re: D4635: tests: fix a couple of drawdag.py references

No idea which is more correct, but drawdag.py is generally paired with `$TESTDIR`. Queued, thanks. ___ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Re: [PATCH 3 of 3] py3: allow run-tests.py to run on Windows

> It looks like part of the problem with not finding these files is TESTDIR > is not defined when running the *.t file. Inside Test.runTest(), > 'TESTDIR' is not in env, osenvironb, or os.environ on py3 (but it is on > py2). I'm assuming this is a bytes vs unicode thing since Linux works,

D4638: changegroup: add functionality to skip adding changelog data to changegroup

pulkit added a comment. I agree this is dirty and I will be very happy to improve this. I can think of folllowing improvements First is this: @@ -837,10 +838,14 @@ class cgpacker(object): size = 0 clstate, deltas = self._generatechangelog(cl,

D4639: narrow: don't send the changelog information when widening without ellipses

pulkit created this revision. Herald added a reviewer: durin42. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY When we widen anon-ellipses narrow copy, the server sends the changelog information of all the changesets. The code was copied

D4638: changegroup: add functionality to skip adding changelog data to changegroup

pulkit created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY In narrow extension, when we have a non-ellipses narrow working copy and we extend it, we pull all the changelog data again and the client tries to reapply all

D4637: tests: add debug output in test-narrow-widen-no-ellipsis.t

pulkit created this revision. Herald added a reviewer: durin42. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY This will help us in understanding the upcoming patches better. REPOSITORY rHG Mercurial REVISION DETAIL

D4636: changegroup: improve the devel-warn to specify changelog was empty

pulkit created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY Right now, the develwarn says "applied empty changegroup" which is not correct because we can send a changegroup without changelog with just manifest and

[Bug 5987] New: exception thrown when cloning with trailing slash

https://bz.mercurial-scm.org/show_bug.cgi?id=5987 Bug ID: 5987 Summary: exception thrown when cloning with trailing slash Product: Mercurial Version: 4.7.1 Hardware: PC OS: All Status: UNCONFIRMED

Re: Wire protocol futures

I liked this post very much, it gave me a lot of information about Mercurial. Thanks for writing it up :) I wish if I could help in achieving this goal of a new wire protocol. On Sat, Sep 1, 2018 at 4:17 AM Gregory Szorc wrote: > This is a long post and I apologize in advance for that. I've