emilymye commented on a change in pull request #13117:
URL: https://github.com/apache/beam/pull/13117#discussion_r505091993
##########
File path: sdks/java/container/license_scripts/pull_licenses_java.py
##########
@@ -140,17 +149,22 @@ def execute(dep):
}
'''
- name = dep['moduleName'].split(':')[1].lower()
+ name = dep['moduleName'].split(':')[1]
version = dep['moduleVersion']
name_version = name + '-' + version
+ # javac is not a runtime dependency
+ if name == 'javac':
+ logging.debug('Skipping', name_version)
+ return
dir_name = '{license_dir}/{name_version}.jar'.format(
license_dir=license_dir, name_version=name_version)
# if auto pulled, directory is existing at {license_dir}
if not os.path.isdir(dir_name):
# skip self dependencies
if dep['moduleName'].startswith('beam'):
- logging.debug('Skippig', name_version)
+ logging.debug('Skipping', name_version)
Review comment:
Do we only skip self-dependencies for non-auto-pulled libraries? Mostly
this function seems very long and complicated to start with, and having the
skipping returns at the top would make me feel a little better.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
