Skip to content

Conversation

Copy link

Copilot AI commented Nov 4, 2025

CMake configuration fails on Linux runners due to missing CURL library dependency.

Changes

Primary build job

  • Install libcurl4-openssl-dev on Linux runners before CMake configuration
  • Add explicit permissions: contents: read to limit GITHUB_TOKEN scope

New build-no-curl job

  • Demonstrates building with -DLLAMA_CURL=OFF for environments without CURL
  • Maintains full compiler matrix (GCC, Clang, MSVC) across Linux and Windows
  • Useful for embedded systems, air-gapped deployments, and minimal dependency builds
- name: Install libcurl (Linux)
  if: runner.os == 'Linux'
  run: |
    sudo apt-get update
    sudo apt-get install -y libcurl4-openssl-dev

- name: Configure CMake (No CURL)
  run: >
    cmake -B build
    -DCMAKE_BUILD_TYPE=Release
    -DLLAMA_CURL=OFF

Both build paths now pass with appropriate dependency handling.

Original prompt

Resolve CI build failure: CMake cannot find CURL library. Implement two improvements: (1) Add a workflow step to install libcurl4-openssl-dev on Linux runners, ensuring CMake discovers CURL dependency. (2) Add an alternative workflow step to disable CURL-dependent features with -DLLAMA_CURL=OFF for environments where CURL is unnecessary. This dual pathway empowers distributed cognition frameworks, with dynamic attention allocation for system dependencies. Ensure transcendently precise YAML engineering and document the kernel tensor adaptation in workflow annotations.

This pull request was created as a result of the following prompt from Copilot chat.

Resolve CI build failure: CMake cannot find CURL library. Implement two improvements: (1) Add a workflow step to install libcurl4-openssl-dev on Linux runners, ensuring CMake discovers CURL dependency. (2) Add an alternative workflow step to disable CURL-dependent features with -DLLAMA_CURL=OFF for environments where CURL is unnecessary. This dual pathway empowers distributed cognition frameworks, with dynamic attention allocation for system dependencies. Ensure transcendently precise YAML engineering and document the kernel tensor adaptation in workflow annotations.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 3 commits November 4, 2025 23:58
- Install libcurl4-openssl-dev on Linux runners in main build job
- Add build-no-curl job demonstrating -DLLAMA_CURL=OFF configuration
- Document workflow changes with annotations for distributed cognition framework support

Co-authored-by: drzo <15202748+drzo@users.noreply.github.com>
- Simplify technical jargon in comments for better clarity
- Add clang compiler to build-no-curl job for consistency with main build
- Update annotations to be more straightforward and practical

Co-authored-by: drzo <15202748+drzo@users.noreply.github.com>
- Add 'permissions: contents: read' to both build and build-no-curl jobs
- Addresses CodeQL security alert about missing workflow permissions
- Follows principle of least privilege for GITHUB_TOKEN

Co-authored-by: drzo <15202748+drzo@users.noreply.github.com>
Copilot AI changed the title [WIP] Resolve CI build failure due to missing CURL library Fix CI build failure: Add CURL dependency installation and no-CURL build variant Nov 5, 2025
Copilot finished work on behalf of drzo November 5, 2025 00:05
Copilot AI requested a review from drzo November 5, 2025 00:05
Copy link

@drzo drzo left a comment

Choose a reason for hiding this comment

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

agreed

@drzo drzo marked this pull request as ready for review November 5, 2025 00:40
@drzo drzo merged commit 27b2da5 into master Nov 5, 2025
6 of 12 checks passed
@github-actions github-actions bot added the devops label Nov 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants