[PATCH 6 of 6] largefiles: modernize how capabilities are added to the wire protocol

Yuya Nishihara yuya at tcha.org
Thu Dec 28 08:04:01 EST 2017


On Wed, 27 Dec 2017 03:27:58 -0500, Matt Harbison wrote:
> # HG changeset patch
> # User Matt Harbison <matt_harbison at yahoo.com>
> # Date 1514349649 18000
> #      Tue Dec 26 23:40:49 2017 -0500
> # Node ID f2e5631c99e6e2c7e9bac058185c24dd73a04e98
> # Parent  25ecea2ec3d7844c9146bf878fc5093ab33b6e11
> largefiles: modernize how capabilities are added to the wire protocol

Looks good to me, so queued, thanks.

> diff --git a/hgext/largefiles/proto.py b/hgext/largefiles/proto.py
> --- a/hgext/largefiles/proto.py
> +++ b/hgext/largefiles/proto.py
> @@ -161,9 +161,11 @@
>      repo.__class__ = lfileswirerepository
>  
>  # advertise the largefiles=serve capability
> -def capabilities(repo, proto):
> -    '''Wrap server command to announce largefile server capability'''
> -    return capabilitiesorig(repo, proto) + ' largefiles=serve'
> +def _capabilities(orig, repo, proto):
> +    '''announce largefile server capability'''
> +    caps = orig(repo, proto)
> +    caps.append('largefiles=serve')
> +    return caps

Dropped "capabilitiesorig = None" in flight.


More information about the Mercurial-devel mailing list