From 2dc852c214fe98863961d6a426dccf762e6bc52e Mon Sep 17 00:00:00 2001 From: Fabian Meumertzheim Date: Thu, 3 Feb 2022 08:53:01 +0100 Subject: [PATCH] Fix fail message construction in cc_shared_library Previously, if there were many unaccounted for libraries, this would simply fail with: ``` Error in join: 'string' is not iterable ``` --- .../builtins_bzl/common/cc/experimental_cc_shared_library.bzl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/starlark/builtins_bzl/common/cc/experimental_cc_shared_library.bzl b/src/main/starlark/builtins_bzl/common/cc/experimental_cc_shared_library.bzl index 75a3f7f38cfbfb..ed81c52ad233ce 100644 --- a/src/main/starlark/builtins_bzl/common/cc/experimental_cc_shared_library.bzl +++ b/src/main/starlark/builtins_bzl/common/cc/experimental_cc_shared_library.bzl @@ -342,7 +342,7 @@ def _throw_error_if_unaccounted_libs(unaccounted_for_libs): libs_message.append(str(unaccounted_lib)) if len(unaccounted_for_libs) > 10: - libs_message = "(and " + str(len(unaccounted_for_libs) - 10) + " others)\n" + libs_message.append("(and " + str(len(unaccounted_for_libs) - 10) + " others)\n") static_deps_message = [] for repo in different_repos: