Skip to content

Commit ed81605

Browse files
authored
[ABLD-158] Switch @openssl// to depend on @xz//:liblzma instead of :lzma (#42545)
It turns out the things that depend on openssl can not build with the :lzma usege. It needs more investigation, but I think the culprit is using the output_shared_library feature of rules-foreign-cc. We probably should just build a regulary library and then add a cc_shared_library on top of that. Doing this now, even without complete understanding, allows us to add curl on top of openssl, and then test a real dependency stack to a binary. Needed by #42546 Co-authored-by: tony.aiuto <tony.aiuto@datadoghq.com>
1 parent bbd22f0 commit ed81605

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

deps/openssl.BUILD.bazel

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ configure_make(
6565
}),
6666
deps = [
6767
"@bzip2//:bz2",
68-
"@xz//:lzma",
68+
"@xz//:liblzma",
6969
"@zlib",
7070
],
7171
)

0 commit comments

Comments
 (0)