[PATCH] D143320: [bazel] Rework zlib dependency

Aaron Siddhartha Mondal via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 6 16:56:24 PST 2023


aaronmondal added a comment.

In D143320#4108196 <https://reviews.llvm.org/D143320#4108196>, @GMNGeoffrey wrote:

> You'd add `-lz` as was here previously and let the system handle it. Definitely don't want to reinvent system package search here :-)

This only works if `-L<path_to_lib>` is set and may cause inconsistencies in some shells and environments (like Nix), where it becomes unclear whether we are linking `-L/usr/lib` or `-L/veryesotericpath`. If that path is unknown or disabled things don't work at all. So far I haven't found a way to "query" for these standard library paths please let me know if there is some some config file or something that I'm not aware of. This is causing me so many headaches.

Regarding the WORKSPACE logic, that sounds good. I'll try to get something like that working but that may take me a while. Tbh I'm not really using WORKSPACES myself and instead using the bzlmod patch series that I tried upstreaming a while ago but I think they got kinda dropped and would probably need some rework at this point. (A working bazel module for a registry is here <https://github.com/eomii/bazel-eomii-registry/tree/main/modules/llvm-project-overlay/17-init-bcr.0> with sample usage here <https://github.com/eomii/rules_ll/blob/d451cb2ab5efb3ea1a86c5a40aca7f545043ec8c/MODULE.bazel#L25>).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143320



More information about the llvm-commits mailing list