diff options
| author | Jonathan Herman <hermanjl@cs.unc.edu> | 2012-10-19 02:58:19 -0400 |
|---|---|---|
| committer | Jonathan Herman <hermanjl@cs.unc.edu> | 2012-10-19 02:58:19 -0400 |
| commit | 6d830d55ccae53dca6f0338dfee1274312c93161 (patch) | |
| tree | 9bef529065ed8f0c5446606279e8d6928bcbdf9d /experiment | |
| parent | d66aa52d719cf7edad8cac20b711e4c16d2899de (diff) | |
Better error handling.
Diffstat (limited to 'experiment')
| -rw-r--r-- | experiment/executable/executable.py | 3 | ||||
| -rw-r--r-- | experiment/experiment.py | 4 |
2 files changed, 6 insertions, 1 deletions
diff --git a/experiment/executable/executable.py b/experiment/executable/executable.py index 09b7370..b964699 100644 --- a/experiment/executable/executable.py +++ b/experiment/executable/executable.py | |||
| @@ -72,3 +72,6 @@ class Executable(object): | |||
| 72 | self.sp.wait() | 72 | self.sp.wait() |
| 73 | if self.sp.returncode != 0: | 73 | if self.sp.returncode != 0: |
| 74 | print >>sys.stderr, "Non-zero return: %s %s" % (self.exec_file, " ".join(self.extra_args)) | 74 | print >>sys.stderr, "Non-zero return: %s %s" % (self.exec_file, " ".join(self.extra_args)) |
| 75 | return 0 | ||
| 76 | else: | ||
| 77 | return 1 | ||
diff --git a/experiment/experiment.py b/experiment/experiment.py index f0c201d..8c88799 100644 --- a/experiment/experiment.py +++ b/experiment/experiment.py | |||
| @@ -136,7 +136,9 @@ class Experiment(object): | |||
| 136 | ret = False | 136 | ret = False |
| 137 | 137 | ||
| 138 | self.log("Waiting for program to finish...") | 138 | self.log("Waiting for program to finish...") |
| 139 | map(methodcaller('wait'), self.executables) | 139 | for e in self.executables: |
| 140 | if not e.wait(): | ||
| 141 | ret = False | ||
| 140 | 142 | ||
| 141 | # And it must be stopped here for the same reason | 143 | # And it must be stopped here for the same reason |
| 142 | if self.overhead_trace: | 144 | if self.overhead_trace: |
