Manouchehri / serf

High-performance asynchronous HTTP client library
https://code.google.com/p/serf
Apache License 2.0
0 stars 0 forks source link

[PATCH] scons check should return non-zero exit staths #116

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. (make a check fail)
2. scons check
3. echo $?

What is the expected output? What do you see instead?

expected: 1
actual: 0

What version of the product are you using? On what operating system?

serf trunk and 1.3.0, GNU/Linux all

Please provide any additional information below.

This is required for packages who run the tests in build systems and need some 
feedback for failed tests, e.g. rpm' %check stanza.

Original issue reported on code.google.com by andreas.stieger@gmx.de on 28 Jul 2013 at 1:15

GoogleCodeExporter commented 9 years ago

Original comment by andreas.stieger@gmx.de on 28 Jul 2013 at 1:15

Attachments:

GoogleCodeExporter commented 9 years ago
Good catch! Fixed in r2081, sys.exit added in two locations.
thanks.
Lieven

Original comment by lieven.govaerts@gmail.com on 28 Jul 2013 at 2:03