https://github.com/python/cpython/commit/b6c92ec419cfa1e4483b072996bab403c9970a5d commit: b6c92ec419cfa1e4483b072996bab403c9970a5d branch: main author: Itamar Oren <itamar...@gmail.com> committer: AA-Turner <9087854+aa-tur...@users.noreply.github.com> date: 2025-04-03T18:37:08+01:00 summary:
gh-132038: Make perf version check in test_perf_profiler more robust (#132039) Should work also if the version string includes a commit hash, like `perf version 6.12.9.g242e6068fd5c` files: M Lib/test/test_perf_profiler.py diff --git a/Lib/test/test_perf_profiler.py b/Lib/test/test_perf_profiler.py index 43e5a7e114b317..1922d8e0bab216 100644 --- a/Lib/test/test_perf_profiler.py +++ b/Lib/test/test_perf_profiler.py @@ -493,7 +493,8 @@ def compile_trampolines_for_all_functions(): def _is_perf_version_at_least(major, minor): # The output of perf --version looks like "perf version 6.7-3" but - # it can also be perf version "perf version 5.15.143" + # it can also be perf version "perf version 5.15.143", or even include + # a commit hash in the version string, like "6.12.9.g242e6068fd5c" try: output = subprocess.check_output(["perf", "--version"], text=True) except (subprocess.CalledProcessError, FileNotFoundError): @@ -501,7 +502,7 @@ def _is_perf_version_at_least(major, minor): version = output.split()[2] version = version.split("-")[0] version = version.split(".") - version = tuple(map(int, version)) + version = tuple(map(int, version[:2])) return version >= (major, minor) _______________________________________________ Python-checkins mailing list -- python-checkins@python.org To unsubscribe send an email to python-checkins-le...@python.org https://mail.python.org/mailman3/lists/python-checkins.python.org/ Member address: arch...@mail-archive.com