diff options
author | Andres Gomez <agomez@igalia.com> | 2020-12-04 11:43:08 +0200 |
---|---|---|
committer | Andres Gomez <agomez@igalia.com> | 2020-12-17 22:32:40 +0200 |
commit | 859b32a97280b0722d04f5f90cd94d0b1392304c (patch) | |
tree | 9fe3cf9f73a62aa4ec4b14fe771f0bffd9e8a480 /framework | |
parent | 902ac9d94096aa4bcf464fe8f10c35bbf80c46de (diff) |
framework/replay: rename dump image to include the checksum
Before, we were just replacing its name with the checksum. We don't
want to lose the original name so we just append the ckecksum.
Signed-off-by: Andres Gomez <agomez@igalia.com>
Acked-by: Eric Anholt <eric@anholt.net>
Reviewed-by: Dylan Baker <dylan@pnwbakers.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/426>
Diffstat (limited to 'framework')
-rw-r--r-- | framework/replay/compare_replay.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/framework/replay/compare_replay.py b/framework/replay/compare_replay.py index a27ca630c..287ddaa18 100644 --- a/framework/replay/compare_replay.py +++ b/framework/replay/compare_replay.py @@ -115,8 +115,8 @@ def _check_trace(trace_path, expected_checksum): result = Result.DIFFER if result is not Result.MATCH or OPTIONS.keep_image: - image_file_dest = path.join(path.dirname(image_file), - '{}.png'.format(checksum)) + root, ext = path.splitext(image_file) + image_file_dest = '{}-{}{}'.format(root, checksum, ext) shutil.move(image_file, image_file_dest) if 'images' in json_result: json_result['images'][0]['image_render'] = image_file_dest |