Initialize terminfo.bzl linkopts to None

The underlying issue here is that line 125 checks if linkopts was assigned (by line 120) but it's not initialized, so that becomes a crash. This was noticed by someone trying to use Docker, so no terminfo library installed.

Reviewed By: GMNGeoffrey

Differential Revision: https://reviews.llvm.org/D118270
This commit is contained in:
jonmeow 2022-01-26 11:02:20 -08:00 committed by Geoffrey Martin-Noble
parent 216002c4bb
commit 4691f00a63
1 changed files with 1 additions and 0 deletions

View File

@ -113,6 +113,7 @@ def _llvm_terminfo_system_impl(repository_ctx):
# these would be provided as lists, but Bazel doesn't currently
# support that. See: https://github.com/bazelbuild/bazel/issues/12178
linkopts_candidates = [[x] for x in repository_ctx.attr.candidate_system_linkopts]
linkopts = None
# For each candidate, try to use it to link our test source file.
for linkopts_candidate in linkopts_candidates: