diff options
author | Uri Simchoni <urisimchoni@gmail.com> | 2015-05-03 22:56:15 +0300 |
---|---|---|
committer | Andrew Bartlett <abartlet@samba.org> | 2015-05-20 11:19:11 +0200 |
commit | c244e7dc09e9c98f678cbe716fe2e640de0bd4c0 (patch) | |
tree | 75630b8201a8a7017edeb4c196965dc37bc85eb4 /buildtools | |
parent | eb696df4936bc48bb8fa7077bb5b45e5e2e6cb01 (diff) | |
download | samba-c244e7dc09e9c98f678cbe716fe2e640de0bd4c0.tar.gz |
waf: improve readability of cross-answers generated by cross-execute
When generating a result for cross-answers from the (retcode, retstring) tuple:
- (0, "output") indicated as "output"
- 1 is interpreted as generic fail code, instead of 255, because most
if not all tests fail with 1 as exit code rather than 255
- For failing test, use NO instead of FAIL, because that's not
necessarily a failure (it could mean that something is NOT
broken)
Signed-off-by: Uri Simchoni <urisimchoni@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Alexander Bokovoy <ab@samba.org>
Diffstat (limited to 'buildtools')
-rw-r--r-- | buildtools/wafsamba/samba_cross.py | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/buildtools/wafsamba/samba_cross.py b/buildtools/wafsamba/samba_cross.py index 9ca2f8d4610..8213e1718d3 100644 --- a/buildtools/wafsamba/samba_cross.py +++ b/buildtools/wafsamba/samba_cross.py @@ -6,7 +6,7 @@ from Configure import conf real_Popen = None ANSWER_UNKNOWN = (254, "") -ANSWER_FAIL = (255, "") +ANSWER_NO = (1, "") ANSWER_OK = (0, "") cross_answers_incomplete = False @@ -33,10 +33,13 @@ def add_answer(ca_file, msg, answer): f.write('%s: OK\n' % msg) elif answer == ANSWER_UNKNOWN: f.write('%s: UNKNOWN\n' % msg) - elif answer == ANSWER_FAIL: - f.write('%s: FAIL\n' % msg) + elif answer == ANSWER_NO: + f.write('%s: NO\n' % msg) else: - f.write('%s: (%d, "%s")\n' % (msg, retcode, retstring)) + if retcode == 0: + f.write('%s: "%s"\n' % (msg, retstring)) + else: + f.write('%s: (%d, "%s")\n' % (msg, retcode, retstring)) f.close() @@ -64,7 +67,7 @@ def cross_answer(ca_file, msg): return ANSWER_UNKNOWN elif ans == "FAIL" or ans == "NO": f.close() - return ANSWER_FAIL + return ANSWER_NO elif ans[0] == '"': f.close() return (0, ans.strip('"')) |