- Apr 14, 2018
-
-
Martin von Zweigbergk authored
The shortest() template function depended on the behavior of revlog._partialmatch() for these types of inputs: * non-hex strings * ambiguous strings * too long strings revlog._partialmatch() seems to return the input unchanged in these cases, but we shouldn't depend on such a low-level function to match the behavior we want in the user-facing template function. Instead, let's handle these cases in the template function and always pass a binary nodeid to _partialmatch(). Differential Revision: https://phab.mercurial-scm.org/D3371
-
- Apr 13, 2018
-
-
Martin von Zweigbergk authored
Both callers were doing this, and resolvehexnodeidprefix() was also working on the unfiltered repo, so it makes more sense to have it all in one place. Differential Revision: https://phab.mercurial-scm.org/D3313
-
- Apr 15, 2018
-
-
Anton Shestakov authored
There used to be 2 spaces between divergent nodes (when not using custom template for divergentnodes) because divergentnodes is a hybrid list, which means it gets ' '.join()ed, but formatnode() already had a space. Now it doesn't, which requires extra effort in writing custom templates for whyunstable, but at least it looks correctly by default. Test output needs to be sorted for stability.
-
Anton Shestakov authored
-
Anton Shestakov authored
These templates demonstrate that {whyunstable} is fully template-friendly, but they don't need to produce such long lines of output.
-
Anton Shestakov authored
-
- Apr 14, 2018
-
-
muxator authored
Bionic Beaver is an LTS release, supported until 2023-04.
-
muxator authored
The list of make targets is likely to evolve over time. This reformats streamlines future patches.
-
muxator authored
This imitates e63dfbbdbd07 and is a small addition to 231690dba9b4 and 5c1283713293
-
- Apr 01, 2018
-
-
Yuya Nishihara authored
IIUC, only reason it was a class is to make the generator restartable, which is now served by the mappinggenerator.
-
Yuya Nishihara authored
_siblings will be converted to a plain function.
-
Yuya Nishihara authored
-
Yuya Nishihara authored
There was actually no lazy stuff.
-
Yuya Nishihara authored
Since a sessionvars object is updated in-place, we can't simply wrap it by mappinglist.
-
Yuya Nishihara authored
-
Yuya Nishihara authored
-
- Apr 13, 2018
-
-
Matt Harbison authored
I'm not bothering with validating PUT requests (for now), because the spec doesn't explicitly call out a Content-Type (though the example transcript does use the sensible 'application/octet-stream').
-
- Feb 25, 2018
-
-
Matt Harbison authored
The aborts weren't killing the server, but this seems cleaner. I'm not sure if it matters to handle the remaining IOError in the test like this, for consistency. The error code still feels wrong (especially if the client is trying to download a corrupt blob) but I don't see anything better in the RFCs, and this is already used elsewhere because the Batch API spec specifically mentioned this as a "Validation Error".
-
- Apr 13, 2018
-
-
Matt Harbison authored
This wasn't appending the '.git/info/lfs' path in this case.
-
Matt Harbison authored
Reporting a 500 and then not leaving any traces on the server seems like a receipe for frustration. There's similar log writing in hgweb.server.do_POST(). That doesn't write directly to the wsgi.errors object, so it doesn't seem worth trying to refactor. It does seem like earlier stack frames are missing for some reason.
-
- Apr 07, 2018
-
-
Matt Harbison authored
The tests are somewhat fragile in that the extension that forces the errors is counting how many times some of the functions are being called, so it depends heavily on the content of the repo. Maybe we can do something clever like load an extension on the client, and have it send over instructions in the HTTP header how to fail. (I'm trying to avoid killing and restarting the server, because Windows seems to have issues with doing that a lot.) But I'd rather fix issues than polish tests before the freeze.
-
- Apr 14, 2018
-
-
Augie Fackler authored
Caught by Yuya in D3326. Differential Revision: https://phab.mercurial-scm.org/D3372
-
Yuya Nishihara authored
We aren't stress testing CPU. $ time ./run-tests.py -l test-check-commit.t --timeout 600 (orig) 162.59s user 17.98s system 101% cpu 2:58.55 total (new) 5.85s user 0.99s system 98% cpu 6.939 total
-
Anton Shestakov authored
-
- Mar 26, 2018
-
-
Anton Shestakov authored
-
- Apr 14, 2018
-
-
Augie Fackler authored
Fixes test-add.t on Python 3. Differential Revision: https://phab.mercurial-scm.org/D3365
-
Augie Fackler authored
# skip-blame just b prefixes Differential Revision: https://phab.mercurial-scm.org/D3366
-
Augie Fackler authored
Both of these tests now pass on Python 3. # skip-blame just b prefixes. So many b prefixes. Differential Revision: https://phab.mercurial-scm.org/D3369
-
Augie Fackler authored
Fortunately for us, this is really only used internally, so we can be lazy about the encoding here. test-wireproto-framing.py now passes on Python 3. Differential Revision: https://phab.mercurial-scm.org/D3368
-
Martin von Zweigbergk authored
We have scmutil.resolvehexnodeidprefix() for resolving a prefix to a full nodeid, so it makes sense to have the inverse method next to it. For now it just delegates to changelog.shortest(), but it will soon also make sure it's called on the unfiltered repo, to match resolvehexnodeidprefix(). Note that the change in show.py also makes it so the conversion from revnum to nodeid is done on the filtered repo, but that should be inconsequential since the revs are all from the filtered repo anyway. Differential Revision: https://phab.mercurial-scm.org/D3370
-
- Apr 13, 2018
-
-
Martin von Zweigbergk authored
I should have copied this from changectx.__init__ into in 35b34202dd3b (context: handle partial nodeids in revsymbol(), 2018-04-08). Differential Revision: https://phab.mercurial-scm.org/D3310
-
- Apr 14, 2018
-
-
Martin von Zweigbergk authored
I'm going to wrap revlog.shortest() in scmutil. I plan on calling it shortesthexnodeidprefix(). resolvehexnodeidprefix() matches that better. Also, "prefix" carries more information than "partial". Differential Revision: https://phab.mercurial-scm.org/D3309
-
- Apr 13, 2018
-
-
Martin von Zweigbergk authored
Differential Revision: https://phab.mercurial-scm.org/D3308
-
Martin von Zweigbergk authored
This patch makes the deprecation warning print the caller of repo.__getitem__. That's not quite correct, since there could also be other callers of changectx.__init__, but it seems like the most helpful stack level in practice. Differential Revision: https://phab.mercurial-scm.org/D3307
-
Martin von Zweigbergk authored
Without this patch, you'll get something like this: <path>/mercurial/util.py:3784: DeprecationWarning: 'util.hgexecutable' is deprecated, use 'utils.procutil.hgexecutable' (but on one line) Differential Revision: https://phab.mercurial-scm.org/D3331
-
- Apr 14, 2018
-
-
Yuya Nishihara authored
And document that it's only for legacy lookup. If we have a repo, we're likely to do more things where that shouldn't be done.
-
Yuya Nishihara authored
AFAIK, the only user was the directaccess extension, which is in core now.
-
- Apr 13, 2018
-
-
Pulkit Goyal authored
The values of opts dict still needed to be converted to bytes. Differential Revision: https://phab.mercurial-scm.org/D3330
-
Augie Fackler authored
Differential Revision: https://phab.mercurial-scm.org/D3338
-
Augie Fackler authored
Differential Revision: https://phab.mercurial-scm.org/D3337
-