The simplest way to fix this would be to just change 'make check' back out of using subunit output.
Pros:
* the failure email will be more concise
* the subunit stream seems reasonably often not to be able to be parsed by subunit so it's not reliably useful
* other people running make check will not be surprised by reams of output
* this will avoid some complexity about checking the output that's sometimes caused pqm to fail open
Cons:
* perhaps people do like subunit? (I like the idea in general but I don't find it very useful with pqm).
* changing it might break something in pqm
The simplest way to fix this would be to just change 'make check' back out of using subunit output.
Pros:
* the failure email will be more concise
* the subunit stream seems reasonably often not to be able to be parsed by subunit so it's not reliably useful
* other people running make check will not be surprised by reams of output
* this will avoid some complexity about checking the output that's sometimes caused pqm to fail open
Cons:
* perhaps people do like subunit? (I like the idea in general but I don't find it very useful with pqm).
* changing it might break something in pqm