This is an automated email from the ASF dual-hosted git repository.

yhu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git


The following commit(s) were added to refs/heads/master by this push:
     new 8801afdbd32 Update to upstream caller (#37270)
8801afdbd32 is described below

commit 8801afdbd321d6de4a9dca23f0416b125a8ff14e
Author: Derrick Williams <[email protected]>
AuthorDate: Fri Jan 9 13:32:23 2026 -0500

    Update to upstream caller (#37270)
---
 sdks/java/container/license_scripts/pull_licenses_java.py | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/sdks/java/container/license_scripts/pull_licenses_java.py 
b/sdks/java/container/license_scripts/pull_licenses_java.py
index 7032b368284..f0c1b48468f 100644
--- a/sdks/java/container/license_scripts/pull_licenses_java.py
+++ b/sdks/java/container/license_scripts/pull_licenses_java.py
@@ -137,7 +137,7 @@ def pull_from_url(file_name, url, dep, no_list, 
use_cache=False):
     if use_cache:
         CACHED_LICENSES.add(os.path.basename(pulled_file_name))
         logging.info(f"Copying {pulled_file_name} -> {file_name}")
-        shutil.copy(pull_file_name, file_name)
+        shutil.copy(pulled_file_name, file_name)
 
 def pull_source_code(base_url, dir_name, dep):
     # base_url example: 
https://repo1.maven.org/maven2/org/mortbay/jetty/jsp-2.1/6.1.14/
@@ -189,7 +189,7 @@ def execute(dep):
         "moduleLicenseUrl": "http://www.antlr.org/license.html";
     }
     '''
-
+    logging.debug("Dep: %s", dep)
     name = dep['moduleName'].split(':')[1]
     version = dep['moduleVersion']
     name_version = name + '-' + version
@@ -218,8 +218,14 @@ def execute(dep):
                 with thread_lock:
                     no_licenses.append(name_version)
                 license_url = 'skip'
-        pull_from_url(dir_name + '/LICENSE', license_url, name_version,
-                      no_licenses, use_cache=use_license_cache)
+
+        # Split the url string by commas in case of multiple/dual licenses
+        # NOTE: If license doesn't have a ',', this is a no-op.
+        # TODO: Do we only download our preferred one?
+        license_urls = [u.strip() for u in license_url.split(',')]
+        for license_url in license_urls:
+            pull_from_url(dir_name + '/LICENSE', license_url, name_version,
+                        no_licenses, use_cache=use_license_cache)
         # pull notice
         try:
             notice_url = dep_config[name][version]['notice']

Reply via email to