Skip to content

GH-50318: [R][CI] Install missing libpng-dev for test-r-linux-as-cran#50328

Merged
raulcd merged 1 commit into
apache:mainfrom
kou:ci-r-clang
Jul 2, 2026
Merged

GH-50318: [R][CI] Install missing libpng-dev for test-r-linux-as-cran#50328
raulcd merged 1 commit into
apache:mainfrom
kou:ci-r-clang

Conversation

@kou

@kou kou commented Jul 2, 2026

Copy link
Copy Markdown
Member

Rationale for this change

We need libpng-dev for the png R package.

What changes are included in this PR?

Install libpng-dev.

Are these changes tested?

Yes.

Are there any user-facing changes?

No.

Copilot AI review requested due to automatic review settings July 2, 2026 02:06
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown

⚠️ GitHub issue #50318 has been automatically assigned in GitHub to PR creator.

@kou

kou commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

@github-actions crossbow submit test-r-linux-as-cran

@github-actions github-actions Bot added the awaiting committer review Awaiting committer review label Jul 2, 2026

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

This PR fixes the test-r-linux-as-cran CI failure by ensuring the Ubuntu/Debian system dependencies include libpng-dev, which is needed to compile the png R package from source in the CRAN-style environment.

Changes:

  • Add libpng-dev to the apt-get install dependency list for the R CI system-deps script.
  • Document libpng as a required system dependency in the script comments.

Comment on lines 44 to 48
apt-get)
apt-get install -y libcurl4-openssl-dev libssl-dev libuv1-dev
apt-get install -y libcurl4-openssl-dev libpng-dev libssl-dev libuv1-dev
;;
apk)
$PACKAGE_MANAGER add curl-dev openssl-dev libuv-dev
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown

Revision: d2645b4

Submitted crossbow builds: ursacomputing/crossbow @ actions-97f1925c38

Task Status
test-r-linux-as-cran GitHub Actions

@kou

kou commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

The job is still failing but the png R package failure is fixed: https://github.com/ursacomputing/crossbow/actions/runs/28560424956/job/84676787021#step:5:17952

  #22 106.8 * DONE (png)

@raulcd raulcd merged commit 54cd7c7 into apache:main Jul 2, 2026
43 checks passed
@raulcd raulcd removed the awaiting committer review Awaiting committer review label Jul 2, 2026
@github-actions github-actions Bot added the awaiting merge Awaiting merge label Jul 2, 2026
@kou kou deleted the ci-r-clang branch July 2, 2026 08:39
raulcd pushed a commit that referenced this pull request Jul 2, 2026
…#50328)

### Rationale for this change

We need `libpng-dev` for the png R package.

### What changes are included in this PR?

Install `libpng-dev`.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* GitHub Issue: #50318

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Raúl Cumplido <raulcumplido@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting merge Awaiting merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants