/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/ignores.py

  • Committer: Robert Collins
  • Date: 2008-01-06 20:04:22 UTC
  • mto: (3221.11.1 StackableBranch)
  • mto: This revision was merged to the branch mainline in revision 3226.
  • Revision ID: robertc@robertcollins.net-20080106200422-x8yz6cxotlzltvwp
The bzrdir format registry now accepts an ``alias`` keyword to
register_metadir, used to indicate that a format name is an alias for
some other format and thus should not be reported when describing the
format. (Robert Collins)
-------------- This line and the fmllowing will be ignored --------------

modified:
  NEWS
  bzrlib/bzrdir.py
  bzrlib/info.py
  bzrlib/tests/test_bzrdir.py
  bzrlib/tests/test_info.py

=== modified file 'NEWS'
--- a/NEWS      2008-01-02 22:30:46 +0000
+++ b/NEWS      2008-01-06 20:04:15 +0000
@@ -135,6 +135,11 @@
     * Patience Diff now supports arbitrary python objects, as long as they
       support ``hash()``. (John Arbash Meinel)
 
+    * The bzrdir format registry now accepts an ``alias`` keyword to
+      register_metadir, used to indicate that a format name is an alias for
+      some other format and thus should not be reported when describing the
+      format. (Robert Collins)
+
   API BREAKS:
 
   TESTING:

=== modified file 'bzrlib/bzrdir.py'
--- a/bzrlib/bzrdir.py  2008-01-02 22:30:46 +0000
+++ b/bzrlib/bzrdir.py  2008-01-06 19:41:29 +0000
@@ -2447,12 +2447,22 @@
     e.g. BzrDirMeta1 with weave repository.  Also, it's more user-oriented.
     """
 
+    def __init__(self):
+        """Create a BzrDirFormatRegistry."""
+        self._aliases = set()
+        super(BzrDirFormatRegistry, self).__init__()
+
+    def aliases(self):
+        """Return a set of the format names which are aliases."""
+        return frozenset(self._aliases)
+
     def register_metadir(self, key,
              repository_format, help, native=True, deprecated=False,
              branch_format=None,
              tree_format=None,
              hidden=False,
-             experimental=False):
+             experimental=False,
+             alias=False):
         """Register a metadir subformat.
 
         These all use a BzrDirMetaFormat1 bzrdir, but can be parameterized
@@ -2491,10 +2501,10 @@
                 bd.repository_format = _load(repository_format)
             return bd
         self.register(key, helper, help, native, deprecated, hidden,
-            experimental)
+            experimental, alias)
 
     def register(self, key, factory, help, native=True, deprecated=False,
-                 hidden=False, experimental=False):
+                 hidden=False, experimental=False, alias=False):
         """Register a BzrDirFormat factory.
         
         The factory must be a callable that takes one parameter: the key.
@@ -2505,11 +2515,15 @@
         """
         registry.Registry.register(self, key, factory, help,
             BzrDirFormatInfo(native, deprecated, hidden, experimental))
+        if alias:
+            self._aliases.add(key)
 
     def register_lazy(self, key, module_name, member_name, help, native=True,
-                      deprecated=False, hidden=False, experimental=False):
+        deprecated=False, hidden=False, experimental=False, alias=False):
         registry.Registry.register_lazy(self, key, module_name, member_name,
             help, BzrDirFormatInfo(native, deprecated, hidden, experimental))
+        if alias:
+            self._aliases.add(key)
 
     def set_default(self, key):
         """Set the 'default' key to be a clone of the supplied key.
@@ -2518,6 +2532,7 @@
         """
         registry.Registry.register(self, 'default', self.get(key),
             self.get_help(key), info=self.get_info(key))
+        self._aliases.add('default')
 
     def set_default_repository(self, key):
         """Set the FormatRegistry default and Repository default.
@@ -2670,6 +2685,7 @@
     tree_format='bzrlib.workingtree.WorkingTreeFormat4',
     hidden=False,
     )
+# The following two formats should always just be aliases.
 format_registry.register_metadir('development',
     'bzrlib.repofmt.pack_repo.RepositoryFormatPackDevelopment0',
     help='Current development format. Can convert data to and from pack-0.92 '
@@ -2681,6 +2697,7 @@
     branch_format='bzrlib.branch.BzrBranchFormat6',
     tree_format='bzrlib.workingtree.WorkingTreeFormat4',
     experimental=True,
+    alias=True,
     )
 format_registry.register_metadir('development-subtree',
     'bzrlib.repofmt.pack_repo.RepositoryFormatPackDevelopment0Subtree',
@@ -2693,7 +2710,9 @@
     branch_format='bzrlib.branch.BzrBranchFormat6',
     tree_format='bzrlib.workingtree.WorkingTreeFormat4',
     experimental=True,
+    alias=True,
     )
+# And the development formats which the will have aliased one of follow:
 format_registry.register_metadir('development0',
     'bzrlib.repofmt.pack_repo.RepositoryFormatPackDevelopment0',
     help='Trivial rename of pack-0.92 to provide a development format. '

=== modified file 'bzrlib/info.py'
--- a/bzrlib/info.py    2007-11-06 09:00:25 +0000
+++ b/bzrlib/info.py    2008-01-06 20:01:30 +0000
@@ -440,7 +440,9 @@
         tree.bzrdir.root_transport.base):
         branch = None
         repository = None
-    for key in bzrdir.format_registry.keys():
+    non_aliases = set(bzrdir.format_registry.keys())
+    non_aliases.difference_update(bzrdir.format_registry.aliases())
+    for key in non_aliases:
         format = bzrdir.format_registry.make_bzrdir(key)
         if isinstance(format, bzrdir.BzrDirMetaFormat1):
             if (tree and format.workingtree_format !=
@@ -457,11 +459,12 @@
         candidates.append(key)
     if len(candidates) == 0:
         return 'unnamed'
-    new_candidates = [c for c in candidates if c != 'default']
-    if len(new_candidates) > 0:
-        candidates = new_candidates
+    candidates.sort()
     new_candidates = [c for c in candidates if not
         bzrdir.format_registry.get_info(c).hidden]
     if len(new_candidates) > 0:
+        # If there are any non-hidden formats that match, only return those to
+        # avoid listing hidden formats except when only a hidden format will
+        # do.
         candidates = new_candidates
     return ' or '.join(candidates)

=== modified file 'bzrlib/tests/test_bzrdir.py'
--- a/bzrlib/tests/test_bzrdir.py       2007-12-21 20:32:22 +0000
+++ b/bzrlib/tests/test_bzrdir.py       2008-01-06 19:45:00 +0000
@@ -170,6 +170,16 @@
         finally:
             bzrdir.format_registry.set_default_repository(old_default)
 
+    def test_aliases(self):
+        a_registry = bzrdir.BzrDirFormatRegistry()
+        a_registry.register('weave', bzrdir.BzrDirFormat6,
+            'Pre-0.8 format.  Slower and does not support checkouts or shared'
+            ' repositories', deprecated=True)
+        a_registry.register('weavealias', bzrdir.BzrDirFormat6,
+            'Pre-0.8 format.  Slower and does not support checkouts or shared'
+            ' repositories', deprecated=True, alias=True)
+        self.assertEqual(frozenset(['weavealias']), a_registry.aliases())
+    
 
 class SampleBranch(bzrlib.branch.Branch):
     """A dummy branch for guess what, dummy use."""

=== modified file 'bzrlib/tests/test_info.py'
--- a/bzrlib/tests/test_info.py 2007-11-26 13:55:51 +0000
+++ b/bzrlib/tests/test_info.py 2008-01-06 20:02:10 +0000
@@ -126,16 +126,22 @@
 
     def test_describe_tree_format(self):
         for key in bzrdir.format_registry.keys():
-            if key == 'default':
+            if key in bzrdir.format_registry.aliases():
                 continue
             self.assertTreeDescription(key)
 
     def test_describe_checkout_format(self):
         for key in bzrdir.format_registry.keys():
-            if key in ('default', 'weave', 'experimental'):
-                continue
-            if key.startswith('experimental-'):
-                # these are typically hidden or aliases for other formats
+            if key in bzrdir.format_registry.aliases():
+                # Aliases will not describe correctly in the UI because the
+                # real format is found.
+                continue
+            # legacy: weave does not support checkouts
+            if key == 'weave':
+                continue
+            if bzrdir.format_registry.get_info(key).experimental:
+                # We don't require that experimental formats support checkouts
+                # or describe correctly in the UI.
                 continue
             expected = None
             if key in ('dirstate', 'dirstate-tags', 'dirstate-with-subtree',
@@ -149,7 +155,7 @@
 
     def test_describe_branch_format(self):
         for key in bzrdir.format_registry.keys():
-            if key == 'default':
+            if key in bzrdir.format_registry.aliases():
                 continue
             expected = None
             if key in ('dirstate', 'knit'):
@@ -158,7 +164,7 @@
 
     def test_describe_repo_format(self):
         for key in bzrdir.format_registry.keys():
-            if key == 'default':
+            if key in bzrdir.format_registry.aliases():
                 continue
             expected = None
             if key in ('dirstate', 'knit', 'dirstate-tags'):

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/ignores.py
 
1
# Copyright (C) 2005, 2006 Canonical Ltd
 
2
#
 
3
# This program is free software; you can redistribute it and/or modify
 
4
# it under the terms of the GNU General Public License as published by
 
5
# the Free Software Foundation; either version 2 of the License, or
 
6
# (at your option) any later version.
 
7
#
 
8
# This program is distributed in the hope that it will be useful,
 
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
 
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
11
# GNU General Public License for more details.
 
12
#
 
13
# You should have received a copy of the GNU General Public License
 
14
# along with this program; if not, write to the Free Software
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
16
 
 
17
"""Lists of ignore files, etc."""
 
18
 
 
19
import errno
 
20
 
 
21
from bzrlib import (
 
22
    config,
 
23
    globbing,
 
24
    )
 
25
 
 
26
# This was the full ignore list for bzr 0.8
 
27
# please keep these sorted (in C locale order) to aid merging
 
28
OLD_DEFAULTS = [
 
29
    '#*#',
 
30
    '*$',
 
31
    '*,v',
 
32
    '*.BAK',
 
33
    '*.a',
 
34
    '*.bak',
 
35
    '*.elc',
 
36
    '*.exe',
 
37
    '*.la',
 
38
    '*.lo',
 
39
    '*.o',
 
40
    '*.obj',
 
41
    '*.orig',
 
42
    '*.py[oc]',
 
43
    '*.so',
 
44
    '*.tmp',
 
45
    '*~',
 
46
    '.#*',
 
47
    '.*.sw[nop]',
 
48
    '.*.tmp',
 
49
    # Our setup tests dump .python-eggs in the bzr source tree root
 
50
    './.python-eggs',
 
51
    '.DS_Store',
 
52
    '.arch-ids',
 
53
    '.arch-inventory',
 
54
    '.bzr.log',
 
55
    '.del-*',
 
56
    '.git',
 
57
    '.hg',
 
58
    '.jamdeps'
 
59
    '.libs',
 
60
    '.make.state',
 
61
    '.sconsign*',
 
62
    '.svn',
 
63
    '.sw[nop]',    # vim editing nameless file
 
64
    '.tmp*',
 
65
    'BitKeeper',
 
66
    'CVS',
 
67
    'CVS.adm',
 
68
    'RCS',
 
69
    'SCCS',
 
70
    'TAGS',
 
71
    '_darcs',
 
72
    'aclocal.m4',
 
73
    'autom4te*',
 
74
    'config.h',
 
75
    'config.h.in',
 
76
    'config.log',
 
77
    'config.status',
 
78
    'config.sub',
 
79
    'stamp-h',
 
80
    'stamp-h.in',
 
81
    'stamp-h1',
 
82
    '{arch}',
 
83
]
 
84
 
 
85
 
 
86
# ~/.bazaar/ignore will be filled out using
 
87
# this ignore list, if it does not exist
 
88
# please keep these sorted (in C locale order) to aid merging
 
89
USER_DEFAULTS = [
 
90
    '*.a',
 
91
    '*.o',
 
92
    '*.py[co]',
 
93
    '*.so',
 
94
    '*.sw[nop]',
 
95
    '*~',
 
96
    '.#*',
 
97
    '[#]*#',
 
98
]
 
99
 
 
100
 
 
101
def parse_ignore_file(f):
 
102
    """Read in all of the lines in the file and turn it into an ignore list"""
 
103
    ignored = set()
 
104
    for line in f.read().decode('utf8').split('\n'):
 
105
        line = line.rstrip('\r\n')
 
106
        if not line or line.startswith('#'):
 
107
            continue
 
108
        ignored.add(globbing.normalize_pattern(line))
 
109
    return ignored
 
110
 
 
111
 
 
112
def get_user_ignores():
 
113
    """Get the list of user ignored files, possibly creating it."""
 
114
    path = config.user_ignore_config_filename()
 
115
    patterns = set(USER_DEFAULTS)
 
116
    try:
 
117
        f = open(path, 'rb')
 
118
    except (IOError, OSError), e:
 
119
        # open() shouldn't return an IOError without errno, but just in case
 
120
        err = getattr(e, 'errno', None)
 
121
        if err not in (errno.ENOENT,):
 
122
            raise
 
123
        # Create the ignore file, and just return the default
 
124
        # We want to ignore if we can't write to the file
 
125
        # since get_* should be a safe operation
 
126
        try:
 
127
            _set_user_ignores(USER_DEFAULTS)
 
128
        except (IOError, OSError), e:
 
129
            if e.errno not in (errno.EPERM,):
 
130
                raise
 
131
        return patterns
 
132
 
 
133
    try:
 
134
        return parse_ignore_file(f)
 
135
    finally:
 
136
        f.close()
 
137
 
 
138
 
 
139
def _set_user_ignores(patterns):
 
140
    """Fill out the user ignore file with the given patterns
 
141
 
 
142
    This may raise an error if it doesn't have permission to
 
143
    write to the user ignore file.
 
144
    This is mostly used for testing, since it would be
 
145
    bad form to rewrite a user's ignore list.
 
146
    bzrlib only writes this file if it does not exist.
 
147
    """
 
148
    ignore_path = config.user_ignore_config_filename()
 
149
    config.ensure_config_dir_exists()
 
150
 
 
151
    # Create an empty file
 
152
    f = open(ignore_path, 'wb')
 
153
    try:
 
154
        for pattern in patterns:
 
155
            f.write(pattern.encode('utf8') + '\n')
 
156
    finally:
 
157
        f.close()
 
158
 
 
159
 
 
160
def add_unique_user_ignores(new_ignores):
 
161
    """Add entries to the user's ignore list if not present.
 
162
 
 
163
    :param new_ignores: A list of ignore patterns
 
164
    :return: The list of ignores that were added
 
165
    """
 
166
    ignored = get_user_ignores()
 
167
    to_add = []
 
168
    for ignore in new_ignores:
 
169
        ignore = globbing.normalize_pattern(ignore)
 
170
        if ignore not in ignored:
 
171
            ignored.add(ignore)
 
172
            to_add.append(ignore)
 
173
 
 
174
    if not to_add:
 
175
        return []
 
176
 
 
177
    f = open(config.user_ignore_config_filename(), 'ab')
 
178
    try:
 
179
        for pattern in to_add:
 
180
            f.write(pattern.encode('utf8') + '\n')
 
181
    finally:
 
182
        f.close()
 
183
 
 
184
    return to_add
 
185
 
 
186
 
 
187
_runtime_ignores = set()
 
188
 
 
189
 
 
190
def add_runtime_ignores(ignores):
 
191
    """Add some ignore patterns that only exists in memory.
 
192
 
 
193
    This is used by some plugins that want bzr to ignore files,
 
194
    but don't want to change a users ignore list.
 
195
    (Such as a conversion script that needs to ignore temporary files,
 
196
    but does not want to modify the project's ignore list.)
 
197
 
 
198
    :param ignores: A list or generator of ignore patterns.
 
199
    :return: None
 
200
    """
 
201
    global _runtime_ignores
 
202
    _runtime_ignores.update(set(ignores))
 
203
 
 
204
 
 
205
def get_runtime_ignores():
 
206
    """Get the current set of runtime ignores."""
 
207
    return _runtime_ignores