On 19/11/24 16:05, Daniel P. Berrangé wrote:
The missing directory separator resulted in the kernel file being
created 1 level higher than expected.
Signed-off-by: Daniel P. Berrangé <berra...@redhat.com>
---
tests/functional/test_mips64el_malta.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/functional/test_mips64el_malta.py
b/tests/functional/test_mips64el_malta.py
index 6c6355b131..24ebcdb9c1 100755
--- a/tests/functional/test_mips64el_malta.py
+++ b/tests/functional/test_mips64el_malta.py
@@ -129,7 +129,7 @@ def do_test_i6400_framebuffer_logo(self, cpu_cores_count):
screendump_path = os.path.join(self.workdir, 'screendump.pbm')
kernel_path_gz = self.ASSET_KERNEL_4_7_0.fetch()
- kernel_path = self.workdir + "vmlinux"
+ kernel_path = self.workdir + "/vmlinux"
Safer is to use os.path.join(), anyway:
Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>
gzip_uncompress(kernel_path_gz, kernel_path)
tuxlogo_path = self.ASSET_TUXLOGO.fetch()
