[PATCH] D115131: [Support] [Debuginfod] Move HTTPClient to Debuginfod library.

[Noah Shutty via Phabricator via llvm-commits]

noajshu added a comment.

In D115131#3173872 <https://reviews.llvm.org/D115131#3173872>, @kpn wrote:

> BTW, for a future ticket: having curl enabled without first checking to see if it is installed creates builds failures on non-Linux hosts. OK, yes, it can be disabled with a cmake argument, but it would be better if the extra cmake argument wasn't needed. A warning at cmake configure time that says what features will be missing without curl is probably a good idea as well.

Thanks, the configuration step does check <https://github.com/llvm/llvm-project/blob/main/llvm/cmake/config-ix.cmake#L162> if curl is installed first before trying to build with it. Does this not work for your build? And does D115131 <https://reviews.llvm.org/D115131> solve the problem?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115131