[PATCH] D143320: [bazel] Rework zlib dependency

[Geoffrey Martin-Noble via Phabricator via llvm-commits]

GMNGeoffrey requested changes to this revision.
GMNGeoffrey added a subscriber: chandlerc.
GMNGeoffrey added a comment.
This revision now requires changes to proceed.

I like that this fixes this to use a proper flag, but support for using the system version was deliberate and static-linking an external archive had interesting licensing implications. Let me dig up the history. Hermeticity is indeed an important consideration, but building with system dependencies is also key functionality. @chandlerc, who added this originally.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D143320/new/

https://reviews.llvm.org/D143320