mirror of
https://github.com/saymrwulf/pytorch.git
synced 2026-05-14 20:57:59 +00:00
Catch and ignore tracebacks for compilation errors (#55986)
Summary: The Python traceback on a cmake invocation is meaningless to most developers, so this PR wraps it in a `try..catch` so we can ignore it and save scrolling through the 20-or-so lines. Pull Request resolved: https://github.com/pytorch/pytorch/pull/55986 Pulled By: driazati Reviewed By: wanchaol Differential Revision: D27769304 fbshipit-source-id: 5889eea03db098d10576290abeeb4600029fb3f2
This commit is contained in:
parent
40d74e6f71
commit
48a7d69946
1 changed files with 8 additions and 2 deletions
|
|
@ -5,7 +5,7 @@
|
|||
import multiprocessing
|
||||
import os
|
||||
import re
|
||||
from subprocess import check_call, check_output
|
||||
from subprocess import check_call, check_output, CalledProcessError
|
||||
import sys
|
||||
import distutils.sysconfig
|
||||
from distutils.version import LooseVersion
|
||||
|
|
@ -137,7 +137,13 @@ class CMake:
|
|||
|
||||
command = [self._cmake_command] + args
|
||||
print(' '.join(command))
|
||||
check_call(command, cwd=self.build_dir, env=env)
|
||||
try:
|
||||
check_call(command, cwd=self.build_dir, env=env)
|
||||
except (CalledProcessError, KeyboardInterrupt) as e:
|
||||
# This error indicates that there was a problem with cmake, the
|
||||
# Python backtrace adds no signal here so skip over it by catching
|
||||
# the error and exiting manually
|
||||
sys.exit(1)
|
||||
|
||||
@staticmethod
|
||||
def defines(args, **kwargs):
|
||||
|
|
|
|||
Loading…
Reference in a new issue