Skip to content

docs: add libcurl-dev install hint for Linux distros #14801

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 24, 2025

Conversation

PouyaGhahramanian
Copy link
Contributor

I hit a CMake configure error: Could NOT find CURL when building on Ubuntu. The build guide mentions that curl is enabled by default and can be disabled, but doesn’t list the packages to install. This PR adds distro-specific install one-liners (Debian/Ubuntu, Fedora/RHEL, Arch) below the curl note to help first-time builders avoid the error. Tested on Ubuntu 22.04: reinstalling libcurl4-openssl-dev allowed CMake to find CURL successfully.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jul 21, 2025
docs/build.md Outdated
@@ -68,6 +68,9 @@ cmake --build build --config Release
cmake --build build-x64-windows-llvm-release
```
- Curl usage is enabled by default and can be turned off with `-DLLAMA_CURL=OFF`. Otherwise you need to install development libraries for libcurl.
- Debian / Ubuntu: `sudo apt-get install libcurl4-openssl-dev`
- Fedora / RHEL: `sudo dnf install libcurl-devel`
- Arch: `sudo pacman -S curl`
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should it be something like libcurl or curl-dev or something else? otherwise we should remove this line if unsure

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for catching that!
I’ve replaced the vague “curl‑dev” wording with the exact dev packages for the main distros:

Debian/Ubuntu -> libcurl4‑openssl‑dev (alt: libcurl4‑gnutls‑dev)

Fedora/RHEL -> libcurl‑devel

Arch -> curl (contains headers)
Sources are linked in the commit message.
Let me know if you’d prefer we drop the Arch line or add any other distro.

@PouyaGhahramanian PouyaGhahramanian force-pushed the docs/add-libcurl-dev-note branch from 1781010 to 4e1196c Compare July 23, 2025 10:24
@ngxson ngxson merged commit 39cffdf into ggml-org:master Jul 24, 2025
2 checks passed
taronaeo pushed a commit to taronaeo/llama.cpp-s390x that referenced this pull request Jul 25, 2025
* docs: add libcurl-dev install hint for Linux distros

Signed-off-by: PouyaGhahramanian <[email protected]>

* Update docs/build.md

---------

Signed-off-by: PouyaGhahramanian <[email protected]>
Co-authored-by: Xuan-Son Nguyen <[email protected]>
gabe-l-hart added a commit to gabe-l-hart/llama.cpp that referenced this pull request Jul 25, 2025
* origin/master:
docs : update HOWTO‑add‑model.md for ModelBase and new model classes (ggml-org#14874)
ggml : remove invalid portPos specifiers from dot files (ggml-org#14838)
context : restore preemptive sched reset when LLAMA_SET_ROWS=0 (ggml-org#14870)
mtmd : fix 32-bit narrowing issue in export-lora and mtmd clip (ggml-org#14503)
rpc : check for null buffers in get/set/copy tensor endpoints (ggml-org#14868)
sched : fix multiple evaluations of the same graph with pipeline parallelism (ggml-org#14855)
musa: upgrade musa sdk to rc4.2.0 (ggml-org#14498)
sync : ggml
cmake : fix usage issues (ggml/1257)
ggml-cpu : remove stdlib include from repack.cpp (ggml/1276)
context : perform output reorder lazily upon access after sync (ggml-org#14853)
chat : fix kimi-k2 chat template (ggml-org#14852)
sycl: fixed semantics of block offset calculation (ggml-org#14814)
llama : fix MiniCPM inference after Granite Four changes (ggml-org#14850)
docs: add libcurl-dev install hint for Linux distros (ggml-org#14801)
metal : fix fusion across different encoders (ggml-org#14849)
sycl: fix undefined variable in work group size check (ggml-org#14843)
convert : text-only support for GLM-4.1V-9B-Thinking (ggml-org#14823)
CUDA: fix overflow in FA, tune performance (ggml-org#14840)
CUDA: fix compilation with GGML_CUDA_F16 (ggml-org#14837)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants