diff options
author | Tomáš Levora <t.levora@partner.samsung.com> | 2019-06-05 10:55:46 +0000 |
---|---|---|
committer | Gerrit Code Review <gerrit@onap.org> | 2019-06-05 10:55:46 +0000 |
commit | d3cb6429d6d3cdcd625906e702140a51fc146847 (patch) | |
tree | 5ee819b12371db16cef4f066530ae46f0d799a02 /build/download/rpm_packages.py | |
parent | c8601f0659f90fc2ca390de6d5d7682d1af190f0 (diff) | |
parent | 455be472dfdd4b3c9d2e1cc3c4962115760383f4 (diff) |
Merge "Add base download script"
Diffstat (limited to 'build/download/rpm_packages.py')
-rwxr-xr-x | build/download/rpm_packages.py | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/build/download/rpm_packages.py b/build/download/rpm_packages.py index 7f9700a3..732af0ea 100755 --- a/build/download/rpm_packages.py +++ b/build/download/rpm_packages.py @@ -33,7 +33,7 @@ log = logging.getLogger(name=__name__) def download(rpm_list, dst_dir): if not base.check_tool('yumdownloader'): log.error('ERROR: yumdownloader is not installed') - return 1 + raise RuntimeError('yumdownloader missing') rpm_set = base.load_list(rpm_list) @@ -41,11 +41,10 @@ def download(rpm_list, dst_dir): log.info('Running command: {}'.format(command)) try: subprocess.check_call(command.split()) - log.info('Downloaded') except subprocess.CalledProcessError as err: - log.error(err.output) - return err.returncode - + log.exception(err.output) + raise err + log.info('Downloaded') def run_cli(): @@ -59,7 +58,11 @@ def run_cli(): logging.basicConfig(stream=sys.stdout, level=logging.INFO, format='%(message)s') - sys.exit(download(args.rpm_list, args.output_dir)) + try: + download(args.rpm_list, args.output_dir) + except (subprocess.CalledProcessError, RuntimeError): + sys.exit(1) + if __name__ == '__main__': |