Patchwork [3,of,7,path-configs] commands.paths: use ui.paths to show paths

login
register
mail settings
Submitter Gregory Szorc
Date Feb. 8, 2015, 1:13 a.m.
Message ID <0f2bea2665040e626a7b.1423358019@vm-ubuntu-main.gateway.sonic.net>
Download mbox | patch
Permalink /patch/7764/
State Superseded
Commit a330660b3d71da0adbb4508c32e64e31feeb18f1
Headers show

Comments

Gregory Szorc - Feb. 8, 2015, 1:13 a.m.
# HG changeset patch
# User Gregory Szorc <gregory.szorc@gmail.com>
# Date 1423342170 28800
#      Sat Feb 07 12:49:30 2015 -0800
# Node ID 0f2bea2665040e626a7b6d039b698def3bf58dd4
# Parent  492682e13efa98be4640530ceed58e1f0b2adf75
commands.paths: use ui.paths to show paths

We have a new abstraction for representing paths. Use it.

Patch

diff --git a/mercurial/commands.py b/mercurial/commands.py
--- a/mercurial/commands.py
+++ b/mercurial/commands.py
@@ -4820,21 +4820,22 @@  def paths(ui, repo, search=None):
 
     Returns 0 on success.
     """
     if search:
-        for name, path in ui.configitems("paths"):
-            if name == search:
-                ui.status("%s\n" % util.hidepassword(path))
+        for path in ui.paths:
+            if path.name == search:
+                ui.status("%s\n" % util.hidepassword(path.url))
                 return
         if not ui.quiet:
             ui.warn(_("not found!\n"))
         return 1
     else:
-        for name, path in ui.configitems("paths"):
+        for path in ui.paths:
             if ui.quiet:
-                ui.write("%s\n" % name)
+                ui.write("%s\n" % path.name)
             else:
-                ui.write("%s = %s\n" % (name, util.hidepassword(path)))
+                ui.write("%s = %s\n" % (path.name,
+                                        util.hidepassword(path.url)))
 
 @command('phase',
     [('p', 'public', False, _('set changeset phase to public')),
      ('d', 'draft', False, _('set changeset phase to draft')),