Skip to contentSkip to conversation

selftest/package: Correct format arguments in test_gdb_hardlink_debug

PATCH Superseded Ola x Nilsson 2 participants 3 messages Dec 11, 2018

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore

Ola x Nilsson

author
#0 lore raw

PATCH 1/1 selftest/package: Correct format arguments in test_gdb_hardlink_debug

Signed-off-by Ola x Nilsson <[email protected]>
---
 meta/lib/oeqa/selftest/cases/package.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
meta/lib/oeqa/selftest/cases/package.py +1 -1
index 0a88dc25b3..b0abc2f4ee 100644
@@ -137,7 +137,7 @@ class PackageTests(OESelftestTestCase):
                 elif "Breakpoint 1, main () at hello.c:4" in l:
                     return True
 
-            self.logger.error("GDB result:\n%s: %s" % output)
+            self.logger.error("GDB result:\n%s: %s", status, output)
             return False
 
         with runqemu('core-image-minimal') as qemu:
Signature · 1 line
2.11.0
#1 lore raw

meta/lib/oeqa/selftest/cases/package.py class PackageTests(OESelftestTestCase): · line 140

Quoted 2 lines · Ola x Nilsson
meta/lib/oeqa/selftest/cases/package.py +1 -1
-            self.logger.error("GDB result:\n%s: %s" % output)
+            self.logger.error("GDB result:\n%s: %s", status, output)

Isn't status an integer so should be %d?

Ross

author
#2 lore raw
Quoted 6 lines · Ross

meta/lib/oeqa/selftest/cases/package.py class PackageTests(OESelftestTestCase): · line 140

Quoted 2 lines · Ola x Nilsson
meta/lib/oeqa/selftest/cases/package.py +1 -1
-            self.logger.error("GDB result:\n%s: %s" % output)
+            self.logger.error("GDB result:\n%s: %s", status, output)

Isn't status an integer so should be %d?

Both seem to work, but I'll resend.

Signature · 1 line
Ola x Nilsson