Merge pull request apache#11983 from ibzib/BEAM-10234

[BEAM-10234] Fix error message for missing licenses.
mransley · Jun 11, 2020 · f5b505a · f5b505a
2 parents f33c85d + 5ae1057
commit f5b505a
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/sdks/python/container/license_scripts/pull_licenses_py.py b/sdks/python/container/license_scripts/pull_licenses_py.py
@@ -115,6 +115,7 @@ def pull_from_url(dep, configs):
  finally:
  shutil.rmtree(cur_temp_dir)
 
+
 if __name__ == "__main__":
  os.makedirs(LICENSE_DIR)
  no_licenses = []
@@ -144,8 +145,12 @@ def pull_from_url(dep, configs):
  'and add entries to sdks/python/container/license_scripts/' \
  'dep_urls_py.yaml.'
  raise RuntimeError(
- 'Could not retrieve licences for packages [{license_list}] in '
+ 'Could not retrieve licences for packages {license_list} in '
  'Python{py_ver} environment. \n {how_to}'.format(
- py_ver=py_ver, license_list=sorted(','.join(no_licenses)), how_to=how_to))
+ py_ver=py_ver,
+ license_list=sorted(no_licenses),
+ how_to=how_to))
  else:
- logging.info('Successfully pulled licenses for {n} dependencies'.format(n=len(dependencies)))
+ logging.info(
+ 'Successfully pulled licenses for {n} dependencies'.format(
+ n=len(dependencies)))