From 07acaa3b97157b9c29246a378d0eb8cef885f5e1 Mon Sep 17 00:00:00 2001 From: Jules Villard Date: Thu, 21 Apr 2016 02:32:47 -0700 Subject: [PATCH] remove bogus logging passing `None` to `utils.stdout()` Summary:`exc.output` can be (always is?) `None`. Other places in the code only print using `traceback.print_exc()` and this was the only place trying to print this extra info (`git grep CalledProcessError`). This caused `utils.stdout()` to raise an exception, which was further confusing. closes #330 Reviewed By: jberdine Differential Revision: D3203896 fb-gh-sync-id: d2988d8 fbshipit-source-id: d2988d8 --- infer/lib/python/inferlib/capture/xcodebuild.py | 1 - 1 file changed, 1 deletion(-) diff --git a/infer/lib/python/inferlib/capture/xcodebuild.py b/infer/lib/python/inferlib/capture/xcodebuild.py index 673a740e0..4f510f93d 100644 --- a/infer/lib/python/inferlib/capture/xcodebuild.py +++ b/infer/lib/python/inferlib/capture/xcodebuild.py @@ -75,5 +75,4 @@ class XcodebuildCapture: except subprocess.CalledProcessError as exc: if self.args.debug: traceback.print_exc() - utils.stdout(exc.output) return exc.returncode