Diff
Modified: trunk/Tools/ChangeLog (292843 => 292844)
--- trunk/Tools/ChangeLog 2022-04-13 23:24:40 UTC (rev 292843)
+++ trunk/Tools/ChangeLog 2022-04-13 23:43:01 UTC (rev 292844)
@@ -1,3 +1,21 @@
+2022-04-13 Jonathan Bedard <jbed...@apple.com>
+
+ [git-webkit] Found branch name instead of hash
+ https://bugs.webkit.org/show_bug.cgi?id=239296
+ <rdar://problem/91720224>
+
+ Reviewed by Michael Catanzaro.
+
+ * Scripts/libraries/webkitscmpy/setup.py: Bump version.
+ * Scripts/libraries/webkitscmpy/webkitscmpy/__init__.py: Ditto.
+ * Scripts/libraries/webkitscmpy/webkitscmpy/local/git.py:
+ (Git.Cache.populate): Add --no-decorate to `git log` command.
+ (Git.commit): Ditto.
+ (Git.commits): Ditto.
+ * Scripts/libraries/webkitscmpy/webkitscmpy/mocks/local/git.py:
+ * Scripts/libraries/webkitscmpy/webkitscmpy/test/git_unittest.py:
+ (TestGit.test_log): Ditto.
+
2022-04-13 Ross Kirsling <ross.kirsl...@sony.com>
Add PlayStation builds to bot watcher's dashboard.
Modified: trunk/Tools/Scripts/libraries/webkitscmpy/setup.py (292843 => 292844)
--- trunk/Tools/Scripts/libraries/webkitscmpy/setup.py 2022-04-13 23:24:40 UTC (rev 292843)
+++ trunk/Tools/Scripts/libraries/webkitscmpy/setup.py 2022-04-13 23:43:01 UTC (rev 292844)
@@ -29,7 +29,7 @@
setup(
name='webkitscmpy',
- version='4.9.1',
+ version='4.9.2',
description='Library designed to interact with git and svn repositories.',
long_description=readme(),
classifiers=[
Modified: trunk/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/__init__.py (292843 => 292844)
--- trunk/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/__init__.py 2022-04-13 23:24:40 UTC (rev 292843)
+++ trunk/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/__init__.py 2022-04-13 23:43:01 UTC (rev 292844)
@@ -46,7 +46,7 @@
"Please install webkitcorepy with `pip install webkitcorepy --extra-index-url <package index URL>`"
)
-version = Version(4, 9, 1)
+version = Version(4, 9, 2)
AutoInstall.register(Package('fasteners', Version(0, 15, 0)))
AutoInstall.register(Package('jinja2', Version(2, 11, 3)))
Modified: trunk/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/local/git.py (292843 => 292844)
--- trunk/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/local/git.py 2022-04-13 23:24:40 UTC (rev 292843)
+++ trunk/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/local/git.py 2022-04-13 23:43:01 UTC (rev 292844)
@@ -130,7 +130,7 @@
kwargs = dict(encoding='utf-8')
self._last_populated[branch] = time.time()
log = subprocess.Popen(
- [self.repo.executable(), 'log', branch],
+ [self.repo.executable(), 'log', branch, '--no-decorate'],
cwd=self.repo.root_path,
stdout=subprocess.PIPE,
stderr=subprocess.PIPE,
@@ -560,7 +560,7 @@
default_branch = self.default_branch
parsed_branch_point = None
- log_format = ['-1'] if include_log else ['-1', '--format=short']
+ log_format = ['-1', '--no-decorate'] if include_log else ['-1', '--no-decorate', '--format=short']
# Determine the `git log` output and branch for a given identifier
if identifier is not None:
@@ -743,7 +743,7 @@
try:
log = None
log = subprocess.Popen(
- [self.executable(), 'log', '--format=fuller', '{}...{}'.format(end.hash, begin.hash)],
+ [self.executable(), 'log', '--format=fuller', '--no-decorate', '{}...{}'.format(end.hash, begin.hash)],
cwd=self.root_path,
stdout=subprocess.PIPE,
stderr=subprocess.PIPE,
Modified: trunk/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/mocks/local/git.py (292843 => 292844)
--- trunk/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/mocks/local/git.py 2022-04-13 23:24:40 UTC (rev 292843)
+++ trunk/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/mocks/local/git.py 2022-04-13 23:43:01 UTC (rev 292844)
@@ -258,7 +258,7 @@
stdout='{}\n'.format(self.find(args[2]).hash),
) if self.find(args[2]) else mocks.ProcessCompletion(returncode=128)
), mocks.Subprocess.Route(
- self.executable, 'log', re.compile(r'.+'), '-1',
+ self.executable, 'log', re.compile(r'.+'), '-1', '--no-decorate',
cwd=self.path,
generator=lambda *args, **kwargs: mocks.ProcessCompletion(
returncode=0,
@@ -283,7 +283,7 @@
),
) if self.find(args[2]) else mocks.ProcessCompletion(returncode=128),
), mocks.Subprocess.Route(
- self.executable, 'log', '--format=fuller', re.compile(r'.+\.\.\..+'),
+ self.executable, 'log', '--format=fuller', '--no-decorate', re.compile(r'.+\.\.\..+'),
cwd=self.path,
generator=lambda *args, **kwargs: mocks.ProcessCompletion(
returncode=0,
@@ -306,7 +306,7 @@
os.path.basename(path),
commit.revision,
)] if git_svn else []),
- )) for commit in list(self.commits_in_range(args[3].split('...')[-1], args[3].split('...')[0]))[:-1]
+ )) for commit in list(self.commits_in_range(args[4].split('...')[-1], args[4].split('...')[0]))[:-1]
])
)
), mocks.Subprocess.Route(
Modified: trunk/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/test/git_unittest.py (292843 => 292844)
--- trunk/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/test/git_unittest.py 2022-04-13 23:24:40 UTC (rev 292843)
+++ trunk/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/test/git_unittest.py 2022-04-13 23:43:01 UTC (rev 292844)
@@ -317,7 +317,7 @@
with mocks.local.Git(self.path, git_svn=True):
self.assertEqual(
run([
- local.Git.executable(), 'log', '--format=fuller', 'remotes/origin/main...1abe25b4',
+ local.Git.executable(), 'log', '--format=fuller', '--no-decorate', 'remotes/origin/main...1abe25b4',
], cwd=self.path, capture_output=True, encoding='utf-8').stdout,
'''commit d8bce26fa65c6fc8f39c17927abb77f69fab82fc
Author: Jonathan Bedard <jbed...@apple.com>
@@ -345,7 +345,7 @@
with mocks.local.Git(self.path, git_svn=True):
self.assertEqual(
run([
- local.Git.executable(), 'log', '--format=fuller', 'branch-b...main',
+ local.Git.executable(), 'log', '--format=fuller', '--no-decorate', 'branch-b...main',
], cwd=self.path, capture_output=True, encoding='utf-8').stdout,
'''commit 790725a6d79e28db2ecdde29548d2262c0bd059d
Author: Jonathan Bedard <jbed...@apple.com>