Skip to content
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

Add xz-utils as dependencies #1677

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

MarioZZJ
Copy link

Description

Fix licensed gem installation failure on ubuntu-latest (probably Ubuntu 24.04) runner. The issue was discovered during recent GitHub Actions ubuntu-latest image upgrade (announcement), where nokogiri compilation fails due to missing xzcat.

Problem

When running metrics action on Ubuntu 24.04 (new ubuntu-latest), licensed gem installation failed with:

ERROR: Failed to build gem native extension.
...
/var/lib/gems/3.1.0/gems/mini_portile2-2.8.8/lib/mini_portile2/mini_portile.rb:557:in `xzcat_exe': xzcat not found (RuntimeError)

Solution

Add xz-utils package to Dockerfile dependencies to provide xzcat command required by nokogiri compilation.

Testing

Successfully tested with action running on ubuntu-latest (workflow run)

Notes

Since GitHub's ubuntu-latest upgrade is rolling out until January 17th, 2025 (see announcement):

  • This PR could be merged after majority of users migrate to Ubuntu 24.04
  • Or affected users can temporarily use runs-on: ubuntu-22.04 as workaround

The licensed gem installation fails due to missing `xzcat` command during `nokogiri` compilation.
Adding `xz-utils` package resolves this dependency issue.
@MarioZZJ
Copy link
Author

@lowlighter Hi, I noticed this issue during recent GitHub Actions ubuntu-latest upgrade process.

Would you mind reviewing this PR?

SN-Koarashi added a commit to SN-Koarashi/metrics that referenced this pull request Dec 28, 2024
Yomisana added a commit to Yomisana/metrics that referenced this pull request Dec 28, 2024
fix: add `xz-utils` dependencies, actions linux system not had this dependencies

refer to lowlighter#1677 PR
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.

1 participant