/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
5967.9.3 by Martin Pool
Explicitly use lazy_regexp where we count on its error reporting behaviour
1
# Copyright (C) 2006-2011 Canonical Ltd
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
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
4183.7.1 by Sabin Iacob
update FSF mailing address
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
16
17
"""Tools for converting globs to regular expressions.
18
19
This module provides functions for converting shell-like globs to regular
20
expressions.
21
"""
22
6379.6.7 by Jelmer Vernooij
Move importing from future until after doc string, otherwise the doc string will disappear.
23
from __future__ import absolute_import
24
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
25
import re
26
6624 by Jelmer Vernooij
Merge Python3 porting work ('py3 pokes')
27
from . import (
5967.9.3 by Martin Pool
Explicitly use lazy_regexp where we count on its error reporting behaviour
28
    lazy_regex,
29
    )
6624 by Jelmer Vernooij
Merge Python3 porting work ('py3 pokes')
30
from .trace import (
5326.2.7 by Parth Malwankar
Globster now mutters regex failure message before changing message
31
    mutter,
5326.2.1 by Parth Malwankar
added InvalidPattern error.
32
    warning,
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
33
    )
34
35
36
class Replacer(object):
37
    """Do a multiple-pattern substitution.
38
39
    The patterns and substitutions are combined into one, so the result of
40
    one replacement is never substituted again. Add the patterns and
41
    replacements via the add method and then call the object. The patterns
42
    must not contain capturing groups.
43
    """
44
6789.1.1 by Martin
Avoid combined ur string prefix when globbing
45
    _expand = lazy_regex.lazy_compile(u'\\\\&')
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
46
47
    def __init__(self, source=None):
48
        self._pat = None
49
        if source:
50
            self._pats = list(source._pats)
51
            self._funs = list(source._funs)
52
        else:
53
            self._pats = []
54
            self._funs = []
55
56
    def add(self, pat, fun):
57
        r"""Add a pattern and replacement.
58
59
        The pattern must not contain capturing groups.
60
        The replacement might be either a string template in which \& will be
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
61
        replaced with the match, or a function that will get the matching text
62
        as argument. It does not get match object, because capturing is
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
63
        forbidden anyway.
64
        """
65
        self._pat = None
66
        self._pats.append(pat)
67
        self._funs.append(fun)
68
69
    def add_replacer(self, replacer):
70
        r"""Add all patterns from another replacer.
71
72
        All patterns and replacements from replacer are appended to the ones
73
        already defined.
74
        """
75
        self._pat = None
76
        self._pats.extend(replacer._pats)
77
        self._funs.extend(replacer._funs)
78
79
    def __call__(self, text):
80
        if not self._pat:
5967.9.5 by Martin Pool
More explicit laziness
81
            self._pat = lazy_regex.lazy_compile(
7143.15.2 by Jelmer Vernooij
Run autopep8.
82
                u'|'.join([u'(%s)' % p for p in self._pats]),
83
                re.UNICODE)
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
84
        return self._pat.sub(self._do_sub, text)
85
86
    def _do_sub(self, m):
87
        fun = self._funs[m.lastindex - 1]
88
        if hasattr(fun, '__call__'):
89
            return fun(m.group(0))
90
        else:
91
            return self._expand.sub(m.group(0), fun)
92
93
94
_sub_named = Replacer()
6789.1.1 by Martin
Avoid combined ur string prefix when globbing
95
_sub_named.add(r'\[:digit:\]', r'\d')
96
_sub_named.add(r'\[:space:\]', r'\s')
97
_sub_named.add(r'\[:alnum:\]', r'\w')
98
_sub_named.add(r'\[:ascii:\]', r'\0-\x7f')
99
_sub_named.add(r'\[:blank:\]', r' \t')
100
_sub_named.add(r'\[:cntrl:\]', r'\0-\x1f\x7f-\x9f')
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
101
102
103
def _sub_group(m):
104
    if m[1] in (u'!', u'^'):
105
        return u'[^' + _sub_named(m[2:-1]) + u']'
106
    return u'[' + _sub_named(m[1:-1]) + u']'
107
108
109
def _invalid_regex(repl):
110
    def _(m):
2135.2.7 by Kent Gibson
Implement JAM's review suggestions.
111
        warning(u"'%s' not allowed within a regular expression. "
112
                "Replacing with '%s'" % (m, repl))
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
113
        return repl
114
    return _
115
116
2298.8.1 by Kent Gibson
Normalise ignore patterns to use '/' path separator.
117
def _trailing_backslashes_regex(m):
2298.8.2 by Kent Gibson
Review fixes for lp86451 patch.
118
    """Check trailing backslashes.
119
120
    Does a head count on trailing backslashes to ensure there isn't an odd
121
    one on the end that would escape the brackets we wrap the RE in.
122
    """
123
    if (len(m) % 2) != 0:
2298.8.1 by Kent Gibson
Normalise ignore patterns to use '/' path separator.
124
        warning(u"Regular expressions cannot end with an odd number of '\\'. "
125
                "Dropping the final '\\'.")
126
        return m[:-1]
127
    return m
128
129
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
130
_sub_re = Replacer()
131
_sub_re.add(u'^RE:', u'')
6798.1.1 by Jelmer Vernooij
Properly escape backslashes.
132
_sub_re.add(u'\\((?!\\?)', u'(?:')
133
_sub_re.add(u'\\(\\?P<.*>', _invalid_regex(u'(?:'))
134
_sub_re.add(u'\\(\\?P=[^)]*\\)', _invalid_regex(u''))
6789.1.1 by Martin
Avoid combined ur string prefix when globbing
135
_sub_re.add(r'\\+$', _trailing_backslashes_regex)
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
136
137
2135.2.2 by Kent Gibson
Ignore pattern matcher (glob.py) patches:
138
_sub_fullpath = Replacer()
7143.15.2 by Jelmer Vernooij
Run autopep8.
139
_sub_fullpath.add(r'^RE:.*', _sub_re)  # RE:<anything> is a regex
140
_sub_fullpath.add(r'\[\^?\]?(?:[^][]|\[:[^]]+:\])+\]',
141
                  _sub_group)  # char group
142
_sub_fullpath.add(r'(?:(?<=/)|^)(?:\.?/)+', u'')  # canonicalize path
143
_sub_fullpath.add(r'\\.', r'\&')  # keep anything backslashed
144
_sub_fullpath.add(r'[(){}|^$+.]', r'\\&')  # escape specials
145
_sub_fullpath.add(r'(?:(?<=/)|^)\*\*+/', r'(?:.*/)?')  # **/ after ^ or /
146
_sub_fullpath.add(r'\*+', r'[^/]*')  # * elsewhere
147
_sub_fullpath.add(r'\?', r'[^/]')  # ? everywhere
2135.2.2 by Kent Gibson
Ignore pattern matcher (glob.py) patches:
148
149
150
_sub_basename = Replacer()
7143.15.2 by Jelmer Vernooij
Run autopep8.
151
_sub_basename.add(r'\[\^?\]?(?:[^][]|\[:[^]]+:\])+\]',
152
                  _sub_group)  # char group
153
_sub_basename.add(r'\\.', r'\&')  # keep anything backslashed
154
_sub_basename.add(r'[(){}|^$+.]', r'\\&')  # escape specials
155
_sub_basename.add(r'\*+', r'.*')  # * everywhere
156
_sub_basename.add(r'\?', r'.')  # ? everywhere
2135.2.2 by Kent Gibson
Ignore pattern matcher (glob.py) patches:
157
158
159
def _sub_extension(pattern):
160
    return _sub_basename(pattern[2:])
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
161
162
163
class Globster(object):
164
    """A simple wrapper for a set of glob patterns.
165
166
    Provides the capability to search the patterns to find a match for
167
    a given filename (including the full path).
168
169
    Patterns are translated to regular expressions to expidite matching.
170
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
171
    The regular expressions for multiple patterns are aggregated into
172
    a super-regex containing groups of up to 99 patterns.
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
173
    The 99 limitation is due to the grouping limit of the Python re module.
174
    The resulting super-regex and associated patterns are stored as a list of
175
    (regex,[patterns]) in _regex_patterns.
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
176
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
177
    For performance reasons the patterns are categorised as extension patterns
178
    (those that match against a file extension), basename patterns
179
    (those that match against the basename of the filename),
180
    and fullpath patterns (those that match against the full path).
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
181
    The translations used for extensions and basenames are relatively simpler
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
182
    and therefore faster to perform than the fullpath patterns.
183
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
184
    Also, the extension patterns are more likely to find a match and
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
185
    so are matched first, then the basename patterns, then the fullpath
186
    patterns.
187
    """
5050.14.2 by Parth Malwankar
_add_patterns is now done in a specific order in Globster
188
    # We want to _add_patterns in a specific order (as per type_list below)
189
    # starting with the shortest and going to the longest.
190
    # As some Python version don't support ordered dicts the list below is
191
    # used to select inputs for _add_pattern in a specific order.
7143.15.2 by Jelmer Vernooij
Run autopep8.
192
    pattern_types = ["extension", "basename", "fullpath"]
5050.14.3 by Parth Malwankar
use dict for managining pattern information
193
194
    pattern_info = {
6809.1.1 by Martin
Apply 2to3 ws_comma fixer
195
        "extension": {
7143.15.2 by Jelmer Vernooij
Run autopep8.
196
            "translator": _sub_extension,
197
            "prefix": r'(?:.*/)?(?!.*/)(?:.*\.)'
5050.14.3 by Parth Malwankar
use dict for managining pattern information
198
        },
6809.1.1 by Martin
Apply 2to3 ws_comma fixer
199
        "basename": {
7143.15.2 by Jelmer Vernooij
Run autopep8.
200
            "translator": _sub_basename,
201
            "prefix": r'(?:.*/)?(?!.*/)'
5050.14.3 by Parth Malwankar
use dict for managining pattern information
202
        },
6809.1.1 by Martin
Apply 2to3 ws_comma fixer
203
        "fullpath": {
7143.15.2 by Jelmer Vernooij
Run autopep8.
204
            "translator": _sub_fullpath,
205
            "prefix": r''
5050.14.3 by Parth Malwankar
use dict for managining pattern information
206
        },
5339.3.1 by Parth Malwankar
'bzr ignore' now fails on bad pattern.
207
    }
208
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
209
    def __init__(self, patterns):
210
        self._regex_patterns = []
5339.3.1 by Parth Malwankar
'bzr ignore' now fails on bad pattern.
211
        pattern_lists = {
6809.1.1 by Martin
Apply 2to3 ws_comma fixer
212
            "extension": [],
213
            "basename": [],
214
            "fullpath": [],
5339.3.1 by Parth Malwankar
'bzr ignore' now fails on bad pattern.
215
        }
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
216
        for pat in patterns:
2298.8.1 by Kent Gibson
Normalise ignore patterns to use '/' path separator.
217
            pat = normalize_pattern(pat)
5339.3.1 by Parth Malwankar
'bzr ignore' now fails on bad pattern.
218
            pattern_lists[Globster.identify(pat)].append(pat)
5050.14.3 by Parth Malwankar
use dict for managining pattern information
219
        pi = Globster.pattern_info
220
        for t in Globster.pattern_types:
221
            self._add_patterns(pattern_lists[t], pi[t]["translator"],
7143.15.2 by Jelmer Vernooij
Run autopep8.
222
                               pi[t]["prefix"])
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
223
224
    def _add_patterns(self, patterns, translator, prefix=''):
225
        while patterns:
5967.9.3 by Martin Pool
Explicitly use lazy_regexp where we count on its error reporting behaviour
226
            grouped_rules = [
227
                '(%s)' % translator(pat) for pat in patterns[:99]]
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
228
            joined_rule = '%s(?:%s)$' % (prefix, '|'.join(grouped_rules))
5967.9.3 by Martin Pool
Explicitly use lazy_regexp where we count on its error reporting behaviour
229
            # Explicitly use lazy_compile here, because we count on its
230
            # nicer error reporting.
231
            self._regex_patterns.append((
232
                lazy_regex.lazy_compile(joined_rule, re.UNICODE),
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
233
                patterns[:99]))
234
            patterns = patterns[99:]
235
236
    def match(self, filename):
237
        """Searches for a pattern that matches the given filename.
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
238
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
239
        :return A matching pattern or None if there is no matching pattern.
240
        """
5326.2.1 by Parth Malwankar
added InvalidPattern error.
241
        try:
242
            for regex, patterns in self._regex_patterns:
243
                match = regex.match(filename)
244
                if match:
7143.15.2 by Jelmer Vernooij
Run autopep8.
245
                    return patterns[match.lastindex - 1]
6729.3.1 by Jelmer Vernooij
Move lazy regex error to breezy.lazy_regex.
246
        except lazy_regex.InvalidPattern as e:
5339.1.1 by Parth Malwankar
fixes errors.InvalidPattern to work on Python2.5
247
            # We can't show the default e.msg to the user as thats for
5326.2.1 by Parth Malwankar
added InvalidPattern error.
248
            # the combined pattern we sent to regex. Instead we indicate to
249
            # the user that an ignore file needs fixing.
5339.1.1 by Parth Malwankar
fixes errors.InvalidPattern to work on Python2.5
250
            mutter('Invalid pattern found in regex: %s.', e.msg)
7290.2.1 by Jelmer Vernooij
Update references to home location.
251
            e.msg = (
252
                "File ~/.config/breezy/ignore or "
253
                ".bzrignore contains error(s).")
5339.3.2 by Parth Malwankar
Globster now prints specific patterns that are bad.
254
            bad_patterns = ''
255
            for _, patterns in self._regex_patterns:
256
                for p in patterns:
257
                    if not Globster.is_pattern_valid(p):
258
                        bad_patterns += ('\n  %s' % p)
5050.14.1 by Parth Malwankar
'bzr ignore' now fails on bad patterns. failing patterns are displayed.
259
            e.msg += bad_patterns
5326.2.1 by Parth Malwankar
added InvalidPattern error.
260
            raise e
2135.2.1 by Kent Gibson
Added glob module to replace broken fnmatch based ignore pattern matching (#57637)
261
        return None
3398.1.1 by Ian Clatworthy
simplify the custom Globster to only care about ordering
262
5339.3.1 by Parth Malwankar
'bzr ignore' now fails on bad pattern.
263
    @staticmethod
264
    def identify(pattern):
265
        """Returns pattern category.
266
267
        :param pattern: normalized pattern.
268
        Identify if a pattern is fullpath, basename or extension
269
        and returns the appropriate type.
270
        """
271
        if pattern.startswith(u'RE:') or u'/' in pattern:
5050.14.3 by Parth Malwankar
use dict for managining pattern information
272
            return "fullpath"
5339.3.1 by Parth Malwankar
'bzr ignore' now fails on bad pattern.
273
        elif pattern.startswith(u'*.'):
5050.14.3 by Parth Malwankar
use dict for managining pattern information
274
            return "extension"
5339.3.1 by Parth Malwankar
'bzr ignore' now fails on bad pattern.
275
        else:
5050.14.3 by Parth Malwankar
use dict for managining pattern information
276
            return "basename"
5339.3.1 by Parth Malwankar
'bzr ignore' now fails on bad pattern.
277
278
    @staticmethod
279
    def is_pattern_valid(pattern):
280
        """Returns True if pattern is valid.
281
282
        :param pattern: Normalized pattern.
283
        is_pattern_valid() assumes pattern to be normalized.
284
        see: globbing.normalize_pattern
285
        """
286
        result = True
7143.15.2 by Jelmer Vernooij
Run autopep8.
287
        translator = Globster.pattern_info[Globster.identify(
288
            pattern)]["translator"]
5339.3.1 by Parth Malwankar
'bzr ignore' now fails on bad pattern.
289
        tpattern = '(%s)' % translator(pattern)
290
        try:
5967.9.5 by Martin Pool
More explicit laziness
291
            re_obj = lazy_regex.lazy_compile(tpattern, re.UNICODE)
7143.15.2 by Jelmer Vernooij
Run autopep8.
292
            re_obj.search("")  # force compile
7143.15.5 by Jelmer Vernooij
More PEP8 fixes.
293
        except lazy_regex.InvalidPattern:
5339.3.1 by Parth Malwankar
'bzr ignore' now fails on bad pattern.
294
            result = False
295
        return result
296
297
4948.5.7 by John Whitley
Terminology change: exclusion => exception.
298
class ExceptionGlobster(object):
299
    """A Globster that supports exception patterns.
7143.15.2 by Jelmer Vernooij
Run autopep8.
300
4948.5.7 by John Whitley
Terminology change: exclusion => exception.
301
    Exceptions are ignore patterns prefixed with '!'.  Exception
7195.5.1 by Martin
Fix remaining whitespace lint in codebase
302
    patterns take precedence over regular patterns and cause a
303
    matching filename to return None from the match() function.
304
    Patterns using a '!!' prefix are highest precedence, and act
4948.5.7 by John Whitley
Terminology change: exclusion => exception.
305
    as regular ignores. '!!' patterns are useful to establish ignores
306
    that apply under paths specified by '!' exception patterns.
4948.5.5 by John Whitley
Add descriptive text to ExcludingGlobster.
307
    """
7143.15.2 by Jelmer Vernooij
Run autopep8.
308
6809.1.1 by Martin
Apply 2to3 ws_comma fixer
309
    def __init__(self, patterns):
4948.5.7 by John Whitley
Terminology change: exclusion => exception.
310
        ignores = [[], [], []]
4948.5.3 by John Whitley
Refactor the exclusion handling functionality out of
311
        for p in patterns:
4948.5.6 by John Whitley
A trial implementation of '!!' syntax for double-negative ignore exclusions.
312
            if p.startswith(u'!!'):
4948.5.7 by John Whitley
Terminology change: exclusion => exception.
313
                ignores[2].append(p[2:])
4948.5.6 by John Whitley
A trial implementation of '!!' syntax for double-negative ignore exclusions.
314
            elif p.startswith(u'!'):
4948.5.7 by John Whitley
Terminology change: exclusion => exception.
315
                ignores[1].append(p[1:])
4948.5.3 by John Whitley
Refactor the exclusion handling functionality out of
316
            else:
4948.5.7 by John Whitley
Terminology change: exclusion => exception.
317
                ignores[0].append(p)
318
        self._ignores = [Globster(i) for i in ignores]
7143.15.2 by Jelmer Vernooij
Run autopep8.
319
4948.5.3 by John Whitley
Refactor the exclusion handling functionality out of
320
    def match(self, filename):
321
        """Searches for a pattern that matches the given filename.
322
323
        :return A matching pattern or None if there is no matching pattern.
324
        """
4948.5.7 by John Whitley
Terminology change: exclusion => exception.
325
        double_neg = self._ignores[2].match(filename)
4948.5.6 by John Whitley
A trial implementation of '!!' syntax for double-negative ignore exclusions.
326
        if double_neg:
327
            return "!!%s" % double_neg
4948.5.7 by John Whitley
Terminology change: exclusion => exception.
328
        elif self._ignores[1].match(filename):
4948.5.3 by John Whitley
Refactor the exclusion handling functionality out of
329
            return None
330
        else:
4948.5.7 by John Whitley
Terminology change: exclusion => exception.
331
            return self._ignores[0].match(filename)
3398.1.1 by Ian Clatworthy
simplify the custom Globster to only care about ordering
332
7143.15.2 by Jelmer Vernooij
Run autopep8.
333
3398.1.1 by Ian Clatworthy
simplify the custom Globster to only care about ordering
334
class _OrderedGlobster(Globster):
335
    """A Globster that keeps pattern order."""
336
337
    def __init__(self, patterns):
338
        """Constructor.
339
340
        :param patterns: sequence of glob patterns
341
        """
342
        # Note: This could be smarter by running like sequences together
343
        self._regex_patterns = []
344
        for pat in patterns:
345
            pat = normalize_pattern(pat)
5050.14.3 by Parth Malwankar
use dict for managining pattern information
346
            t = Globster.identify(pat)
347
            self._add_patterns([pat], Globster.pattern_info[t]["translator"],
7143.15.2 by Jelmer Vernooij
Run autopep8.
348
                               Globster.pattern_info[t]["prefix"])
3398.1.1 by Ian Clatworthy
simplify the custom Globster to only care about ordering
349
2298.8.1 by Kent Gibson
Normalise ignore patterns to use '/' path separator.
350
5967.9.5 by Martin Pool
More explicit laziness
351
_slashes = lazy_regex.lazy_compile(r'[\\/]+')
7143.15.2 by Jelmer Vernooij
Run autopep8.
352
353
2298.8.1 by Kent Gibson
Normalise ignore patterns to use '/' path separator.
354
def normalize_pattern(pattern):
355
    """Converts backslashes in path patterns to forward slashes.
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
356
2298.8.2 by Kent Gibson
Review fixes for lp86451 patch.
357
    Doesn't normalize regular expressions - they may contain escapes.
2298.8.1 by Kent Gibson
Normalise ignore patterns to use '/' path separator.
358
    """
4948.5.4 by John Whitley
bzrlib.globbing.normalize_pattern needed fix to avoid mangling ignore
359
    if not (pattern.startswith('RE:') or pattern.startswith('!RE:')):
4792.4.1 by Gordon Tyler
Fixed globbing.normalize_pattern to not strip '/' down to '' and normalize multiple slashes.
360
        pattern = _slashes.sub('/', pattern)
361
    if len(pattern) > 1:
362
        pattern = pattern.rstrip('/')
363
    return pattern