Mercurial > hg
annotate hgext/narrow/TODO.rst @ 39771:a063786c89fb
narrow: remove narrowrevlog
Core now automatically enables ellipsis support on revlogs when
repositories have narrow enabled. So, we no longer need to globally
register the revlog flag as part of activating the narrow extension
and this code can be deleted.
A side effect of this change is that repositories will now raise an
error on encountering an ellipsis flag when the narrow extension is
loaded. Previously, loading the narrow extension on a non-narrow repo
could result in silent usage of the ellipsis flag. This could lead
to undetected bugs. I think the new behavior is more correct.
Differential Revision: https://phab.mercurial-scm.org/D4649
author | Gregory Szorc <gregory.szorc@gmail.com> |
---|---|
date | Tue, 18 Sep 2018 15:32:11 -0700 |
parents | 10a8472f6662 |
children | 937ce75ea18c |
rev | line source |
---|---|
36104 | 1 Integration with the share extension needs improvement. Right now |
2 we've seen some odd bugs, and the way we modify the contents of the | |
3 .hg/shared file is unfortunate. See wrappostshare() and unsharenarrowspec(). | |
4 | |
5 Resolve commentary on narrowrepo.wraprepo.narrowrepository.status | |
6 about the filtering of status being done at an awkward layer. This | |
7 came up the import to hgext, but nobody's got concrete improvement | |
8 ideas as of then. | |
9 | |
39771
a063786c89fb
narrow: remove narrowrevlog
Gregory Szorc <gregory.szorc@gmail.com>
parents:
39544
diff
changeset
|
10 Address commentary in manifest.excludedmanifestrevlog.add - |
36104 | 11 specifically we should improve the collaboration with core so that |
12 add() never gets called on an excluded directory and we can improve | |
13 the stand-in to raise a ProgrammingError. | |
14 | |
15 Reason more completely about rename-filtering logic in | |
16 narrowfilelog. There could be some surprises lurking there. | |
17 | |
18 Formally document the narrowspec format. Unify with sparse, if at all | |
19 possible. For bonus points, unify with the server-specified narrowspec | |
20 format. | |
21 | |
22 narrowrepo.setnarrowpats() or narrowspec.save() need to make sure | |
23 they're holding the wlock. |