Skip to content

Install libuv headers for R CI#12173

Merged
RAMitchell merged 1 commit into
dmlc:masterfrom
RAMitchell:codex/fix-r-libuv-ci
Apr 20, 2026
Merged

Install libuv headers for R CI#12173
RAMitchell merged 1 commit into
dmlc:masterfrom
RAMitchell:codex/fix-r-libuv-ci

Conversation

@RAMitchell

Copy link
Copy Markdown
Member

Summary

Fix R CI dependency installation by installing libuv1-dev in the Linux R test jobs.

Recent R CI failures on both this PR and master show the R package fs failing to configure because libuv is unavailable via pkg-config:

Package 'libuv', required by 'virtual:world', not found
Configuration failed because libuv was not found.
ERROR: configuration failed for package ‘fs’

The workflow already installs cmake, which previously allowed fs to build its bundled libuv, but the current dependency resolution now requires the system libuv development package. Installing libuv1-dev provides the headers and libuv.pc.

Changes

  • Add libuv1-dev to the Ubuntu R test system dependencies.
  • Add libuv1-dev to the Debian-container R test system dependencies.
  • Keep cmake installed as a fallback for fs bundled-libuv builds.

Testing

  • git diff --check
  • Parsed .github/workflows/r_tests.yml with PyYAML.
  • Commit hooks passed, including check yaml.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the R CI workflow to install libuv development headers so the R package fs can configure successfully (via pkg-config) during Linux R test jobs.

Changes:

  • Install libuv1-dev in the Ubuntu R test job.
  • Install libuv1-dev in the Debian-container R test job.
  • Adjust the cmake comment to reflect that it’s only needed if fs builds bundled libuv.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/r_tests.yml
@RAMitchell
RAMitchell marked this pull request as ready for review April 20, 2026 10:30
@RAMitchell
RAMitchell merged commit c2559a5 into dmlc:master Apr 20, 2026
83 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants