Patchwork [05,of,24] commandserver: report capabilities sorted

login
register
mail settings
Submitter Mads Kiilerich
Date Dec. 16, 2012, 10:34 p.m.
Message ID <cba5d81f70e88ebd6647.1355697240@localhost6.localdomain6>
Download mbox | patch
Permalink /patch/129/
State Accepted
Commit 4b09e6f7d7de2352e975a49ecf1f87ad2dfad3d8
Headers show

Comments

Mads Kiilerich - Dec. 16, 2012, 10:34 p.m.
# HG changeset patch
# User Mads Kiilerich <mads at kiilerich.com>
# Date 1355276294 -3600
# Node ID cba5d81f70e88ebd6647ce7d67467da6c3cd31b1
# Parent  98197a95d1d11271802d2afbf471909977483649
commandserver: report capabilities sorted
Bryan O'Sullivan - Dec. 17, 2012, 8:56 p.m.
On Sun, Dec 16, 2012 at 2:34 PM, Mads Kiilerich <mads at kiilerich.com> wrote:

> commandserver: report capabilities sorted
>

Looks good to me.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20121217/33956f68/attachment.html>

Patch

diff --git a/mercurial/commandserver.py b/mercurial/commandserver.py
--- a/mercurial/commandserver.py
+++ b/mercurial/commandserver.py
@@ -220,7 +220,7 @@ 
                     'getencoding' : getencoding}
 
     def serve(self):
-        hellomsg = 'capabilities: ' + ' '.join(self.capabilities.keys())
+        hellomsg = 'capabilities: ' + ' '.join(sorted(self.capabilities))
         hellomsg += '\n'
         hellomsg += 'encoding: ' + encoding.encoding