diff options
author | Björn Baumbach <bb@sernet.de> | 2014-04-25 22:05:54 +0200 |
---|---|---|
committer | Michael Adam <obnox@samba.org> | 2014-05-03 01:51:08 +0200 |
commit | a0ab8cb53712cf77cae1d46f49d8eb56e6d5703b (patch) | |
tree | b1cf9f860cfa08e27d2d839a196e7baaabf1065f /wscript | |
parent | 4b4f4e0f4fafc72da7ab8e594595adcba9a383b6 (diff) | |
download | samba-a0ab8cb53712cf77cae1d46f49d8eb56e6d5703b.tar.gz |
waf: fetch and use some exit codes of called processes
Without this patch for example "make ctags" reports "Success",
also if ctags fails or is not available.
Signed-off-by: Björn Baumbach <bb@sernet.de>
Reviewed-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Diffstat (limited to 'wscript')
-rw-r--r-- | wscript | 20 |
1 files changed, 15 insertions, 5 deletions
@@ -218,7 +218,9 @@ def etags(ctx): source_root = os.path.dirname(Utils.g_module.root_path) cmd = 'rm -f %s/TAGS && (find %s -name "*.[ch]" | egrep -v \.inst\. | xargs -n 100 etags -a)' % (source_root, source_root) print("Running: %s" % cmd) - os.system(cmd) + status = os.system(cmd) + if os.WEXITSTATUS(status): + raise Utils.WafError('etags failed') def ctags(ctx): "build 'tags' file using ctags" @@ -226,7 +228,9 @@ def ctags(ctx): source_root = os.path.dirname(Utils.g_module.root_path) cmd = 'ctags --python-kinds=-i $(find %s -name "*.[ch]" | grep -v "*_proto\.h" | egrep -v \.inst\.) $(find %s -name "*.py")' % (source_root, source_root) print("Running: %s" % cmd) - os.system(cmd) + status = os.system(cmd) + if os.WEXITSTATUS(status): + raise Utils.WafError('ctags failed') # putting this here enabled build in the list # of commands in --help @@ -249,14 +253,18 @@ def pydoctor(ctx): cmd='PYTHONPATH=%s pydoctor --introspect-c-modules --project-name=Samba --project-url=http://www.samba.org --make-html --docformat=restructuredtext --add-package bin/python/samba --add-module %s --add-module %s --add-module %s' % ( bp, mpaths['tdb'], mpaths['ldb'], mpaths['talloc'], mpaths['ntdb']) print("Running: %s" % cmd) - os.system(cmd) + status = os.system(cmd) + if os.WEXITSTATUS(status): + raise Utils.WafError('pydoctor failed') def pep8(ctx): '''run pep8 validator''' cmd='PYTHONPATH=bin/python pep8 -r bin/python/samba' print("Running: %s" % cmd) - os.system(cmd) + status = os.system(cmd) + if os.WEXITSTATUS(status): + raise Utils.WafError('pep8 failed') def wafdocs(ctx): @@ -270,7 +278,9 @@ def wafdocs(ctx): for f in list: cmd += ' --add-module %s' % f print("Running: %s" % cmd) - os.system(cmd) + status = os.system(cmd) + if os.WEXITSTATUS(status): + raise Utils.WafError('wafdocs failed') def dist(): |