Merge ~pieq/bugit/+git/qabro:fix-1903229-acpidump-arm into bugit:master

[Pierre Equoy]

See linked bug and commit message for more info.

The original code raised an exception within qabro resulting in no logs being attached.

[Vic Liu (zongminl)]

lgtm +1

review: Approve

diff --git a/qabro/bug_assistant.py b/qabro/bug_assistant.py
index f8630e4..7af347c 100644
--- a/qabro/bug_assistant.py
+++ b/qabro/bug_assistant.py
@@ -340,8 +340,14 @@ class AttachmentAssistant:
         p = os.getenv('PATH')
         if p:
             command = ["sudo", "env", "PATH={}".format(p), "acpidump-acpica"]
-            output = subprocess.check_output(command)
-            self.attachments['acpidump.log'] = output
+            process = subprocess.run(command,
+                                     stdout=subprocess.PIPE,
+                                     stderr=subprocess.PIPE)
+            if process.returncode == 0:
+                self.attachments['acpidump.log'] = process.stdout
+            else:
+                print("Error while running acpidump:")
+                print(process.stderr)
         else:
             print("$PATH missing. Cannot run acpidump!")