Changeset 267895 in webkit


Ignore:
Timestamp:
Oct 2, 2020 1:44:34 PM (4 years ago)
Author:
Jonathan Bedard
Message:

[webkitscmpy] Remove "i" from canonical identifier
​https://bugs.webkit.org/show_bug.cgi?id=217238
<rdar://problem/69888017>

Reviewed by Dewei Zhu.

  • Scripts/libraries/webkitscmpy/webkitscmpy/commit.py:

(Commit):
(Commit.pretty_print): Remove "i" from canonical identifier, add branch point.
(Commit.repr): Remove "i" from canonical identifier.

  • Scripts/libraries/webkitscmpy/webkitscmpy/test/commit_unittest.py:

(TestCommit.test_parse_revision):
(TestCommit.test_parse_identifier):
(TestCommit.test_parse):
(TestCommit.test_pretty_print):

Location:
trunk/Tools
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/Tools/ChangeLog

    r267892 r267895  
     12020-10-02  Jonathan Bedard  <jbedard@apple.com>
     2
     3        [webkitscmpy] Remove "i" from canonical identifier
     4        https://bugs.webkit.org/show_bug.cgi?id=217238
     5        <rdar://problem/69888017>
     6
     7        Reviewed by Dewei Zhu.
     8
     9        * Scripts/libraries/webkitscmpy/webkitscmpy/commit.py:
     10        (Commit):
     11        (Commit.pretty_print): Remove "i" from canonical identifier, add branch point.
     12        (Commit.__repr__): Remove "i" from canonical identifier.
     13        * Scripts/libraries/webkitscmpy/webkitscmpy/test/commit_unittest.py:
     14        (TestCommit.test_parse_revision):
     15        (TestCommit.test_parse_identifier):
     16        (TestCommit.test_parse):
     17        (TestCommit.test_pretty_print):
     18
    1192020-10-02  Philippe Normand  <pnormand@igalia.com>
    220

  • trunk/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/__init__.py

    r267853 r267895  
    4747    )
    4848
    49 version = Version(0, 0, 7)
     49version = Version(0, 0, 8)
    5050
    5151from webkitscmpy.contributor import Contributor

  • trunk/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/commit.py

    r267815 r267895  
    3131    HASH_RE = re.compile(r'^[a-f0-9A-F]+$')
    3232    REVISION_RE = re.compile(r'^[Rr]?(?P<revision>\d+)$')
    33     IDENTIFIER_RE = re.compile(r'^[Ii]?((?P<branch_point>\d+)\.)?(?P<identifier>-?\d+)(@(?P<branch>\S+))?$')
     33    IDENTIFIER_RE = re.compile(r'^((?P<branch_point>\d+)\.)?(?P<identifier>-?\d+)(@(?P<branch>\S+))?$')
    3434    NUMBER_RE = re.compile(r'^-?\d*$')
    3535
    … …  
    196196            result += '\n'
    197197        if self.identifier:
    198             result += '    identifier: i{}'.format(self.identifier)
    199             if self.identifier:
     198            result += '    identifier: {}'.format(self.identifier)
     199            if self.branch:
    200200                result += ' on {}'.format(self.branch)
     201            if self.branch_point:
     202                result += ' branched from {}'.format(self.branch_point)
    201203            result += '\n'
    202204        if self.author:
    … …  
    214216    def __repr__(self):
    215217        if self.branch_point and self.identifier and self.branch:
    216             return 'i{}.{}@{}'.format(self.branch_point, self.identifier, self.branch)
     218            return '{}.{}@{}'.format(self.branch_point, self.identifier, self.branch)
    217219        if self.identifier and self.branch:
    218             return 'i{}@{}'.format(self.identifier, self.branch)
     220            return '{}@{}'.format(self.identifier, self.branch)
    219221        if self.revision:
    220222            return 'r{}'.format(self.revision)
    … …  
    222224            return self.hash[:12]
    223225        if self.identifier:
    224             return 'i{}'.format(self.identifier)
     226            return str(self.identifier)
    225227        raise ValueError('Incomplete commit format')
    226228

  • trunk/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/test/commit_unittest.py

    r267815 r267895  
    4747        self.assertEqual(266896, Commit._parse_revision(266896))
    4848
    49         self.assertEqual(None, Commit._parse_revision('i1234'))
    5049        self.assertEqual(None, Commit._parse_revision('c3bd784f8b88bd03'))
    5150        self.assertEqual(None, Commit._parse_revision('0'))
    … …  
    5554
    5655    def test_parse_identifier(self):
    57         self.assertEqual((None, 1234, None), Commit._parse_identifier('i1234'))
    58         self.assertEqual((None, 1234, None), Commit._parse_identifier('I1234'))
    5956        self.assertEqual((None, 1234, None), Commit._parse_identifier('1234'))
    6057        self.assertEqual((None, 1234, None), Commit._parse_identifier(1234))
    6158
    62         self.assertEqual((None, 1234, 'main'), Commit._parse_identifier('i1234@main'))
    63         self.assertEqual((None, 1234, 'main'), Commit._parse_identifier('I1234@main'))
    6459        self.assertEqual((None, 1234, 'main'), Commit._parse_identifier('1234@main'))
    65 
    66         self.assertEqual((None, 1234, 'eng/bug'), Commit._parse_identifier('i1234@eng/bug'))
    67         self.assertEqual((None, 1234, 'eng/bug'), Commit._parse_identifier('I1234@eng/bug'))
    6860        self.assertEqual((None, 1234, 'eng/bug'), Commit._parse_identifier('1234@eng/bug'))
    69 
    70         self.assertEqual((1234, 1, 'eng/bug'), Commit._parse_identifier('i1234.1@eng/bug'))
    71         self.assertEqual((1234, 1, 'eng/bug'), Commit._parse_identifier('I1234.1@eng/bug'))
    7261        self.assertEqual((1234, 1, 'eng/bug'), Commit._parse_identifier('1234.1@eng/bug'))
    7362
    … …  
    7564        self.assertEqual((None, -1, None), Commit._parse_identifier('-1'))
    7665
    77         self.assertEqual((None, 0, 'eng/bug'), Commit._parse_identifier('i0@eng/bug'))
    78         self.assertEqual((None, 0, 'eng/bug'), Commit._parse_identifier('I0@eng/bug'))
    7966        self.assertEqual((None, 0, 'eng/bug'), Commit._parse_identifier('0@eng/bug'))
    80 
    81         self.assertEqual((None, -1, 'eng/bug'), Commit._parse_identifier('i-1@eng/bug'))
    82         self.assertEqual((None, -1, 'eng/bug'), Commit._parse_identifier('I-1@eng/bug'))
    8367        self.assertEqual((None, -1, 'eng/bug'), Commit._parse_identifier('-1@eng/bug'))
    8468
    85         self.assertEqual(None, Commit._parse_identifier('i1234-invalid'))
     69        self.assertEqual(None, Commit._parse_identifier('1234-invalid'))
    8670        self.assertEqual(None, Commit._parse_identifier('r266896'))
    8771        self.assertEqual(None, Commit._parse_identifier('c3bd784f8b88bd03'))
    … …  
    8973
    9074    def test_parse(self):
    91         self.assertEqual(Commit.parse('i123@main'), Commit(identifier=123, branch='main'))
    92         self.assertEqual(Commit.parse('i123'), Commit(identifier=123))
     75        self.assertEqual(Commit.parse('123@main'), Commit(identifier=123, branch='main'))
    9376        self.assertEqual(Commit.parse('123'), Commit(identifier=123))
    9477
    … …  
    10386        self.assertEqual(
    10487            Commit(
    105                 identifier='i123@master',
     88                identifier='123@master',
    10689                hash='c3bd784f8b88bd03f64467ddd3304ed8be28acbe',
    10790                author=Contributor('Jonathan Bedard', ['jbedard@apple.com']),
    10891                message='NOT PRINTED',
    10992            ).pretty_print(),
    110             '''i123@master
     93            '''123@master
    11194    git hash: c3bd784f8b88 on master
    112     identifier: i123 on master
     95    identifier: 123 on master
    11396    by Jonathan Bedard <jbedard@apple.com>
    11497''',
    … …  
    117100        self.assertEqual(
    118101            Commit(
    119                 identifier='i123@trunk',
     102                identifier='123@trunk',
    120103                revision='r123',
    121104                author=Contributor('Jonathan Bedard', ['jbedard@apple.com']),
    … …  
    123106                message='NOT PRINTED',
    124107            ).pretty_print(),
    125             '''i123@trunk
     108            '''123@trunk
    126109    SVN revision: r123 on trunk
    127     identifier: i123 on trunk
     110    identifier: 123 on trunk
    128111    by Jonathan Bedard <jbedard@apple.com> @ {}
    129112'''.format(datetime.utcfromtimestamp(1000)),
    … …  
    132115        self.assertEqual(
    133116            Commit(
    134                 identifier='i123@trunk',
     117                identifier='123.1@branch-a',
     118                revision='r124',
     119                author=Contributor('Jonathan Bedard', ['jbedard@apple.com']),
     120                timestamp=1000,
     121                message='NOT PRINTED',
     122            ).pretty_print(),
     123            '''123.1@branch-a
     124    SVN revision: r124 on branch-a
     125    identifier: 1 on branch-a branched from 123
     126    by Jonathan Bedard <jbedard@apple.com> @ {}
     127'''.format(datetime.utcfromtimestamp(1000)),
     128        )
     129
     130        self.assertEqual(
     131            Commit(
     132                identifier='123@trunk',
    135133                revision='r123',
    136134                author=Contributor('Jonathan Bedard', ['jbedard@apple.com']),
    … …  
    138136                message='PRINTED\n',
    139137            ).pretty_print(message=True),
    140             '''i123@trunk
     138            '''123@trunk
    141139    SVN revision: r123 on trunk
    142     identifier: i123 on trunk
     140    identifier: 123 on trunk
    143141    by Jonathan Bedard <jbedard@apple.com> @ {}
    144142
    … …  
    148146
    149147    def test_repr(self):
    150         self.assertEqual(str(Commit(identifier=123, branch='main')), 'i123@main')
    151         self.assertEqual(str(Commit(branch_point=1234, identifier=1, branch='eng/1234')), 'i1234.1@eng/1234')
    152         self.assertEqual(str(Commit(identifier=123)), 'i123')
     148        self.assertEqual(str(Commit(identifier=123, branch='main')), '123@main')
     149        self.assertEqual(str(Commit(branch_point=1234, identifier=1, branch='eng/1234')), '1234.1@eng/1234')
     150        self.assertEqual(str(Commit(identifier=123)), '123')
    153151
    154152        self.assertEqual(str(Commit(revision=123)), 'r123')
Note: See TracChangeset for help on using the changeset viewer.